qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] virtio-vga: enable for i386
@ 2015-07-13 11:58 Gerd Hoffmann
  2015-09-11  7:30 ` Michael Tokarev
  0 siblings, 1 reply; 5+ messages in thread
From: Gerd Hoffmann @ 2015-07-13 11:58 UTC (permalink / raw)
  To: qemu-devel; +Cc: qemu-trivial, Paolo Bonzini, Gerd Hoffmann

Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-trivial@nongnu.org
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
---
 default-configs/i386-softmmu.mak | 1 +
 1 file changed, 1 insertion(+)

diff --git a/default-configs/i386-softmmu.mak b/default-configs/i386-softmmu.mak
index 48b5762..6f042e2 100644
--- a/default-configs/i386-softmmu.mak
+++ b/default-configs/i386-softmmu.mak
@@ -7,6 +7,7 @@ CONFIG_QXL=$(CONFIG_SPICE)
 CONFIG_VGA_ISA=y
 CONFIG_VGA_CIRRUS=y
 CONFIG_VMWARE_VGA=y
+CONFIG_VIRTIO_VGA=y
 CONFIG_VMMOUSE=y
 CONFIG_SERIAL=y
 CONFIG_PARALLEL=y
-- 
1.8.3.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] virtio-vga: enable for i386
  2015-07-13 11:58 [Qemu-devel] [PATCH] virtio-vga: enable for i386 Gerd Hoffmann
@ 2015-09-11  7:30 ` Michael Tokarev
  2015-09-11  9:13   ` Gerd Hoffmann
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Tokarev @ 2015-09-11  7:30 UTC (permalink / raw)
  To: Gerd Hoffmann, qemu-devel; +Cc: qemu-trivial, Paolo Bonzini

13.07.2015 14:58, Gerd Hoffmann wrote:
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: qemu-trivial@nongnu.org
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>

Gerd, -trivial received 3 patches like this enabling various virtio
bits for various architectures.  I never got any reviewed-by or ACK
on these patches, so they're not applied still.  Maybe you can add
some explanation as to why you think it's a good idea, or else it
doesn't seem like a trivial change? :)

Thanks,

/mjt

>  default-configs/i386-softmmu.mak | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/default-configs/i386-softmmu.mak b/default-configs/i386-softmmu.mak
> index 48b5762..6f042e2 100644
> --- a/default-configs/i386-softmmu.mak
> +++ b/default-configs/i386-softmmu.mak
> @@ -7,6 +7,7 @@ CONFIG_QXL=$(CONFIG_SPICE)
>  CONFIG_VGA_ISA=y
>  CONFIG_VGA_CIRRUS=y
>  CONFIG_VMWARE_VGA=y
> +CONFIG_VIRTIO_VGA=y
>  CONFIG_VMMOUSE=y
>  CONFIG_SERIAL=y
>  CONFIG_PARALLEL=y
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] virtio-vga: enable for i386
  2015-09-11  7:30 ` Michael Tokarev
@ 2015-09-11  9:13   ` Gerd Hoffmann
  2015-09-11  9:23     ` Michael Tokarev
  0 siblings, 1 reply; 5+ messages in thread
From: Gerd Hoffmann @ 2015-09-11  9:13 UTC (permalink / raw)
  To: Michael Tokarev; +Cc: qemu-trivial, Paolo Bonzini, qemu-devel

On Fr, 2015-09-11 at 10:30 +0300, Michael Tokarev wrote:
> 13.07.2015 14:58, Gerd Hoffmann wrote:
> > Cc: Paolo Bonzini <pbonzini@redhat.com>
> > Cc: qemu-trivial@nongnu.org
> > Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> 
> Gerd, -trivial received 3 patches like this enabling various virtio
> bits for various architectures.  I never got any reviewed-by or ACK
> on these patches, so they're not applied still.  Maybe you can add
> some explanation as to why you think it's a good idea, or else it
> doesn't seem like a trivial change? :)

This one just syncs x86_64 and i386.  Dunno about the other ones, have a
pointer?

cheers,
  Gerd

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] virtio-vga: enable for i386
  2015-09-11  9:13   ` Gerd Hoffmann
@ 2015-09-11  9:23     ` Michael Tokarev
  2015-09-11  9:56       ` Paolo Bonzini
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Tokarev @ 2015-09-11  9:23 UTC (permalink / raw)
  To: Gerd Hoffmann; +Cc: qemu-trivial, Paolo Bonzini, qemu-devel

11.09.2015 12:13, Gerd Hoffmann пишет:
> On Fr, 2015-09-11 at 10:30 +0300, Michael Tokarev wrote:
>> 13.07.2015 14:58, Gerd Hoffmann wrote:
>>> Cc: Paolo Bonzini <pbonzini@redhat.com>
>>> Cc: qemu-trivial@nongnu.org
>>> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
>>
>> Gerd, -trivial received 3 patches like this enabling various virtio
>> bits for various architectures.  I never got any reviewed-by or ACK
>> on these patches, so they're not applied still.  Maybe you can add
>> some explanation as to why you think it's a good idea, or else it
>> doesn't seem like a trivial change? :)
> 
> This one just syncs x86_64 and i386.  Dunno about the other ones, have a
> pointer?

Ok, applied this one to -trivial (with a commit comment :)

The other 2 are actually 2 incarnations of the same patch,
http://patchwork.ozlabs.org/patch/494250/

Since the replies hasn't been seen at qemu-trivial@ I haven't
found the replies.  Now I did, and it looks like the patch
needs some more work ;)

Thanks,

/mjt

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Qemu-devel] [PATCH] virtio-vga: enable for i386
  2015-09-11  9:23     ` Michael Tokarev
@ 2015-09-11  9:56       ` Paolo Bonzini
  0 siblings, 0 replies; 5+ messages in thread
From: Paolo Bonzini @ 2015-09-11  9:56 UTC (permalink / raw)
  To: Michael Tokarev, Gerd Hoffmann; +Cc: qemu-trivial, qemu-devel



On 11/09/2015 11:23, Michael Tokarev wrote:
> Ok, applied this one to -trivial (with a commit comment :)
> 
> The other 2 are actually 2 incarnations of the same patch,
> http://patchwork.ozlabs.org/patch/494250/
> 
> Since the replies hasn't been seen at qemu-trivial@ I haven't
> found the replies.  Now I did, and it looks like the patch
> needs some more work ;)

I think the sys/ioctl.h issue has been fixed now:

    $ grep -Fr ioctl.h include/standard-headers/
    $

Paolo

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2015-09-11  9:56 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-13 11:58 [Qemu-devel] [PATCH] virtio-vga: enable for i386 Gerd Hoffmann
2015-09-11  7:30 ` Michael Tokarev
2015-09-11  9:13   ` Gerd Hoffmann
2015-09-11  9:23     ` Michael Tokarev
2015-09-11  9:56       ` Paolo Bonzini

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).