public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Stephen Warren <swarren@wwwdotorg.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory
Date: Wed, 12 Nov 2014 11:11:33 -0700	[thread overview]
Message-ID: <5463A2D5.1030906@wwwdotorg.org> (raw)
In-Reply-To: <20141112164538.17607380133@gemini.denx.de>

On 11/12/2014 09:45 AM, Wolfgang Denk wrote:
> Dear Lukasz,
>
> In message <20141112152949.1fe6ce3c@amdc2363> you wrote:
>>
>>> But this is a bug.  Papering over is not a good idea.  It should be
>>> analyzed, reported, and finally fixed.
>>
>> I've debugged the script with strace.
>
> Thanks a lot for that!
>
>> The problem is with umount() syscall:
> ...
>> I need to debug the syscall in the kernel.
>
> I have a gut feeling that this might be a problem somewhere with the
> USB stuff that is involved here.  Maybe enabling some degug messages
> for the USB subsystem gives some more information?

If this really is related to USB only, it may be worth asking on the 
linux-usb mailing list to see if anyone already unerstands the issue.

Irrespective of all this though, even if there is a kernel issue and it 
can be fixed, I still think we should apply this patch, because the 
issue currently affects people. At the very least it will until some 
kernel fix is rolled out to everyone's distro, which will be a while no 
matter what.

  reply	other threads:[~2014-11-12 18:11 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-06 10:23 [U-Boot] [PATCH] test: ums: Add sleep before unmount directory Lukasz Majewski
2014-11-06 17:00 ` Stephen Warren
2014-11-06 21:30   ` Lukasz Majewski
2014-11-07 13:05 ` [U-Boot] [PATCH v2] " Lukasz Majewski
2014-11-07 15:28   ` Marek Vasut
2014-11-07 16:52     ` Lukasz Majewski
2014-11-07 21:02       ` Marek Vasut
2014-11-07 15:33   ` Stephen Warren
2014-11-07 21:54   ` Wolfgang Denk
2014-11-07 22:00     ` Stephen Warren
2014-11-07 23:03       ` Wolfgang Denk
2014-11-12 14:29         ` Lukasz Majewski
2014-11-12 16:45           ` Wolfgang Denk
2014-11-12 18:11             ` Stephen Warren [this message]
2014-11-12 21:49               ` Lukasz Majewski
2014-11-14 10:43                 ` Lukasz Majewski

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=5463A2D5.1030906@wwwdotorg.org \
    --to=swarren@wwwdotorg.org \
    --cc=u-boot@lists.denx.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox