On 8/26/21 10:45 AM, Niklas Cassel wrote: > From: Niklas Cassel <niklas.cassel@xxxxxxx> > > Hello, > > I noticed a bug when testing cmdprio_percentage with io_uring. > Turns out that this was because sqe->ioprio contained old data. > > Fixed one additional field that could contain old data (sqe->flags), > as well as one field that was always cleared _after_ the flags for > this field had been set (sqe->rw_flags). > > There might be additional fields that might contain old data, e.g. > sqe->sync_range_flags and sqe->fsync_flags. However, I assume that > io_uring ignores those flags for read and write commands, however, > they might still be able to interfere with non-read/write commands. Thanks, looks good, applied. -- Jens Axboe