2022-07-21 19:52:00

by Kristen Carlson Accardi

[permalink] [raw]
Subject: [PATCH] selftests/sgx: Ignore OpenSSL 3.0 deprecated functions warning

OpenSSL 3.0 deprecates some of the functions used in the SGX
selftests, causing build errors on new distros. For now ignore
the warnings until support for the functions is no longer
available.

Signed-off-by: Kristen Carlson Accardi <[email protected]>
---
tools/testing/selftests/sgx/sigstruct.c | 3 +++
1 file changed, 3 insertions(+)

diff --git a/tools/testing/selftests/sgx/sigstruct.c b/tools/testing/selftests/sgx/sigstruct.c
index 50c5ab1aa6fa..bb191b70141a 100644
--- a/tools/testing/selftests/sgx/sigstruct.c
+++ b/tools/testing/selftests/sgx/sigstruct.c
@@ -17,6 +17,9 @@
#include "defines.h"
#include "main.h"

+/* OpenSSL 3.0 has deprecated some functions. For now just ignore the warnings. */
+#pragma GCC diagnostic ignored "-Wdeprecated-declarations"
+
struct q1q2_ctx {
BN_CTX *bn_ctx;
BIGNUM *m;
--
2.36.1


2022-07-21 20:23:06

by Dave Hansen

[permalink] [raw]
Subject: Re: [PATCH] selftests/sgx: Ignore OpenSSL 3.0 deprecated functions warning

On 7/21/22 12:40, Kristen Carlson Accardi wrote:
> OpenSSL 3.0 deprecates some of the functions used in the SGX
> selftests, causing build errors on new distros. For now ignore
> the warnings until support for the functions is no longer
> available.

Are there some better functions we should be moving to?

2022-07-21 20:33:40

by Dave Hansen

[permalink] [raw]
Subject: Re: [PATCH] selftests/sgx: Ignore OpenSSL 3.0 deprecated functions warning

On 7/21/22 13:08, Kristen Carlson Accardi wrote:
> On Thu, 2022-07-21 at 13:02 -0700, Dave Hansen wrote:
>> On 7/21/22 12:40, Kristen Carlson Accardi wrote:
>>> OpenSSL 3.0 deprecates some of the functions used in the SGX
>>> selftests, causing build errors on new distros. For now ignore
>>> the warnings until support for the functions is no longer
>>> available.
>> Are there some better functions we should be moving to?
> I looked into this actually as my first choice, but the problem I had
> was that the recommended new functions weren't available on other
> OpenSSL libraries, and we'd have to add compatibility macros and stuff,
> so it seemed to me that the less complicated thing to do was wait till
> OpenSSL 1 was just not widely deployed anymore. This strategy is also
> being used in other scripts in the kernel.

Sounds sane to me. Thanks for the explanation.

2022-07-21 20:36:41

by Kristen Carlson Accardi

[permalink] [raw]
Subject: Re: [PATCH] selftests/sgx: Ignore OpenSSL 3.0 deprecated functions warning

On Thu, 2022-07-21 at 13:02 -0700, Dave Hansen wrote:
> On 7/21/22 12:40, Kristen Carlson Accardi wrote:
> > OpenSSL 3.0 deprecates some of the functions used in the SGX
> > selftests, causing build errors on new distros. For now ignore
> > the warnings until support for the functions is no longer
> > available.
>
> Are there some better functions we should be moving to?

I looked into this actually as my first choice, but the problem I had
was that the recommended new functions weren't available on other
OpenSSL libraries, and we'd have to add compatibility macros and stuff,
so it seemed to me that the less complicated thing to do was wait till
OpenSSL 1 was just not widely deployed anymore. This strategy is also
being used in other scripts in the kernel.


2022-07-28 08:29:41

by Jarkko Sakkinen

[permalink] [raw]
Subject: Re: [PATCH] selftests/sgx: Ignore OpenSSL 3.0 deprecated functions warning

On Thu, Jul 21, 2022 at 12:40:40PM -0700, Kristen Carlson Accardi wrote:
> OpenSSL 3.0 deprecates some of the functions used in the SGX
> selftests, causing build errors on new distros. For now ignore
> the warnings until support for the functions is no longer
> available.
>
> Signed-off-by: Kristen Carlson Accardi <[email protected]>
> ---
> tools/testing/selftests/sgx/sigstruct.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/tools/testing/selftests/sgx/sigstruct.c b/tools/testing/selftests/sgx/sigstruct.c
> index 50c5ab1aa6fa..bb191b70141a 100644
> --- a/tools/testing/selftests/sgx/sigstruct.c
> +++ b/tools/testing/selftests/sgx/sigstruct.c
> @@ -17,6 +17,9 @@
> #include "defines.h"
> #include "main.h"
>
> +/* OpenSSL 3.0 has deprecated some functions. For now just ignore the warnings. */

Perhaps, even prefix with "FIXME:" prefix?

> +#pragma GCC diagnostic ignored "-Wdeprecated-declarations"
> +
> struct q1q2_ctx {
> BN_CTX *bn_ctx;
> BIGNUM *m;
> --
> 2.36.1
>

BR, Jarkko