On Mon, Apr 09, 2018 at 05:39:15PM +0200, Christoph Hellwig wrote:
> Same numerical value (for now at least), but a much better documentation
> of intent.
> @@ -499,7 +499,7 @@ int sg_scsi_ioctl(struct request_queue *q, struct gendisk *disk, fmode_t mode,
> break;
> }
>
> - if (bytes && blk_rq_map_kern(q, rq, buffer, bytes, __GFP_RECLAIM)) {
> + if (bytes && blk_rq_map_kern(q, rq, buffer, bytes, GFP_NOIO)) {
We don't seem to have grabbed any locks between the line which allocates
memory using GFP_USER (line 446) and here, so I don't see why we should
prohibit I/O?
