1. Patchew
  2. QEMU
  3. View series

[Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile

Max Reitz posted 1 patch 7 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170308181833.32110-1-mreitz@redhat.com
Test checkpatch passed
Test docker passed
block/archipelago.c | 1 +
1 file changed, 1 insertion(+)
[Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile
Posted by Max Reitz 7 years, 1 month ago 
In order to use error_setg() and similar functions, we need to include
qapi/error.h.

Signed-off-by: Max Reitz <mreitz@redhat.com>
---
 block/archipelago.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/block/archipelago.c b/block/archipelago.c
index 2449cfc702..01c4ff2f7d 100644
--- a/block/archipelago.c
+++ b/block/archipelago.c
@@ -55,6 +55,7 @@
 #include "block/block_int.h"
 #include "qemu/error-report.h"
 #include "qemu/thread.h"
+#include "qapi/error.h"
 #include "qapi/qmp/qint.h"
 #include "qapi/qmp/qstring.h"
 #include "qapi/qmp/qjson.h"
-- 
2.12.0
Re: [Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile
Posted by Eric Blake 7 years, 1 month ago 
On 03/08/2017 12:18 PM, Max Reitz wrote:
> In order to use error_setg() and similar functions, we need to include
> qapi/error.h.
> 
> Signed-off-by: Max Reitz <mreitz@redhat.com>
> ---
>  block/archipelago.c | 1 +
>  1 file changed, 1 insertion(+)

Reviewed-by: Eric Blake <eblake@redhat.com>

Does this mean our automated buildbots aren't building archipelago by
default?

> 
> diff --git a/block/archipelago.c b/block/archipelago.c
> index 2449cfc702..01c4ff2f7d 100644
> --- a/block/archipelago.c
> +++ b/block/archipelago.c
> @@ -55,6 +55,7 @@
>  #include "block/block_int.h"
>  #include "qemu/error-report.h"
>  #include "qemu/thread.h"
> +#include "qapi/error.h"
>  #include "qapi/qmp/qint.h"
>  #include "qapi/qmp/qstring.h"
>  #include "qapi/qmp/qjson.h"
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Re: [Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile
Posted by Max Reitz 7 years, 1 month ago 
On 08.03.2017 19:25, Eric Blake wrote:
> On 03/08/2017 12:18 PM, Max Reitz wrote:
>> In order to use error_setg() and similar functions, we need to include
>> qapi/error.h.
>>
>> Signed-off-by: Max Reitz <mreitz@redhat.com>
>> ---
>>  block/archipelago.c | 1 +
>>  1 file changed, 1 insertion(+)
> 
> Reviewed-by: Eric Blake <eblake@redhat.com>
> 
> Does this mean our automated buildbots aren't building archipelago by
> default?

Well, at least patchew didn't catch that I definitely broke archipelago
in my bdrv_truncate() errp series...

[CC-ing Fam]

Max
Re: [Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile
Posted by Eric Blake 7 years, 1 month ago 
On 03/08/2017 12:25 PM, Eric Blake wrote:
> On 03/08/2017 12:18 PM, Max Reitz wrote:
>> In order to use error_setg() and similar functions, we need to include
>> qapi/error.h.
>>
>> Signed-off-by: Max Reitz <mreitz@redhat.com>
>> ---
>>  block/archipelago.c | 1 +
>>  1 file changed, 1 insertion(+)
> 
> Reviewed-by: Eric Blake <eblake@redhat.com>
> 
> Does this mean our automated buildbots aren't building archipelago by
> default?
> 

Oh, I meant to add:

Should probably mention that it was commit da34e65 that introduced the
problem (if I'm right?).  Does that mean we have not been compiling
archipelago.c since Mar 2016?

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Re: [Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile
Posted by Max Reitz 7 years, 1 month ago 
On 08.03.2017 19:28, Eric Blake wrote:
> On 03/08/2017 12:25 PM, Eric Blake wrote:
>> On 03/08/2017 12:18 PM, Max Reitz wrote:
>>> In order to use error_setg() and similar functions, we need to include
>>> qapi/error.h.
>>>
>>> Signed-off-by: Max Reitz <mreitz@redhat.com>
>>> ---
>>>  block/archipelago.c | 1 +
>>>  1 file changed, 1 insertion(+)
>>
>> Reviewed-by: Eric Blake <eblake@redhat.com>
>>
>> Does this mean our automated buildbots aren't building archipelago by
>> default?
>>
> 
> Oh, I meant to add:
> 
> Should probably mention that it was commit da34e65 that introduced the
> problem (if I'm right?).  Does that mean we have not been compiling
> archipelago.c since Mar 2016?

At least here on my machine it does indeed compile before that commit
and fails afterwards. So, yes, maybe we actually have not. :-)

Max
Re: [Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile
Posted by Kevin Wolf 7 years, 1 month ago 
Am 08.03.2017 um 19:32 hat Max Reitz geschrieben:
> On 08.03.2017 19:28, Eric Blake wrote:
> > On 03/08/2017 12:25 PM, Eric Blake wrote:
> >> On 03/08/2017 12:18 PM, Max Reitz wrote:
> >>> In order to use error_setg() and similar functions, we need to include
> >>> qapi/error.h.
> >>>
> >>> Signed-off-by: Max Reitz <mreitz@redhat.com>
> >>> ---
> >>>  block/archipelago.c | 1 +
> >>>  1 file changed, 1 insertion(+)
> >>
> >> Reviewed-by: Eric Blake <eblake@redhat.com>
> >>
> >> Does this mean our automated buildbots aren't building archipelago by
> >> default?
> >>
> > 
> > Oh, I meant to add:
> > 
> > Should probably mention that it was commit da34e65 that introduced the
> > problem (if I'm right?).  Does that mean we have not been compiling
> > archipelago.c since Mar 2016?
> 
> At least here on my machine it does indeed compile before that commit
> and fails afterwards. So, yes, maybe we actually have not. :-)

In that case, maybe we should rather remove the driver because very
obviously nobody is interested in it, not even the maintainer. Keeping
things around that nobody tests (and consequently, nobody uses) is just
a useless maintenance burden.

Kevin
Re: [Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile
Posted by Eric Blake 7 years, 1 month ago 
On 03/08/2017 12:46 PM, Kevin Wolf wrote:
>>> Oh, I meant to add:
>>>
>>> Should probably mention that it was commit da34e65 that introduced the
>>> problem (if I'm right?).  Does that mean we have not been compiling
>>> archipelago.c since Mar 2016?
>>
>> At least here on my machine it does indeed compile before that commit
>> and fails afterwards. So, yes, maybe we actually have not. :-)
> 
> In that case, maybe we should rather remove the driver because very
> obviously nobody is interested in it, not even the maintainer. Keeping
> things around that nobody tests (and consequently, nobody uses) is just
> a useless maintenance burden.

We've released 2.6, 2.7, and 2.8 with no complaints about failure to
compile. Sounds like a good candidate for removal in 2.9.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Re: [Qemu-devel] [PATCH for-2.9] block/archipelago: Make it compile
Posted by Philippe Mathieu-Daudé 7 years, 1 month ago 
On 03/08/2017 03:25 PM, Eric Blake wrote:
> On 03/08/2017 12:18 PM, Max Reitz wrote:
>> In order to use error_setg() and similar functions, we need to include
>> qapi/error.h.
>>
>> Signed-off-by: Max Reitz <mreitz@redhat.com>
>> ---
>>  block/archipelago.c | 1 +
>>  1 file changed, 1 insertion(+)
>
> Reviewed-by: Eric Blake <eblake@redhat.com>

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

>
> Does this mean our automated buildbots aren't building archipelago by
> default?
>
>>
>> diff --git a/block/archipelago.c b/block/archipelago.c
>> index 2449cfc702..01c4ff2f7d 100644
>> --- a/block/archipelago.c
>> +++ b/block/archipelago.c
>> @@ -55,6 +55,7 @@
>>  #include "block/block_int.h"
>>  #include "qemu/error-report.h"
>>  #include "qemu/thread.h"
>> +#include "qapi/error.h"
>>  #include "qapi/qmp/qint.h"
>>  #include "qapi/qmp/qstring.h"
>>  #include "qapi/qmp/qjson.h"
>>
>

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.