[Top][All Lists]

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

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

From: BALATON Zoltan
Subject: Re: [Qemu-ppc] [PATCH v2] hw/ide: Emulate SiI3112 SATA controller
Date: Tue, 19 Dec 2017 01:34:00 +0100 (CET)
User-agent: Alpine 2.21 (BSF 202 2017-01-01)

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:


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.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]