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

[Stefano Stabellini <stefano.stabellini@eu.citrix.com>]

On Tue, 23 Jun 2015, Xu, Quan wrote:
> > -----Original Message-----
> > From: Stefano Stabellini [mailto:stefano.stabellini@eu.citrix.com]
> > Sent: Tuesday, June 23, 2015 12:37 AM
> > To: Xu, Quan
> > Cc: stefano.stabellini@eu.citrix.com; qemu-devel@nongnu.org;
> > stefanb@linux.vnet.ibm.com; eblake@redhat.com; wei.liu2@citrix.com;
> > dgdegra@tycho.nsa.gov; xen-devel@lists.xen.org
> > 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 <quan.xu@intel.com>
> > 
> > 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.