[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PULL 0/2] Block job patches
From: |
Peter Maydell |
Subject: |
Re: [Qemu-block] [PULL 0/2] Block job patches |
Date: |
Fri, 14 Aug 2015 18:06:25 +0100 |
On 14 August 2015 at 15:55, Jeff Cody <address@hidden> wrote:
> On Fri, Aug 14, 2015 at 03:51:03PM +0100, Peter Maydell wrote:
>> Your pull req tag has not only these two commits in it,
>> but also a merge commit ("Merge branch 'block-next' into HEAD).
>> Why is that?
> I was trying to keep a commit id stable (for 'block/mirror: limit qiov
> to IOV_MAX elements'), so it could be used for a downstream backport
> before this patch actually hit the official upstream repo.
>
> Does this cause you any issues (i.e., should I rebase and submit a new
> pull request)?
No particular problems, I just like to flag up things I'm not
expecting in case they're accidental. Following discussion
on IRC, I've merged this and pushed it to master.
thanks
-- PMM