Do a sweep and remove most WARNS=6 settingsRepeating the default WARNS here makes it slightly more difficult toexperiment with default WARNS changes, e.g. if we did something absolutelybananas an
Do a sweep and remove most WARNS=6 settingsRepeating the default WARNS here makes it slightly more difficult toexperiment with default WARNS changes, e.g. if we did something absolutelybananas and introduced a WARNS=7 and wanted to try lifting the default tothat.Drop most of them; there is one in the blake2 kernel module, but I suspectit should be dropped -- the default WARNS in the rest of the build doesn'tcurrently apply to kernel modules, and I haven't put too much thought intowhether it makes sense to make it so.
show more ...
Re-enable proc_test:symbol_lookup after r360979.PR: 244732MFC after: 2 weeksSponsored by: The FreeBSD Foundation
Temporarily skip 2 failing tests after llvm10 importPR: 244732Sponsored by: The FreeBSD Foundation
Detach from the child process before completing the test.Otherwise the child will receive SIGTRAP if the parent exits first.
Disable optimization of the libproc test program.Dead code elimination may remove symbols that are required by the tests.Reported by: Jenkins, via asomersMFC after: 1 week
Add an envp argument to proc_create().This is needed to support dtrace's -x setenv option.MFC after: 2 weeks
Eliminate the last user of basename_r() in the base system.In this case it's fairly easy to make use of basename().
Refine symtab sorting in libproc.Add some rules to more closely match what illumos does when an addressresolves to multiple symbols:- prefer non-local symbols- prefer symbols with fewer leading
Refine symtab sorting in libproc.Add some rules to more closely match what illumos does when an addressresolves to multiple symbols:- prefer non-local symbols- prefer symbols with fewer leading underscores and no leading '$'Add some regression tests to verify these rules.
DIRDEPS_BUILD: Update dependencies.Sponsored by: Dell EMC Isilon
Increase WARNS for libproc testsATF tests have a default WARNS of 0, unlike other usermode programs.Reviewed by: ngie, julianMFC after: 3 weeksSponsored by: Spectra Logic CorporationDiffer
Increase WARNS for libproc testsATF tests have a default WARNS of 0, unlike other usermode programs.Reviewed by: ngie, julianMFC after: 3 weeksSponsored by: Spectra Logic CorporationDifferential Revision: https://reviews.freebsd.org/D9933
libproc: Match prefixes when looking up mapped object by name.When looking up an object by name, allow prefix matches if no direct matchis found. This allows one to, for example, match libc entry
libproc: Match prefixes when looking up mapped object by name.When looking up an object by name, allow prefix matches if no direct matchis found. This allows one to, for example, match libc entry probes with: # dtrace -n 'pid$target:libc.so::entry' -c ./fooinstead of requiring "libc.so.7" or a glob.Also remove proc_obj2map() as it currently just duplicates thefunctionality of proc_name2map(). It's supposed to take a Solarislink-map ID as a paramter, but support for this isn't implemented andisn't required to support DTrace's pid provider.
DIRDEPS_BUILD: Update dependencies.Sponsored by: EMC / Isilon Storage Division
Merge ^/user/ngie/release-pkg-fix-tests to unbreak how test files are installedafter r298107Summary of changes:- Replace all instances of FILES/TESTS with ${PACKAGE}FILES. This ensures that na
Merge ^/user/ngie/release-pkg-fix-tests to unbreak how test files are installedafter r298107Summary of changes:- Replace all instances of FILES/TESTS with ${PACKAGE}FILES. This ensures that namespacing is kept with FILES appropriately, and that this shouldn't need to be repeated if the namespace changes -- only the definition of PACKAGE needs to be changed- Allow PACKAGE to be overridden by callers instead of forcing it to always be `tests`. In the event we get to the point where things can be split up enough in the base system, it would make more sense to group the tests with the blocks they're a part of, e.g. byacc with byacc-tests, etc- Remove PACKAGE definitions where possible, i.e. where FILES wasn't used previously.- Remove unnecessary TESTSPACKAGE definitions; this has been elided into bsd.tests.mk- Remove unnecessary BINDIRs used previously with ${PACKAGE}FILES; ${PACKAGE}FILESDIR is now automatically defined in bsd.test.mk.- Fix installation of files under data/ subdirectories in lib/libc/tests/hash and lib/libc/tests/net/getaddrinfo- Remove unnecessary .include <bsd.own.mk>s (some opportunistic cleanup)Document the proposed changes in share/examples/tests/tests/... via examplesso it's clear that ${PACKAGES}FILES is the suggested way forward in terms ofreplacing FILES. share/mk/bsd.README didn't seem like the appropriate methodof communicating that info.MFC after: never probablyX-MFC with: r298107PR: 209114Relnotes: yesTested with: buildworld, installworld, checkworld; buildworld, packageworldSponsored by: EMC / Isilon Storage Division
DIRDEPS_BUILD: Connect MK_TESTS.Sponsored by: EMC / Isilon Storage Division
Avoid adding the '-release' suffix to non-debug kernel packages.Sponsored by: The FreeBSD Foundation
Remove WARNS inherited fine by ../Makefile.inc.Sponsored by: EMC / Isilon Storage Division
Fix LDADD/DPADD that should be LIBADD.Sponsored by: EMC / Isilon Storage Division
Refactor the test/ Makefiles after recent changes to bsd.test.mk (r289158) andnetbsd-tests.test.mk (r289151)- Eliminate explicit OBJTOP/SRCTOP setting- Convert all ad hoc NetBSD test integration
Refactor the test/ Makefiles after recent changes to bsd.test.mk (r289158) andnetbsd-tests.test.mk (r289151)- Eliminate explicit OBJTOP/SRCTOP setting- Convert all ad hoc NetBSD test integration over to netbsd-tests.test.mk- Remove unnecessary TESTSDIR setting- Use SRCTOP where possible for clarityMFC after: 2 weeksSponsored by: EMC / Isilon Storage Divison
Enable libproc symbol_lookup tests on arm64This reverts part of r286863, as the kernel support required by thesetests was added in r287105.PR: 202305Sponsored by: The FreeBSD Foundation
On arm64 disable three tests that hang or panicEach issue has a PR open to track. This workaround allows us to run thetests to investigate the failures and avoid any new regressions.PR: 202304,
On arm64 disable three tests that hang or panicEach issue has a PR open to track. This workaround allows us to run thetests to investigate the failures and avoid any new regressions.PR: 202304, 202305, 202307Reviewed by: ngieSponsored by: The FreeBSD FoundationDifferential Revision: https://reviews.freebsd.org/D3378
Ensure that we don't try to demangle a symbol name if we failed to lookup the symbol. Add a test to exercise this code path.Reviewed by: adrian
Hook up support for userland CTF support in DTrace. This required somemodifications to libproc to support fetching the CTF info for a given file.With this change, dtrace(1) is able to resolve type
Hook up support for userland CTF support in DTrace. This required somemodifications to libproc to support fetching the CTF info for a given file.With this change, dtrace(1) is able to resolve type info for function andUSDT probe arguments, and function return values. In particular, the args[n]syntax should now work for referencing arguments of userland probes,provided that the requisite CTF info is available.The uctf tests pass if the test programs are compiled with CTF info. Thecurrent infrastructure around the DTrace test suite doesn't support thisyet.Differential Revision: https://reviews.freebsd.org/D891MFC after: 1 monthRelnotes: yesSponsored by: EMC / Isilon Storage Division
Add some ATF tests for libproc.Differential Revision: D710Reviewed by: jmmv, ngie, rpaulo