>>> On 08.03.16 at 13:39, <quan...@intel.com> wrote:
> On March 08, 2016 8:29pm, <dario.faggi...@citrix.com> wrote:
>> On Tue, 2016-03-08 at 19:09 +0800, Quan Xu wrote:
>> > Signed-off-by: Quan Xu <quan...@intel.com>
>> > CC: Keir Fraser <k...@xen.org>
>> > CC: Jan Beulich <jbeul...@suse.com>
>> > CC: Andrew Cooper <andrew.coop...@citrix.com>
>> > CC: Suravee Suthikulpanit <suravee.suthikulpa...@amd.com>
>> > CC: Aravind Gopalakrishnan <aravind.gopalakrish...@amd.com>
>> > CC: Feng Wu <feng...@intel.com>
>> > CC: Kevin Tian <kevin.t...@intel.com>
>> > CC: Dario Faggioli <dario.faggi...@citrix.com>
>> >
>> I've gone through the code, and it looks fine.
>> 
>> However, when trying to apply the patch, on top of this morning's staging, I 
>> got
>> this:
>> 
> Oh, sorry, it is not against this morning's staging.
> I would try to send out patch against this morning's staging soon. Thanks.

Well, with e.g.

>> [dario@Solace xen.git] $ patch -p1 <
>> \[PATCH_2_2\]_IOMMU_spinlock\:_Make_the_pcidevs_lock_a_recursive_one.
>> mbox
>> patching file xen/arch/x86/domctl.c
>> Hunk #1 succeeded at 472 (offset 45 lines).
>> Hunk #2 succeeded at 497 (offset 45 lines).
>> patching file xen/arch/x86/hvm/vmsi.c
>> Hunk #1 succeeded at 388 with fuzz 1.
>> Hunk #2 succeeded at 446 with fuzz 1 (offset 3 lines).

... this it must have been quite old a tree - this file didn't change
in the last 4 months. I consider it rather unfriendly to post such
a patch without RFC tag, and without stating that it's against a
stale tree. Was the recent v6 of the 5-patch series this way too?
If so, I'm glad I didn't spend time looking at it yet.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to