On Tue, 23 Jun 2015, Xu, Quan wrote: > > -----Original Message----- > > From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com] > > Sent: Tuesday, June 23, 2015 12:37 AM > > To: Xu, Quan > > Cc: stefano.stabell...@eu.citrix.com; qemu-devel@nongnu.org; > > stef...@linux.vnet.ibm.com; ebl...@redhat.com; wei.l...@citrix.com; > > dgde...@tycho.nsa.gov; xen-de...@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...@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.
