Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed #140 accel_crypto_key_create missing Key2 #141

Merged
merged 2 commits into from
Feb 9, 2023
Merged

Conversation

glimchb
Copy link
Member

@glimchb glimchb commented Feb 9, 2023

Key2 is marked as optional but SPDK mandates it in the code still...
https://spdk.io/doc/jsonrpc.html#rpc_accel_crypto_key_create

spdk_1             | [2023-02-09 15:52:03.456606] accel_sw.c: 707:sw_accel_create_aes_xts: *ERROR*: key size 16 is not equal to key2 size 0 or is 0

Signed-off-by: Boris Glimcher 36732377+glimchb@users.noreply.github.com

@glimchb glimchb requested a review from a team as a code owner February 9, 2023 19:13
@codecov
Copy link

codecov bot commented Feb 9, 2023

Codecov Report

Merging #141 (51deb7d) into main (6134ccb) will increase coverage by 0.00%.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##             main     #141   +/-   ##
=======================================
  Coverage   49.67%   49.67%           
=======================================
  Files           4        4           
  Lines        1224     1226    +2     
=======================================
+ Hits          608      609    +1     
- Misses        598      599    +1     
  Partials       18       18           
Impacted Files Coverage Δ
pkg/server/middleend.go 66.12% <33.33%> (-0.18%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

glimchb and others added 2 commits February 9, 2023 23:01
Signed-off-by: Boris Glimcher <36732377+glimchb@users.noreply.github.com>
Signed-off-by: Boris Glimcher <Boris.Glimcher@emc.com>
Signed-off-by: Boris Glimcher <Boris.Glimcher@emc.com>
@glimchb glimchb merged commit 65d75cb into main Feb 9, 2023
@glimchb glimchb deleted the glimchb-patch-1 branch February 9, 2023 21:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SPDK: accel_crypto_key_create: json response error: failed to create DEK, rc -22
2 participants