lkml.org 
[lkml]   [2020]   [Dec]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 674/717] iio: buffer: Fix demux update
    Date
    From: Nuno Sá <nuno.sa@analog.com>

    commit 19ef7b70ca9487773c29b449adf0c70f540a0aab upstream.

    When updating the buffer demux, we will skip a scan element from the
    device in the case `in_ind != out_ind` and we enter the while loop.
    in_ind should only be refreshed with `find_next_bit()` in the end of the
    loop.

    Note, to cause problems we need a situation where we are skippig over
    an element (channel not enabled) that happens to not have the same size
    as the next element. Whilst this is a possible situation we haven't
    actually identified any cases in mainline where it happens as most drivers
    have consistent channel storage sizes with the exception of the timestamp
    which is the last element and hence never skipped over.

    Fixes: 5ada4ea9be16 ("staging:iio: add demux optionally to path from device to buffer")
    Signed-off-by: Nuno Sá <nuno.sa@analog.com>
    Link: https://lore.kernel.org/r/20201112144323.28887-1-nuno.sa@analog.com
    Cc: <Stable@vger.kernel.org>
    Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/iio/industrialio-buffer.c | 6 +++---
    1 file changed, 3 insertions(+), 3 deletions(-)

    --- a/drivers/iio/industrialio-buffer.c
    +++ b/drivers/iio/industrialio-buffer.c
    @@ -865,12 +865,12 @@ static int iio_buffer_update_demux(struc
    indio_dev->masklength,
    in_ind + 1);
    while (in_ind != out_ind) {
    - in_ind = find_next_bit(indio_dev->active_scan_mask,
    - indio_dev->masklength,
    - in_ind + 1);
    length = iio_storage_bytes_for_si(indio_dev, in_ind);
    /* Make sure we are aligned */
    in_loc = roundup(in_loc, length) + length;
    + in_ind = find_next_bit(indio_dev->active_scan_mask,
    + indio_dev->masklength,
    + in_ind + 1);
    }
    length = iio_storage_bytes_for_si(indio_dev, in_ind);
    out_loc = roundup(out_loc, length);

    \
     
     \ /
      Last update: 2020-12-28 15:33    [W:4.097 / U:0.228 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site