[Qemu-devel] [PATCH v3 13/13] xen: Rename xen_be_del_xendev

2016-10-25 Thread Emil Condrea
Prepare xen_be_del_xendev to be shared with frontends: * xen_be_del_xendev -> xen_pv_del_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/xen/xen_backend.c

[Qemu-devel] [PATCH v3 11/13] xen: Rename xen_be_evtchn_event

2016-10-25 Thread Emil Condrea
Prepare xen_be_evtchn_event to be shared with frontends: * xen_be_evtchn_event -> xen_pv_evtchn_event Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/xen/xen_backend.c

[Qemu-devel] [PATCH v3 10/13] xen: Rename xen_be_send_notify

2016-10-25 Thread Emil Condrea
Prepare xen_be_send_notify to be shared with frontends: * xen_be_send_notify -> xen_pv_send_notify Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/block/xen_disk.c

[Qemu-devel] [PATCH v3 09/13] xen: Rename xen_be_unbind_evtchn

2016-10-25 Thread Emil Condrea
Prepare xen_be_unbind_evtchn to be shared with frontends: * xen_be_unbind_evtchn -> xen_pv_unbind_evtchn Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/block/xen_dis

[Qemu-devel] [PATCH v3 04/13] xen: Move xenstore_update to xen_pvdev.c

2016-10-25 Thread Emil Condrea
* xenstore_update -> xen_pvdev.c Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 30 +++--- hw/xen/xen_pvdev.c | 23 +++ include/hw/xen/xen_backend.h | 3 +++ include/hw/xen/xen_pvdev.h

[Qemu-devel] [PATCH v3 01/13] xen: Fix coding style errors

2016-10-25 Thread Emil Condrea
Fixes the following errors: * ERROR: line over 90 characters * ERROR: code indent should never use tabs * ERROR: space prohibited after that open square bracket '[' * ERROR: do not initialise statics to 0 or NULL * ERROR: "(foo*)" should be "(foo *)" Signed-off-by: Em

[Qemu-devel] [PATCH v3 12/13] xen: Rename xen_be_find_xendev

2016-10-25 Thread Emil Condrea
Prepare xen_be_find_xendev to be shared with frontends: * xen_be_find_xendev -> xen_pv_find_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/display/xenfb.c

[Qemu-devel] [PATCH v3 00/13] Refactor common part of xen backend and frontend

2016-10-25 Thread Emil Condrea
to xen_pvdev file. --- Changes in v3: * align second line with first parameter Changes in v2: * fixed parameter alignment when spliting long lines * removed patches for creating xen_frontend Emil Condrea (13): xen: Fix coding style errors xen: Fix coding style warnings xen: Create a new file

[Qemu-devel] [PATCH v3 06/13] xen: Prepare xendev qtail to be shared with frontends

2016-10-25 Thread Emil Condrea
* move xendevs qtail to xen_pvdev.c * change xen_be_get_xendev to use a new function: xen_pv_insert_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 51 +-- hw/xen/xen_pvdev.c

[Qemu-devel] [PATCH v3 08/13] xen: Rename xen_be_printf to xen_pv_printf

2016-10-25 Thread Emil Condrea
Prepare xen_be_printf to be used by both backend and frontends: * xen_be_printf -> xen_pv_printf Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> --- hw/block/xen_disk.c| 58 +++-

[Qemu-devel] [PATCH v3 05/13] xen: Move evtchn functions to xen_pvdev.c

2016-10-25 Thread Emil Condrea
The name of the functions moved: * xen_be_evtchn_event * xen_be_unbind_evtchn * xen_be_send_notify Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 35 --- hw/xen/xen_pvdev.c

[Qemu-devel] [PATCH v3 07/13] xen: Move xenstore cleanup and mkdir functions

2016-10-24 Thread Emil Condrea
The name of the functions moved to xen_pvdev.c: * xenstore_cleanup_dir * xen_config_cleanup * xenstore_mkdir Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> --- hw/xen/xen_ba

[Qemu-devel] [PATCH v3 02/13] xen: Fix coding style warnings

2016-10-24 Thread Emil Condrea
Fixes: * WARNING: line over 80 characters Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c | 3 ++- hw/char/xen_console.c| 3 ++- hw/display/xenfb.c | 6 -- hw/net/xen_nic.c | 12 hw/xen/xen_bac

[Qemu-devel] [PATCH v3 03/13] xen: Create a new file xen_pvdev.c

2016-10-24 Thread Emil Condrea
The purpose of the new file is to store generic functions shared by frontend and backends such as xenstore operations, xendevs. Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/Makefile.objs | 2 +- hw/xen/x

Re: [Qemu-devel] [PATCH 02/15] xen: Fix coding style warnings

2016-10-13 Thread Emil Condrea
, Anthony PERARD <anthony.per...@citrix.com> wrote: > On Thu, Oct 13, 2016 at 07:04:56AM +0300, Emil Condrea wrote: >> On Tue, Oct 11, 2016 at 5:20 PM, Anthony PERARD >> <anthony.per...@citrix.com> wrote: >> > On Tue, Oct 04, 2016 at 09:43:31AM +0300, Emil Condrea wrote:

[Qemu-devel] [PATCH v2 13/13] xen: Rename xen_be_del_xendev

2016-10-13 Thread Emil Condrea
Prepare xen_be_del_xendev to be shared with frontends: * xen_be_del_xendev -> xen_pv_del_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/xen/xen_backend.c

[Qemu-devel] [PATCH v2 08/13] xen: Rename xen_be_printf to xen_pv_printf

2016-10-13 Thread Emil Condrea
Prepare xen_be_printf to be used by both backend and frontends: * xen_be_printf -> xen_pv_printf Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> --- hw/block/xen_disk.c| 58 +++-

[Qemu-devel] [PATCH v2 11/13] xen: Rename xen_be_evtchn_event

2016-10-13 Thread Emil Condrea
Prepare xen_be_evtchn_event to be shared with frontends: * xen_be_evtchn_event -> xen_pv_evtchn_event Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/xen/xen_backend.c

Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed

2016-10-13 Thread Emil Condrea
As you suggested, I've dropped the all patches for xen_frontend. Emil On Wed, Oct 12, 2016 at 2:00 PM, Paolo Bonzini <pbonz...@redhat.com> wrote: > > > On 09/10/2016 21:50, Emil Condrea wrote: >> On Tue, Oct 4, 2016 at 11:06 AM, Paolo Bonzini <pbonz...@redhat.com>

[Qemu-devel] [PATCH v2 12/13] xen: Rename xen_be_find_xendev

2016-10-13 Thread Emil Condrea
Prepare xen_be_find_xendev to be shared with frontends: * xen_be_find_xendev -> xen_pv_find_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/display/xenfb.c

[Qemu-devel] [PATCH v2 09/13] xen: Rename xen_be_unbind_evtchn

2016-10-13 Thread Emil Condrea
Prepare xen_be_unbind_evtchn to be shared with frontends: * xen_be_unbind_evtchn -> xen_pv_unbind_evtchn Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/block/xen_dis

Re: [Qemu-devel] [PATCH v2 00/13] Refactor common part of xen backend and frontend

2016-10-13 Thread Emil Condrea
+ cc Paolo Bonzini On Thu, Oct 13, 2016 at 9:01 AM, Emil Condrea <emilcond...@gmail.com> wrote: > This patch series was splitted from QEMU:Xen stubdom vTPM for HVM virtual > machine > http://markmail.org/message/fkix7g3a5zdj7lvr > > It contains a reorganization of xen

[Qemu-devel] [PATCH v2 10/13] xen: Rename xen_be_send_notify

2016-10-13 Thread Emil Condrea
Prepare xen_be_send_notify to be shared with frontends: * xen_be_send_notify -> xen_pv_send_notify Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> Reviewed-by: Quan Xu <xuqu...@huawei.com> --- hw/block/xen_disk.c

[Qemu-devel] [PATCH v2 05/13] xen: Move evtchn functions to xen_pvdev.c

2016-10-13 Thread Emil Condrea
The name of the functions moved: * xen_be_evtchn_event * xen_be_unbind_evtchn * xen_be_send_notify Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 35 --- hw/xen/xen_pvdev.c

[Qemu-devel] [PATCH v2 03/13] xen: Create a new file xen_pvdev.c

2016-10-13 Thread Emil Condrea
The purpose of the new file is to store generic functions shared by frontend and backends such as xenstore operations, xendevs. Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/Makefile.objs | 2 +- hw/xen/x

[Qemu-devel] [PATCH v2 06/13] xen: Prepare xendev qtail to be shared with frontends

2016-10-13 Thread Emil Condrea
* move xendevs qtail to xen_pvdev.c * change xen_be_get_xendev to use a new function: xen_pv_insert_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 51 +-- hw/xen/xen_pvdev.c

[Qemu-devel] [PATCH v2 07/13] xen: Move xenstore cleanup and mkdir functions

2016-10-13 Thread Emil Condrea
The name of the functions moved to xen_pvdev.c: * xenstore_cleanup_dir * xen_config_cleanup * xenstore_mkdir Signed-off-by: Emil Condrea <emilcond...@gmail.com> Acked-by: Anthony PERARD <anthony.per...@citrix.com> --- hw/xen/xen_ba

[Qemu-devel] [PATCH v2 02/13] xen: Fix coding style warnings

2016-10-13 Thread Emil Condrea
Fixes: * WARNING: line over 80 characters Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c | 3 ++- hw/char/xen_console.c| 6 -- hw/display/xenfb.c | 30 -- hw/net/xen_nic.c | 12 --

[Qemu-devel] [PATCH v2 01/13] xen: Fix coding style errors

2016-10-13 Thread Emil Condrea
Fixes the following errors: * ERROR: line over 90 characters * ERROR: code indent should never use tabs * ERROR: space prohibited after that open square bracket '[' * ERROR: do not initialise statics to 0 or NULL * ERROR: "(foo*)" should be "(foo *)" Signed-off-by: Em

[Qemu-devel] [PATCH v2 00/13] Refactor common part of xen backend and frontend

2016-10-13 Thread Emil Condrea
to xen_pvdev file. --- Changes in v2: * fixed parameter alignment when spliting long lines * removed patches for creating xen_frontend Emil Condrea (13): xen: Fix coding style errors xen: Fix coding style warnings xen: Create a new file xen_pvdev.c xen: Move xenstore_update to xen_pvdev.c xen

[Qemu-devel] [PATCH v2 04/13] xen: Move xenstore_update to xen_pvdev.c

2016-10-13 Thread Emil Condrea
* xenstore_update -> xen_pvdev.c Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 30 +++--- hw/xen/xen_pvdev.c | 23 +++ include/hw/xen/xen_backend.h | 3 +++ include/hw/xen/xen_pvdev.h

Re: [Qemu-devel] [PATCH 02/15] xen: Fix coding style warnings

2016-10-12 Thread Emil Condrea
On Tue, Oct 11, 2016 at 5:20 PM, Anthony PERARD <anthony.per...@citrix.com> wrote: > On Tue, Oct 04, 2016 at 09:43:31AM +0300, Emil Condrea wrote: >> Fixes: >> * WARNING: line over 80 characters >> >> Signed-off-by: Emil Condrea <emilcond...@gmail.com> >&

Re: [Qemu-devel] [PATCH 01/15] xen: Fix coding style errors

2016-10-12 Thread Emil Condrea
warnings" patch. On Tue, Oct 11, 2016 at 4:56 PM, Anthony PERARD <anthony.per...@citrix.com> wrote: > On Tue, Oct 04, 2016 at 09:43:30AM +0300, Emil Condrea wrote: >> Fixes the following errors: >> * ERROR: line over 90 characters > > It's 80 ;), and there are a f

Re: [Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed

2016-10-09 Thread Emil Condrea
On Tue, Oct 4, 2016 at 11:06 AM, Paolo Bonzini <pbonz...@redhat.com> wrote: > > > On 04/10/2016 08:43, Emil Condrea wrote: >> xen_be_frontend_changed -> xen_fe_frontend_changed > > This is not correct. The front-end is implemented in the guest domain, >

Re: [Qemu-devel] [v9 00/19] QEMU:Xen stubdom vTPM for HVM virtual machine(QEMU Part)

2016-10-04 Thread Emil Condrea
1PM +0300, Emil Condrea wrote: >> Emil Condrea (19): >> xen: Create a new file xen_pvdev.c >> xen: Create a new file xen_frontend.c >> xen: Move xenstore_update to xen_pvdev.c >> xen: Move evtchn functions to xen_pvdev.c >> xen: Prepare xendev qtail

[Qemu-devel] [PATCH 13/15] xen: Rename xen_be_find_xendev

2016-10-04 Thread Emil Condrea
Prepare xen_be_find_xendev to be shared with frontends: * xen_be_find_xendev -> xen_pv_find_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/display/xenfb.c | 4 ++-- hw/xen/xen_backend.c | 2 +- hw/xen/xen_pvdev.c | 2 +- include/hw/xen/xen_pv

[Qemu-devel] [PATCH 09/15] xen: Rename xen_be_printf to xen_pv_printf

2016-10-04 Thread Emil Condrea
Prepare xen_be_printf to be used by both backend and frontends: * xen_be_printf -> xen_pv_printf Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c| 58 +++--- hw/char/xen_console.c | 10 hw/displa

[Qemu-devel] [PATCH 10/15] xen: Rename xen_be_unbind_evtchn

2016-10-04 Thread Emil Condrea
Prepare xen_be_unbind_evtchn to be shared with frontends: * xen_be_unbind_evtchn -> xen_pv_unbind_evtchn Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c| 2 +- hw/char/xen_console.c | 2 +- hw/display/xenfb.c | 2 +- hw/net/

[Qemu-devel] [PATCH 11/15] xen: Rename xen_be_send_notify

2016-10-04 Thread Emil Condrea
Prepare xen_be_send_notify to be shared with frontends: * xen_be_send_notify -> xen_pv_send_notify Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c| 4 ++-- hw/char/xen_console.c | 4 ++-- hw/display/xenfb.c | 8 hw/net/

[Qemu-devel] [PATCH 14/15] xen: Rename xen_be_del_xendev

2016-10-04 Thread Emil Condrea
Prepare xen_be_del_xendev to be shared with frontends: * xen_be_del_xendev -> xen_pv_del_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 2 +- hw/xen/xen_pvdev.c | 2 +- include/hw/xen/xen_pvdev.h | 2 +- 3 files changed, 3 inserti

[Qemu-devel] [PATCH 07/15] xen: Prepare xendev qtail to be shared with frontends

2016-10-04 Thread Emil Condrea
* move xendevs qtail to xen_pvdev.c * change xen_be_get_xendev to use a new function: xen_pv_insert_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 51 +- hw/xen/xen_pvdev.c

[Qemu-devel] [PATCH 03/15] xen: Create a new file xen_pvdev.c

2016-10-04 Thread Emil Condrea
The purpose of the new file is to store generic functions shared by frontend and backends such as xenstore operations, xendevs. Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/Makefile.objs | 2 +- hw/xen/x

[Qemu-devel] [PATCH 12/15] xen: Rename xen_be_evtchn_event

2016-10-04 Thread Emil Condrea
Prepare xen_be_evtchn_event to be shared with frontends: * xen_be_evtchn_event -> xen_pv_evtchn_event Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 2 +- hw/xen/xen_pvdev.c | 2 +- include/hw/xen/xen_pvdev.h | 2 +- 3 files changed, 3 i

[Qemu-devel] [PATCH 08/15] xen: Move xenstore cleanup and mkdir functions

2016-10-04 Thread Emil Condrea
The name of the functions moved to xen_pvdev.c: * xenstore_cleanup_dir * xen_config_cleanup * xenstore_mkdir Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 49 - hw/xen/xen_pvdev.c

[Qemu-devel] [PATCH 06/15] xen: Move evtchn functions to xen_pvdev.c

2016-10-04 Thread Emil Condrea
The name of the functions moved: * xen_be_evtchn_event * xen_be_unbind_evtchn * xen_be_send_notify Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 35 --- hw/xen/xen_pvdev.c

[Qemu-devel] [PATCH 05/15] xen: Move xenstore_update to xen_pvdev.c

2016-10-04 Thread Emil Condrea
* xenstore_update -> xen_pvdev.c * xenstore_update_fe -> xen_frontend.c Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 43 +-- hw/xen/xen_frontend.c | 18 ++ hw/xen

[Qemu-devel] [PATCH 01/15] xen: Fix coding style errors

2016-10-04 Thread Emil Condrea
Fixes the following errors: * ERROR: line over 90 characters * ERROR: code indent should never use tabs * ERROR: space prohibited after that open square bracket '[' * ERROR: do not initialise statics to 0 or NULL * ERROR: "(foo*)" should be "(foo *)" Signed-off-by: Em

[Qemu-devel] [PATCH 04/15] xen: Create a new file xen_frontend.c

2016-10-04 Thread Emil Condrea
Its purpose is to store frontend related functions. Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c | 1 + hw/display/xenfb.c| 1 + hw/net/xen_nic.c | 1 + hw/u

[Qemu-devel] [PATCH 15/15] xen: Rename xen_be_frontend_changed

2016-10-04 Thread Emil Condrea
xen_be_frontend_changed -> xen_fe_frontend_changed Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 2 +- hw/xen/xen_frontend.c | 4 ++-- include/hw/xen/xen_frontend.h | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git

[Qemu-devel] [PATCH 02/15] xen: Fix coding style warnings

2016-10-04 Thread Emil Condrea
Fixes: * WARNING: line over 80 characters Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c | 3 ++- hw/char/xen_console.c| 6 -- hw/display/xenfb.c | 30 -- hw/net/xen_nic.c | 12 --

[Qemu-devel] [PATCH 00/15] Refactor common part of xen backend and frontend

2016-10-04 Thread Emil Condrea
to xen_pvdev file. Emil Condrea (15): xen: Fix coding style errors xen: Fix coding style warnings xen: Create a new file xen_pvdev.c xen: Create a new file xen_frontend.c xen: Move xenstore_update to xen_pvdev.c xen: Move evtchn functions to xen_pvdev.c xen: Prepare xendev qtail to be shared

Re: [Qemu-devel] [Xen-devel] [PATCH 15/19] Qemu-Xen-vTPM: Xen frontend driver infrastructure

2016-08-07 Thread Emil Condrea
On Mon, Jul 25, 2016 at 7:01 PM, Anthony PERARD <anthony.per...@citrix.com> wrote: > > On Sun, Jul 10, 2016 at 02:47:46PM +0300, Emil Condrea wrote: > > This patch adds infrastructure for xen front drivers living in qemu, > > so drivers don't need to implement common s

Re: [Qemu-devel] [v9 00/19] QEMU:Xen stubdom vTPM for HVM virtual machine(QEMU Part)

2016-07-31 Thread Emil Condrea
Thanks Anthony for the feedback. I will send v10 containing fixes for patches 1 to 12 + additional patches for fixing code style issues for moved code. On Mon, Jul 25, 2016 at 5:09 PM, Anthony PERARD <anthony.per...@citrix.com> wrote: > On Sun, Jul 10, 2016 at 02:47:31PM +0300, Emi

Re: [Qemu-devel] [Xen-devel] [PATCH 04/19] xen: Move evtchn functions to xen_pvdev.c

2016-07-31 Thread Emil Condrea
On Mon, Jul 25, 2016 at 4:53 PM, Anthony PERARD <anthony.per...@citrix.com> wrote: > On Sun, Jul 10, 2016 at 02:47:35PM +0300, Emil Condrea wrote: > > The name of the functions moved: > > * xen_be_evtchn_event > > * xen_be_unbind_evtchn > > * xen_be_send_n

Re: [Qemu-devel] [Xen-devel] [PATCH 01/19] xen: Create a new file xen_pvdev.c

2016-07-22 Thread Emil Condrea
Sure, I will continue to send revisions until it is approved upstream. On Jul 22, 2016 5:24 PM, "Quan Xu" wrote: > Anthony, thanks for your explaination. > IMO, patch 1 and patch 2 need your detailed review.. IMO the reset > patches are good in general.. > Emil, if patch 1

Re: [Qemu-devel] [Xen-devel] [PATCH 01/19] xen: Create a new file xen_pvdev.c

2016-07-18 Thread Emil Condrea
s to store generic functions shared by > frontendand backends such as xenstore operations, xendevs. > > > > s/frontendand/front end and/ > > Please wrap your commit message lines. Since 'git log' displays logs > with indentation, wrapping around 72 characters is ideal. > > > Sig

Re: [Qemu-devel] [Xen-devel] [PATCH 01/19] xen: Create a new file xen_pvdev.c

2016-07-17 Thread Emil Condrea
xen_backend with refactoring in the same patch. Eventually this can be done in another patch later. > > > The purpose of the new file is to store generic functions shared by frontend > and backends such as xenstore operations, xendevs. > > Signed-off-by: Quan Xu <quan.xu@

[Qemu-devel] [PATCH 18/19] Qemu-Xen-vTPM: Qemu vTPM xenstubdoms backend

2016-07-10 Thread Emil Condrea
, "-tpmdev xenstubdoms,id=xenvtpm0 -device tpm-tis,tpmdev=xenvtpm0" Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> Reviewed-by: Stefan Berger <stef...@linux.vnet.ibm.com> --- Changes in v9: * added reset_tpm_establis

[Qemu-devel] [PATCH 17/19] Qemu-Xen-vTPM: Move tpm_passthrough_is_selftest() into tpm_util.c

2016-07-10 Thread Emil Condrea
Also rename it to tpm_util_is_selftest(). Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> Reviewed-by: Stefan Berger <stef...@linux.vnet.ibm.com> --- hw/tpm/Makefile.objs | 2 +- hw/tpm/tpm_passthrough.c | 13 +

[Qemu-devel] [PATCH 16/19] Qemu-Xen-vTPM: Register Xen stubdom vTPM frontend driver

2016-07-10 Thread Emil Condrea
[...] -->xen_fe_register("vtpm", ...) -->xenstore_fe_scan() -->xen_fe_try_init(ops) --> XenDevOps.init() -->xen_fe_get_xendev() --> XenDevOps.alloc() -->xen_fe_check() -->xen_fe_try_initialise() --> XenDevOp

[Qemu-devel] [PATCH 15/19] Qemu-Xen-vTPM: Xen frontend driver infrastructure

2016-07-10 Thread Emil Condrea
channel between the virtual machines. Call xen_fe_register() function to register XenDevOps, and make sure, XenDevOps's flags is DEVOPS_FLAG_FE, which is flag bit to point out the XenDevOps is Xen frontend. Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <

[Qemu-devel] [PATCH 13/19] xen: Distinguish between frontend and backend devops

2016-07-10 Thread Emil Condrea
xen_be_check_state should not be called for frontends Use DEVOPS_FLAG_FE flag to distinguish a frontend. Signed-off-by: Emil Condrea <emilcond...@gmail.com> Signed-off-by: Quan Xu <quan...@intel.com> --- Changes in v9: * Removed not needed strstr from xenstore_update_be It was le

[Qemu-devel] [PATCH 14/19] Qemu-Xen-vTPM: Support for Xen stubdom vTPM command line options

2016-07-10 Thread Emil Condrea
Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> Reviewed-by: Eric Blake <ebl...@redhat.com> --- Changes in v9: * Replace `type` with `struct` as required by 895a2a80e * Change `qpm_query_tpm_inst` as required by ce21131a0 --- con

[Qemu-devel] [PATCH 12/19] xen: Rename xen_be_frontend_changed

2016-07-10 Thread Emil Condrea
xen_be_frontend_changed -> xen_fe_frontend_changed Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 2 +- hw/xen/xen_frontend.c | 4 ++-- include/hw/xen/xen_frontend.h | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git

[Qemu-devel] [PATCH 07/19] xen: Rename xen_be_unbind_evtchn

2016-07-10 Thread Emil Condrea
Prepare xen_be_unbind_evtchn to be shared with frontends: * xen_be_unbind_evtchn -> xen_pv_unbind_evtchn Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c| 2 +- hw/char/xen_console.c | 2 +- hw/display/xenfb.c | 2 +- hw/net/

[Qemu-devel] [PATCH 11/19] xen: Rename xen_be_del_xendev

2016-07-10 Thread Emil Condrea
Prepare xen_be_del_xendev to be shared with frontends: * xen_be_del_xendev -> xen_pv_del_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 2 +- hw/xen/xen_pvdev.c | 2 +- include/hw/xen/xen_pvdev.h | 2 +- 3 files changed, 3 inserti

[Qemu-devel] [PATCH 10/19] xen: Rename xen_be_find_xendev

2016-07-10 Thread Emil Condrea
Prepare xen_be_find_xendev to be shared with frontends: * xen_be_find_xendev -> xen_pv_find_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/display/xenfb.c | 4 ++-- hw/xen/xen_backend.c | 2 +- hw/xen/xen_pvdev.c | 2 +- include/hw/xen/xen_pv

[Qemu-devel] [PATCH 08/19] xen: Rename xen_be_send_notify

2016-07-10 Thread Emil Condrea
Prepare xen_be_send_notify to be shared with frontends: * xen_be_send_notify -> xen_pv_send_notify Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c| 4 ++-- hw/char/xen_console.c | 4 ++-- hw/display/xenfb.c | 8 hw/net/

[Qemu-devel] [PATCH 04/19] xen: Move evtchn functions to xen_pvdev.c

2016-07-10 Thread Emil Condrea
The name of the functions moved: * xen_be_evtchn_event * xen_be_unbind_evtchn * xen_be_send_notify Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 37 + hw/xen/xen_pvdev.c

[Qemu-devel] [PATCH 19/19] Qemu-Xen-vTPM: QEMU machine class is initialized before tpm_init()

2016-07-10 Thread Emil Condrea
make sure QEMU machine class is initialized and QEMU has registered Xen stubdom vTPM driver when call tpm_init() Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> Reviewed-by: Stefan Berger <stef...@linux.vnet.ibm.com&

[Qemu-devel] [PATCH 09/19] xen: Rename xen_be_evtchn_event

2016-07-10 Thread Emil Condrea
Prepare xen_be_evtchn_event to be shared with frontends: * xen_be_evtchn_event -> xen_pv_evtchn_event Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 2 +- hw/xen/xen_pvdev.c | 2 +- include/hw/xen/xen_pvdev.h | 2 +- 3 files changed, 3 i

[Qemu-devel] [PATCH 02/19] xen: Create a new file xen_frontend.c

2016-07-10 Thread Emil Condrea
Its purpose is to store frontend related functions. Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c | 1 + hw/display/xenfb.c| 1 + hw/net/xen_nic.c | 1 + hw/xen/M

[Qemu-devel] [PATCH 03/19] xen: Move xenstore_update to xen_pvdev.c

2016-07-10 Thread Emil Condrea
* xenstore_update -> xen_pvdev.c * xenstore_update_fe -> xen_frontend.c Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 43 +-- hw/xen/xen_frontend.c | 18 ++ hw/xen

[Qemu-devel] [PATCH 01/19] xen: Create a new file xen_pvdev.c

2016-07-10 Thread Emil Condrea
The purpose of the new file is to store generic functions shared by frontend and backends such as xenstore operations, xendevs. Signed-off-by: Quan Xu <quan...@intel.com> Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/Makefile.objs | 2 +- hw/xen/x

[Qemu-devel] [PATCH 06/19] xen: Rename xen_be_printf to xen_pv_printf

2016-07-10 Thread Emil Condrea
Prepare xen_be_printf to be used by both backend and frontends: * xen_be_printf -> xen_pv_printf Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/block/xen_disk.c| 52 +++--- hw/char/xen_console.c | 10 - hw/displa

[Qemu-devel] [v9 00/19] QEMU:Xen stubdom vTPM for HVM virtual machine(QEMU Part)

2016-07-10 Thread Emil Condrea
) Emil Condrea (19): xen: Create a new file xen_pvdev.c xen: Create a new file xen_frontend.c xen: Move xenstore_update to xen_pvdev.c xen: Move evtchn functions to xen_pvdev.c xen: Prepare xendev qtail to be shared with frontends xen: Rename xen_be_printf to xen_pv_printf xen: Rename

[Qemu-devel] [PATCH 05/19] xen: Prepare xendev qtail to be shared with frontends

2016-07-10 Thread Emil Condrea
* move xendevs qtail to xen_pvdev.c * change xen_be_get_xendev to use a new function: xen_pv_insert_xendev Signed-off-by: Emil Condrea <emilcond...@gmail.com> --- hw/xen/xen_backend.c | 70 +--- hw/xen/xen_pvdev.c

Re: [Qemu-devel] [PATCH] replaced get_ticks_per_sec() with constant

2015-07-08 Thread Emil Condrea
On Mon, Jul 6, 2015 at 6:11 PM, Stefan Hajnoczi stefa...@gmail.com wrote: On Sun, Jul 5, 2015 at 7:15 AM, Emil Condrea emilcond...@gmail.com wrote: Stefan, do you know if this patch was applied? It hasn't been applied, sorry. It slipped through the cracks because it touches a lot of files

Re: [Qemu-devel] [PATCH] replaced get_ticks_per_sec() with constant

2015-07-05 Thread Emil Condrea
Stefan, do you know if this patch was applied? Thanks On Tue, Mar 31, 2015 at 8:16 PM, Emil Condrea emilcond...@gmail.com wrote: Thanks, Stefan. On Mon, Mar 30, 2015 at 4:01 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Sat, Mar 28, 2015 at 06:22:11PM +0200, Emil Condrea wrote

Re: [Qemu-devel] [PATCH] replaced get_ticks_per_sec() with constant

2015-03-31 Thread Emil Condrea
Thanks, Stefan. On Mon, Mar 30, 2015 at 4:01 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Sat, Mar 28, 2015 at 06:22:11PM +0200, Emil Condrea wrote: This replaces all calls to get_ticks_per_sec() with NSEC_PER_SEC defined in /include/qemu/timer.h Signed-off-by: Emil Condrea

[Qemu-devel] [PATCH] replaced get_ticks_per_sec() with constant

2015-03-28 Thread Emil Condrea
This replaces all calls to get_ticks_per_sec() with NSEC_PER_SEC defined in /include/qemu/timer.h Signed-off-by: Emil Condrea emilcond...@gmail.com --- audio/audio.c | 2 +- audio/noaudio.c| 4 ++-- audio/spiceaudio.c | 2 +- audio/wavaudio.c | 2