2018-04-06 0:03 GMT+09:00 Don Zickus <dzickus@xxxxxxxxxx>: > On Thu, Apr 05, 2018 at 11:29:11AM +0900, Masahiro Yamada wrote: >> 2018-04-05 4:56 GMT+09:00 Don Zickus <dzickus@xxxxxxxxxx>: >> > We at Red Hat/Fedora have generally tried to have a per file breakdown of >> > every config option we set. This makes it easy for us to add new options >> > when they are exposed and keep a changelog of why they were set. >> > >> > A Fedora example is here: >> > https://src.fedoraproject.org/cgit/rpms/kernel.git/tree/configs/fedora/generic >> > >> > Using various merge scripts, we build up a config file and run it through >> > 'make listnewconfig' and 'make oldnoconfig'. The idea is to print out new >> > config options that haven't been manually set and use the default until >> > a patch is posted to set it properly. >> > >> > To speed things up, it would be nice to make it easier to generate a >> > patch to post the default setting. The output of 'make listnewconfig' >> > has two issues that limit us: >> > >> > - it doesn't provide the default value >> > - it doesn't provide the new 'choice' options that get flagged in >> > 'oldconfig' >> >> >> So, 'listnewconfig' is a subset of 'listnewdefconfig'. >> >> I wonder if we could extend 'listnewconfig' to meet your demands >> instead of adding the new target. > > Sure. I am open to whatever you think is appropriate. :-) I prefer extending the existing target. >> >> >> What for '#choice' comment ? > > This was something silly added by me to make it obvious that > CONFIG_KERNEL_XZ doesn't needed to be decided, it just needs us to re-review > our current default to see if we should keep it or switch to something new. > > We can drop that piece of the patch. I wasn't sure if that separating > 'choices' from 'configs|menuconfigs' is interesting or not. I think it is better to drop '#choice'. -- Best Regards Masahiro Yamada -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html