[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
--enable-xen on gitlab CI? (was Re: [PATCH 09/36] qdev: Make qdev_get_prop_ptr() get Object* arg)
- To: Marc-André Lureau <marcandre.lureau@xxxxxxxxx>, Thomas Huth <thuth@xxxxxxxxxx>, Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>, Alex Bennée <alex.bennee@xxxxxxxxxx>
- From: Eduardo Habkost <ehabkost@xxxxxxxxxx>
- Date: Fri, 30 Oct 2020 07:35:16 -0400
- Authentication-results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=ehabkost@xxxxxxxxxx
- Cc: Wainer dos Santos Moschetta <wainersm@xxxxxxxxxx>, QEMU <qemu-devel@xxxxxxxxxx>, Matthew Rosato <mjrosato@xxxxxxxxxxxxx>, Paul Durrant <paul@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, "open list:Block layer core" <qemu-block@xxxxxxxxxx>, Stefan Berger <stefanb@xxxxxxxxxxxxxxxxxx>, David Hildenbrand <david@xxxxxxxxxx>, Markus Armbruster <armbru@xxxxxxxxxx>, Halil Pasic <pasic@xxxxxxxxxxxxx>, Christian Borntraeger <borntraeger@xxxxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx, Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>, Thomas Huth <thuth@xxxxxxxxxx>, Alex Williamson <alex.williamson@xxxxxxxxxx>, Paolo Bonzini <pbonzini@xxxxxxxxxx>, John Snow <jsnow@xxxxxxxxxx>, Richard Henderson <rth@xxxxxxxxxxx>, Kevin Wolf <kwolf@xxxxxxxxxx>, "Daniel P. Berrange" <berrange@xxxxxxxxxx>, Cornelia Huck <cohuck@xxxxxxxxxx>, Qemu-s390x list <qemu-s390x@xxxxxxxxxx>, Max Reitz <mreitz@xxxxxxxxxx>, Igor Mammedov <imammedo@xxxxxxxxxx>
- Delivery-date: Fri, 30 Oct 2020 11:35:35 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Fri, Oct 30, 2020 at 11:29:25AM +0400, Marc-André Lureau wrote:
> On Fri, Oct 30, 2020 at 2:07 AM Eduardo Habkost <ehabkost@xxxxxxxxxx> wrote:
>
> > Make the code more generic and not specific to TYPE_DEVICE.
> >
> > Signed-off-by: Eduardo Habkost <ehabkost@xxxxxxxxxx>
> >
>
> Nice cleanup!, but fails to build atm
>
> ../hw/block/xen-block.c:403:9: error: ‘dev’ undeclared (first use in this
> function); did you mean ‘vdev’?
> 403 | if (dev->realized) {
Thanks for catching it!
What is necessary to make sure we have a CONFIG_XEN=y job in
gitlab CI? Maybe just including xen-devel in some of the
container images is enough?
--
Eduardo
|