Re: [Qemu-devel] [PATCH v6 0/6] Xen PCI passthru: Convert to realize()

2016-01-21 Thread Cao jin
Hi Stefano, Just FYI: The series have been reviewed by Eric, I guess it could be picked up;) On 01/17/2016 08:13 PM, Cao jin wrote: v6 changelog: 1. split modification of xen_host_pci_sysfs_path() into a separate new patch as 1/6 shows. 2. 'bug' fix of qemu_strtoul(), in patch 2/6 &

Re: [Qemu-devel] [PATCH v6 0/6] Xen PCI passthru: Convert to realize()

2016-01-21 Thread Stefano Stabellini
Hi Cao, I appreciate the reminder, but it looks like Eric hasn't reviewed patch 3/5. Am I wrong? On Thu, 21 Jan 2016, Cao jin wrote: > Hi Stefano, > > Just FYI: The series have been reviewed by Eric, I guess it could be > picked up;) > > On 01/17/2016 08:13 PM, Cao jin wrote: > > v6

Re: [Qemu-devel] [PATCH v6 0/6] Xen PCI passthru: Convert to realize()

2016-01-21 Thread Stefano Stabellini
On Thu, 21 Jan 2016, Eric Blake wrote: > On 01/21/2016 08:41 AM, Stefano Stabellini wrote: > > Hi Cao, > > > > I appreciate the reminder, but it looks like Eric hasn't reviewed patch > > 3/5. Am I wrong? > > I've done it now. Thanks for your patience, and for letting me jump in > with a late

Re: [Qemu-devel] [PATCH v6 0/6] Xen PCI passthru: Convert to realize()

2016-01-21 Thread Eric Blake
On 01/21/2016 08:41 AM, Stefano Stabellini wrote: > Hi Cao, > > I appreciate the reminder, but it looks like Eric hasn't reviewed patch > 3/5. Am I wrong? I've done it now. Thanks for your patience, and for letting me jump in with a late review (since I remember you were about to push an earlier

Re: [Qemu-devel] [PATCH v6 0/6] Xen PCI passthru: Convert to realize()

2016-01-21 Thread Cao jin
On 01/21/2016 11:41 PM, Stefano Stabellini wrote: Hi Cao, I appreciate the reminder, but it looks like Eric hasn't reviewed patch 3/5. Am I wrong? Seems I made a little mistake:-[ Already see your pull request, Thank you:) And thanks for Eric`s review:) -- Yours Sincerely, Cao jin

Re: [Qemu-devel] [PATCH v6 0/6] Xen PCI passthru: Convert to realize()

2016-01-18 Thread Cao jin
Seems I forgot to CC Eric, but I checked my command line history, I do include Eric...weird On 01/17/2016 08:13 PM, Cao jin wrote: v6 changelog: 1. split modification of xen_host_pci_sysfs_path() into a separate new patch as 1/6 shows. 2. 'bug' fix of qemu_strtoul(), in patch 2/6 & 3/6 3.

Re: [Qemu-devel] [PATCH v6 0/6] Xen PCI passthru: Convert to realize()

2016-01-18 Thread Markus Armbruster
Cao jin writes: > Seems I forgot to CC Eric, but I checked my command line history, I do > include Eric...weird Probably the broken duplicate suppression feature. Eric, are you sure the reduction in e-mail for you is worth the occasional confusion for others?

[Qemu-devel] [PATCH v6 0/6] Xen PCI passthru: Convert to realize()

2016-01-17 Thread Cao jin
v6 changelog: 1. split modification of xen_host_pci_sysfs_path() into a separate new patch as 1/6 shows. 2. 'bug' fix of qemu_strtoul(), in patch 2/6 & 3/6 3. Grammar fix in patch 4/6 4. 'msg' --> 'message' in commit message. Cao jin (6): Change xen_host_pci_sysfs_path() to return void