Re: [Qemu-ppc] [PATCH v2] hw/ide: Emulate SiI3112 SATA controller

[BALATON Zoltan]

On Mon, 18 Dec 2017, John Snow wrote:
> On 12/18/2017 07:30 AM, BALATON Zoltan wrote:
> > On Mon, 18 Dec 2017, David Gibson wrote:
> > > On Sat, Dec 16, 2017 at 11:42:39PM +0100, BALATON Zoltan wrote:
> > > > This is a common generic PCI SATA controller that is also used in PCs
> > > > but more importantly guests running on the Sam460ex board prefer this
> > > > card and have a driver for it (unlike for other SATA controllers
> > > > already emulated).
> > > >
> > > > Signed-off-by: BALATON Zoltan <address@hidden>
> > >
> > > I don't know enough about IDE to give this any more than a quick check
> > > for anything obviously bogus looking.? So,
> > >
> > > Reviewed-by: David Gibson <address@hidden>
> > >
> > > John, can you take this through your tree, or should I take it through
> > > the ppc tree?
> >
> > I think he said before it can go via ppc tree:
> >
> > http://lists.nongnu.org/archive/html/qemu-devel/2017-08/msg04480.html
>
> Yep, I tend to use Acked-by to indicate that it's okay for you to stage.
> I'll give it another look-over in a minute, but I'm fine with adding the
> model in general, even if it's not quite complete just yet.

Maybe it's not complete yet but it should work to some degree. I've tested 
it with Linux on i386-softmmu and on PPC Sam460ex with u-boot and some 
guests (both on x86_64 host) and these could boot from an ide-cd model 
connected to this controller so at least basic PIO and UDMA modes should 
work. Maybe some more advanced features are missing but don't know if 
there are any drivers using those.

On i386 the bios does not have a driver so one cannot boot from this 
controller only, so I've tested it with -kernel and -initrd options but 
root filesystem was then read from CD connected to this controller and it 
could read it and start. I haven't tested it more than that though. Maybe 
the card's original option ROM that would allow booting from BIOS might 
not work with this device model but I think we could not distribute that 
anyway for legal reasons so I did not try that. On the Sam460ex which is 
my target for this the u-boot firmware has a driver and can boot from this 
controller as well as clients can use it.

As of now this patch only adds it to ppcemb and will only be created by 
default on the sam460ex machine so not very likely someone would try to 
use it on something else without recompiling QEMU.

Regards,
BALATON Zoltan