[lkml]   [2008]   [Jul]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[RFC] (almost) booting allyesconfig -- please don't poke super-io without request_region
    After the following patch to mark the isa region busy and applying a few
    patches[1], I was able to kexec boot into an all-yes-config kernel from
    linux-next, with the following KCONFIG_ALLCONFIG file:

    # CONFIG_CRASH_DUMP is not set
    # CONFIG_NSC_FIR is not set
    # CONFIG_SMC_IRCC_FIR is not set
    # CONFIG_ALI_FIR is not set
    # CONFIG_TCG_NSC is not set
    # CONFIG_TCG_ATMEL is not set
    # CONFIG_SENSORS_F71805F is not set
    # CONFIG_SENSORS_F71882FG is not set
    # CONFIG_SENSORS_F75375S is not set
    # CONFIG_SENSORS_IT87 is not set
    # CONFIG_SENSORS_PC87360 is not set
    # CONFIG_SENSORS_PC87427 is not set
    # CONFIG_SENSORS_DME1737 is not set
    # CONFIG_SENSORS_SMSC47M1 is not set
    # CONFIG_SENSORS_SMSC47B397 is not set
    # CONFIG_SENSORS_VT1211 is not set
    # CONFIG_SENSORS_W83627HF is not set
    # CONFIG_SENSORS_W83627EHF is not set

    While the first two might not be required, and the third is just
    selecting the right platform, it would be nice to get the drivers
    to play as well as the rest of the kernel.

    The drivers all are for super-io chips, either irda/FIR drivers or hwmon,
    and poke at isa ports without checking request_region.

    It would be great if these 17 super-io drivers would check for resource
    availability before poking at io ports.

    [not that I expect this to merge as it, but maybe insert-resource or something]

    [1] 3 section layout patches and the MTD DISKONCHIP patch,
    see linuxppc-dev for full list

    diff --git a/arch/powerpc/platforms/pseries/pci.c b/arch/powerpc/platforms/pseries/pci.c
    index 2c6ded2..db54620 100644
    --- a/arch/powerpc/platforms/pseries/pci.c
    +++ b/arch/powerpc/platforms/pseries/pci.c
    @@ -58,8 +58,10 @@ DECLARE_PCI_FIXUP_HEADER(PCI_ANY_ID, PCI_ANY_ID, pcibios_name_device);

    static void __init pSeries_request_regions(void)
    - if (!isa_io_base)
    + if (!isa_io_base) {
    + request_region(0x0,0x400,"isa");
    + }


     \ /
      Last update: 2008-07-10 23:17    [W:0.023 / U:56.992 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site