User Details
- User Since
- Jul 1 2014, 6:02 PM (515 w, 5 d)
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,
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
- 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
Explain why we can't use readelf -D and add symbol visibility.
Mon, Apr 29
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