Re: [PATCH 1/2] Add Legacy IDE mode support for SB600 SATA

[Conke Hu <conke.hu@amd.com>]

On Sun, 2006-11-05 at 12:45 +0000, Alan Cox wrote:
> Ar Sul, 2006-11-05 am 20:17 +0800, ysgrifennodd Conke Hu:
> >     1. The SATA configuration option "Legacy IDE mode" (as well as
> > Native IDE mode) in SB600 BIOS is ONLY for old OS, and it is not useful
> > any longer for new Linux kernels.
> 
> Some users choose to use old drivers for the feeling of security and
> reduction of change. Remember there are users out there who have to go
> through a formal verification process to switch the driver they use.
> 
  Consider that the behavior itself of using the new motherboard, SB600,
means a lot of change, so even if the user still use IDE driver, he also
a need verification process.

> >     3. "This should only be done if AHCI is configured into the kernel,
> > so wants a #ifdef check adding".
> >     Alan, this fix should always be done whether AHCI is configured into
> > kernel or not, even when AHCI is not configured at all. Because:
> >     a). Without it, the SB600 SATA controller will appear as an IDE,
> > which may misguide user to try to load legacy IDE driver (or other IDE
> 
> This is not neccessarily misguided. They may want to do this.
  But the SB600 SATA controller is really an AHCI controller. And that
will lose high performance (and cannot use NCQ).

> 
> >     b). We have a RAID driver (close source) for SB600 SATA 
> 
> Thats your problem. Some day the lawyers can find out just how legal
> that is.
  No, even if there would be a open source RAID driver, it still could
no run.

> 
> You are right that most users will want to use the AHCI layer and that
> if AHCI is compiled in then we should switch to AHCI. In the case the
> kernel has no AHCI support compiled in the legacy driver support should
> continue to work for it. This is how other vendors products such as
> Jmicron are handled.
> 
  The case of Jmicron is not the same as ours. They use different pci
functions in the SATA controller.

> NAK reasserted.
  OK, I am rewriting the patch based on the following considerations:
  1. move these code to ahci driver, and add new code to PATA driver.
  2. add new options to kernel configuration, and users can choose IDE
driver or AHCI driver to support SB600 SATA when it is in IDE mode.
  

> Alan
> 
At last, thank you again for your opinions and suggestion. if any other
advice, please feel free to contact me.


Best regards,
Conke Hu