kbuild - pending stuff, and Maintainer news - Kernel

This is a discussion on kbuild - pending stuff, and Maintainer news - Kernel ; On Tue, Jul 01, 2008 at 08:41:22AM +0200, Ingo Molnar wrote: > > * Sam Ravnborg wrote: > > > We could hit a situation where hex values > > are stored internally in kconfig without 0x prefix, so if ...

+ Reply to Thread
Page 2 of 2 FirstFirst 1 2
Results 21 to 23 of 23

Thread: kbuild - pending stuff, and Maintainer news

  1. Re: [PATCH 21/24] kconfig: prefix hex values with 0x in .config

    On Tue, Jul 01, 2008 at 08:41:22AM +0200, Ingo Molnar wrote:
    >
    > * Sam Ravnborg wrote:
    >
    > > We could hit a situation where hex values
    > > are stored internally in kconfig without 0x prefix, so if we see
    > > one of these add 0x prefix when we write value to .config.
    > > The real fix is to make sure we always prefix hex
    > > values with 0x in kconfig.
    > >
    > > Signed-off-by: Sam Ravnborg
    > > Cc: Roman Zippel
    > > ---
    > > arch/x86/boot/compressed/relocs.c | 2 +-
    > > scripts/kconfig/confdata.c | 2 +-
    > > 2 files changed, 2 insertions(+), 2 deletions(-)
    > >
    > > diff --git a/arch/x86/boot/compressed/relocs.c b/arch/x86/boot/compressed/relocs.c
    > > index edaadea..88d27f6 100644
    > > --- a/arch/x86/boot/compressed/relocs.c
    > > +++ b/arch/x86/boot/compressed/relocs.c
    > > @@ -244,7 +244,7 @@ static void read_ehdr(FILE *fp)
    > > static void read_shdrs(FILE *fp)
    > > {
    > > int i;
    > > - if (ehdr.e_shnum > MAX_SHDRS) {
    > > + if (1 || ehdr.e_shnum > MAX_SHDRS) {
    > > die("%d section headers supported: %d\n",
    > > ehdr.e_shnum, MAX_SHDRS);
    > > }

    >
    > note: stale x86 change slipped into this patch, please do not commit it
    > to the kbuild tree. (i suspect it happened when you sent me the
    > MAX_SHDRS debug patch yesterday)


    Crap - thanks for noticing.
    I will rebase tonight when I get home.

    Sam
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  2. Re: [PATCH 08/24] kbuild: make clean removes *.o.* as well


    On Monday 2008-06-30 23:35, Sam Ravnborg wrote:

    >From: Alexey Dobriyan
    >
    >Those are left presumably from aborted ccache(1) compilations:
    >
    > arch/x86/kernel/.tmp_io_apic_64.o.T5veul
    > arch/x86/kvm/.tmp_x86.o.SZWn69
    > arch/x86/mm/.tmp_pgtable.o.sL1LTf
    > drivers/ieee1394/.tmp_ieee1394_transactions.o.bUj6o1
    > drivers/infiniband/hw/mlx4/.tmp_main.o.vy0ep6


    I'd suggest using '.tmp_*.o.*' instead of '*.o.*'.
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  3. Re: [PATCH 08/24] kbuild: make clean removes *.o.* as well

    On Tue, Jul 01, 2008 at 02:10:26PM +0200, Jan Engelhardt wrote:
    >
    > On Monday 2008-06-30 23:35, Sam Ravnborg wrote:
    >
    > >From: Alexey Dobriyan
    > >
    > >Those are left presumably from aborted ccache(1) compilations:
    > >
    > > arch/x86/kernel/.tmp_io_apic_64.o.T5veul
    > > arch/x86/kvm/.tmp_x86.o.SZWn69
    > > arch/x86/mm/.tmp_pgtable.o.sL1LTf
    > > drivers/ieee1394/.tmp_ieee1394_transactions.o.bUj6o1
    > > drivers/infiniband/hw/mlx4/.tmp_main.o.vy0ep6

    >
    > I'd suggest using '.tmp_*.o.*' instead of '*.o.*'.


    Agreed. Fixed and pushed out a new set of patches.

    Sam
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

+ Reply to Thread
Page 2 of 2 FirstFirst 1 2