Re: [PATCH] SCSI: Add SG_GET_SG_MAX_SEGMENT_SIZE ioctl to sg device

From: Cong Meng
Date: Tue Jul 03 2012 - 03:53:30 EST


On Mon, 2012-07-02 at 12:18 +0100, James Bottomley wrote:
> On Mon, 2012-07-02 at 17:20 +0800, Cong Meng wrote:
> > Add a new ioctl cmd to get the max segment size. It's useful in
> > virtualization environment for guest to know the queue limits, so
> > that the guest kicks off legal scsi command to a sg device.
>
> What's wrong with simply looking in the /sys for this? Plus, if you can

Adding this ioctl is a big convenience for the programs, qemu
in my case, to which parameter that likes '/dev/sgX' is given.
With this ioctl, those programs only need to issue an ioctl to
get the queue limit. Otherwise, 'dev/sgX' is needed to convert
to /sys path, then open and read it.
In addition, the potential variation and exceptional situation
of sys FS, EX mount path, are needed to consider as well.
Is my justification acceptable?

> come up with a justifiable reason, this needs to be in block as well so
> we can send it to all SCSI devices regardless of what ULD people use.
I can do it.

Thanks.
Cong.
>
> James
>
>


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/