* [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64'
@ 2019-11-08 23:09 Revanth Rajashekar
2019-11-14 16:14 ` Rajashekar, Revanth
2019-11-18 16:49 ` Jens Axboe
0 siblings, 2 replies; 6+ messages in thread
From: Revanth Rajashekar @ 2019-11-08 23:09 UTC (permalink / raw)
To: linux-block; +Cc: Scott Bauer, Jonathan Derrick, Jens Axboe, Revanth Rajashekar
In function 'activate_lsp', rather than hard-coding the
short atom header(0x83), we need to let the function
'add_short_atom_header' append the header based on the
parameter being appended.
The paramete has been defined in Section 3.1.2.1 of
https://trustedcomputinggroup.org/wp-content/uploads/TCG_Storage-Opal_Feature_Set_Single_User_Mode_v1-00_r1-00-Final.pdf
Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>
---
block/opal_proto.h | 4 ++++
block/sed-opal.c | 6 +-----
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/block/opal_proto.h b/block/opal_proto.h
index 736e67c3e7c5..325cbba2465f 100644
--- a/block/opal_proto.h
+++ b/block/opal_proto.h
@@ -205,6 +205,10 @@ enum opal_lockingstate {
OPAL_LOCKING_LOCKED = 0x03,
};
+enum opal_parameter {
+ OPAL_SUM_SET_LIST = 0x060000,
+};
+
/* Packets derived from:
* TCG_Storage_Architecture_Core_Spec_v2.01_r1.00
* Secion: 3.2.3 ComPackets, Packets & Subpackets
diff --git a/block/sed-opal.c b/block/sed-opal.c
index b2cacc9ddd11..880cc57a5f6b 100644
--- a/block/sed-opal.c
+++ b/block/sed-opal.c
@@ -1886,7 +1886,6 @@ static int activate_lsp(struct opal_dev *dev, void *data)
{
struct opal_lr_act *opal_act = data;
u8 user_lr[OPAL_UID_LENGTH];
- u8 uint_3 = 0x83;
int err, i;
err = cmd_start(dev, opaluid[OPAL_LOCKINGSP_UID],
@@ -1899,10 +1898,7 @@ static int activate_lsp(struct opal_dev *dev, void *data)
return err;
add_token_u8(&err, dev, OPAL_STARTNAME);
- add_token_u8(&err, dev, uint_3);
- add_token_u8(&err, dev, 6);
- add_token_u8(&err, dev, 0);
- add_token_u8(&err, dev, 0);
+ add_token_u64(&err, dev, OPAL_SUM_SET_LIST);
add_token_u8(&err, dev, OPAL_STARTLIST);
add_token_bytestring(&err, dev, user_lr, OPAL_UID_LENGTH);
--
2.17.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64'
2019-11-08 23:09 [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64' Revanth Rajashekar
@ 2019-11-14 16:14 ` Rajashekar, Revanth
2019-11-14 17:28 ` Jens Axboe
2019-11-18 16:49 ` Jens Axboe
1 sibling, 1 reply; 6+ messages in thread
From: Rajashekar, Revanth @ 2019-11-14 16:14 UTC (permalink / raw)
To: linux-block; +Cc: Scott Bauer, Jonathan Derrick, Jens Axboe
Hi,
On 11/8/2019 4:09 PM, Revanth Rajashekar wrote:
> In function 'activate_lsp', rather than hard-coding the
> short atom header(0x83), we need to let the function
> 'add_short_atom_header' append the header based on the
> parameter being appended.
>
> The paramete has been defined in Section 3.1.2.1 of
> https://trustedcomputinggroup.org/wp-content/uploads/TCG_Storage-Opal_Feature_Set_Single_User_Mode_v1-00_r1-00-Final.pdf
>
> Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>
> ---
> block/opal_proto.h | 4 ++++
> block/sed-opal.c | 6 +-----
> 2 files changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/block/opal_proto.h b/block/opal_proto.h
> index 736e67c3e7c5..325cbba2465f 100644
> --- a/block/opal_proto.h
> +++ b/block/opal_proto.h
> @@ -205,6 +205,10 @@ enum opal_lockingstate {
> OPAL_LOCKING_LOCKED = 0x03,
> };
>
> +enum opal_parameter {
> + OPAL_SUM_SET_LIST = 0x060000,
> +};
> +
> /* Packets derived from:
> * TCG_Storage_Architecture_Core_Spec_v2.01_r1.00
> * Secion: 3.2.3 ComPackets, Packets & Subpackets
> diff --git a/block/sed-opal.c b/block/sed-opal.c
> index b2cacc9ddd11..880cc57a5f6b 100644
> --- a/block/sed-opal.c
> +++ b/block/sed-opal.c
> @@ -1886,7 +1886,6 @@ static int activate_lsp(struct opal_dev *dev, void *data)
> {
> struct opal_lr_act *opal_act = data;
> u8 user_lr[OPAL_UID_LENGTH];
> - u8 uint_3 = 0x83;
> int err, i;
>
> err = cmd_start(dev, opaluid[OPAL_LOCKINGSP_UID],
> @@ -1899,10 +1898,7 @@ static int activate_lsp(struct opal_dev *dev, void *data)
> return err;
>
> add_token_u8(&err, dev, OPAL_STARTNAME);
> - add_token_u8(&err, dev, uint_3);
> - add_token_u8(&err, dev, 6);
> - add_token_u8(&err, dev, 0);
> - add_token_u8(&err, dev, 0);
> + add_token_u64(&err, dev, OPAL_SUM_SET_LIST);
>
> add_token_u8(&err, dev, OPAL_STARTLIST);
> add_token_bytestring(&err, dev, user_lr, OPAL_UID_LENGTH);
Kind remainder to review this patch and throw your comments if any :)
Thank you
Revanth Rajashekar
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64'
2019-11-14 16:14 ` Rajashekar, Revanth
@ 2019-11-14 17:28 ` Jens Axboe
2019-11-14 18:19 ` Rajashekar, Revanth
2019-11-18 16:47 ` Derrick, Jonathan
0 siblings, 2 replies; 6+ messages in thread
From: Jens Axboe @ 2019-11-14 17:28 UTC (permalink / raw)
To: Rajashekar, Revanth, linux-block; +Cc: Scott Bauer, Jonathan Derrick
On 11/14/19 9:14 AM, Rajashekar, Revanth wrote:
> Hi,
>
> On 11/8/2019 4:09 PM, Revanth Rajashekar wrote:
>> In function 'activate_lsp', rather than hard-coding the
>> short atom header(0x83), we need to let the function
>> 'add_short_atom_header' append the header based on the
>> parameter being appended.
>>
>> The paramete has been defined in Section 3.1.2.1 of
>> https://trustedcomputinggroup.org/wp-content/uploads/TCG_Storage-Opal_Feature_Set_Single_User_Mode_v1-00_r1-00-Final.pdf
>>
>> Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>
>> ---
>> block/opal_proto.h | 4 ++++
>> block/sed-opal.c | 6 +-----
>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/block/opal_proto.h b/block/opal_proto.h
>> index 736e67c3e7c5..325cbba2465f 100644
>> --- a/block/opal_proto.h
>> +++ b/block/opal_proto.h
>> @@ -205,6 +205,10 @@ enum opal_lockingstate {
>> OPAL_LOCKING_LOCKED = 0x03,
>> };
>>
>> +enum opal_parameter {
>> + OPAL_SUM_SET_LIST = 0x060000,
>> +};
>> +
>> /* Packets derived from:
>> * TCG_Storage_Architecture_Core_Spec_v2.01_r1.00
>> * Secion: 3.2.3 ComPackets, Packets & Subpackets
>> diff --git a/block/sed-opal.c b/block/sed-opal.c
>> index b2cacc9ddd11..880cc57a5f6b 100644
>> --- a/block/sed-opal.c
>> +++ b/block/sed-opal.c
>> @@ -1886,7 +1886,6 @@ static int activate_lsp(struct opal_dev *dev, void *data)
>> {
>> struct opal_lr_act *opal_act = data;
>> u8 user_lr[OPAL_UID_LENGTH];
>> - u8 uint_3 = 0x83;
>> int err, i;
>>
>> err = cmd_start(dev, opaluid[OPAL_LOCKINGSP_UID],
>> @@ -1899,10 +1898,7 @@ static int activate_lsp(struct opal_dev *dev, void *data)
>> return err;
>>
>> add_token_u8(&err, dev, OPAL_STARTNAME);
>> - add_token_u8(&err, dev, uint_3);
>> - add_token_u8(&err, dev, 6);
>> - add_token_u8(&err, dev, 0);
>> - add_token_u8(&err, dev, 0);
>> + add_token_u64(&err, dev, OPAL_SUM_SET_LIST);
>>
>> add_token_u8(&err, dev, OPAL_STARTLIST);
>> add_token_bytestring(&err, dev, user_lr, OPAL_UID_LENGTH);
>
> Kind remainder to review this patch and throw your comments if any :)
I'll be happy to queue it up for 5.5, but Scott/Jonathan should
review/ack it first.
--
Jens Axboe
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64'
2019-11-14 17:28 ` Jens Axboe
@ 2019-11-14 18:19 ` Rajashekar, Revanth
2019-11-18 16:47 ` Derrick, Jonathan
1 sibling, 0 replies; 6+ messages in thread
From: Rajashekar, Revanth @ 2019-11-14 18:19 UTC (permalink / raw)
To: Jens Axboe, linux-block; +Cc: Scott Bauer, Jonathan Derrick
On 11/14/2019 10:28 AM, Jens Axboe wrote:
> On 11/14/19 9:14 AM, Rajashekar, Revanth wrote:
>> Hi,
>>
>> On 11/8/2019 4:09 PM, Revanth Rajashekar wrote:
>>> In function 'activate_lsp', rather than hard-coding the
>>> short atom header(0x83), we need to let the function
>>> 'add_short_atom_header' append the header based on the
>>> parameter being appended.
>>>
>>> The paramete has been defined in Section 3.1.2.1 of
>>> https://trustedcomputinggroup.org/wp-content/uploads/TCG_Storage-Opal_Feature_Set_Single_User_Mode_v1-00_r1-00-Final.pdf
>>>
>>> Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>
>>> ---
>>> block/opal_proto.h | 4 ++++
>>> block/sed-opal.c | 6 +-----
>>> 2 files changed, 5 insertions(+), 5 deletions(-)
>>>
>>> diff --git a/block/opal_proto.h b/block/opal_proto.h
>>> index 736e67c3e7c5..325cbba2465f 100644
>>> --- a/block/opal_proto.h
>>> +++ b/block/opal_proto.h
>>> @@ -205,6 +205,10 @@ enum opal_lockingstate {
>>> OPAL_LOCKING_LOCKED = 0x03,
>>> };
>>>
>>> +enum opal_parameter {
>>> + OPAL_SUM_SET_LIST = 0x060000,
>>> +};
>>> +
>>> /* Packets derived from:
>>> * TCG_Storage_Architecture_Core_Spec_v2.01_r1.00
>>> * Secion: 3.2.3 ComPackets, Packets & Subpackets
>>> diff --git a/block/sed-opal.c b/block/sed-opal.c
>>> index b2cacc9ddd11..880cc57a5f6b 100644
>>> --- a/block/sed-opal.c
>>> +++ b/block/sed-opal.c
>>> @@ -1886,7 +1886,6 @@ static int activate_lsp(struct opal_dev *dev, void *data)
>>> {
>>> struct opal_lr_act *opal_act = data;
>>> u8 user_lr[OPAL_UID_LENGTH];
>>> - u8 uint_3 = 0x83;
>>> int err, i;
>>>
>>> err = cmd_start(dev, opaluid[OPAL_LOCKINGSP_UID],
>>> @@ -1899,10 +1898,7 @@ static int activate_lsp(struct opal_dev *dev, void *data)
>>> return err;
>>>
>>> add_token_u8(&err, dev, OPAL_STARTNAME);
>>> - add_token_u8(&err, dev, uint_3);
>>> - add_token_u8(&err, dev, 6);
>>> - add_token_u8(&err, dev, 0);
>>> - add_token_u8(&err, dev, 0);
>>> + add_token_u64(&err, dev, OPAL_SUM_SET_LIST);
>>>
>>> add_token_u8(&err, dev, OPAL_STARTLIST);
>>> add_token_bytestring(&err, dev, user_lr, OPAL_UID_LENGTH);
>> Kind remainder to review this patch and throw your comments if any :)
> I'll be happy to queue it up for 5.5, but Scott/Jonathan should
> review/ack it first.
Sure.
Thank you :)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64'
2019-11-14 17:28 ` Jens Axboe
2019-11-14 18:19 ` Rajashekar, Revanth
@ 2019-11-18 16:47 ` Derrick, Jonathan
1 sibling, 0 replies; 6+ messages in thread
From: Derrick, Jonathan @ 2019-11-18 16:47 UTC (permalink / raw)
To: Rajashekar, Revanth, linux-block, axboe; +Cc: sbauer
On Thu, 2019-11-14 at 10:28 -0700, Jens Axboe wrote:
> On 11/14/19 9:14 AM, Rajashekar, Revanth wrote:
> > Hi,
> >
> > On 11/8/2019 4:09 PM, Revanth Rajashekar wrote:
> > > In function 'activate_lsp', rather than hard-coding the
> > > short atom header(0x83), we need to let the function
> > > 'add_short_atom_header' append the header based on the
> > > parameter being appended.
> > >
> > > The paramete has been defined in Section 3.1.2.1 of
> > > https://trustedcomputinggroup.org/wp-content/uploads/TCG_Storage-Opal_Feature_Set_Single_User_Mode_v1-00_r1-00-Final.pdf
> > >
> > > Signed-off-by: Revanth Rajashekar <revanth.rajashekar@intel.com>
> > > ---
> > > block/opal_proto.h | 4 ++++
> > > block/sed-opal.c | 6 +-----
> > > 2 files changed, 5 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/block/opal_proto.h b/block/opal_proto.h
> > > index 736e67c3e7c5..325cbba2465f 100644
> > > --- a/block/opal_proto.h
> > > +++ b/block/opal_proto.h
> > > @@ -205,6 +205,10 @@ enum opal_lockingstate {
> > > OPAL_LOCKING_LOCKED = 0x03,
> > > };
> > >
> > > +enum opal_parameter {
> > > + OPAL_SUM_SET_LIST = 0x060000,
> > > +};
> > > +
> > > /* Packets derived from:
> > > * TCG_Storage_Architecture_Core_Spec_v2.01_r1.00
> > > * Secion: 3.2.3 ComPackets, Packets & Subpackets
> > > diff --git a/block/sed-opal.c b/block/sed-opal.c
> > > index b2cacc9ddd11..880cc57a5f6b 100644
> > > --- a/block/sed-opal.c
> > > +++ b/block/sed-opal.c
> > > @@ -1886,7 +1886,6 @@ static int activate_lsp(struct opal_dev *dev, void *data)
> > > {
> > > struct opal_lr_act *opal_act = data;
> > > u8 user_lr[OPAL_UID_LENGTH];
> > > - u8 uint_3 = 0x83;
> > > int err, i;
> > >
> > > err = cmd_start(dev, opaluid[OPAL_LOCKINGSP_UID],
> > > @@ -1899,10 +1898,7 @@ static int activate_lsp(struct opal_dev *dev, void *data)
> > > return err;
> > >
> > > add_token_u8(&err, dev, OPAL_STARTNAME);
> > > - add_token_u8(&err, dev, uint_3);
> > > - add_token_u8(&err, dev, 6);
> > > - add_token_u8(&err, dev, 0);
> > > - add_token_u8(&err, dev, 0);
> > > + add_token_u64(&err, dev, OPAL_SUM_SET_LIST);
> > >
> > > add_token_u8(&err, dev, OPAL_STARTLIST);
> > > add_token_bytestring(&err, dev, user_lr, OPAL_UID_LENGTH);
> >
> > Kind remainder to review this patch and throw your comments if any :)
>
> I'll be happy to queue it up for 5.5, but Scott/Jonathan should
> review/ack it first.
>
Looks like it should work the same
Reviewed-by: Jon Derrick <jonathan.derrick@intel.com>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64'
2019-11-08 23:09 [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64' Revanth Rajashekar
2019-11-14 16:14 ` Rajashekar, Revanth
@ 2019-11-18 16:49 ` Jens Axboe
1 sibling, 0 replies; 6+ messages in thread
From: Jens Axboe @ 2019-11-18 16:49 UTC (permalink / raw)
To: Revanth Rajashekar, linux-block; +Cc: Scott Bauer, Jonathan Derrick
On 11/8/19 4:09 PM, Revanth Rajashekar wrote:
> In function 'activate_lsp', rather than hard-coding the
> short atom header(0x83), we need to let the function
> 'add_short_atom_header' append the header based on the
> parameter being appended.
>
> The paramete has been defined in Section 3.1.2.1 of
> https://trustedcomputinggroup.org/wp-content/uploads/TCG_Storage-Opal_Feature_Set_Single_User_Mode_v1-00_r1-00-Final.pdf
Applied, thanks.
--
Jens Axboe
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-11-18 16:49 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-11-08 23:09 [PATCH] block: sed-opal: Introduce SUM_SET_LIST parameter and append it using 'add_token_u64' Revanth Rajashekar
2019-11-14 16:14 ` Rajashekar, Revanth
2019-11-14 17:28 ` Jens Axboe
2019-11-14 18:19 ` Rajashekar, Revanth
2019-11-18 16:47 ` Derrick, Jonathan
2019-11-18 16:49 ` Jens Axboe
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.