From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:39637) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TfUkN-0005P5-OP for qemu-devel@nongnu.org; Mon, 03 Dec 2012 07:03:45 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TfUkE-0002Li-0i for qemu-devel@nongnu.org; Mon, 03 Dec 2012 07:03:39 -0500 Received: from mail-bk0-f45.google.com ([209.85.214.45]:41389) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TfUkD-0002Le-OA for qemu-devel@nongnu.org; Mon, 03 Dec 2012 07:03:29 -0500 Received: by mail-bk0-f45.google.com with SMTP id jk13so925603bkc.4 for ; Mon, 03 Dec 2012 04:03:28 -0800 (PST) Date: Mon, 3 Dec 2012 20:03:10 +0800 From: walimis Message-ID: <20121203120309.GD6029@pek-lwang0-d1> References: <1354153398-13569-1-git-send-email-peter.crosthwaite@xilinx.com> <20121129020041.GB3127@walimis-DELL> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH 1.3] ehci-sysbus: Attach DMA context. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Crosthwaite Cc: david.holsgrove@xilinx.com, john.williams@xilinx.com, qemu-devel@nongnu.org, kraxel@redhat.com On Thu, Nov 29, 2012 at 12:05:14PM +1000, Peter Crosthwaite wrote: >On Thu, Nov 29, 2012 at 12:00 PM, walimis wrote: >> On Thu, Nov 29, 2012 at 11:43:18AM +1000, Peter Crosthwaite wrote: >>>This was left as NULL on the initial merge due to debate on the mailing list on >>>how to handle DMA contexts for sysbus devices. Patch >>>9e11908f12f92e31ea94dc2a4c962c836cba9f2a was later merged to fix OHCI. This is the, >>>equivalent fix for sysbus EHCI. >> >> I have also found this issue, but it's not the cause that xilinx >> ehci can't work with usb-storage disk. Do you have any update >> for xilinx ehci? >> > >Hi Liming, > >I haven't got around to looking into that one yet unfortunately. No >updates just yet - ill let you know if it resolves. It could very well >be a Linux bug as well so it needs to be investigated from both sides >of the fence. As said in another mail, I found that the root cause is that xilinx_zynq has two EHCI controller. If we use usb-storage disk, the disk will be attached to the second EHCI controller, which the kernel uses the first EHCI controller by default. For now, qemu doesn't support two EHCI controller, could we remove the second EHCI from xilinx_zynq? Liming Wang > >>> >>>Signed-off-by: Peter Crosthwaite >> >> Tested-by: Liming Wang >> > >Thanks. > >Regards, >Peter > >> Liming Wang >> >>>--- >>> hw/usb/hcd-ehci-sysbus.c | 1 + >>> 1 files changed, 1 insertions(+), 0 deletions(-) >>> >>>diff --git a/hw/usb/hcd-ehci-sysbus.c b/hw/usb/hcd-ehci-sysbus.c >>>index 1584079..803df92 100644 >>>--- a/hw/usb/hcd-ehci-sysbus.c >>>+++ b/hw/usb/hcd-ehci-sysbus.c >>>@@ -45,6 +45,7 @@ static int usb_ehci_sysbus_initfn(SysBusDevice *dev) >>> >>> s->capsbase = 0x100; >>> s->opregbase = 0x140; >>>+ s->dma = &dma_context_memory; >>> >>> usb_ehci_initfn(s, DEVICE(dev)); >>> sysbus_init_irq(dev, &s->irq); >>>-- >>>1.7.0.4 >>> >>> >>