On Fri, Mar 30, 2012 at 1:34 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Fri, 2012-03-30 at 13:30 -0700, Luis R. Rodriguez wrote: >> On Fri, Mar 30, 2012 at 4:11 AM, Johannes Berg >> <johannes@xxxxxxxxxxxxxxxx> wrote: >> > From: Johannes Berg <johannes.berg@xxxxxxxxx> >> > >> > Doing the blanket "export" statement at the beginning >> > of the generated file confuses the kernel's Makefiles >> > and causes extreme build slowdowns. >> > >> > Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx> >> > --- >> >> Nice, how did you figure this out BTW!? > > Random monkey approach ;-) > > I noticed it was slow and saw that it was due to some weird shit from > the kernel Makefile.lib being executed when it shouldn't be. > > Since I have the compat *result* in git, I could easily see the diff > between the old compat.git/compat-wireless.git and the new versions. And > then I basically tried to identify the changes that caused this problem, > my best guess was that it had to be the Makefile so I gradually changed > that back to the previous version until the problem went away ... and > then identified which changes were responsible. Wow, nice find.. so we should be able to make compat-wireless builds just as fast too there. There config.mk does the same wide export hackery. Looks like the 3.4 stable releases will have huge improvements, now to just address 2.6.31 and below for compat-wireless: Trying kernel 2.6.31-02063113-generic [FAILED] Trying kernel 2.6.30-02063010-generic [FAILED] Trying kernel 2.6.29-02062906-generic [FAILED] Trying kernel 2.6.28-02062810-generic [FAILED] Trying kernel 2.6.27-020627-generic [FAILED] Trying kernel 2.6.26-020626-generic [FAILED] Trying kernel 2.6.25-020625-generic [FAILED] Trying kernel 2.6.24-020624-generic [FAILED] PS. Please Cc: lf_driver_backport@xxxxxxxxxxxxxxxxxxxxxxxxxx on future patches. Luis -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
