Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors

From	Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
Newsgroups	linux.kernel
Subject	Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors
Date	2017-08-12 18:10 +0200
Message-ID	<udH98-327-21@gated-at.bofh.it> (permalink)
References	<ud53P-4ev-1@gated-at.bofh.it> <udiNr-4xf-11@gated-at.bofh.it> <udDRU-Ji-9@gated-at.bofh.it> <udEbf-Rc-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Sat, 2017-08-12 at 14:58 +0200, Bastien Nocera wrote:
> On Sat, 2017-08-12 at 13:16 +0100, Jonathan Cameron wrote:
> > 
> > On Fri, 11 Aug 2017 16:04:30 +0200
> > Bastien Nocera <hadess@hadess.net> wrote:
> > 
> > > 
> > > Woot!
> > > 
[...]

> > > > 
> > > > Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.i
> > > > nt
> > > > el.com>  
> > > Tested-by: Bastien Nocera <hadess@hadess.net>
> > > 
> > > I'm still chasing a couple of bugs in the user-space side of
> > > things
> > > caused by the removal of the timeout.
> > > 
> > > Thanks!
> > Is it worth me sitting on this for a week or so to see if it deals
> > with
> > all the reported issues around this?
> > 
> > Or are you happy that the test set you have is sufficient to verify
> > it?
> I'd rather have it merged ASAP. The fact that it didn't need to sleep
> for 3 seconds allowed me to find a couple of problems in GNOME's use
> of
> this feature, with the sensor showing up before the desktop has
> started. Those will likely be taken care of next week as well.
> 
> I've tested it on a couple of machines, and it's working as expected.
> 
> > 
> > Definitely good to put this one to bed finally! 
> And no one's happier than me in this one. I can concentrate on bugs I
> wrote myself now ;)
I will resubmit patch by adding Testedby and removing [RFT].

Thanks,
Srinivas

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user space powering up sensors Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> - 2017-08-11 01:30 +0200
  Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user  space powering up sensors Bastien Nocera <hadess@hadess.net> - 2017-08-11 16:10 +0200
    Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user  space powering up sensors Jonathan Cameron <jic23@kernel.org> - 2017-08-12 14:40 +0200
      Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user  space powering up sensors Bastien Nocera <hadess@hadess.net> - 2017-08-12 15:00 +0200
        Re: [RFT][PATCH] iio: hid-sensor-trigger: Fix the race with user  space powering up sensors Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> - 2017-08-12 18:10 +0200

csiph-web