From: Stefan Hajnoczi <stefanha@gmail.com>
To: Markus Armbruster <armbru@redhat.com>
Cc: qemu-devel@nongnu.org, stefanha@redhat.com
Subject: Re: [PULL 00/28] QAPI patches patches for 2022-10-25
Date: Tue, 25 Oct 2022 16:07:27 -0400 [thread overview]
Message-ID: <CAJSP0QVkhzTw6_n+UgEVYDBhMqFuve1oVT2SnorwtMarWHgDcA@mail.gmail.com> (raw)
In-Reply-To: <20221025080541.271547-1-armbru@redhat.com>
The following CI failures were reported. I think they may be related
to this pull request:
cc -m64 -mcx16 -Ilibblock.fa.p -I. -I.. -Iqapi -Itrace -Iui
-Iui/shader -Iblock -I/usr/include/p11-kit-1
-I/usr/include/x86_64-linux-gnu -I/usr/include/uuid
-I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
-fdiagnostics-color=auto -Wall -Winvalid-pch -Werror -std=gnu11 -O2 -g
-isystem /builds/qemu-project/qemu/linux-headers -isystem
linux-headers -iquote . -iquote /builds/qemu-project/qemu -iquote
/builds/qemu-project/qemu/include -pthread -U_FORTIFY_SOURCE
-D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -D_LARGEFILE_SOURCE
-Wstrict-prototypes -Wredundant-decls -Wundef -Wwrite-strings
-Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv
-Wold-style-declaration -Wold-style-definition -Wtype-limits
-Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers
-Wempty-body -Wnested-externs -Wendif-labels -Wexpansion-to-defined
-Wimplicit-fallthrough=2 -Wno-missing-include-dirs
-Wno-shift-negative-value -Wno-psabi -fstack-protector-strong -fPIE
-D_FILE_OFFSET_BITS=64 -D__USE_FILE_OFFSET64 -D__USE_LARGEFILE64
-DUSE_POSIX_ACLS=1 -MD -MQ libblock.fa.p/block_rbd.c.o -MF
libblock.fa.p/block_rbd.c.o.d -o libblock.fa.p/block_rbd.c.o -c
../block/rbd.c
../block/rbd.c: In function ‘qemu_rbd_do_create’:
../block/rbd.c:545:15: error: ‘BlockdevCreateOptionsRbd’ {aka ‘struct
BlockdevCreateOptionsRbd’} has no member named ‘has_encrypt’; did you
mean ‘encrypt’?
545 | if (opts->has_encrypt) {
| ^~~~~~~~~~~
| encrypt
https://gitlab.com/qemu-project/qemu/-/jobs/3225215747
I think this will solve it:
diff --cc block/rbd.c
index f826410f40,d809b8e0a3..3aa6aae0e0
--- a/block/rbd.c
+++ b/block/rbd.c
@@@ -542,7 -542,7 +542,7 @@@ static int qemu_rbd_do_create(BlockdevC
}
#ifndef LIBRBD_SUPPORTS_ENCRYPTION
-- if (opts->has_encrypt) {
++ if (opts->encrypt) {
error_setg(errp, "RBD library does not support image encryption");
return -ENOTSUP;
}
cc -m64 -mcx16 -Ilibqemuutil.a.p -I. -I.. -Iqapi -Itrace -Iui/shader
-I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include
-fdiagnostics-color=auto -Wall -Winvalid-pch -Werror -std=gnu11 -O2 -g
-isystem /builds/qemu-project/qemu/linux-headers -isystem
linux-headers -iquote . -iquote /builds/qemu-project/qemu -iquote
/builds/qemu-project/qemu/include -iquote
/builds/qemu-project/qemu/tcg/i386 -pthread -U_FORTIFY_SOURCE
-D_FORTIFY_SOURCE=2 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64
-D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wundef
-Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common
-fwrapv -Wold-style-declaration -Wold-style-definition -Wtype-limits
-Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers
-Wempty-body -Wnested-externs -Wendif-labels -Wexpansion-to-defined
-Wimplicit-fallthrough=2 -Wno-missing-include-dirs
-Wno-shift-negative-value -Wno-psabi -fstack-protector-strong -fPIE
-MD -MQ libqemuutil.a.p/stubs_qdev.c.o -MF
libqemuutil.a.p/stubs_qdev.c.o.d -o libqemuutil.a.p/stubs_qdev.c.o -c
../stubs/qdev.c
../stubs/qdev.c:18:6: error: conflicting types for
'qapi_event_send_device_deleted'
18 | void qapi_event_send_device_deleted(bool has_device,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from ../stubs/qdev.c:16:
./qapi/qapi-events-qdev.h:20:6: note: previous declaration of
'qapi_event_send_device_deleted' was here
20 | void qapi_event_send_device_deleted(const char *device, const char *path);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../stubs/qdev.c:25:6: error: conflicting types for
'qapi_event_send_device_unplug_guest_error'
25 | void qapi_event_send_device_unplug_guest_error(bool has_device,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from ../stubs/qdev.c:16:
./qapi/qapi-events-qdev.h:22:6: note: previous declaration of
'qapi_event_send_device_unplug_guest_error' was here
22 | void qapi_event_send_device_unplug_guest_error(const char
*device, const char *path);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
https://gitlab.com/qemu-project/qemu/-/jobs/3225215726
next prev parent reply other threads:[~2022-10-25 20:08 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-25 8:05 [PULL 00/28] QAPI patches patches for 2022-10-25 Markus Armbruster
2022-10-25 8:05 ` [PULL 01/28] docs/devel/qapi-code-gen: Update example to match current code Markus Armbruster
2022-10-25 8:05 ` [PULL 02/28] qapi: Tidy up whitespace in generated code Markus Armbruster
2022-10-25 8:05 ` [PULL 03/28] docs/devel/qapi-code-gen: Extend example for next commit's change Markus Armbruster
2022-10-25 8:05 ` [PULL 04/28] qapi: Start to elide redundant has_FOO in generated C Markus Armbruster
2022-10-25 8:05 ` [PULL 05/28] qapi tests: Elide " Markus Armbruster
2022-10-25 8:05 ` [PULL 06/28] qapi acpi: " Markus Armbruster
2022-10-25 8:05 ` [PULL 07/28] qapi audio: " Markus Armbruster
2022-10-25 8:05 ` [PULL 08/28] qapi block: " Markus Armbruster
2022-10-25 8:05 ` [PULL 09/28] qapi chardev: " Markus Armbruster
2022-10-25 8:05 ` [PULL 10/28] qapi crypto: " Markus Armbruster
2022-10-25 8:05 ` [PULL 11/28] qapi dump: " Markus Armbruster
2022-10-25 8:05 ` [PULL 12/28] qapi job: " Markus Armbruster
2022-10-25 8:05 ` [PULL 13/28] qapi machine: " Markus Armbruster
2022-10-25 8:05 ` [PULL 14/28] qapi migration: " Markus Armbruster
2022-10-25 8:05 ` [PULL 15/28] qapi misc: " Markus Armbruster
2022-10-25 8:05 ` [PULL 16/28] qapi net: " Markus Armbruster
2022-10-25 8:05 ` [PULL 17/28] qapi pci: " Markus Armbruster
2022-10-25 8:05 ` [PULL 18/28] qapi qdev qom: " Markus Armbruster
2022-10-25 8:05 ` [PULL 19/28] qapi replay: " Markus Armbruster
2022-10-25 8:05 ` [PULL 20/28] qapi rocker: " Markus Armbruster
2022-10-25 8:05 ` [PULL 21/28] qapi run-state: " Markus Armbruster
2022-10-25 8:05 ` [PULL 22/28] qapi stats: " Markus Armbruster
2022-10-25 8:05 ` [PULL 23/28] qapi tpm: " Markus Armbruster
2022-10-25 8:05 ` [PULL 24/28] qapi transaction: " Markus Armbruster
2022-10-25 8:05 ` [PULL 25/28] qapi ui: " Markus Armbruster
2022-10-25 8:05 ` [PULL 26/28] qapi virtio: " Markus Armbruster
2022-10-25 8:05 ` [PULL 27/28] qapi qga: " Markus Armbruster
2022-10-25 8:05 ` [PULL 28/28] qapi: Drop temporary logic to support conversion step by step Markus Armbruster
2022-10-25 15:40 ` [PULL 00/28] QAPI patches patches for 2022-10-25 Stefan Hajnoczi
2022-10-25 19:00 ` Markus Armbruster
2022-10-25 20:07 ` Stefan Hajnoczi [this message]
2022-10-26 18:43 ` Markus Armbruster
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAJSP0QVkhzTw6_n+UgEVYDBhMqFuve1oVT2SnorwtMarWHgDcA@mail.gmail.com \
--to=stefanha@gmail.com \
--cc=armbru@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).