[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 20/67] ahci: clear aiocb in ncq_cb
From: |
Michael Roth |
Subject: |
Re: [Qemu-devel] [PATCH 20/67] ahci: clear aiocb in ncq_cb |
Date: |
Wed, 21 Dec 2016 10:56:57 -0600 |
User-agent: |
alot/0.3.6 |
Quoting John Snow (2016-12-19 17:07:17)
> Hi, some more considerations:
>
>
> e7bd708ec85e40fd51569bb90c52d6613ffd8f45
> atapi: classify read_cd as conditionally returning data
>
> This fixes problems with certain operating systems being unable to read
> from the CDROM.
applied.
>
>
> 99868af3d0a75cf6a515a9aa81bf0d7bcb39eadb
> dma-helpers: explicitly pass alignment into DMA helpers
>
> I believe this does fix MACIO for some platforms, but I'm not sure how
> annoying it would be to backport due to the DMA alignment changes. Low
> priority.
Seems to apply cleanly, but change seems mechanical. I assume the macio fix
comes from the follow-up be1e343995ef81fc05d9a4e1ec263ca171d842e7.
both applied.
>
>
> c47ee043dc2cc85da710e87524144a720598c096
> block-backend: Always notify on blk_eject
>
> This fixes a legitimate bug in some versions of libvirt.
applied, but modified to avoid the need for 2d76e724 since that
requires qapi api changes.
>
>
> ca44141d5fb801dd5903102acefd0f2d8e8bb6a1
> ide: Fix memory leak in ide_register_restart_cb()
>
> Not important, but not risky or hard to backport either. Skippable.
applied.
>
>
> 9da82227caa74fb6fbea224dad91fe5b7cc115a5
> ide: fix DMA register transitions
>
> This is only theoretical. Safely skipped.
Slight chance some OSs might not deal with this as expected so I'd opt
to leave it out if there haven't been any observed bugs around it.
>
>
> e305a16510afa74eec20390479e349402e55ef4c portio: keep references on portio
>
> Doesn't look important as it doesn't actually FIX a leak.
Agreed, but went ahead and pulled it in since it's a context dependency for
ca44141.
>
>
> There were some fixes for Transactional Block Jobs as well in the 2.8
> window, but as we still haven't quite finished nailing down incremental
> backups, it may not be too important to backport to 2.7 -- I don't think
> anyone is using this yet.
>
> The series in question is:
> https://lists.nongnu.org/archive/html/qemu-devel/2016-11/msg01392.html
There are some interdependencies between this and bae8196+ that I don't
trust myself to properly untangle. And backporting bae8196 seems risky.
I think I'd need a direct backport for 2.7, but at this point there's not
really any time left for all that so I'll opt to leave these out for
2.7.1.
Thanks for all the suggestions.
>
> On 12/14/2016 07:44 PM, Michael Roth wrote:
> > From: John Snow <address@hidden>
> >
> > Similar to existing fixes for IDE (87ac25fd) and ATAPI (7f951b2d), the
> > AIOCB must be cleared in the callback. Otherwise, we may accidentally
> > try to reset a dangling pointer in bdrv_aio_cancel() from a port reset.
> >
> > Signed-off-by: John Snow <address@hidden>
> > Reviewed-by: Stefan Hajnoczi <address@hidden>
> > Message-id: address@hidden
> > Signed-off-by: John Snow <address@hidden>
> > (cherry picked from commit df403bc58859c893ebd0accda07678e84d15dc5d)
> > Signed-off-by: Michael Roth <address@hidden>
> > ---
> > hw/ide/ahci.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c
> > index f3438ad..63ead21 100644
> > --- a/hw/ide/ahci.c
> > +++ b/hw/ide/ahci.c
> > @@ -948,6 +948,7 @@ static void ncq_cb(void *opaque, int ret)
> > NCQTransferState *ncq_tfs = (NCQTransferState *)opaque;
> > IDEState *ide_state = &ncq_tfs->drive->port.ifs[0];
> >
> > + ncq_tfs->aiocb = NULL;
> > if (ret == -ECANCELED) {
> > return;
> > }
> >
>
> --
> —js
>
- [Qemu-devel] [PATCH 13/67] vmw_pvscsi: check page count while initialising descriptor rings, (continued)
- [Qemu-devel] [PATCH 13/67] vmw_pvscsi: check page count while initialising descriptor rings, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 12/67] scsi-disk: change disk serial length from 20 to 36, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 15/67] scsi: mptconfig: fix misuse of MPTSAS_CONFIG_PACK, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 14/67] scsi: mptconfig: fix an assert expression, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 16/67] crypto: ensure XTS is only used with ciphers with 16 byte blocks, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 18/67] scsi-disk: Cleaning up around tray open state, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 17/67] iothread: Stop threads before main() quits, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 19/67] virtio-scsi: Don't abort when media is ejected, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 20/67] ahci: clear aiocb in ncq_cb, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 01/67] linux-headers: update, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 21/67] s390x/css: handle cssid 255 correctly, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 23/67] qcow2: fix encryption during cow of sectors, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 24/67] iscsi: Fix divide-by-zero regression on raw SG devices, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 22/67] vfio/pci: Fix regression in MSI routing configuration, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 25/67] block: reintroduce bdrv_flush_all, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 26/67] qemu: use bdrv_flush_all for vm_stop et al, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 30/67] qht: fix unlock-after-free segfault upon resizing, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 27/67] block-backend: remove blk_flush_all, Michael Roth, 2016/12/14
- [Qemu-devel] [PATCH 02/67] hw/ppc/spapr: Move code related to "ibm, pa-features" to a separate function, Michael Roth, 2016/12/14