Re: [PATCH] st: Do not rewind for SG_IO

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2014-02-06 at 08:50 -0500, Martin K. Petersen wrote:
> >>>>> "Hannes" == Hannes Reinecke <hare@xxxxxxx> writes:
> 
> >> My patch provides both the original VPD 0x83 and 0x80 bits as well as
> >> a handle identical to /sbin/scsi_id.
> >> 
> Hannes> Bah, don't do that.  That should better be handled by udev
> Hannes> rules.  I've got a set of patches moving from scsi_id to sg_inq,
> Hannes> which can be easily adapted to using sysfs directly.
> 
> I just want to get out of the "userland sending random SCSI commands"
> business. That is a world of pain right now.

Well, in the words of Bill Clinton "I feel your pain".  However, I need
to know we won't pick up that same whole world of pain in the kernel.
Remember it's not just SCSI devices, it's also ATA devices and
potentially other block devices ... then there's blkid and all the weird
things it does. Then there's transport identifiers for multi-path
reservations and so on.

Convince me this path won't have us shifting a whole bunch of weird from
user space to the kernel without any reduction in the weird factor.
Remember the point is not what can we do for all the nicely behaved
SCSI-3 devices, it's what do we do for everything.

James

> I wanted to provide a handle that was guaranteed to be compatible with
> existing tooling. If you have worked around that in udev rules I guess
> that's OK with me.
> 


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




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux