Re: [PATCH 45/86] pata_legacy: do not probe extra portsautomatically if PCI is not present

From: Alan Cox
Date: Wed Nov 25 2009 - 13:10:16 EST


> > NAK. You made a rather peculiar set of changes in the IDE layer but the
>
> Please give commit numbers and describe such 'peculiar set of changes in
> the IDE layer' in more details or skip such baseless, unjust and pointless

Disabling what can be a safe probe

> comments from your mails.

The old IDE code didn't do the full PCI check that libata did at the time
there were public complaints. Libata legacy also doesn't fall for the bug
where legacy loads and blocks PCI adapters as it has a smart check in it,
including for weirdness like the CS55x0 and the MPIIX.

So I'm firmly of the opinion that
- We should leave it as is
- If it causes a problem we should look at the exact case involved.
Possibly it would need a CONFIG_X86 but thats sort of implied as
embedded use pata_platform in preference.

The fact it just works by default is very very important.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/