For non-ports contributions we should really discourage submissions via bugzilla unless it's what a specific committer you're working with wants. Generally, bugzilla is where patches go to die.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Today
Yesterday
SNDST_UNVLBUF_MAX is probably excessively large, but is fine as a safety measure.
Fri, May 17
More comments.
You need to decide the type of data based on the command passed before you examine it so you'll need a case statement before the tests. I'd also tend not to assign the arg/arg32 until you've decided the type. Additionally, data will be NULL for at least SNDSTIOC_REFRESH_DEVS and SNDSTIOC_FLUSH_USER_DEVS,
In D45233#1031909, @dim wrote:This is fine for now, but may be can put some #ifdefs in the version maps? Or some more fine-grained gcc version checks?
I noticed we mostly don't use printf in share/mk so switched to the ${:U@} trick. When doing that I noticed that bsd.symver.mk finds it's awk script with a path search so I switched to that model.
Thu, May 16
Seems reasonable
Tue, May 14
Mon, May 13
Fri, May 10
Thanks for picking this up, you went a similarly direction to what I was thinking, but I've been at a conference and traveling home so distracted.
Thu, May 9
I've created a tracking bug https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=278879.
Wed, May 8
In D45086#1029238, @des wrote:Would you like me to commandeer this?
- sort arch list
- examine X_COMPILER_TYPE if defined and COMPILER_TYPE iff X_COMPILER_TYPE is not defined.
Sat, May 4
I think I've addressed everything. There might be argument for just using a pragma push/pop to disable the warnings instead of adding checks, but I've already spent more time then this code is worth.
Fri, May 3
Fix strftime log message. Prefer sizeof(buffer) to bare MAXPATHLEN.
Graturiously check the return of sprintf because glibc's headers
require it when compiling with gcc.
The easiest way to address this would probably be a cross toolchain wrapper script that picks the right compiler when -m32 is on the command line, but just disabling it for now seems reasonable. (I'm trying to test make tinderbox with gcc13 and this was one of the issues I encountered.)
Thu, May 2
actually install symref.awk
Eliminate the use of printf
Avoid a couple compiler invocations to create empty object files
These symbols are amd64-only
Wed, May 1
With D45052 committed, this passes a make CROSS_TOOLCHAIN=llvm18 -DWITHOUT_TOOLCHAIN -DMAKE_JUST_WORLDS -j40 tinderbox run. I'll kick off a full run over night in case there are issues in the toolchain bits, but I think this is ready to land. We have had several regressions since I first posted this review so getting this in is clearly valuable (admittedly, a number of which were my fault in merging patches I failed to notice had gotten out of date.)
Rebase, bump doc date
This seems to be the result of botched testing. D45052 replaces it.
Tue, Apr 30
Move extendxftf2 and trunctfxf2 removal to D45034
I've moved the extendxftf2 and trunctfxf2 removal to D45034.
@arichardson made a big change upstream that impacted the upstream conditional for extendxftf2 and trunctfxf2 prior to llvm18's release. I might want to make that change a separate commit.
Only complain about missing symref file in the current directory.
Otherwise, it's too noisy in lib/libc due to libraries in the tests tree.
Exclude private symbols
In D44271#1026327, @kib wrote:In D44271#1026288, @brooks wrote:In D44271#1025583, @kib wrote:Private versioning namespace should be filtered out, I think.
I think we want to know when that set changes even if we don't consider it part of the ABI, but I probably could be persuaded otherwise. I might want to make it optional so libraries can opt in and others opt out.
At least it should be split into two parts somehow, either textually in the same file, or putting the FBSDprivate into aux file at all.
Explain why we can't use readelf -D and add symbol visibility.
Mon, Apr 29
In D44271#1025583, @kib wrote:Private versioning namespace should be filtered out, I think.
Review suggestions from @sjg
Sat, Apr 27
typo fix
- Rewrite with readelf and an awk script. Weak vs global and symbol type are now tracked. It's also a bit less grotty as readelf implementations are more similar.
- Make the symref files dot-files so they clutter the tree less.
Fri, Apr 26
I'm going to switch to using readelf