* [Qemu-devel] [PATCH v2 0/2] fix building complaint @ 2016-08-30 4:10 Gonglei 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 1/2] crypto: " Gonglei 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint Gonglei 0 siblings, 2 replies; 14+ messages in thread From: Gonglei @ 2016-08-30 4:10 UTC (permalink / raw) To: qemu-devel; +Cc: berrange, dmitry, Gonglei Gonglei (2): crypto: fix building complaint e1000: fix buliding complaint crypto/tlscredsx509.c | 2 +- hw/net/e1000e_core.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) -- 1.7.12.4 ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Qemu-devel] [PATCH v2 1/2] crypto: fix building complaint 2016-08-30 4:10 [Qemu-devel] [PATCH v2 0/2] fix building complaint Gonglei @ 2016-08-30 4:10 ` Gonglei 2016-08-30 14:14 ` Eric Blake 2016-09-05 10:14 ` Daniel P. Berrange 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint Gonglei 1 sibling, 2 replies; 14+ messages in thread From: Gonglei @ 2016-08-30 4:10 UTC (permalink / raw) To: qemu-devel; +Cc: berrange, dmitry, Gonglei gnutls commit 846753877d renamed LIBGNUTLS_VERSION_NUMBER to GNUTLS_VERSION_NUMBER. If using gnutls before that verion, we'll get the below warning: crypto/tlscredsx509.c:618:5: warning: "GNUTLS_VERSION_NUMBER" is not defined Signed-off-by: Gonglei <arei.gonglei@huawei.com> --- crypto/tlscredsx509.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c index 520d34d..f2fd80f 100644 --- a/crypto/tlscredsx509.c +++ b/crypto/tlscredsx509.c @@ -615,7 +615,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, } if (cert != NULL && key != NULL) { -#if GNUTLS_VERSION_NUMBER >= 0x030111 +#if defined(GNUTLS_VERSION_NUMBER) && GNUTLS_VERSION_NUMBER >= 0x030111 char *password = NULL; if (creds->passwordid) { password = qcrypto_secret_lookup_as_utf8(creds->passwordid, -- 1.7.12.4 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/2] crypto: fix building complaint 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 1/2] crypto: " Gonglei @ 2016-08-30 14:14 ` Eric Blake 2016-08-30 14:21 ` Peter Maydell 2016-09-05 10:14 ` Daniel P. Berrange 1 sibling, 1 reply; 14+ messages in thread From: Eric Blake @ 2016-08-30 14:14 UTC (permalink / raw) To: Gonglei, qemu-devel; +Cc: dmitry [-- Attachment #1: Type: text/plain, Size: 1375 bytes --] On 08/29/2016 11:10 PM, Gonglei wrote: > gnutls commit 846753877d renamed LIBGNUTLS_VERSION_NUMBER to GNUTLS_VERSION_NUMBER. > If using gnutls before that verion, we'll get the below warning: s/verion/version/ > crypto/tlscredsx509.c:618:5: warning: "GNUTLS_VERSION_NUMBER" is not defined > The C preprocessor is well-defined to treat an unknown macro name as 0. Which compiler are you using that complains? Is this something that we consistently work around, or is it better to figure out how to fix your compiler to quit complaining? > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > --- > crypto/tlscredsx509.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c > index 520d34d..f2fd80f 100644 > --- a/crypto/tlscredsx509.c > +++ b/crypto/tlscredsx509.c > @@ -615,7 +615,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, > } > > if (cert != NULL && key != NULL) { > -#if GNUTLS_VERSION_NUMBER >= 0x030111 > +#if defined(GNUTLS_VERSION_NUMBER) && GNUTLS_VERSION_NUMBER >= 0x030111 At any rate, this is the correct way to silence the warning (even if verbose), if we do want it, so: Reviewed-by: Eric Blake <eblake@redhat.com> -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 604 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/2] crypto: fix building complaint 2016-08-30 14:14 ` Eric Blake @ 2016-08-30 14:21 ` Peter Maydell 0 siblings, 0 replies; 14+ messages in thread From: Peter Maydell @ 2016-08-30 14:21 UTC (permalink / raw) To: Eric Blake; +Cc: Gonglei, QEMU Developers, Dmitry Fleytman On 30 August 2016 at 15:14, Eric Blake <eblake@redhat.com> wrote: > On 08/29/2016 11:10 PM, Gonglei wrote: >> gnutls commit 846753877d renamed LIBGNUTLS_VERSION_NUMBER to GNUTLS_VERSION_NUMBER. >> If using gnutls before that verion, we'll get the below warning: > > s/verion/version/ > >> crypto/tlscredsx509.c:618:5: warning: "GNUTLS_VERSION_NUMBER" is not defined >> > > The C preprocessor is well-defined to treat an unknown macro name as 0. > Which compiler are you using that complains? Is this something that we > consistently work around, or is it better to figure out how to fix your > compiler to quit complaining? This is gcc's -Wundef, which we explicitly enable in configure. (The rationale is the usual one, ie that almost all the time this happens it's by accident and so it's worth putting in the extra effort to avoid the odd case where you were doing it deliberately.) thanks -- PMM ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/2] crypto: fix building complaint 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 1/2] crypto: " Gonglei 2016-08-30 14:14 ` Eric Blake @ 2016-09-05 10:14 ` Daniel P. Berrange 2016-09-05 10:50 ` Gonglei (Arei) 1 sibling, 1 reply; 14+ messages in thread From: Daniel P. Berrange @ 2016-09-05 10:14 UTC (permalink / raw) To: Gonglei; +Cc: qemu-devel, dmitry On Tue, Aug 30, 2016 at 12:10:52PM +0800, Gonglei wrote: > gnutls commit 846753877d renamed LIBGNUTLS_VERSION_NUMBER to GNUTLS_VERSION_NUMBER. > If using gnutls before that verion, we'll get the below warning: > crypto/tlscredsx509.c:618:5: warning: "GNUTLS_VERSION_NUMBER" is not defined > > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > --- > crypto/tlscredsx509.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c > index 520d34d..f2fd80f 100644 > --- a/crypto/tlscredsx509.c > +++ b/crypto/tlscredsx509.c > @@ -615,7 +615,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, > } > > if (cert != NULL && key != NULL) { > -#if GNUTLS_VERSION_NUMBER >= 0x030111 > +#if defined(GNUTLS_VERSION_NUMBER) && GNUTLS_VERSION_NUMBER >= 0x030111 > char *password = NULL; > if (creds->passwordid) { > password = qcrypto_secret_lookup_as_utf8(creds->passwordid, Other places in this file still use LIBGNUTLS_VERSION_NUMBER, which is good, because gnutls 3.x still defines LIBGNUTLS_VERSION_NUMBER for back compat. The tests/test-crypto-tlscredsx509.c file also needs a similar fix. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/2] crypto: fix building complaint 2016-09-05 10:14 ` Daniel P. Berrange @ 2016-09-05 10:50 ` Gonglei (Arei) 2016-09-05 11:04 ` Daniel P. Berrange 0 siblings, 1 reply; 14+ messages in thread From: Gonglei (Arei) @ 2016-09-05 10:50 UTC (permalink / raw) To: Daniel P. Berrange; +Cc: qemu-devel > -----Original Message----- > From: Daniel P. Berrange [mailto:berrange@redhat.com] > Sent: Monday, September 05, 2016 6:15 PM > To: Gonglei (Arei) > Cc: qemu-devel@nongnu.org; dmitry@daynix.com > Subject: Re: [PATCH v2 1/2] crypto: fix building complaint > > On Tue, Aug 30, 2016 at 12:10:52PM +0800, Gonglei wrote: > > gnutls commit 846753877d renamed LIBGNUTLS_VERSION_NUMBER to > GNUTLS_VERSION_NUMBER. > > If using gnutls before that verion, we'll get the below warning: > > crypto/tlscredsx509.c:618:5: warning: "GNUTLS_VERSION_NUMBER" is not > defined > > > > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > > --- > > crypto/tlscredsx509.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c > > index 520d34d..f2fd80f 100644 > > --- a/crypto/tlscredsx509.c > > +++ b/crypto/tlscredsx509.c > > @@ -615,7 +615,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 > *creds, > > } > > > > if (cert != NULL && key != NULL) { > > -#if GNUTLS_VERSION_NUMBER >= 0x030111 > > +#if defined(GNUTLS_VERSION_NUMBER) && > GNUTLS_VERSION_NUMBER >= 0x030111 > > char *password = NULL; > > if (creds->passwordid) { > > password = > qcrypto_secret_lookup_as_utf8(creds->passwordid, > > Other places in this file still use LIBGNUTLS_VERSION_NUMBER, which is > good, because gnutls 3.x still defines LIBGNUTLS_VERSION_NUMBER for > back compat. > You mean using LIBGNUTLS_VERSION_NUMBER directly here? That's ok. > The tests/test-crypto-tlscredsx509.c file also needs a similar fix. > Sorry? Regards, -Gonglei > Regards, > Daniel > -- > |: http://berrange.com -o- > http://www.flickr.com/photos/dberrange/ :| > |: http://libvirt.org -o- > http://virt-manager.org :| > |: http://autobuild.org -o- > http://search.cpan.org/~danberr/ :| > |: http://entangle-photo.org -o- > http://live.gnome.org/gtk-vnc :| ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/2] crypto: fix building complaint 2016-09-05 10:50 ` Gonglei (Arei) @ 2016-09-05 11:04 ` Daniel P. Berrange 2016-09-05 12:02 ` Gonglei (Arei) 0 siblings, 1 reply; 14+ messages in thread From: Daniel P. Berrange @ 2016-09-05 11:04 UTC (permalink / raw) To: Gonglei (Arei); +Cc: qemu-devel On Mon, Sep 05, 2016 at 10:50:54AM +0000, Gonglei (Arei) wrote: > > > > -----Original Message----- > > From: Daniel P. Berrange [mailto:berrange@redhat.com] > > Sent: Monday, September 05, 2016 6:15 PM > > To: Gonglei (Arei) > > Cc: qemu-devel@nongnu.org; dmitry@daynix.com > > Subject: Re: [PATCH v2 1/2] crypto: fix building complaint > > > > On Tue, Aug 30, 2016 at 12:10:52PM +0800, Gonglei wrote: > > > gnutls commit 846753877d renamed LIBGNUTLS_VERSION_NUMBER to > > GNUTLS_VERSION_NUMBER. > > > If using gnutls before that verion, we'll get the below warning: > > > crypto/tlscredsx509.c:618:5: warning: "GNUTLS_VERSION_NUMBER" is not > > defined > > > > > > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > > > --- > > > crypto/tlscredsx509.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c > > > index 520d34d..f2fd80f 100644 > > > --- a/crypto/tlscredsx509.c > > > +++ b/crypto/tlscredsx509.c > > > @@ -615,7 +615,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 > > *creds, > > > } > > > > > > if (cert != NULL && key != NULL) { > > > -#if GNUTLS_VERSION_NUMBER >= 0x030111 > > > +#if defined(GNUTLS_VERSION_NUMBER) && > > GNUTLS_VERSION_NUMBER >= 0x030111 > > > char *password = NULL; > > > if (creds->passwordid) { > > > password = > > qcrypto_secret_lookup_as_utf8(creds->passwordid, > > > > Other places in this file still use LIBGNUTLS_VERSION_NUMBER, which is > > good, because gnutls 3.x still defines LIBGNUTLS_VERSION_NUMBER for > > back compat. > > > You mean using LIBGNUTLS_VERSION_NUMBER directly here? That's ok. Yes, > > The tests/test-crypto-tlscredsx509.c file also needs a similar fix. > > > Sorry? It also uses the GNUTLS_VERSION_NUMBER constant instead of LIBGNUTLS_VERSION_NUMBER Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/2] crypto: fix building complaint 2016-09-05 11:04 ` Daniel P. Berrange @ 2016-09-05 12:02 ` Gonglei (Arei) 2016-09-05 12:07 ` Daniel P. Berrange 0 siblings, 1 reply; 14+ messages in thread From: Gonglei (Arei) @ 2016-09-05 12:02 UTC (permalink / raw) To: Daniel P. Berrange; +Cc: qemu-devel > -----Original Message----- > From: Daniel P. Berrange [mailto:berrange@redhat.com] > Sent: Monday, September 05, 2016 7:04 PM > To: Gonglei (Arei) > Cc: qemu-devel@nongnu.org > Subject: Re: [PATCH v2 1/2] crypto: fix building complaint > > On Mon, Sep 05, 2016 at 10:50:54AM +0000, Gonglei (Arei) wrote: > > > > > > > -----Original Message----- > > > From: Daniel P. Berrange [mailto:berrange@redhat.com] > > > Sent: Monday, September 05, 2016 6:15 PM > > > To: Gonglei (Arei) > > > Cc: qemu-devel@nongnu.org; dmitry@daynix.com > > > Subject: Re: [PATCH v2 1/2] crypto: fix building complaint > > > > > > On Tue, Aug 30, 2016 at 12:10:52PM +0800, Gonglei wrote: > > > > gnutls commit 846753877d renamed LIBGNUTLS_VERSION_NUMBER to > > > GNUTLS_VERSION_NUMBER. > > > > If using gnutls before that verion, we'll get the below warning: > > > > crypto/tlscredsx509.c:618:5: warning: "GNUTLS_VERSION_NUMBER" is > not > > > defined > > > > > > > > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > > > > --- > > > > crypto/tlscredsx509.c | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c > > > > index 520d34d..f2fd80f 100644 > > > > --- a/crypto/tlscredsx509.c > > > > +++ b/crypto/tlscredsx509.c > > > > @@ -615,7 +615,7 @@ > qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 > > > *creds, > > > > } > > > > > > > > if (cert != NULL && key != NULL) { > > > > -#if GNUTLS_VERSION_NUMBER >= 0x030111 > > > > +#if defined(GNUTLS_VERSION_NUMBER) && > > > GNUTLS_VERSION_NUMBER >= 0x030111 > > > > char *password = NULL; > > > > if (creds->passwordid) { > > > > password = > > > qcrypto_secret_lookup_as_utf8(creds->passwordid, > > > > > > Other places in this file still use LIBGNUTLS_VERSION_NUMBER, which is > > > good, because gnutls 3.x still defines LIBGNUTLS_VERSION_NUMBER for > > > back compat. > > > > > You mean using LIBGNUTLS_VERSION_NUMBER directly here? That's ok. > > Yes, > > > > The tests/test-crypto-tlscredsx509.c file also needs a similar fix. > > > > > Sorry? > > It also uses the GNUTLS_VERSION_NUMBER constant instead of > LIBGNUTLS_VERSION_NUMBER > What about the below patch? diff --git a/crypto/init.c b/crypto/init.c index 1e564d9..16e099b 100644 --- a/crypto/init.c +++ b/crypto/init.c @@ -59,8 +59,7 @@ #if (defined(CONFIG_GCRYPT) && \ (!defined(CONFIG_GNUTLS) || \ - !defined(GNUTLS_VERSION_NUMBER) || \ - (GNUTLS_VERSION_NUMBER < 0x020c00)) && \ + (LIBGNUTLS_VERSION_NUMBER < 0x020c00)) && \ (!defined(GCRYPT_VERSION_NUMBER) || \ (GCRYPT_VERSION_NUMBER < 0x010600))) #define QCRYPTO_INIT_GCRYPT_THREADS diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c index 520d34d..50eb54f 100644 --- a/crypto/tlscredsx509.c +++ b/crypto/tlscredsx509.c @@ -615,7 +615,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, } if (cert != NULL && key != NULL) { -#if GNUTLS_VERSION_NUMBER >= 0x030111 +#if LIBGNUTLS_VERSION_NUMBER >= 0x030111 char *password = NULL; if (creds->passwordid) { password = qcrypto_secret_lookup_as_utf8(creds->passwordid, @@ -630,7 +630,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, password, 0); g_free(password); -#else /* GNUTLS_VERSION_NUMBER < 0x030111 */ +#else /* LIBGNUTLS_VERSION_NUMBER < 0x030111 */ if (creds->passwordid) { error_setg(errp, "PKCS8 decryption requires GNUTLS >= 3.1.11"); goto cleanup; @@ -638,7 +638,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, ret = gnutls_certificate_set_x509_key_file(creds->data, cert, key, GNUTLS_X509_FMT_PEM); -#endif /* GNUTLS_VERSION_NUMBER < 0x030111 */ +#endif if (ret < 0) { error_setg(errp, "Cannot load certificate '%s' & key '%s': %s", cert, key, gnutls_strerror(ret)); diff --git a/tests/crypto-tls-x509-helpers.h b/tests/crypto-tls-x509-helpers.h index 356b49c..a8faa92 100644 --- a/tests/crypto-tls-x509-helpers.h +++ b/tests/crypto-tls-x509-helpers.h @@ -26,7 +26,6 @@ #if !(defined WIN32) && \ defined(CONFIG_TASN1) && \ - defined(LIBGNUTLS_VERSION_NUMBER) && \ (LIBGNUTLS_VERSION_NUMBER >= 0x020600) # define QCRYPTO_HAVE_TLS_TEST_SUPPORT #endif > Regards, > Daniel > -- > |: http://berrange.com -o- > http://www.flickr.com/photos/dberrange/ :| > |: http://libvirt.org -o- > http://virt-manager.org :| > |: http://autobuild.org -o- > http://search.cpan.org/~danberr/ :| > |: http://entangle-photo.org -o- > http://live.gnome.org/gtk-vnc :| ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 1/2] crypto: fix building complaint 2016-09-05 12:02 ` Gonglei (Arei) @ 2016-09-05 12:07 ` Daniel P. Berrange 0 siblings, 0 replies; 14+ messages in thread From: Daniel P. Berrange @ 2016-09-05 12:07 UTC (permalink / raw) To: Gonglei (Arei); +Cc: qemu-devel On Mon, Sep 05, 2016 at 12:02:42PM +0000, Gonglei (Arei) wrote: > > > -----Original Message----- > > From: Daniel P. Berrange [mailto:berrange@redhat.com] > > Sent: Monday, September 05, 2016 7:04 PM > > To: Gonglei (Arei) > > Cc: qemu-devel@nongnu.org > > Subject: Re: [PATCH v2 1/2] crypto: fix building complaint > > > > On Mon, Sep 05, 2016 at 10:50:54AM +0000, Gonglei (Arei) wrote: > > > > > > > > > > -----Original Message----- > > > > From: Daniel P. Berrange [mailto:berrange@redhat.com] > > > > Sent: Monday, September 05, 2016 6:15 PM > > > > To: Gonglei (Arei) > > > > Cc: qemu-devel@nongnu.org; dmitry@daynix.com > > > > Subject: Re: [PATCH v2 1/2] crypto: fix building complaint > > > > > > > > On Tue, Aug 30, 2016 at 12:10:52PM +0800, Gonglei wrote: > > > > > gnutls commit 846753877d renamed LIBGNUTLS_VERSION_NUMBER to > > > > GNUTLS_VERSION_NUMBER. > > > > > If using gnutls before that verion, we'll get the below warning: > > > > > crypto/tlscredsx509.c:618:5: warning: "GNUTLS_VERSION_NUMBER" is > > not > > > > defined > > > > > > > > > > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > > > > > --- > > > > > crypto/tlscredsx509.c | 2 +- > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c > > > > > index 520d34d..f2fd80f 100644 > > > > > --- a/crypto/tlscredsx509.c > > > > > +++ b/crypto/tlscredsx509.c > > > > > @@ -615,7 +615,7 @@ > > qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 > > > > *creds, > > > > > } > > > > > > > > > > if (cert != NULL && key != NULL) { > > > > > -#if GNUTLS_VERSION_NUMBER >= 0x030111 > > > > > +#if defined(GNUTLS_VERSION_NUMBER) && > > > > GNUTLS_VERSION_NUMBER >= 0x030111 > > > > > char *password = NULL; > > > > > if (creds->passwordid) { > > > > > password = > > > > qcrypto_secret_lookup_as_utf8(creds->passwordid, > > > > > > > > Other places in this file still use LIBGNUTLS_VERSION_NUMBER, which is > > > > good, because gnutls 3.x still defines LIBGNUTLS_VERSION_NUMBER for > > > > back compat. > > > > > > > You mean using LIBGNUTLS_VERSION_NUMBER directly here? That's ok. > > > > Yes, > > > > > > The tests/test-crypto-tlscredsx509.c file also needs a similar fix. > > > > > > > Sorry? > > > > It also uses the GNUTLS_VERSION_NUMBER constant instead of > > LIBGNUTLS_VERSION_NUMBER > > > > What about the below patch? > > diff --git a/crypto/init.c b/crypto/init.c > index 1e564d9..16e099b 100644 > --- a/crypto/init.c > +++ b/crypto/init.c > @@ -59,8 +59,7 @@ > > #if (defined(CONFIG_GCRYPT) && \ > (!defined(CONFIG_GNUTLS) || \ > - !defined(GNUTLS_VERSION_NUMBER) || \ > - (GNUTLS_VERSION_NUMBER < 0x020c00)) && \ > + (LIBGNUTLS_VERSION_NUMBER < 0x020c00)) && \ > (!defined(GCRYPT_VERSION_NUMBER) || \ > (GCRYPT_VERSION_NUMBER < 0x010600))) > #define QCRYPTO_INIT_GCRYPT_THREADS > diff --git a/crypto/tlscredsx509.c b/crypto/tlscredsx509.c > index 520d34d..50eb54f 100644 > --- a/crypto/tlscredsx509.c > +++ b/crypto/tlscredsx509.c > @@ -615,7 +615,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, > } > > if (cert != NULL && key != NULL) { > -#if GNUTLS_VERSION_NUMBER >= 0x030111 > +#if LIBGNUTLS_VERSION_NUMBER >= 0x030111 > char *password = NULL; > if (creds->passwordid) { > password = qcrypto_secret_lookup_as_utf8(creds->passwordid, > @@ -630,7 +630,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, > password, > 0); > g_free(password); > -#else /* GNUTLS_VERSION_NUMBER < 0x030111 */ > +#else /* LIBGNUTLS_VERSION_NUMBER < 0x030111 */ > if (creds->passwordid) { > error_setg(errp, "PKCS8 decryption requires GNUTLS >= 3.1.11"); > goto cleanup; > @@ -638,7 +638,7 @@ qcrypto_tls_creds_x509_load(QCryptoTLSCredsX509 *creds, > ret = gnutls_certificate_set_x509_key_file(creds->data, > cert, key, > GNUTLS_X509_FMT_PEM); > -#endif /* GNUTLS_VERSION_NUMBER < 0x030111 */ > +#endif > if (ret < 0) { > error_setg(errp, "Cannot load certificate '%s' & key '%s': %s", > cert, key, gnutls_strerror(ret)); > diff --git a/tests/crypto-tls-x509-helpers.h b/tests/crypto-tls-x509-helpers.h > index 356b49c..a8faa92 100644 > --- a/tests/crypto-tls-x509-helpers.h > +++ b/tests/crypto-tls-x509-helpers.h > @@ -26,7 +26,6 @@ > > #if !(defined WIN32) && \ > defined(CONFIG_TASN1) && \ > - defined(LIBGNUTLS_VERSION_NUMBER) && \ > (LIBGNUTLS_VERSION_NUMBER >= 0x020600) > # define QCRYPTO_HAVE_TLS_TEST_SUPPORT > #endif Yep, that looks reasonable. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| ^ permalink raw reply [flat|nested] 14+ messages in thread
* [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint 2016-08-30 4:10 [Qemu-devel] [PATCH v2 0/2] fix building complaint Gonglei 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 1/2] crypto: " Gonglei @ 2016-08-30 4:10 ` Gonglei 2016-08-31 12:58 ` Dmitry Fleytman 1 sibling, 1 reply; 14+ messages in thread From: Gonglei @ 2016-08-30 4:10 UTC (permalink / raw) To: qemu-devel; +Cc: berrange, dmitry, Gonglei hw/net/e1000e_core.c:56: warning: e1000e_set_interrupt_cause declared inline after being called hw/net/e1000e_core.c:56: warning: previous declaration of e1000e_set_interrupt_cause was here Signed-off-by: Gonglei <arei.gonglei@huawei.com> --- hw/net/e1000e_core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c index badb1fe..825e169 100644 --- a/hw/net/e1000e_core.c +++ b/hw/net/e1000e_core.c @@ -2168,7 +2168,7 @@ e1000e_update_interrupt_state(E1000ECore *core) } } -static inline void +static void e1000e_set_interrupt_cause(E1000ECore *core, uint32_t val) { trace_e1000e_irq_set_cause_entry(val, core->mac[ICR]); -- 1.7.12.4 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint Gonglei @ 2016-08-31 12:58 ` Dmitry Fleytman 2016-09-09 7:15 ` Gonglei (Arei) 0 siblings, 1 reply; 14+ messages in thread From: Dmitry Fleytman @ 2016-08-31 12:58 UTC (permalink / raw) To: Gonglei; +Cc: qemu-devel, berrange Reviewed-by: Dmitry Fleytman <dmitry@daynix.com> > On 30 Aug 2016, at 07:10 AM, Gonglei <arei.gonglei@huawei.com> wrote: > > hw/net/e1000e_core.c:56: warning: e1000e_set_interrupt_cause declared inline after being called > hw/net/e1000e_core.c:56: warning: previous declaration of e1000e_set_interrupt_cause was here > > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > --- > hw/net/e1000e_core.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c > index badb1fe..825e169 100644 > --- a/hw/net/e1000e_core.c > +++ b/hw/net/e1000e_core.c > @@ -2168,7 +2168,7 @@ e1000e_update_interrupt_state(E1000ECore *core) > } > } > > -static inline void > +static void > e1000e_set_interrupt_cause(E1000ECore *core, uint32_t val) > { > trace_e1000e_irq_set_cause_entry(val, core->mac[ICR]); > -- > 1.7.12.4 > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint 2016-08-31 12:58 ` Dmitry Fleytman @ 2016-09-09 7:15 ` Gonglei (Arei) 2016-09-13 6:00 ` Dmitry Fleytman 0 siblings, 1 reply; 14+ messages in thread From: Gonglei (Arei) @ 2016-09-09 7:15 UTC (permalink / raw) To: Dmitry Fleytman, jasowang; +Cc: qemu-devel, berrange Who can pick up this patch? Dmitry or Jason? Thanks! Regards, -Gonglei > -----Original Message----- > From: Dmitry Fleytman [mailto:dmitry@daynix.com] > Sent: Wednesday, August 31, 2016 8:59 PM > To: Gonglei (Arei) > Cc: qemu-devel@nongnu.org; berrange@redhat.com > Subject: Re: [PATCH v2 2/2] e1000: fix buliding complaint > > Reviewed-by: Dmitry Fleytman <dmitry@daynix.com> > > > On 30 Aug 2016, at 07:10 AM, Gonglei <arei.gonglei@huawei.com> wrote: > > > > hw/net/e1000e_core.c:56: warning: e1000e_set_interrupt_cause declared > inline after being called > > hw/net/e1000e_core.c:56: warning: previous declaration of > e1000e_set_interrupt_cause was here > > > > Signed-off-by: Gonglei <arei.gonglei@huawei.com> > > --- > > hw/net/e1000e_core.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c > > index badb1fe..825e169 100644 > > --- a/hw/net/e1000e_core.c > > +++ b/hw/net/e1000e_core.c > > @@ -2168,7 +2168,7 @@ e1000e_update_interrupt_state(E1000ECore > *core) > > } > > } > > > > -static inline void > > +static void > > e1000e_set_interrupt_cause(E1000ECore *core, uint32_t val) > > { > > trace_e1000e_irq_set_cause_entry(val, core->mac[ICR]); > > -- > > 1.7.12.4 > > > > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint 2016-09-09 7:15 ` Gonglei (Arei) @ 2016-09-13 6:00 ` Dmitry Fleytman 2016-09-22 6:30 ` Jason Wang 0 siblings, 1 reply; 14+ messages in thread From: Dmitry Fleytman @ 2016-09-13 6:00 UTC (permalink / raw) To: jasowang; +Cc: qemu-devel, berrange, Gonglei (Arei) > On 9 Sep 2016, at 10:15 AM, Gonglei (Arei) <arei.gonglei@huawei.com> wrote: > > Who can pick up this patch? Dmitry or Jason? Thanks! Jason, would you please? > > > Regards, > -Gonglei > > >> -----Original Message----- >> From: Dmitry Fleytman [mailto:dmitry@daynix.com] >> Sent: Wednesday, August 31, 2016 8:59 PM >> To: Gonglei (Arei) >> Cc: qemu-devel@nongnu.org; berrange@redhat.com >> Subject: Re: [PATCH v2 2/2] e1000: fix buliding complaint >> >> Reviewed-by: Dmitry Fleytman <dmitry@daynix.com> >> >>> On 30 Aug 2016, at 07:10 AM, Gonglei <arei.gonglei@huawei.com> wrote: >>> >>> hw/net/e1000e_core.c:56: warning: e1000e_set_interrupt_cause declared >> inline after being called >>> hw/net/e1000e_core.c:56: warning: previous declaration of >> e1000e_set_interrupt_cause was here >>> >>> Signed-off-by: Gonglei <arei.gonglei@huawei.com> >>> --- >>> hw/net/e1000e_core.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c >>> index badb1fe..825e169 100644 >>> --- a/hw/net/e1000e_core.c >>> +++ b/hw/net/e1000e_core.c >>> @@ -2168,7 +2168,7 @@ e1000e_update_interrupt_state(E1000ECore >> *core) >>> } >>> } >>> >>> -static inline void >>> +static void >>> e1000e_set_interrupt_cause(E1000ECore *core, uint32_t val) >>> { >>> trace_e1000e_irq_set_cause_entry(val, core->mac[ICR]); >>> -- >>> 1.7.12.4 >>> >>> > ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint 2016-09-13 6:00 ` Dmitry Fleytman @ 2016-09-22 6:30 ` Jason Wang 0 siblings, 0 replies; 14+ messages in thread From: Jason Wang @ 2016-09-22 6:30 UTC (permalink / raw) To: Dmitry Fleytman; +Cc: qemu-devel, berrange, Gonglei (Arei) On 2016年09月13日 14:00, Dmitry Fleytman wrote: >> >On 9 Sep 2016, at 10:15 AM, Gonglei (Arei)<arei.gonglei@huawei.com> wrote: >> > >> >Who can pick up this patch? Dmitry or Jason? Thanks! > Jason, would you please? > Picked in my tree. Thanks ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2016-09-22 6:30 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-08-30 4:10 [Qemu-devel] [PATCH v2 0/2] fix building complaint Gonglei 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 1/2] crypto: " Gonglei 2016-08-30 14:14 ` Eric Blake 2016-08-30 14:21 ` Peter Maydell 2016-09-05 10:14 ` Daniel P. Berrange 2016-09-05 10:50 ` Gonglei (Arei) 2016-09-05 11:04 ` Daniel P. Berrange 2016-09-05 12:02 ` Gonglei (Arei) 2016-09-05 12:07 ` Daniel P. Berrange 2016-08-30 4:10 ` [Qemu-devel] [PATCH v2 2/2] e1000: fix buliding complaint Gonglei 2016-08-31 12:58 ` Dmitry Fleytman 2016-09-09 7:15 ` Gonglei (Arei) 2016-09-13 6:00 ` Dmitry Fleytman 2016-09-22 6:30 ` Jason Wang
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.