RE: out of range LBA using sg_raw

From: Kashyap Desai
Date: Wed Mar 08 2017 - 11:50:06 EST

> -----Original Message-----
> From: Martin K. Petersen [mailto:martin.petersen@xxxxxxxxxx]
> Sent: Wednesday, March 08, 2017 10:03 PM
> To: Kashyap Desai
> Cc: Christoph Hellwig; linux-kernel@xxxxxxxxxxxxxxx; linux-
> scsi@xxxxxxxxxxxxxxx
> Subject: Re: out of range LBA using sg_raw
> >>>>> "Kashyap" == Kashyap Desai <kashyap.desai@xxxxxxxxxxxx> writes:
> Kashyap,
> Kashyap> I am just curious to know how badly we have to scrutinize each
> Kashyap> packet before sending to Fast Path as we are in IO path and
> Kashyap> recommend only important checks to be added.
> As Christoph pointed out, when the fast path is in use you assume the
role of
> the SCSI device. And therefore it is your responsibility to ensure that
the VD's
> capacity and other relevant constraints are being honored. Just like the
> firmware and any attached disks would.

Martin -

Agree on this point. I am planning to study all possible such sanity in
driver for VD and not trying to fix one specific scenario as described
Do you think fix in this area is good for kernel-stable as well OR just
keep in linux-next as it is not so severe considering real time exposure ?
Trying to understand priority and severity of this issue.

> It is a feature that there is no sanity checking in the sg interface.
> The intent is to be able to pass through commands directly to a device
> have the device act upon them. Including fail them if they don't make
> sense.

Understood as sg_raw is not design to sanity check.

> PS. I'm really no fan of the fast path. It's super messy to have the VD
> handled in two different places.
> --
> Martin K. Petersen Oracle Linux Engineering