kgdb now handles kernel module state internally, so the asf tool serves no purpose.
PR: 229046 Relnotes: yes Sponsored by: The FreeBSD Foundation
Differential D15827
Remove obsolete asf(8) emaste on Jun 15 2018, 4:57 PM. Authored by Tags None Referenced Files
Details
kgdb now handles kernel module state internally, so the asf tool serves no purpose. PR: 229046 Relnotes: yes Sponsored by: The FreeBSD Foundation
Diff Detail
Event TimelineComment Actions Does it make since to first commit a deprecation notice to this, which can be merged to stable/11, then actually remove it from ^/head after that? This avoids the need for a direct commit to stable/11 Comment Actions LGTM. Doing the deprecation notice in the manpage seems like a good idea, but won't effect this review. Comment Actions @brooks a fully sgree, infact it can be done in a seperate review, and the asf binary should spit out a warning, not just a man page change. Comment Actions perhaps this for deprecation notice: Index: usr.sbin/asf/asf.8 =================================================================== --- usr.sbin/asf/asf.8 (revision 335216) +++ usr.sbin/asf/asf.8 (working copy) @@ -37,6 +37,15 @@ .Op Fl o Ar outfile .Op Fl X Ar suffix .Op Ar modules-path Op Ar outfile +.Sh DEPRECATION NOTICE +The +.Nm +utility is not present in +.Fx 12.0 +and later. +The +.Xr kgdb 1 +debugger handles kernel module state internally. .Sh DESCRIPTION By default, .Nm Comment Actions I actually worry less about man page DEPRECATION, and more about having the binary spit something out so anyone using it for who knows what gets a heads up that it is/may be going away soon. Comment Actions Spitting out a warning is less useful since the output generates a GDB script (though stderr would perhaps be ok). That said, this probably hasn't actually worked in several years and has rotted since kgdb grew native module support back in 7.1 release. I'm surprised it was x86 only. That seems a bit odd as there was nothing about this tool that was MD. All the arch-specific bits live in the objdump binary it invoked. |