[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] util/meson.build: fix fdmon-io_uring build
From: |
Stefano Garzarella |
Subject: |
Re: [PATCH] util/meson.build: fix fdmon-io_uring build |
Date: |
Fri, 21 Aug 2020 18:50:34 +0200 |
On Fri, Aug 21, 2020 at 06:46:15PM +0200, Philippe Mathieu-Daudé wrote:
> On 8/21/20 6:23 PM, Stefano Garzarella wrote:
> > On Fri, Aug 21, 2020 at 06:12:45PM +0200, Philippe Mathieu-Daudé wrote:
> >> Hi Stefano,
> >>
> >> On 8/21/20 5:48 PM, Stefano Garzarella wrote:
> >>> libqemuutil.a build fails with this error:
> >>>
> >>> /usr/bin/ld: libqemuutil.a(util_fdmon-io_uring.c.o): in function
> >>> `get_sqe':
> >>> qemu/build/../util/fdmon-io_uring.c:83: undefined reference to
> >>> `io_uring_get_sqe'
> >>> /usr/bin/ld: qemu/build/../util/fdmon-io_uring.c:92: undefined
> >>> reference to `io_uring_submit'
> >>> /usr/bin/ld: qemu/build/../util/fdmon-io_uring.c:96: undefined
> >>> reference to `io_uring_get_sqe'
> >>> /usr/bin/ld: libqemuutil.a(util_fdmon-io_uring.c.o): in function
> >>> `fdmon_io_uring_wait':
> >>> qemu/build/../util/fdmon-io_uring.c:289: undefined reference to
> >>> `io_uring_submit_and_wait'
> >>> /usr/bin/ld: libqemuutil.a(util_fdmon-io_uring.c.o): in function
> >>> `fdmon_io_uring_setup':
> >>> qemu/build/../util/fdmon-io_uring.c:328: undefined reference to
> >>> `io_uring_queue_init'
> >>> /usr/bin/ld: libqemuutil.a(util_fdmon-io_uring.c.o): in function
> >>> `fdmon_io_uring_destroy':
> >>> qemu/build/../util/fdmon-io_uring.c:343: undefined reference to
> >>> `io_uring_queue_exit'
> >>> collect2: error: ld returned 1 exit status
> >>
> >> Can you add a gitlab job to reproduce this? (Or at least explain
> >> how to reproduce, so we add that job later). Thanks!
> >
> > I think the only prerequisite is to install 'liburing-devel' package
> > (Fedora 32) on the build system, and then fdmon-io_uring.c will be built.
>
> Oh easy then, simply add it to PACKAGES in
> tests/docker/dockerfiles/fedora.docker :)
Yeah, I just found it ;-)
I'll send a patch.
Thanks,
Stefano