linux-kernel - Re: CONFIG_X86_ES7000=y, CONFIG_X86_GENERICARCH=n, CONFIG

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <adamyz34mim.fsf@cisco.com>
Date:	Wed, 13 Jun 2007 13:39:29 -0700
From:	Roland Dreier <rdreier@...co.com>
To:	vgoyal@...ibm.com
Cc:	linux-kernel@...r.kernel.org,
	"Natalie.Protasevich@...sys.com" <Natalie.Protasevich@...sys.com>,
	akpm@...ux-foundation.org
Subject: Re: CONFIG_X86_ES7000=y, CONFIG_X86_GENERICARCH=n, CONFIG_ACPI=y build broken

 > Can you please try the attached patch? I have compiled it with CONFIG_ES7000=y
 > and CONFIG_X86_GENERICARCH=n. But don't have any ES7000 machine to test it.

I actually don't have anything remotely like ES7000 iether.  I just
found the problem while tracking down another randconfig failure.

 > Subarch build procedure is not straightforward and this patch adds to the
 > misery. Please suggest if there is a better way of handling things.

 > +/*
 > + * This file also gets compiled if CONFIG_X86_GENERICARCH is set. Generic
 > + * arch already has got following function definitions (asm-generic/es7000.c)
 > + * hence no need to define these for that case.
 > + */
 > +#ifndef CONFIG_X86_GENERICARCH

Seems it would be cleaner to figure out some way to build es7000.c for
if CONFIG_X86_ES7000 is set?  Or just move them here all the time?

 > --- linux-2.6.22-rc4-git4/include/asm-i386/mach-es7000/mach_mpparse.h~i386-es7000-build-breakage-fix	2007-06-13 22:52:14.000000000 +0530
 > +++ linux-2.6.22-rc4-git4-vivek/include/asm-i386/mach-es7000/mach_mpparse.h	2007-06-13 22:52:14.000000000 +0530
 > @@ -18,6 +18,12 @@ extern int parse_unisys_oem (char *oempt
 >  extern int find_unisys_acpi_oem_table(unsigned long *oem_addr);
 >  extern void setup_unisys(void);
 >  
 > +#ifndef CONFIG_X86_GENERICARCH
 > +extern int acpi_madt_oem_check(char *oem_id, char *oem_table_id);
 > +extern int mps_oem_check(struct mp_config_table *mpc, char *oem,
 > +				char *productid);
 > +#endif

It seems that this #ifndef is not needed -- even if there is another
declaration of these functions that is visible, the signatures should
match so it should be OK.

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