Re: CONFIG_X86_ES7000=y, CONFIG_X86_GENERICARCH=n, CONFIG_ACPI=y build broken

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


 > Can you please try the attached patch? I have compiled it with
 > 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
 > misery. Please suggest if there is a better way of handling things.

 > +/*
 > + * This file also gets compiled if CONFIG_X86_GENERICARCH is set.
 > + * arch already has got following function definitions
 > + * hence no need to define these for that case.
 > + */

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?

 > ---
s7000-build-breakage-fix        2007-06-13 22:52:14.000000000 +0530
 > +++
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);
 > +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.
I just finished my test builds with and without GENERIC, so I can
suggest just move all functions to arch/i386/mach-es7000/es7000plat.c
- es7000_sw_apic() ,es7000_enable_apic_mode(),acpi_madt_oem_check(),
mps_oem_check() and make them non-static - from
arch/i386/mach-generic/es7000.c and I think I externed couple things
in asm/mach-es7000/mach_apic.h. So the file
arch/i386/mach-generic/es7000.c looks very simple:
/*  * APIC driver for the Unisys ES7000 chipset.
#include <linux/threads.h>
#include <linux/cpumask.h>
#include <asm/smp.h>
#include <asm/mpspec.h>
#include <asm/genapic.h>
#include <asm/fixmap.h>
#include <asm/apicdef.h>
#include <linux/kernel.h>
#include <linux/string.h>
#include <linux/smp.h>
#include <linux/init.h>
#include <asm/mach-es7000/mach_apicdef.h>
#include <asm/mach-es7000/mach_apic.h>
#include <asm/mach-es7000/mach_ipi.h>
#include <asm/mach-es7000/mach_mpparse.h>
#include <asm/mach-es7000/mach_wakecpu.h>

static int probe_es7000(void)
       /* probed later in mptable/ACPI hooks */
       return 0;

struct genapic apic_es7000 = APIC_INIT("es7000", probe_es7000);

and no ifdefs. Sorry for this skimpy description without actual patch...
I can send a patch later on, or you guys can try it yourselves :)
sorry have such a tight time this afternoon, only can back to my
computer after 7-8 pm...
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at
Please read the FAQ at

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux