All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lukasz Majewski <l.majewski@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory
Date: Fri, 14 Nov 2014 11:43:46 +0100	[thread overview]
Message-ID: <20141114114346.440b7673@amdc2363> (raw)
In-Reply-To: <20141112224954.4ec02949@jawa>

Hi Lukasz,

> On Wed, 12 Nov 2014 11:11:33 -0700
> Stephen Warren <swarren@wwwdotorg.org> wrote:
> 
> > 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.
> 
> Frankly speaking, I'm personally curious why such things happen. I may
> look into the issue in my "spare :-)" time.
> 
> In spite of this, I agree with Stephen:
> 
> > 
> > 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.
> 
> +1
> 
> The problem seems to be with long time running linux. The error seems
> to be orthogonal to our u-boot's UMS work.
> 
> > _______________________________________________
> > U-Boot mailing list
> > U-Boot at lists.denx.de
> > http://lists.denx.de/mailman/listinfo/u-boot
> 
> Best regards,
> Lukasz Majewski

Applied to u-boot-dfu tree. Thanks.

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

      reply	other threads:[~2014-11-14 10:43 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
2014-11-12 21:49               ` Lukasz Majewski
2014-11-14 10:43                 ` Lukasz Majewski [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=20141114114346.440b7673@amdc2363 \
    --to=l.majewski@samsung.com \
    --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 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.