[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
 
    
     |