[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 2/7] Qemu-Xen-vTPM: Create a new file xen_pvd
From: |
Xu, Quan |
Subject: |
Re: [Qemu-devel] [PATCH v8 2/7] Qemu-Xen-vTPM: Create a new file xen_pvdev.c |
Date: |
Tue, 23 Jun 2015 06:43:43 +0000 |
> -----Original Message-----
> From: Stefano Stabellini [mailto:address@hidden
> Sent: Tuesday, June 23, 2015 12:37 AM
> To: Xu, Quan
> Cc: address@hidden; address@hidden;
> address@hidden; address@hidden; address@hidden;
> address@hidden; address@hidden
> Subject: Re: [PATCH v8 2/7] Qemu-Xen-vTPM: Create a new file xen_pvdev.c
>
> On Sun, 17 May 2015, Quan Xu wrote:
> > for some common part of xen frontend and backend, such as xendevs
> > queue and xenstore update functions.
> >
> > Signed-off-by: Quan Xu <address@hidden>
>
> Hi Quan,
>
> could you please separate out the code movement from any other changes?
> This patch would become two patches: the first would only move code from
> xen_backend.c to xen_pvdev.c, no other changes except for the ones actually
> required to build the code (Makefile.objs). The second patch would rename
> xen_be_find_xendev to xen_find_xendev and any other changes to the code
> that you making here. That way I can very easily go and look only at the
> things
> you actually modify.
>
> Thanks,
>
> Stefano
>
>
Stefano, thanks for your comment.
Now I am focusing on ' VT-d async invalidation for Device-TLB' feature as high
priority.
I will fix all of your comments when I send out v1 of ' VT-d async invalidation
for Device-TLB' feature.
Quan