Re: [PATCH] vmw_pvscsi: fixup tagging

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

 



On Fri, Oct 17, 2014 at 06:34:48PM +0000, Arvind Kumar wrote:
> Hi Christoph,
> 
> Thanks for the change. Sorry for the delay. The change looks fine to me. I just have a question.
> 
> The comment in include/scsi/scsi_cmnd.h says:
> 
> struct scsi_cmnd {
> ...
>              unsigned char tag; /* SCSI-II queued command tag */
> }
> 
> Is that comment not right? Should we update that too?

The comment looks very confusing and I'm fine with a patch that just
removes the comment.  That being said Hannes has patches to remove the
whole field which I'd like to merge once he fixes up the remaining
issues, so it might not worth the effort.

--
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