On Mon, 2018-10-01 at 10:12 -0700, Nick Desaulniers wrote: > On Mon, Oct 1, 2018 at 9:16 AM Evan Green <[email protected]> wrote: > > > > From: Robb Glasser <[email protected]> > > > > sg_ioctl could be spammed by requests, leading to a double free in > > __free_pages. This protects the entry points of sg_ioctl where the > > memory could be corrupted by a double call to __free_pages if multiple > > requests are happening concurrently. > > > > Signed-off-by: Robb Glasser <[email protected]> > > Signed-off-by: Nick Desaulniers <[email protected]> > > Signed-off-by: Evan Green <[email protected]> > > Cc: [email protected] > > > > --- > > Reposting this patch from last summer, as it looks like it fell in between > > the cracks. > > Christoph, do you still feel strongly about: > https://lkml.org/lkml/2017/8/5/75 ?
I don't know how Christoph feels about it, but serializing all SG I/O seems like a regression to me. If one sg command hangs I usually try to send another sg command to the same SCSI device from another shell to get more information about the nature of the hang. Serializing all SG I/O would make that impossible. Bart.

