Messages in this thread |  | | From | Kees Cook <> | Date | Thu, 22 Feb 2018 11:57:18 -0800 | Subject | Re: [PATCH v3] kconfig.h: Include compiler types to avoid missed struct attributes |
| |
On Thu, Feb 22, 2018 at 10:04 AM, Linus Torvalds <torvalds@linux-foundation.org> wrote: > On Thu, Feb 22, 2018 at 9:41 AM, Kees Cook <keescook@chromium.org> wrote: >> >> Updated to include Tested-by. Linus, this looks ready to go. > > Ok, applied. > > I'm a bit worried that this ends up bypassing our automatic dependency > generation. > > Lookie here (in a fully built tree): > > find . -name '*.o.cmd' | > xargs grep -L linux/compiler_types.h | > xargs grep -l linux/kconfig.h | > while read i; do > j=$(echo $i | sed 's/\.o.cmd$/\.c/' | sed 's:/\.:/:'); > test -f $j && echo $j; > done > > shows that a number of files don't end up depending on that header > file, even though it's included (that "grep -l linux/kconfig,h" > triggers on the command itself having that "-include linux/kconfig.h" > line). > > It looks like "gcc -M" just doesn't list any files that get included > on the command line with "-include".
Hmm. But does that mean deps for kconfig.h are broken too? That seems silly. I'll take a look...
-Kees
-- Kees Cook Pixel Security
|  |