26 Jul, 2017
1 commit
-
The FC-NVME spec hasn't locked down on the format string for TRADDR.
Currently the spec is lobbying for "nn-:pn-"
where the wwn's are hex values but not prefixed by 0x.Most implementations so far expect a string format of
"nn-0x:pn-0x" to be used. The transport
uses the match_u64 parser which requires a leading 0x prefix to set
the base properly. If it's not there, a match will either fail or return
a base 10 value.The resolution in T11 is pushing out. Therefore, to fix things now and
to cover any eventuality and any implementations already in the field,
this patch adds support for both formats.The change consists of replacing the token matching routine with a
routine that validates the fixed string format, and then builds
a local copy of the hex name with a 0x prefix before calling
the system parser.Note: the same parser routine exists in both the initiator and target
transports. Given this is about the only "shared" item, we chose to
replicate rather than create an interdendency on some shared code.Signed-off-by: James Smart
Signed-off-by: Christoph Hellwig
10 Jul, 2017
1 commit
-
Revises the Create Association LS for the amount of pad expected in 1.16.
Add defines for the minimum lengths that a target can accept (e.g. variable
pad lengths)Signed-off-by: James Smart
Signed-off-by: Sagi Grimberg
05 Jun, 2017
1 commit
-
Signed-off-by: Christoph Hellwig
Reviewed-by: Amir Goldstein
Reviewed-by: Andy Shevchenko
04 Apr, 2017
1 commit
-
Update FC-NVME definitions to match FC-NVME r1.14 (16-020vB) plus
change voted in by 2/22 FC-NVME Adhoc (see HOSTID below).Includes the following:
- Addition of "status_code" field to ERSP IU
- Addition of FC-NVME LS RJT reason_codes and reason_explanations
- CreateAssociation payload, HostID field shortened to 16 bytesSigned-off-by: James Smart
Reviewed-by: Christoph Hellwig
Signed-off-by: Sagi Grimberg
Signed-off-by: Jens Axboe
06 Dec, 2016
1 commit
-
- Formats for Cmd, Data, Rsp IUs
- Formats FC-4 LS definitions
- Add to MAINTAINERS fileSigned-off-by: James Smart
Reviewed-by: Christoph Hellwig
Reviewed-by: Jay Freyensee
Reviewed-by: Johannes Thumshirn
Signed-off-by: Christoph Hellwig