From: Thomas Huth <thuth@redhat.com>
To: Alexander Bulekov <alxndr@bu.edu>, qemu-devel@nongnu.org
Cc: "Stefan Hajnoczi" <stefanha@redhat.com>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
"Mauro Matteo Cascella" <mcascell@redhat.com>,
"Peter Xu" <peterx@redhat.com>,
"Jason Wang" <jasowang@redhat.com>,
"David Hildenbrand" <david@redhat.com>,
"Gerd Hoffmann" <kraxel@redhat.com>,
"Laurent Vivier" <lvivier@redhat.com>,
"Bandan Das" <bsd@redhat.com>,
"Edgar E . Iglesias" <edgar.iglesias@gmail.com>,
"Darren Kenny" <darren.kenny@oracle.com>,
"Bin Meng" <bin.meng@windriver.com>,
"Paolo Bonzini" <pbonzini@redhat.com>,
"Michael S . Tsirkin" <mst@redhat.com>,
"Marcel Apfelbaum" <marcel.apfelbaum@gmail.com>,
"Daniel P . Berrangé" <berrange@redhat.com>,
"Eduardo Habkost" <eduardo@habkost.net>,
"Jon Maloy" <jmaloy@redhat.com>, "Siqi Chen" <coc.cyqh@gmail.com>
Subject: Re: [PATCH v7 0/6] memory: prevent dma-reentracy issues
Date: Mon, 13 Mar 2023 15:02:30 +0100 [thread overview]
Message-ID: <c61ecf23-d055-01a8-b2d7-37367c4c1f63@redhat.com> (raw)
In-Reply-To: <20230313082417.827484-1-alxndr@bu.edu>
On 13/03/2023 09.24, Alexander Bulekov wrote:
> v6 -> v7:
> - Fix bad qemu_bh_new_guarded calls found by Thomas (Patch 4)
> - Add an MR-specific flag to disable reentrancy (Patch 5)
> - Disable reentrancy checks for lsi53c895a's RAM-like MR (Patch 6)
>
> Patches 5 and 6 need review. I left the review-tags for Patch 4,
> however a few of the qemu_bh_new_guarded calls have changed.
Hi Alexander,
there seems to be another issue with one of the avocado tests:
make -j8 qemu-system-aarch64
make check-venv
./tests/venv/bin/avocado run \
tests/avocado/boot_linux_console.py:BootLinuxConsole.test_aarch64_raspi3_atf
... works fine for me with the master branch, but it fails
for me after applying your patch series.
Can you reproduce that failure?
Thomas
next prev parent reply other threads:[~2023-03-13 14:03 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-13 8:24 [PATCH v7 0/6] memory: prevent dma-reentracy issues Alexander Bulekov
2023-03-13 8:24 ` [PATCH v7 1/6] " Alexander Bulekov
2023-03-13 8:45 ` Philippe Mathieu-Daudé
2023-03-13 9:15 ` Alexander Bulekov
2023-03-13 9:16 ` Alexander Bulekov
2023-03-13 10:06 ` Philippe Mathieu-Daudé
2023-03-13 8:24 ` [PATCH v7 2/6] async: Add an optional reentrancy guard to the BH API Alexander Bulekov
2023-03-13 8:24 ` [PATCH v7 3/6] checkpatch: add qemu_bh_new/aio_bh_new checks Alexander Bulekov
2023-03-13 8:24 ` [PATCH v7 4/6] hw: replace most qemu_bh_new calls with qemu_bh_new_guarded Alexander Bulekov
2023-03-13 8:36 ` Thomas Huth
2023-03-13 8:24 ` [PATCH v7 5/6] memory: Allow disabling re-entrancy checking per-MR Alexander Bulekov
2023-03-13 8:36 ` Thomas Huth
2023-03-13 12:21 ` Darren Kenny
2023-03-13 8:24 ` [PATCH v7 6/6] lsi53c895a: disable reentrancy detection for script RAM Alexander Bulekov
2023-03-13 8:38 ` Thomas Huth
2023-03-13 12:23 ` Darren Kenny
2023-04-21 11:30 ` Michael Tokarev
2023-03-13 14:02 ` Thomas Huth [this message]
2023-03-13 14:52 ` [PATCH v7 0/6] memory: prevent dma-reentracy issues Alexander Bulekov
2023-03-13 15:41 ` Philippe Mathieu-Daudé
2023-03-13 16:08 ` Peter Maydell
2023-03-13 17:20 ` Alexander Bulekov
2023-03-13 16:18 ` Thomas Huth
2023-03-13 20:39 ` Philippe Mathieu-Daudé
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=c61ecf23-d055-01a8-b2d7-37367c4c1f63@redhat.com \
--to=thuth@redhat.com \
--cc=alxndr@bu.edu \
--cc=berrange@redhat.com \
--cc=bin.meng@windriver.com \
--cc=bsd@redhat.com \
--cc=coc.cyqh@gmail.com \
--cc=darren.kenny@oracle.com \
--cc=david@redhat.com \
--cc=edgar.iglesias@gmail.com \
--cc=eduardo@habkost.net \
--cc=jasowang@redhat.com \
--cc=jmaloy@redhat.com \
--cc=kraxel@redhat.com \
--cc=lvivier@redhat.com \
--cc=marcel.apfelbaum@gmail.com \
--cc=mcascell@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peterx@redhat.com \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@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).