qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: John Snow <jsnow@redhat.com>
To: Michael Tokarev <mjt@tls.msk.ru>
Cc: qemu-devel <qemu-devel@nongnu.org>,
	Thomas Huth <thuth@redhat.com>,
	 Laurent Vivier <lvivier@redhat.com>,
	Qemu-block <qemu-block@nongnu.org>,
	 Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: [PULL for-6.2 0/7] Ide patches
Date: Thu, 7 Sep 2023 12:54:16 -0400	[thread overview]
Message-ID: <CAFn=p-aueHBXMFHgw=Y8XYyeaFZKFRc8vJHQ6PQ8gNSqPcii8Q@mail.gmail.com> (raw)
In-Reply-To: <7ef011cd-09e7-c5f9-dc9d-173ff8f431c8@tls.msk.ru>

[-- Attachment #1: Type: text/plain, Size: 1165 bytes --]

On Thu, Sep 7, 2023, 12:49 PM Michael Tokarev <mjt@tls.msk.ru> wrote:

> 07.09.2023 06:42, John Snow wrote:
>
> > ----------------------------------------------------------------
> > IDE Pull request
> >
> > ----------------------------------------------------------------
> >
> > Niklas Cassel (7):
> >    hw/ide/core: set ERR_STAT in unsupported command completion
> >    hw/ide/ahci: write D2H FIS when processing NCQ command
> >    hw/ide/ahci: simplify and document PxCI handling
> >    hw/ide/ahci: PxSACT and PxCI is cleared when PxCMD.ST is cleared
> >    hw/ide/ahci: PxCI should not get cleared when ERR_STAT is set
> >    hw/ide/ahci: fix ahci_write_fis_sdb()
> >    hw/ide/ahci: fix broken SError handling
>
> Is anything here stable-worthy?
>
> /mjt
>

Yes, assuming it doesn't break anything.

I can't give IDE the testing it deserves anymore, but I trust Niklas. I
don't have good test suites for *inside* linux/windows guests so I am
admittedly relying on qtests and for people to bark if something regressed.

I'd say to tentatively add them to your list and if we find regressions
during this window, we can exclude them from a stable release.

>

[-- Attachment #2: Type: text/html, Size: 1870 bytes --]

  reply	other threads:[~2023-09-07 16:55 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-09-07  3:42 [PULL for-6.2 0/7] Ide patches John Snow
2023-09-07  3:42 ` [PULL for-6.2 1/7] hw/ide/core: set ERR_STAT in unsupported command completion John Snow
2023-09-07  3:42 ` [PULL for-6.2 2/7] hw/ide/ahci: write D2H FIS when processing NCQ command John Snow
2023-09-07  3:42 ` [PULL for-6.2 3/7] hw/ide/ahci: simplify and document PxCI handling John Snow
2023-09-07  3:42 ` [PULL for-6.2 4/7] hw/ide/ahci: PxSACT and PxCI is cleared when PxCMD.ST is cleared John Snow
2023-09-07  3:42 ` [PULL for-6.2 5/7] hw/ide/ahci: PxCI should not get cleared when ERR_STAT is set John Snow
2023-09-07  3:42 ` [PULL for-6.2 6/7] hw/ide/ahci: fix ahci_write_fis_sdb() John Snow
2023-09-07  3:42 ` [PULL for-6.2 7/7] hw/ide/ahci: fix broken SError handling John Snow
2023-09-07  3:43 ` [PULL for-6.2 0/7] Ide patches John Snow
2023-09-07  6:08   ` Philippe Mathieu-Daudé
2023-09-07 15:43 ` Stefan Hajnoczi
2023-09-07 16:49 ` Michael Tokarev
2023-09-07 16:54   ` John Snow [this message]
2023-09-07 18:16     ` Michael Tokarev
2023-09-08  7:53       ` Philippe Mathieu-Daudé
2023-09-08  8:06         ` Michael Tokarev
2023-09-08  8:26           ` Michael Tokarev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAFn=p-aueHBXMFHgw=Y8XYyeaFZKFRc8vJHQ6PQ8gNSqPcii8Q@mail.gmail.com' \
    --to=jsnow@redhat.com \
    --cc=lvivier@redhat.com \
    --cc=mjt@tls.msk.ru \
    --cc=pbonzini@redhat.com \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).