Re: Removing stuff from NOTES because it is in GENEREIC

On Fri, 27-Mar-2026 at 22:33:46 -0600, Warner Losh wrote:
> On Fri, Mar 27, 2026 at 2:54 AM Andre Albsmeier <mail@fbsd2.e4m.org> wrote:
> 
> > On Thu, 26-Mar-2026 at 12:31:44 +0100, Dag-Erling Smørgrav wrote:
> > > Andre Albsmeier <mail@fbsd2.e4m.org> writes:
> > > > Is it correct to remove something from NOTES because it is in GENERIC?
> > > >
> > > > My impression was that NOTES contains ALL what can be configured into
> > > > a kernel (whether it makes sense or not). Am I wrong here?
> > >
> > > You are correct.  NOTES is intended to have all options that can be
> > > enabled, plus non-default values for all parameters that can be tuned.
> >
> > OK, thanks to you and Warner for clarifying.
> >
> > When checking sys/amd64/conf/GENERIC on -HEAD, I found the following
> > items which are not in sys/{.,x86,amd64}/conf/NOTES:
> >
> > device ocs_fc
> > device tws
> > device ufshci
> > device aq
> > device vge
> >
> 
> These are needed. I'll commit.
> 
> 
> > option EXTERR_STRINGS
> > option IPSEC_OFFLOAD
> > option FIB_ALGO
> > option PRINTF_BUFR_SIZE
> > option KDTRACE_FRAME
> > option DDB_CTF
> > option RACCT_DEFAULT_TO_DISABLED
> > option AH_AR5416_INTERRUPT_MITIGATION
> >
> > Should they be added?
> >
> 
> These are in GENERIC and not LINT, so we get things done both ways.

Sorry, I don't understand entirely. Most options we find in GENERIC
are also in one of the NOTES. So why shouldn't those above not be
included as well?

Or, in other words: Why, just as an example, do we have

options         IEEE80211_DEBUG
options         IEEE80211_SUPPORT_MESH
options         USB_DEBUG

in NOTES when they are already in GENERIC?

	-Andre