On Tue, 2017-02-14 at 10:56 -0800, Bart Van Assche wrote: > @@ -2666,7 +2661,7 @@ static int srp_slave_alloc(struct scsi_device *sdev) > struct srp_device *srp_dev = target->srp_host->srp_dev; > struct ib_device *ibdev = srp_dev->dev; > > - if (!(ibdev->attrs.device_cap_flags & IB_DEVICE_SG_GAPS_REG)) > + if (true) > blk_queue_virt_boundary(sdev->request_queue, > ~srp_dev->mr_page_mask); Hello Doug, This patch introduces a new warning when building with W=1, namely that variable ibdev is not used. The build bot figured this out. Do you want me to address this? Thanks, Bart.