Re: [Qemu-devel] [PATCH v8 2/7] Qemu-Xen-vTPM: Create a new file xen_pvdev.c

[Stefano Stabellini]

On Tue, 23 Jun 2015, Xu, Quan wrote:
> > -----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.

Sure, no problem.