On Mon, Mar 14, 2022 at 01:28:38PM +0100, Greg KH wrote:
> On Mon, Mar 14, 2022 at 11:50:18AM +0000, Niklas Cassel wrote:
> > Hello Christoph, stable,
> >
> > I recently saw a crash caused by the kintegrityd workqueue that could only
> > be reproduced on older kernels.
> > A null pointer dereference in function bio_integrity_verify_fn.
> >
> > The fix in Linus's tree for this:
> > 3df49967f6f1 ("block: flush the integrity workqueue in blk_integrity_unregister")
> > was first merged in v5.15.
> >
> > The fix has been backported to v5.10 LTS branch in:
> > 1ef68b84bc11 ("block: flush the integrity workqueue in blk_integrity_unregister")
> >
> > The fix doesn't have a fixes tag, but from inspecting the code,
> > I don't understand why this was only backported to v5.10, AFAICT it should
> > at least have been backported to v5.4, v4.19 and v4.14 LTS as well.
> >
> > Original series:
> > https://lore.kernel.org/all/20210914070657.87677-3-hch@xxxxxx/
> >
> > The blk_flush_integrity() call that actually fixes the crash should be
> > trivial to backport/add before clearing the flag and doing the memset.
>
> A backported patch series would be great to have, to show that you have
> tested that it works properly.
Hello Greg,
Unfortunately, I don't have access to the machine. I was only provided
a kernel crash dump to diagnose the crash.
I guess I was hoping for someone more familiar with the integrity stuff
to backport it. Both patch 1 and 3 are unrelated to the NULL pointer crash,
and because of various refactoring, I'm not sure if patch 1 and 3 are even
applicable for older kernel versions.
Kind regards,
Niklas
