From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Paul Menzel <pmenzel+linux-input@molgen.mpg.de>
Cc: Michael Tokarev <mjt@tls.msk.ru>,
linux-input@vger.kernel.org, linux-kernel@vger.kernel.org,
Ben Caradoc-Davies <ben@transient.nz>,
it+linux-input@molgen.mpg.de
Subject: Re: Commit *driver core: emit uevents when device is bound to a driver* causes regression
Date: Fri, 6 Apr 2018 15:27:20 +0200 [thread overview]
Message-ID: <20180406132720.GA16189@kroah.com> (raw)
In-Reply-To: <22ee8cb0-5793-6ad6-d8e0-3e83a8c1361c@molgen.mpg.de>
On Fri, Apr 06, 2018 at 03:24:39PM +0200, Paul Menzel wrote:
> Dear Greg,
>
>
> On 04/06/18 15:18, Greg Kroah-Hartman wrote:
> > On Fri, Apr 06, 2018 at 02:20:40PM +0200, Paul Menzel wrote:
>
> > > Commit 1455cf8 (driver core: emit uevents when device is bound to a
> > > driver) [1], introduced in Linux 4.14-rc1, causes a regression in user
> > > space.
> > >
> > > After disconnecting USB devices, they are still shown as plugged in [2][3].
> > >
> > > > I seem to be having a similar issue, but with an iPhone X using USB
> > > > to provide internet access. Even after the phone is disconnected, it
> > > > still shows up as "loaded active plugged". The only way to fix this
> > > > is to reboot or use a different USB port.
> > >
> > > `git bisect` shows that commit 1455cf8 (driver core: emit uevents when
> > > device is bound to a driver) is at fault [4].
> > >
> > > As user space should not break with newer Linux kernels, do you know what to
> > > do about that or can you think of a fix?
> >
> > I thought we fixed this with follow-on patches, are you sure you are
> > using the latest 4.14.y kernel that has been released?
>
> Here we are using Linux 4.14.30. We can try newer versions next week.
Ah, no, 4.14.30 should be fine.
I saw the patch listed in the debian bug report, so this only happens
for the MTP type devices? That was a really odd "filter" that it was
trying to use before.
So this is only for userspace programs, and one specifically? And for
only one type of device? What device, MTP devices?
thanks,
greg k-h
next prev parent reply other threads:[~2018-04-06 13:27 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-06 12:20 Commit *driver core: emit uevents when device is bound to a driver* causes regression Paul Menzel
2018-04-06 12:47 ` Ben Caradoc-Davies
2018-04-06 13:18 ` Greg Kroah-Hartman
2018-04-06 13:24 ` Paul Menzel
2018-04-06 13:27 ` Greg Kroah-Hartman [this message]
2018-04-06 18:26 ` Dmitry Torokhov
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=20180406132720.GA16189@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=ben@transient.nz \
--cc=it+linux-input@molgen.mpg.de \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mjt@tls.msk.ru \
--cc=pmenzel+linux-input@molgen.mpg.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.