Re: [PATCH] iio: hi8435: fix race between event enable and event generation

From	Nikita Yushchenko <nikita.yoush@cogentembedded.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] iio: hi8435: fix race between event enable and event generation
Date	2017-05-24 22:00 +0200
Message-ID	<tKKBQ-6yB-5@gated-at.bofh.it> (permalink)
References	<tKd3c-sK-5@gated-at.bofh.it> <tKKiu-6rO-17@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

>> Add locking to avoid interference between reading/processing current
>> sence in event enable and event generation paths.
>>
>> Signed-off-by: Nikita Yushchenko <nikita.yoush@cogentembedded.com>
> Make sense.  Could you give a little more detail on the seriousness
> of this bug?  I need to assess whether we just queue it up for the
> next merge window or push it forward quicker as a bug fix and
> whether to mark it for stable or not.
> 
> You probably have a better idea of how much it matters than I do!

Race was introduced by my previous patch, and noticed by Vladimir.
It is not in any released tree.

Possible race impact is loss of event or duplicate event.

Nikita

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

Thread

[PATCH] iio: hi8435: fix race between event enable and event generation Nikita Yushchenko <nikita.yoush@cogentembedded.com> - 2017-05-23 10:10 +0200
  Re: [PATCH] iio: hi8435: fix race between event enable and event  generation Jonathan Cameron <jic23@kernel.org> - 2017-05-24 21:40 +0200
    Re: [PATCH] iio: hi8435: fix race between event enable and event  generation Nikita Yushchenko <nikita.yoush@cogentembedded.com> - 2017-05-24 22:00 +0200

csiph-web