[LU-8602] Support GSS crypto code with linux 4.6 kernels

• Type: Improvement
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.12.0
• Environment: Any system using GSS and a linux kernel 4.6 and above.
• Assignee: James A Simmons
• Reporter: James A Simmons

Description

Currently the GSS code for Lustre directly uses the linux crypto API. This code uses struct crypto_hash which has now been removed in newer kernels for struct crypto_ahash. Libcfs crypto API has moved to this new kernel API but it doesn't have support for the algos that GSS wants to use. So the first question to ask is does GSS move to libcfs crypto API and we expand support in the libcfs crypto API to AES and DEC or do we ignore libcfs crypto API and just use the linux crypto api directly and use the newer ahash api.

Issue Links

• is related to: LU-9073 SSK: lgss_sk generates keys with invalid HMAC and Crypto algorithms
• is related to: LU-9245 lgss_sk may unsafely overwrite nodemap and fsname fields in keys
• is related to: LU-9430 logic errors in lgss_sk code
• is related to: LU-9795 SSK test failures in many suites when SHARED_KEY is enabled
• is related to: LU-11635 GSS build for client-only
• is related to: LU-3289 IU Shared Secret Key authentication and encryption
• is related to: LU-8560 Support for linux 4.6 kernels

Comments

[James A Simmons]

Sebastien I got everything to work on Ubuntu/Debian

[Gerrit Updater]

Sebastien Buisson ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33592
Subject: LU-8602 gss: support OpenSSL 1.1
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4cb1c93ba786b8378a808cd7a863f46b45eab238

[Gerrit Updater]

Sebastien Buisson ([e-mail]) uploaded a new patch: https://review.whamcloud.com/33493
Subject: LU-8602 gss: get rid of cfs_crypto_hash_desc
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 3ef2ea65e9701f8cee19d987c9927fe1e12779b9

[Peter Jones]

James

Have you tried this recently? How does GSS behave with Ubuntu 18.04 clients?

Peter

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/31191/
Subject: LU-8602 gss: autoconf check missing "test" keyword
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 4dd55cb2bcffd681117b8513a91908afe0647108

[Gerrit Updater]

Olaf Faaland-LLNL ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31191
Subject: LU-8602 gss: autoconf check missing "test" keyword
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 9b73c8b86b67f7f78b97c95ac81ff2d5f7a0bc69

[Jeremy Filizetti]

The patch at https://review.whamcloud.com/31095 breaks the ability to build SSK due to a script error:

checking for krb5_derive_key in -lgssapi_krb5... no
./configure: line 21783: xyes: command not found
checking whether OpenSSL has functions needed for SSK... no

This is due to the second part of the if statement after the &&:
AS_IF([test "x$gss_conf_test" = xsuccess && "x$enable_gss" != xno], [

This should probably be
AS_IF([test "x$gss_conf_test" = xsuccess && test "x$enable_gss" != xno], [

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/31095/
Subject: LU-8602 gss: Fix autoconf check for crypto_hash
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: f6329102615e111490c7955a0a9e8b8610f8a244

[Gerrit Updater]

Olaf Faaland-LLNL ([e-mail]) uploaded a new patch: https://review.whamcloud.com/31095
Subject: LU-8602 gss: Fix autoconf check for crypto_hash
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 4d35ca99ac1a1e23c3ccf30303e88e5e7d063d79

[Olaf Faaland]

There is an bug in the autoconf code for LC_CONFIG_GSS; if LC_HAVE_CRYPTO_HASH sets enable_gss to no, but gss_conf_test==success, enable_gss is set back to yes and the build fails.

Should I submit a patch? That code will go away when "gss: Properly port gss to newer crypto api" is landed, but I assume the latter will not be backported to 2.10 whereas the autoconf fix could be.

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch https://review.whamcloud.com/25199/
Subject: LU-8602 libcfs: call proper crypto algo when keys are passed in
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 53e967746f37d5f0322bfc984af3ce1c9517079b