[0/5] Block patches

Message ID 20180213173429.19501-1-stefanha@redhat.com
State New
Headers show

Pull-request

git://github.com/stefanha/qemu.git tags/block-pull-request

Message

Stefan Hajnoczi Feb. 13, 2018, 5:34 p.m.
The following changes since commit fb68096da3d35e64c88cd610c1fa42766c58e92a:

  Revert "tests: use memfd in vhost-user-test" (2018-02-13 09:51:52 +0000)

are available in the Git repository at:

  git://github.com/stefanha/qemu.git tags/block-pull-request

for you to fetch changes up to 64b01feca991e5b19a5d750ef77cdca92b68bdbb:

  misc: fix spelling (2018-02-13 15:38:17 +0000)

----------------------------------------------------------------

----------------------------------------------------------------

Fam Zheng (2):
  Add a git-publish configuration file
  README: Document 'git-publish' workflow

Marc-André Lureau (1):
  misc: fix spelling

Stefan Hajnoczi (1):
  vl: pause vcpus before stopping iothreads

Wolfgang Bumiller (1):
  ratelimit: don't align wait time with slices

 include/qemu/ratelimit.h   | 11 ++++-----
 util/qemu-coroutine-lock.c |  2 +-
 vl.c                       | 12 ++++++++--
 .gitpublish                | 58 ++++++++++++++++++++++++++++++++++++++++++++++
 README                     | 31 ++++++++++++++++++++++++-
 5 files changed, 104 insertions(+), 10 deletions(-)
 create mode 100644 .gitpublish

Comments

Fam Zheng Feb. 14, 2018, 12:48 a.m. | #1
On Tue, 02/13 17:34, Stefan Hajnoczi wrote:
> The following changes since commit fb68096da3d35e64c88cd610c1fa42766c58e92a:
> 
>   Revert "tests: use memfd in vhost-user-test" (2018-02-13 09:51:52 +0000)
> 
> are available in the Git repository at:
> 
>   git://github.com/stefanha/qemu.git tags/block-pull-request
> 
> for you to fetch changes up to 64b01feca991e5b19a5d750ef77cdca92b68bdbb:
> 
>   misc: fix spelling (2018-02-13 15:38:17 +0000)

Did you mean "PULL" in the subject?

Fam
Stefan Hajnoczi Feb. 14, 2018, 1:25 p.m. | #2
On Wed, Feb 14, 2018 at 08:48:20AM +0800, Fam Zheng wrote:
> On Tue, 02/13 17:34, Stefan Hajnoczi wrote:
> > The following changes since commit fb68096da3d35e64c88cd610c1fa42766c58e92a:
> > 
> >   Revert "tests: use memfd in vhost-user-test" (2018-02-13 09:51:52 +0000)
> > 
> > are available in the Git repository at:
> > 
> >   git://github.com/stefanha/qemu.git tags/block-pull-request
> > 
> > for you to fetch changes up to 64b01feca991e5b19a5d750ef77cdca92b68bdbb:
> > 
> >   misc: fix spelling (2018-02-13 15:38:17 +0000)
> 
> Did you mean "PULL" in the subject?

git-publish is just obeying your profile settings:

+[gitpublishprofile "default"]
+base = master
+prefix = PATCH
 ^---- :)

I used PULL myself in the past but am happy with using whatever we
decide the standard should be.

Shall we remove prefix from the profiles (it defaults to "PATCH" for
regular patches and "PULL" for pull requests)?

Stefan
Fam Zheng Feb. 14, 2018, 1:58 p.m. | #3
On Wed, Feb 14, 2018 at 9:25 PM, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> On Wed, Feb 14, 2018 at 08:48:20AM +0800, Fam Zheng wrote:
>> On Tue, 02/13 17:34, Stefan Hajnoczi wrote:
>> > The following changes since commit fb68096da3d35e64c88cd610c1fa42766c58e92a:
>> >
>> >   Revert "tests: use memfd in vhost-user-test" (2018-02-13 09:51:52 +0000)
>> >
>> > are available in the Git repository at:
>> >
>> >   git://github.com/stefanha/qemu.git tags/block-pull-request
>> >
>> > for you to fetch changes up to 64b01feca991e5b19a5d750ef77cdca92b68bdbb:
>> >
>> >   misc: fix spelling (2018-02-13 15:38:17 +0000)
>>
>> Did you mean "PULL" in the subject?
>
> git-publish is just obeying your profile settings:
>
> +[gitpublishprofile "default"]
> +base = master
> +prefix = PATCH
>  ^---- :)
>
> I used PULL myself in the past but am happy with using whatever we
> decide the standard should be.
>
> Shall we remove prefix from the profiles (it defaults to "PATCH" for
> regular patches and "PULL" for pull requests)?

Yes, this is a bug. In my ~/.gitconfig I had "prefix = PATCH ..." for
all other qemu-* profiles but not the main one, "qemu". When I wrote
the patch I didn't realize it is so for a reason, and "fixed" it!

Fam
Stefan Hajnoczi Feb. 15, 2018, 9:39 a.m. | #4
On Wed, Feb 14, 2018 at 09:58:27PM +0800, Fam Zheng wrote:
> On Wed, Feb 14, 2018 at 9:25 PM, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> > On Wed, Feb 14, 2018 at 08:48:20AM +0800, Fam Zheng wrote:
> >> On Tue, 02/13 17:34, Stefan Hajnoczi wrote:
> >> > The following changes since commit fb68096da3d35e64c88cd610c1fa42766c58e92a:
> >> >
> >> >   Revert "tests: use memfd in vhost-user-test" (2018-02-13 09:51:52 +0000)
> >> >
> >> > are available in the Git repository at:
> >> >
> >> >   git://github.com/stefanha/qemu.git tags/block-pull-request
> >> >
> >> > for you to fetch changes up to 64b01feca991e5b19a5d750ef77cdca92b68bdbb:
> >> >
> >> >   misc: fix spelling (2018-02-13 15:38:17 +0000)
> >>
> >> Did you mean "PULL" in the subject?
> >
> > git-publish is just obeying your profile settings:
> >
> > +[gitpublishprofile "default"]
> > +base = master
> > +prefix = PATCH
> >  ^---- :)
> >
> > I used PULL myself in the past but am happy with using whatever we
> > decide the standard should be.
> >
> > Shall we remove prefix from the profiles (it defaults to "PATCH" for
> > regular patches and "PULL" for pull requests)?
> 
> Yes, this is a bug. In my ~/.gitconfig I had "prefix = PATCH ..." for
> all other qemu-* profiles but not the main one, "qemu". When I wrote
> the patch I didn't realize it is so for a reason, and "fixed" it!

Please resend your series once this email thread has run its course.

I am dropping it from the pull request because I'd like to merge the
other fixes in this pull request as soon as possible.

Stefan