[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <52D28807.2000004@dev.mellanox.co.il>
Date: Sun, 12 Jan 2014 14:18:15 +0200
From: Sagi Grimberg <sagig@....mellanox.co.il>
To: "Martin K. Petersen" <martin.petersen@...cle.com>,
"Nicholas A. Bellinger" <nab@...erainc.com>
CC: target-devel <target-devel@...r.kernel.org>,
linux-scsi <linux-scsi@...r.kernel.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
Christoph Hellwig <hch@....de>, Hannes Reinecke <hare@...e.de>,
Sagi Grimberg <sagig@...lanox.com>,
Or Gerlitz <ogerlitz@...lanox.com>,
Nicholas Bellinger <nab@...ux-iscsi.org>
Subject: Re: [PATCH 09/14] target/configfs: Expose protection device attributes
<SNIP>
> nab> This patch adds support for exposing DIF protection device
> nab> attributes via configfs. This includes:
>
> nab> pi_prot_type: Protection Type (0, 1, 3 currently support)
> nab> pi_prot_version: Protection Version (DIF v1 currently supported)
>
> What's DIF v2?
>
> nab> pi_guard_type: Guard Type (1=DIF CRC, 2=IP CRC)
>
> The IP checksum is only supported by DIX between OS and initiator, not
> by the target. I guess we could signal to the initiator via a
> vendor-private VPD that IP checksum is supported directly. But now what
> we have hardware-accelerated T10 CRC I don't think it's a big deal.
shouldn't it stick around if it is not deprecated yet, the transport is
required to support ip-csum->CRC conversion anyhow.
> (scsi_debug supports IP checksum because it's both initiator and
> target).
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists