qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* Re: [Qemu-devel] uhci: cancel delay for unregistered queues
       [not found]       ` <511CD72D.80502@redhat.com>
@ 2013-04-25 10:20         ` Hans de Goede
  0 siblings, 0 replies; only message in thread
From: Hans de Goede @ 2013-04-25 10:20 UTC (permalink / raw)
  To: Gerd Hoffmann; +Cc: Jan Kiszka, qemu-devel

Hi,

On 02/14/2013 01:23 PM, Gerd Hoffmann wrote:
>    Hi,
>
>>> The only fix I can think of is allocating a receive buffer inside
>>> the UHCI code for read transfers, and copy the result over to the
>>> guest memory when we are re-scanning the schedule, encounter the
>>> completed td and are going to signal its completion to the guest.
>
>> I could take a shot at writing a patch with the proposed fix, but I
>> was sort of waiting on a reaction on the proposal first.
>>
>> Gerd what do you think of my proposed fix ?
>
> That should fix it indeed.  Yes, for uhci we can live with the overhead,
> it's low bandwidth anyway.  No, I don't have a better idea.  Patch is
> very welcome.

Ok, so a bit late I must admit, but I've finally gotten around to writing
the promised fix for this.

I'm going to send it right after this mail.

Jan, can you give it a spin and see if it fixes your issue ?

Regards,

Hans

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2013-04-25 10:17 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <511A53D9.2020704@siemens.com>
     [not found] ` <511A71E3.7050006@redhat.com>
     [not found]   ` <511ABBC4.5030208@redhat.com>
     [not found]     ` <511CD09C.6020400@redhat.com>
     [not found]       ` <511CD72D.80502@redhat.com>
2013-04-25 10:20         ` [Qemu-devel] uhci: cancel delay for unregistered queues Hans de Goede

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).