From: "Alex Bennée" <alex.bennee@linaro.org>
To: Markus Armbruster <armbru@redhat.com>
Cc: Gustavo Romero <gustavo.romero@linaro.org>,
qemu-devel@nongnu.org, qemu-arm@nongnu.org, philmd@linaro.org,
thuth@redhat.com, Bill Mills <bill.mills@linaro.org>
Subject: Re: [RESEND][PATCH v3 0/7] Add ivshmem-flat device
Date: Tue, 07 Jan 2025 17:02:31 +0000 [thread overview]
Message-ID: <874j2ad0bc.fsf@draig.linaro.org> (raw)
In-Reply-To: <87r05e8zv4.fsf@pond.sub.org> (Markus Armbruster's message of "Tue, 07 Jan 2025 15:25:51 +0100")
Markus Armbruster <armbru@redhat.com> writes:
> Gustavo Romero <gustavo.romero@linaro.org> writes:
>
>> This is a resend of the series:
>>
>> https://lore.kernel.org/qemu-devel/20240222222218.2261956-1-gustavo.romero@linaro.org/
>>
>> rebased on the current master. The series was sent about 9 months ago and
>> remains relevant. Besides addressing the longstanding issue:
>>
>> https://gitlab.com/qemu-project/qemu/-/issues/1134
>>
>> it has generated interest in the community at least twice since its last
>> version, from different contexts:
>>
>> https://lists.nongnu.org/archive/html/qemu-discuss/2024-05/msg00003.html
>> https://lists.nongnu.org/archive/html/qemu-devel/2024-09/msg00374.html
>>
>> This suggests the series is being used out-of-tree in various contexts, such
>> as experiments with heterogeneous architectures.
>>
>> But due to the fact it relies on sysbus, which is marked for future removal,
>> some maintainers objected to accepting the patchset, causing it to be held in
>> the ML.
>
> Actually, I inquired about the use cases, and was told it's for OpenAMP.
> I challenged the use of ivshmem for that purpose in some detail[*], but
> got no reply.
>
>> However, given the ongoing community interest and since currently there
>> isn't a better way on QEMU than using sysbus for the wiring needs of this
>> device (e.g. to wire the device to a CPU IRQ input line), I'd kindly like to ask
>> maintainers to reconsider its acceptance.
>
> [*] https://lore.kernel.org/qemu-devel/87zfth4psf.fsf@pond.sub.org/
Yes the principle use case is modelling asymmetric set ups with two sets
of CPU cores attached only by a piece of shared RAM with maybe a
signalling an IRQ. As the CPUs are doing the work we want to test both
sides (VirtIO device and driver) rather than provide an emulation.
Once we have a single QEMU binary that is dynamically configurable and
supports heterogeneous setups then we can model it within QEMU itself.
However until that point 2 QEMU's and some shared memory is the easiest
way to test such things.
This is currently being worked on as part of the larger HVAC project:
https://linaro.atlassian.net/wiki/spaces/HVAC/overview
where we are working on a new VirtIO transport layer (virtio-msg) that
doesn't have the issues associated with lack of trapped configuration
space.
>
> [...]
--
Alex Bennée
Virtualisation Tech Lead @ Linaro
prev parent reply other threads:[~2025-01-07 17:02 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-16 14:18 [RESEND][PATCH v3 0/7] Add ivshmem-flat device Gustavo Romero
2024-12-16 14:18 ` [PATCH v3 1/7] hw/misc/ivshmem-flat: " Gustavo Romero
2024-12-31 18:01 ` Philippe Mathieu-Daudé
2024-12-31 19:10 ` Philippe Mathieu-Daudé
2024-12-16 14:18 ` [PATCH v3 2/7] hw/misc/ivshmem-flat: Allow device to wire itself on sysbus Gustavo Romero
2024-12-16 14:18 ` [PATCH v3 3/7] hw/arm: Allow some machines to use the ivshmem-flat device Gustavo Romero
2024-12-16 14:18 ` [PATCH v3 4/7] hw/misc/ivshmem: Rename ivshmem to ivshmem-pci Gustavo Romero
2024-12-16 14:18 ` [PATCH v3 5/7] tests/qtest: Reorganize common code in ivshmem-test Gustavo Romero
2024-12-16 14:18 ` [PATCH v3 6/7] tests/qtest: Add API functions to capture IRQ toggling Gustavo Romero
2025-01-07 18:35 ` Philippe Mathieu-Daudé
2025-01-07 18:57 ` Philippe Mathieu-Daudé
2024-12-16 14:18 ` [PATCH v3 7/7] tests/qtest: Add ivshmem-flat test Gustavo Romero
2025-01-07 14:25 ` [RESEND][PATCH v3 0/7] Add ivshmem-flat device Markus Armbruster
2025-01-07 17:02 ` Alex Bennée [this message]
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=874j2ad0bc.fsf@draig.linaro.org \
--to=alex.bennee@linaro.org \
--cc=armbru@redhat.com \
--cc=bill.mills@linaro.org \
--cc=gustavo.romero@linaro.org \
--cc=philmd@linaro.org \
--cc=qemu-arm@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.