All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paul Bolle <pebolle@tiscali.nl>
To: Stan Behrens <lists.nongnu.org@sbeh.de>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [BUG] -usb makes qemu fight against linux
Date: Fri, 24 Apr 2009 12:44:17 +0200	[thread overview]
Message-ID: <1240569857.24225.19.camel@localhost.localdomain> (raw)
In-Reply-To: <1240567553.24225.11.camel@localhost.localdomain>

On Fri, 2009-04-24 at 12:05 +0200, Paul Bolle wrote:
> On Fri, 2009-03-06 at 09:54 +0100, Stan Behrens wrote:
> This seems related to this discussion:
> http://lists.gnu.org/archive/html/qemu-devel/2008-10/msg01320.html (in a
> thread regarding a preliminary patch to implement ehci).
> 
> Could you try the following patch? It was created after finally
> discovering the discussion mentioned above (which I hardly understand,
> which in turn leads to an impressive, but possibly nonsensical commit
> message for this patch). It seems to fix a similar issue I ran into
> (while trying some advanced feature of a USB memory stick I happen to
> have.)

I just found a slightly different and, I guess, better solution here:
http://lists.gnu.org/archive/html/qemu-devel/2008-10/msg01326.html (I
seem to have missed this messages in that archive as it is, for some
reason, a separate thread).

I simply copied it from that address into this mail. It will probably
not apply without manual adjustments.


Paul Bolle
---
Index: usb-linux.c
===================================================================
--- usb-linux.c (revision 5571)
+++ usb-linux.c (working copy)
@@ -68,7 +68,7 @@
 static int usb_host_find_device(int *pbus_num, int *paddr,
                                 char *product_name, int
product_name_size,
                                 const char *devname);
-//#define DEBUG
+// #define DEBUG
 
 #ifdef DEBUG
 #define dprintf printf
@@ -276,7 +276,9 @@
 
             case -EPIPE:
                 set_halt(s, p->devep);
-                /* fall through */
+               p->len = USB_RET_STALL;
+               break;
+
             default:
                 p->len = USB_RET_NAK;
                 break;

      reply	other threads:[~2009-04-24 10:44 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-06  8:54 [Qemu-devel] [BUG] -usb makes qemu fight against linux Stan Behrens
2009-04-24 10:05 ` Paul Bolle
2009-04-24 10:44   ` Paul Bolle [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1240569857.24225.19.camel@localhost.localdomain \
    --to=pebolle@tiscali.nl \
    --cc=lists.nongnu.org@sbeh.de \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.