Hello All! On 2013-09-22 01:21 +0200, Yann E. MORIN spake thusly: > Stephen, All, > > Please, test this patch to fix the allmodconfig issue reported by > Stephen. > > From my little testing (some randconfig followed by silentoldconfig, as > well as allmodconfig), this patch seems to fix the issue without any > regression I could ientify. > > But since this is sensitive code, I'd like some feedback before this > gets applied. Ping? :-) I wanted to push this right after -rc3 is out. If any one has an issue with that patch, you've got a few days to chime in! ;-) Regards, Yann E. MORIN. > On 2013-09-22 01:17 +0200, Yann E. MORIN spake thusly: > > From: "Yann E. MORIN" <yann.morin.1998@xxxxxxx> > > > > Reported-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> > > Signed-off-by: "Yann E. MORIN" <yann.morin.1998@xxxxxxx> > > Cc: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> > > Cc: Sam Ravnborg <sam@xxxxxxxxxxxx> > > Cc: Michal Marek <mmarek@xxxxxxx> > > --- > > scripts/kconfig/confdata.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/scripts/kconfig/confdata.c b/scripts/kconfig/confdata.c > > index 87f7238..0cbfc67 100644 > > --- a/scripts/kconfig/confdata.c > > +++ b/scripts/kconfig/confdata.c > > @@ -399,8 +399,6 @@ setsym: > > free(line); > > fclose(in); > > > > - if (modules_sym) > > - sym_calc_value(modules_sym); > > return 0; > > } > > > > @@ -1175,6 +1173,10 @@ bool conf_set_all_new_symbols(enum conf_def_mode mode) > > sym->def[S_DEF_USER].tri = yes; > > break; > > case def_mod: > > + /* Note: although modules_sym must be 'yes' to > > + * enable tristates, 'mod' is promoted to 'yes' > > + * when applied to a boolean, which modules_sym > > + * is. So we need not special-case it here. */ > > sym->def[S_DEF_USER].tri = mod; > > break; > > case def_no: -- .-----------------.--------------------.------------------.--------------------. | Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: | | +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ | | +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no | | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. | '------------------------------^-------^------------------^--------------------' -- 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
