On Sep 11, 2008, at 2:32 AM, Christof Schmitt wrote:
On Wed, Sep 10, 2008 at 07:04:01AM -0700, Seokmann Ju wrote: [...]+struct fc_frame_header { [...] + u32 ct_cmd[1]; +};I was just reading he patch and i am wondering why the fc_frame_header is included here. There was a discussion in August and the conclusion was to only include the common ct_hdr in the LLD interface: http://marc.info/?t=121908849700001&r=1&w=2 The zfcp driver does not care about the FC headers, this is hidden in the System z FCP HBA. I would prefer that the interface to the LLD does not include the frame_header, otherwise userspace would fill the frame_header and zfcp would simply discard it.
Thanks for the pointing out. I will submit the updated patch, soon. Seokmann -- 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
