Re: [Qemu-devel] Re: [PATCH] Implement a virtio GPU transport

2010-11-01 Thread Avi Kivity
On 10/28/2010 03:52 PM, Ian Molton wrote: > On 28/10/10 15:24, Avi Kivity wrote: >>> The caller is intended to block as the host must perform GL rendering >>> before allowing the guests process to continue. >> >> Why is that? Can't we pipeline the process? > > No, not really. the guest may call

Re: [Qemu-devel] Re: [PATCH] Implement a virtio GPU transport

2010-11-01 Thread Anthony Liguori
On 11/01/2010 05:42 AM, Avi Kivity wrote: > On 10/28/2010 03:52 PM, Ian Molton wrote: >> On 28/10/10 15:24, Avi Kivity wrote: The caller is intended to block as the host must perform GL rendering before allowing the guests process to continue. >>> >>> Why is that? Can't we pipeline the

Re: [Qemu-devel] Re: [PATCH] Implement a virtio GPU transport

2010-11-01 Thread Anthony Liguori
On 11/01/2010 06:53 AM, Alon Levy wrote: >> The alternative proposal is Spice which so far noone has mentioned. >> Right now, Spice seems to be taking the right approach to guest 3d >> support. >> >> > While we (speaking as part of the SPICE developers) want to have the same > support in our

Re: [Qemu-devel] Re: [PATCH] Implement a virtio GPU transport

2010-11-01 Thread Ian Molton
On 01/11/10 13:21, Anthony Liguori wrote: > On 11/01/2010 05:42 AM, Avi Kivity wrote: >> On 10/28/2010 03:52 PM, Ian Molton wrote: >>> On 28/10/10 15:24, Avi Kivity wrote: >> Waiting for a response is fine, but can't the guest issue a second >> batch while waiting for the first? > The other scena

Re: [Qemu-devel] Re: [PATCH] Implement a virtio GPU transport

2010-11-01 Thread Ian Molton
On 01/11/10 10:42, Avi Kivity wrote: >> No, not really. the guest may call for the scene to be rendered at >> any time and we have to wait for that to happen before we can >> return the data to it. > > Waiting for a response is fine, but can't the guest issue a second > batch while waiting for the

Re: [Qemu-devel] Re: [PATCH] Implement a virtio GPU transport

2010-11-01 Thread Anthony Liguori
On 11/01/2010 10:49 AM, Ian Molton wrote: > On 01/11/10 13:21, Anthony Liguori wrote: >> On 11/01/2010 05:42 AM, Avi Kivity wrote: >>> On 10/28/2010 03:52 PM, Ian Molton wrote: On 28/10/10 15:24, Avi Kivity wrote: > >>> Waiting for a response is fine, but can't the guest issue a second >>> bat

Re: [PATCH] xen: xenfs: privcmd: check put_user() return code

2010-11-01 Thread Ian Campbell
On Fri, 2010-10-29 at 18:52 +0100, Jeremy Fitzhardinge wrote: > On 10/29/2010 10:44 AM, Ian Campbell wrote: > > On Fri, 2010-10-29 at 18:18 +0100, Jeremy Fitzhardinge wrote: > >> On 10/28/2010 04:39 AM, Vasiliy Kulikov wrote: > >>> put_user() may fail. In this case propagate error code from > >>>

Re: [PATCH] xen: xenfs: privcmd: check put_user() return code

2010-11-01 Thread Ian Campbell
On Fri, 2010-10-29 at 18:18 +0100, Jeremy Fitzhardinge wrote: > On 10/28/2010 04:39 AM, Vasiliy Kulikov wrote: > > put_user() may fail. In this case propagate error code from > > privcmd_ioctl_mmap_batch(). > > Thanks for looking at this. I'm in two minds about this; the existing > logic is such

Re: [Qemu-devel] Re: [PATCH] Implement a virtio GPU transport

2010-11-01 Thread Alon Levy
- "Anthony Liguori" wrote: > On 10/29/2010 06:18 AM, Rusty Russell wrote: > >> Fixed - updated patch tested and attached. > >> > > OK. FWIW, I think this is an awesome idea. > > Paravirtual OpenGL or the actual proposed implementation? Have you > looked at the actual code? > > If I

[PATCH 08/10] staging: hv: Convert camel cased struct fields in ring_buffer.h to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased struct fields in ring_buffer.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 10 +++--- drivers/staging/hv/ring_buffer.c | 58 - drivers/staging/hv

[PATCH 05/10] staging: hv: Convert camel cased functions in hv.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased functions in hv.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/connection.c |4 +- drivers/staging/hv/hv.c | 42 -- drivers/staging/hv/hv.h |

[PATCH 10/10] staging: hv: Convert camel cased functions in ring_buffer.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased functions in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 40 ++-- drivers/staging/hv/ring_buffer.c | 123 +++-- drivers/staging/

[PATCH 02/10] staging: hv: Convert camel cased struct fields in hv_api.h to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased struct fields in hv_api.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 32 ++-- drivers/staging/hv/channel_mgmt.c |6 +- drivers/staging/hv/connection.c |4 +- drivers/stagi

[PATCH 04/10] staging: hv: Convert camel cased local variables in hv.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in hv.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c | 170 +++--- 1 files changed, 85 insertions(+), 85 deletions(-) diff --git a/drivers/

[PATCH 03/10] staging: hv: Convert camel cased struct fields in hv.h to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased struct fields in hv.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c| 95 +++- drivers/staging/hv/hv.h| 20 +- drivers/staging/hv/vmbus.c |

[PATCH 07/10] staging: hv: Convert camel cased functions in osd.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased functions in osd.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 20 +- drivers/staging/hv/channel_mgmt.c | 12 +- drivers/staging/hv/connection.c | 14 ++

[PATCH 09/10] staging: hv: Convert camel cased local variables in ring_buffer.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 +- drivers/staging/hv/ring_buffer.c | 337 +++--- drivers/staging/hv/

[PATCH 06/10] staging: hv: Convert camel cased local variables in osd.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in osd.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/osd.c | 40 1 files changed, 20 insertions(+), 20 deletions(-) diff --git a/drivers/stag

[PATCH 06/10] staging: hv: Convert camel cased local variables in osd.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in osd.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/osd.c | 40 1 files changed, 20 insertions(+), 20 deletions(-) diff --git a/drivers/stag

[PATCH 07/10] staging: hv: Convert camel cased functions in osd.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased functions in osd.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 20 +- drivers/staging/hv/channel_mgmt.c | 12 +- drivers/staging/hv/connection.c | 14 ++

[PATCH 04/10] staging: hv: Convert camel cased local variables in hv.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in hv.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c | 170 +++--- 1 files changed, 85 insertions(+), 85 deletions(-) diff --git a/drivers/

[PATCH 09/10] staging: hv: Convert camel cased local variables in ring_buffer.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 +- drivers/staging/hv/ring_buffer.c | 337 +++--- drivers/staging/hv/

[PATCH 03/10] staging: hv: Convert camel cased struct fields in hv.h to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased struct fields in hv.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c| 95 +++- drivers/staging/hv/hv.h| 20 +- drivers/staging/hv/vmbus.c |

Re: [PATCH 03/10] staging: hv: Convert camel cased struct fields in hv.h to lower cases

2010-11-01 Thread Greg KH
On Mon, Nov 01, 2010 at 09:53:45AM -0700, Haiyang Zhang wrote: > From: Haiyang Zhang > > Convert camel cased struct fields in hv.h to lower cases > > Signed-off-by: Haiyang Zhang > Signed-off-by: Hank Janssen Why did I get two different copies of some of these patches? Are they different? >

RE: [PATCH 03/10] staging: hv: Convert camel cased struct fields in hv.h to lower cases

2010-11-01 Thread Haiyang Zhang
> From: Greg KH [mailto:gre...@suse.de] > Sent: Monday, November 01, 2010 1:43 PM > Why did I get two different copies of some of these patches? Are they > different? They are the same. Initially, I sent all 10 patches out at the same time, but I only saw 5 of them in my mail box. So I re-sent t

Re: [PATCH 03/10] staging: hv: Convert camel cased struct fields in hv.h to lower cases

2010-11-01 Thread Greg KH
On Mon, Nov 01, 2010 at 06:27:04PM +, Haiyang Zhang wrote: > > From: Greg KH [mailto:gre...@suse.de] > > Sent: Monday, November 01, 2010 1:43 PM > > Why did I get two different copies of some of these patches? Are they > > different? > > They are the same. > Initially, I sent all 10 patches o

[PATCH 08/10] staging: hv: Convert camel cased struct fields in ring_buffer.h to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased struct fields in ring_buffer.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 10 +++--- drivers/staging/hv/ring_buffer.c | 58 - drivers/staging/hv

[PATCH 10/10] staging: hv: Convert camel cased functions in ring_buffer.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased functions in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 40 ++-- drivers/staging/hv/ring_buffer.c | 123 +++-- drivers/staging/

[PATCH 05/10] staging: hv: Convert camel cased functions in hv.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased functions in hv.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/connection.c |4 +- drivers/staging/hv/hv.c | 42 -- drivers/staging/hv/hv.h |

[PATCH 04/10] staging: hv: Convert camel cased local variables in hv.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in hv.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c | 172 +++--- 1 files changed, 86 insertions(+), 86 deletions(-) diff --git a/drivers/

[PATCH 09/10] staging: hv: Convert camel cased local variables in ring_buffer.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in ring_buffer.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c |4 +- drivers/staging/hv/ring_buffer.c | 337 +++--- drivers/staging/hv/

[PATCH 07/10] staging: hv: Convert camel cased functions in osd.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased functions in osd.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 20 +- drivers/staging/hv/channel_mgmt.c | 12 +- drivers/staging/hv/connection.c | 14 ++

[PATCH 06/10] staging: hv: Convert camel cased local variables in osd.c to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased local variables in osd.c to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/osd.c | 40 1 files changed, 20 insertions(+), 20 deletions(-) diff --git a/drivers/stag

[PATCH 02/10] staging: hv: Convert camel cased struct fields in hv_api.h to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased struct fields in hv_api.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/channel.c | 32 ++-- drivers/staging/hv/channel_mgmt.c |6 +- drivers/staging/hv/connection.c |4 +- drivers/stagi

RE: [PATCH 03/10] staging: hv: Convert camel cased struct fields in hv.h to lower cases

2010-11-01 Thread Haiyang Zhang
> From: Greg KH [mailto:gre...@suse.de] > Sent: Monday, November 01, 2010 2:39 PM > How about you fix this patch in the series and resend them? > > And if you look at the kernel, there is no "g_" for global variable > names, so I don't want to see that as part of the history at all. > > I'll drop

[PATCH 03/10] staging: hv: Convert camel cased struct fields in hv.h to lower cases

2010-11-01 Thread Haiyang Zhang
From: Haiyang Zhang Convert camel cased struct fields in hv.h to lower cases Signed-off-by: Haiyang Zhang Signed-off-by: Hank Janssen --- drivers/staging/hv/hv.c| 95 +++- drivers/staging/hv/hv.h| 20 +- drivers/staging/hv/vmbus.c |