All of lore.kernel.org
 help / color / mirror / Atom feed
From: Noboru Iwamatsu <n_iwamatsu@jp.fujitsu.com>
To: xen-devel@lists.xensource.com
Subject: Re: anyone using pvusb?
Date: Wed, 09 Sep 2009 11:28:16 +0900	[thread overview]
Message-ID: <4AA712C0.2080807@jp.fujitsu.com> (raw)
In-Reply-To: <AEC6C66638C05B468B556EA548C1A77D0177D0FF@trantor>

Hi James,

> I am wondering if maybe it is just a quirk of that model or something
> that the first usb packet that gets sent times out or something and
> needs to be unlinked. When I look at what happens under Linux, the
> frontend tries to unlink it but fails. Is it normally the hcd or the
> device driver that is responsible for maintaining timeouts and retrying
> etc?

HCD's urb_dequeue() function that is called from hcd_unlink_urb()
has to be responsible for dequeueing the urb from the actual HCD
hardware and giving back the urb within the next time of the HCD
hardware interrupt.

If no urb returns for a given length of time after usbcore or
device drivers unlinked the urb, they might identify the HCD as broken.

About the PVUSB's implementation of urb_dequeue(), it's incomplete.

usbfront can only dequeue the urb that is in the frontend, and cannot
dequeue the urb already transferred to the backend.

So, if the device goes into stalled state in the backend (probably
triggered from wrong urb), The frontend can do nothing.

I think main cause of the flash drive failure is the above wrong urb,
but I will solve the urb-canceling issue too.

Thanks,
Noboru.

  reply	other threads:[~2009-09-09  2:28 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-03 10:42 anyone using pvusb? James Harper
2009-09-03 12:09 ` Keir Fraser
2009-09-04  0:49   ` Noboru Iwamatsu
2009-09-04  1:53     ` James Harper
2009-09-04  3:09       ` Noboru Iwamatsu
2009-09-04  5:49         ` James Harper
2009-09-04  2:40     ` James Harper
2009-09-04  6:06       ` Noboru Iwamatsu
2009-09-04  6:21         ` James Harper
2009-09-04  6:36           ` Noboru Iwamatsu
2009-09-04 10:43             ` Noboru Iwamatsu
2009-09-04 10:49               ` James Harper
2009-09-04 12:23                 ` Noboru Iwamatsu
2009-09-07 11:38                   ` James Harper
2009-09-09  2:28                     ` Noboru Iwamatsu [this message]
2009-09-10 16:43                       ` Ian Jackson
2009-09-04 16:49 ` Neobiker
2009-09-05  1:22   ` James Harper

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=4AA712C0.2080807@jp.fujitsu.com \
    --to=n_iwamatsu@jp.fujitsu.com \
    --cc=xen-devel@lists.xensource.com \
    /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.