Re: [GIT PULL] virtio: features, fixes

From: Michael S. Tsirkin
Date: Tue Mar 19 2024 - 15:24:59 EST


On Tue, Mar 19, 2024 at 11:03:44AM -0700, Linus Torvalds wrote:
> On Tue, 19 Mar 2024 at 00:41, Michael S. Tsirkin <mst@xxxxxxxxxx> wrote:
> >
> > virtio: features, fixes
> >
> > Per vq sizes in vdpa.
> > Info query for block devices support in vdpa.
> > DMA sync callbacks in vduse.
> >
> > Fixes, cleanups.
>
> Grr. I thought the merge message was a bit too terse, but I let it slide.
>
> But only after pushing it out do I notice that not only was the pull
> request message overly terse, you had also rebased this all just
> moments before sending the pull request and didn't even give a hit of
> a reason for that.
>
> So I missed that, and the merge is out now, but this was NOT OK.
>
> Yes, rebasing happens. But last-minute rebasing needs to be explained,
> not some kind of nasty surprise after-the-fact.
>
> And that pull request explanation was really borderline even *without*
> that issue.
>
> Linus

OK thanks Linus and sorry. I did that rebase for testing then I thought
hey history looks much nicer now why don't I switch to that. Just goes
to show not to do this thing past midnight, I write better merge
messages at sane hours, too.

--
MST