[stgt] [PATCH] Allow backends to specify which opcodes they support
ronniesahlberg at gmail.com
Fri Oct 11 01:30:17 CEST 2013
I first had a 32-byte bitmap. That was horrible.
I could change it to have one value per line with a comment containing
both the opcode number and the name :
/* 0x00 TEST_UNIT_READY */ SUPPORTED,
What about that?
On Thu, Oct 10, 2013 at 3:32 PM, Dan Mick <dan.mick at inktank.com> wrote:
> I like this in general, but it sure would be nice to have a more-readable
> way to express command support. I'm not full of ideas but both for errors
> in creation and examination it would be great.
> On 10/09/2013 07:43 PM, Ronnie Sahlberg wrote:
>> Tomo, List
>> Please find a patch that adds an optional "which opceds are supported"
>> to the backend template.
>> RDWR and SHEEPDOG are modified to use these, and the opcodes that are NOT
>> supported by sheepdog are zeroed out so that tgtd will now respond
>> correctly with INVALID_OP_CODE if they are sent to a sheepdog backed LUN.
>> Next will be RBD but there are pending patches to that backend so I will
>> and update to it later.
>> Please comment/review/merge
>> ronnie sahlberg
>> To unsubscribe from this list: send the line "unsubscribe stgt" in
>> the body of a message to majordomo at vger.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo at vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
More information about the stgt