From: Zhao Liu <zhao1.liu@intel.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
"Philippe Mathieu-Daudé" <philmd@linaro.org>,
qemu-devel <qemu-devel@nongnu.org>,
qemu-rust@nongnu.org, "Alex Bennée" <alex.bennee@linaro.org>,
"Marc-André Lureau" <marcandre.lureau@redhat.com>
Subject: Re: [PULL 25/41] rust: qom: put class_init together from multiple ClassInitImpl<>
Date: Tue, 7 Jan 2025 23:55:52 +0800 [thread overview]
Message-ID: <Z31OiMcYT/66AGDt@intel.com> (raw)
In-Reply-To: <CABgObfZ7FFW5yhA2u6Rt5=MndrLsF=BnNhgE73LAyUWUGF=yXQ@mail.gmail.com>
On Mon, Jan 06, 2025 at 04:21:16PM +0100, Paolo Bonzini wrote:
> Date: Mon, 6 Jan 2025 16:21:16 +0100
> From: Paolo Bonzini <pbonzini@redhat.com>
> Subject: Re: [PULL 25/41] rust: qom: put class_init together from multiple
> ClassInitImpl<>
>
> Il lun 6 gen 2025, 14:32 Peter Maydell <peter.maydell@linaro.org> ha
> scritto:
>
> > I think here I agree with Philippe that we might as well
> > provide only the new API to Rust devices.
> >
>
> Ok, I wasn't thinking of doing that because there isn't right now an easy
> way to add interfaces to Rust-defined classes. However, all devices are
> Resettable and Device isn't defined in Rust, so it's not hard to add a
> ResettableImpl trait in rust/qemu-api/srv/qdev.rs, and initialize it for
> all devices.
>
> If anybody wants to do it as an exercise, I am happy to help, otherwise I
> can prepare a patch too.
>
If possible and if no one else wants to practice, I would also like to
give it a try (I'll add it to my list).
Zhao
next prev parent reply other threads:[~2025-01-07 15:37 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-19 8:31 [PULL 00/41] Rust, qdev, target/i386 changes for 2024-12-19 Paolo Bonzini
2024-12-19 8:31 ` [PULL 01/41] migration: Constify migration_properties Paolo Bonzini
2024-12-19 8:31 ` [PULL 02/41] hw/ide: Constify sysbus_ahci_properties Paolo Bonzini
2024-12-19 8:31 ` [PULL 03/41] target/ppc: Remove empty property list Paolo Bonzini
2024-12-19 8:31 ` [PULL 04/41] target/s390x: Use s390x_cpu_properties for system mode only Paolo Bonzini
2024-12-19 8:31 ` [PULL 05/41] hw/pci-host/astro: Remove empty Property list Paolo Bonzini
2024-12-19 8:31 ` [PULL 06/41] hw/ppc: Only register spapr_nvdimm_properties if CONFIG_LIBPMEM Paolo Bonzini
2024-12-19 8:31 ` [PULL 07/41] hw/tricore: Remove empty Property lists Paolo Bonzini
2024-12-19 8:31 ` [PULL 08/41] hw/s390x: " Paolo Bonzini
2024-12-19 8:31 ` [PULL 09/41] hw/xen: " Paolo Bonzini
2024-12-19 8:31 ` [PULL 10/41] hw/sparc: " Paolo Bonzini
2024-12-19 8:31 ` [PULL 11/41] hw/virtio: " Paolo Bonzini
2024-12-19 8:31 ` [PULL 12/41] include/hw/qdev-core: Detect most empty Property lists at compile time Paolo Bonzini
2024-12-19 8:32 ` [PULL 13/41] hw/core: Introduce device_class_set_props_n Paolo Bonzini
2024-12-19 8:32 ` [PULL 14/41] migration: Use device_class_set_props_n Paolo Bonzini
2024-12-19 8:32 ` [PULL 15/41] hw/scsi/megasas: " Paolo Bonzini
2024-12-19 8:32 ` [PULL 16/41] hw/arm/armsse: " Paolo Bonzini
2024-12-19 8:32 ` [PULL 17/41] rust/qemu-api: " Paolo Bonzini
2024-12-19 8:32 ` [PULL 18/41] hw/core: Replace device_class_set_props with a macro Paolo Bonzini
2024-12-19 8:32 ` [PULL 19/41] target/riscv: Do not abuse DEFINE_PROP_END_OF_LIST Paolo Bonzini
2024-12-19 8:32 ` [PULL 20/41] include/hw/qdev-properties: Remove DEFINE_PROP_END_OF_LIST Paolo Bonzini
2024-12-19 8:32 ` [PULL 21/41] include/hw/qdev-properties: Shrink struct Property Paolo Bonzini
2024-12-19 8:32 ` [PULL 22/41] hw/core/qdev-properties: Constify Property argument to object_field_prop_ptr Paolo Bonzini
2024-12-19 8:32 ` [PULL 23/41] hw/core/qdev-properties: Constify Property argument to PropertyInfo.print Paolo Bonzini
2024-12-19 8:32 ` [PULL 24/41] Constify all opaque Property pointers Paolo Bonzini
2024-12-19 8:32 ` [PULL 25/41] rust: qom: put class_init together from multiple ClassInitImpl<> Paolo Bonzini
2025-01-02 17:04 ` Philippe Mathieu-Daudé
2025-01-03 9:02 ` Zhao Liu
2025-01-03 8:48 ` Philippe Mathieu-Daudé
2025-01-06 11:53 ` Paolo Bonzini
2025-01-06 13:32 ` Peter Maydell
2025-01-06 15:21 ` Paolo Bonzini
2025-01-07 15:55 ` Zhao Liu [this message]
2025-01-07 15:50 ` Philippe Mathieu-Daudé
2025-01-07 16:03 ` Paolo Bonzini
2025-01-07 16:24 ` Philippe Mathieu-Daudé
2025-01-07 16:29 ` Paolo Bonzini
2025-01-07 16:43 ` Philippe Mathieu-Daudé
2024-12-19 8:32 ` [PULL 26/41] rust: qom: add possibility of overriding unparent Paolo Bonzini
2024-12-19 8:32 ` [PULL 27/41] rust: rename qemu-api modules to follow C code a bit more Paolo Bonzini
2024-12-19 8:32 ` [PULL 28/41] rust: re-export C types from qemu-api submodules Paolo Bonzini
2024-12-19 8:32 ` [PULL 29/41] rust: tests: allow writing more than one test Paolo Bonzini
2024-12-19 8:32 ` [PULL 30/41] rust: qom: add casting functionality Paolo Bonzini
2024-12-19 8:32 ` [PULL 31/41] rust: qom: add initial subset of methods on Object Paolo Bonzini
2024-12-19 8:32 ` [PULL 32/41] rust: qemu-api: add a module to wrap functions and zero-sized closures Paolo Bonzini
2024-12-19 8:32 ` [PULL 33/41] kvm: consistently return 0/-errno from kvm_convert_memory Paolo Bonzini
2024-12-19 8:32 ` [PULL 34/41] target/i386: Reset TSCs of parked vCPUs too on VM reset Paolo Bonzini
2024-12-19 8:32 ` [PULL 35/41] rust: pl011: fix declaration of LineControl bits Paolo Bonzini
2024-12-19 8:32 ` [PULL 36/41] rust: pl011: match break logic of C version Paolo Bonzini
2024-12-19 8:32 ` [PULL 37/41] rust: pl011: always use reset() method on registers Paolo Bonzini
2024-12-19 8:32 ` [PULL 38/41] rust: pl011: fix break errors and definition of Data struct Paolo Bonzini
2024-12-19 8:32 ` [PULL 39/41] rust: pl011: extend registers to 32 bits Paolo Bonzini
2024-12-19 8:32 ` [PULL 40/41] rust: pl011: fix migration stream Paolo Bonzini
2024-12-19 8:32 ` [PULL 41/41] rust: pl011: simplify handling of the FIFO enabled bit in LCR Paolo Bonzini
2024-12-19 15:12 ` [PULL 00/41] Rust, qdev, target/i386 changes for 2024-12-19 Richard Henderson
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=Z31OiMcYT/66AGDt@intel.com \
--to=zhao1.liu@intel.com \
--cc=alex.bennee@linaro.org \
--cc=marcandre.lureau@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=philmd@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-rust@nongnu.org \
/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.