[PULL] virtio: last minute bugfix

[PULL] virtio: last minute bugfix

[Michael S. Tsirkin]

We had a lot of trouble agreeing on a fix, but apparently there's
finally a consensus.

The following changes since commit 39dae59d66acd86d1de24294bd2f343fd5e7a625:

  Linux 4.14-rc8 (2017-11-05 13:05:14 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git tags/for_linus

for you to fetch changes up to 823b45b4f154384941d57e124ed726847e293b36:

  virtio_balloon: fix deadlock on OOM (2017-11-07 17:24:59 +0200)

----------------------------------------------------------------
virtio: bugfix

Fixes a deadlock encountered with the virtio balloon.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>

----------------------------------------------------------------
Michael S. Tsirkin (1):
      virtio_balloon: fix deadlock on OOM

 drivers/virtio/virtio_balloon.c    | 30 ++++++++++++++++++++++--------
 include/linux/balloon_compaction.h | 38 +++++++++++++++++++++++++++++++++++++-
 mm/balloon_compaction.c            | 27 +++++++++++++++++++++------
 3 files changed, 80 insertions(+), 15 deletions(-)

Re: [PULL] virtio: last minute bugfix

[Linus Torvalds]

On Tue, Nov 7, 2017 at 7:28 AM, Michael S. Tsirkin <mst@redhat.com> wrote:
> We had a lot of trouble agreeing on a fix, but apparently there's
> finally a consensus.

This is big and doesn't have a sign-off on the commit itself, even if
you then do it in the tag.

That's not how this is all supposed to work.

I guess I'll take it, but please don't do things like this to me.

             Linus

Re: [PULL] virtio: last minute bugfix

[Linus Torvalds]

On Tue, Nov 7, 2017 at 9:23 AM, Linus Torvalds
<torvalds@linux-foundation.org> wrote:
>
> I guess I'll take it, but please don't do things like this to me.

Oh no I wont.

The garbage you sent me doesn't even compile cleanly, and is utter shite.

Not acceptable for last-minute bugfixes, and you're now on my shit-list.

                Linus

Re: [PULL] virtio: last minute bugfix

[Michael S. Tsirkin]

On Tue, Nov 07, 2017 at 09:29:59AM -0800, Linus Torvalds wrote:
> On Tue, Nov 7, 2017 at 9:23 AM, Linus Torvalds
> <torvalds@linux-foundation.org> wrote:
> >
> > I guess I'll take it, but please don't do things like this to me.
> 
> Oh no I wont.
> 
> The garbage you sent me doesn't even compile cleanly, and is utter shite.
> 
> Not acceptable for last-minute bugfixes, and you're now on my shit-list.
> 
>                 Linus

Sorry about that.

I'll investigate what went wrong.

Will be more careful not to cut corners next time around, just follow
the standard procedure.

-- 
MST

Re: [PULL] virtio: last minute bugfix

[Michael S. Tsirkin]

On Tue, Nov 07, 2017 at 08:13:10PM +0200, Michael S. Tsirkin wrote:
> On Tue, Nov 07, 2017 at 09:29:59AM -0800, Linus Torvalds wrote:
> > On Tue, Nov 7, 2017 at 9:23 AM, Linus Torvalds
> > <torvalds@linux-foundation.org> wrote:
> > >
> > > I guess I'll take it, but please don't do things like this to me.
> > 
> > Oh no I wont.
> > 
> > The garbage you sent me doesn't even compile cleanly, and is utter shite.
> > 
> > Not acceptable for last-minute bugfixes, and you're now on my shit-list.
> > 
> >                 Linus
> 
> Sorry about that.
> 
> I'll investigate what went wrong.
> 
> Will be more careful not to cut corners next time around, just follow
> the standard procedure.

All right, my local tests didn't fail on new warnings, and I didn't give
the zero day infrastructure enough time to do it's job.

Lesson hopefully learned - don't rush it, give tools the time to do
their job.

Wei, you'll want to respin your 4.15 patchset on top of my fixed tree.
At this point the fix will only land in 4.15, sorry about that.

Thanks everyone.

> -- 
> MST

Re: [PULL] virtio: last minute bugfix

[Wei Wang]

On 11/08/2017 03:23 AM, Michael S. Tsirkin wrote:
> On Tue, Nov 07, 2017 at 08:13:10PM +0200, Michael S. Tsirkin wrote:
>> On Tue, Nov 07, 2017 at 09:29:59AM -0800, Linus Torvalds wrote:
>>> On Tue, Nov 7, 2017 at 9:23 AM, Linus Torvalds
>>> <torvalds@linux-foundation.org> wrote:
>>>> I guess I'll take it, but please don't do things like this to me.
>>> Oh no I wont.
>>>
>>> The garbage you sent me doesn't even compile cleanly, and is utter shite.
>>>
>>> Not acceptable for last-minute bugfixes, and you're now on my shit-list.
>>>
>>>                  Linus
>> Sorry about that.
>>
>> I'll investigate what went wrong.
>>
>> Will be more careful not to cut corners next time around, just follow
>> the standard procedure.
> All right, my local tests didn't fail on new warnings, and I didn't give
> the zero day infrastructure enough time to do it's job.
>
> Lesson hopefully learned - don't rush it, give tools the time to do
> their job.
>
> Wei, you'll want to respin your 4.15 patchset on top of my fixed tree.
> At this point the fix will only land in 4.15, sorry about that.
>
> Thanks everyone.
>

OK, I'll use the fixed tree. Thanks, Michael.


Best,
Wei