lkml.org 
[lkml]   [2014]   [Jan]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 07/14] target/sbc: Add P_TYPE + PROT_EN bits to READ_CAPACITY_16
On 01/09/2014 10:21 PM, Nicholas A. Bellinger wrote:
>> What about FORMAT_UNIT emulation?
>
> Would certainly be useful to have..
>
>> The backstore protection configuration is done at the target side via
>> configfs/targetcli, if you publish DIF support in
>> INQUERY_EVPD/READ_CAPACITY you need to accept protection information format?
>
> Mmmm, these two bits bits are following what scsi_debug is currently
> exposing minus FORMAT_UNIT support..?
>
> MKP..?

Yes, don't you need FORMAT UNIT because protection information is going
to mean the pi-enabled lun will need to report less blocks? The ramdisk
backstore changes in this series allocate extra space for PI info, but
my understanding was that especially for emulation with block and fileio
backstores, everything needs to go in the same amount of space.

Furthermore, if we want PI info stored along with the blocks, then block
and fileio backstore formats are no longer going to be 1:1 -- requiring
offset calculations, non-aligned read-modify-write, and all that
unpleasantness to be handled?

I've looked at the patchset and the code looks good to me, so I guess
I'm trying to understand the scope of future work needed on the
backstore side for this support to be functional.

Regards -- Andy



\
 
 \ /
  Last update: 2014-01-10 21:21    [W:0.131 / U:0.612 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site