From: "Cédric Le Goater" <clg@redhat.com>
To: Avihai Horon <avihaih@nvidia.com>,
qemu-devel@nongnu.org,
Alex Williamson <alex.williamson@redhat.com>
Cc: Eric Auger <eric.auger@redhat.com>,
Zhenzhong Duan <zhenzhong.duan@intel.com>,
John Levon <john.levon@nutanix.com>,
Joao Martins <joao.m.martins@oracle.com>
Subject: Re: [PATCH for-10.1 v2 27/37] vfio: Make vfio_devices_query_dirty_bitmap() static
Date: Mon, 7 Apr 2025 09:03:29 +0200 [thread overview]
Message-ID: <1a7f1d26-279d-4bb4-b5d4-6f28c24e2cc2@redhat.com> (raw)
In-Reply-To: <74f8e5b1-a52e-49b3-b69e-52b3b59fcfd1@nvidia.com>
On 4/2/25 14:43, Avihai Horon wrote:
>
> On 26/03/2025 9:51, Cédric Le Goater wrote:
>> External email: Use caution opening links or attachments
>>
>>
>> vfio_devices_query_dirty_bitmap() is only used in "container-base.c".
>> Also, rename to vfio_container_devices_query_dirty_bitmap() to reflect
>> with the prefix 'vfio_container_devices_' that it simply loops over
>> the container's device list.
>>
>> This rename also frees the vfio_devices_query_dirty_bitmap() name
>> which we will reuse in the next changes.
>
> Nit: this sentence can be dropped, vfio_devices_query_dirty_bitmap() is not used in the following changes IIUC.
I was referring to vfio_container_query_dirty_bitmap() but I agree
the sentence is not that useful. Will drop.
>>
>> Signed-off-by: Cédric Le Goater <clg@redhat.com>
>
> Reviewed-by: Avihai Horon <avihaih@nvidia.com>
>
Since lore didn't receive the email :
Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Thanks,
C.
next prev parent reply other threads:[~2025-04-07 7:04 UTC|newest]
Thread overview: 102+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-03-26 7:50 [PATCH for-10.1 v2 00/37] vfio: Spring cleanup Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 01/37] vfio: Move vfio_mig_active() into migration.c Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 02/37] vfio: Rename vfio_reset_bytes_transferred() Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 03/37] vfio: Introduce a new header file for external migration services Cédric Le Goater
2025-03-27 9:32 ` John Levon
2025-03-27 12:07 ` Prasad Pandit
[not found] ` <d22dc2c7-b552-4341-a3db-fe99a1b52ef9@nvidia.com>
2025-04-07 6:37 ` Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 04/37] vfio: Make vfio_un/block_multiple_devices_migration() static Cédric Le Goater
2025-03-27 9:31 ` John Levon
2025-03-26 7:50 ` [PATCH for-10.1 v2 05/37] vfio: Make vfio_viommu_preset() static Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 06/37] vfio: Introduce a new header file for internal migration services Cédric Le Goater
2025-03-27 10:38 ` John Levon
2025-03-27 12:16 ` Prasad Pandit
[not found] ` <8667eb45-f908-4442-983f-fb1a9ac08e2f@nvidia.com>
2025-04-07 6:43 ` Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 07/37] vfio: Move vfio_device_state_is_running/precopy() into migration.c Cédric Le Goater
2025-03-27 9:50 ` John Levon
2025-03-27 12:12 ` Prasad Pandit
2025-03-26 7:50 ` [PATCH for-10.1 v2 08/37] vfio: Introduce a new header file for VFIOdisplay declarations Cédric Le Goater
2025-03-27 8:40 ` Duan, Zhenzhong
2025-03-26 7:50 ` [PATCH for-10.1 v2 09/37] vfio: Move VFIOHostDMAWindow definition into spapr.c Cédric Le Goater
2025-03-27 8:40 ` Duan, Zhenzhong
2025-03-26 7:50 ` [PATCH for-10.1 v2 10/37] vfio: Introduce a new header file for VFIOIOMMUFD declarations Cédric Le Goater
2025-03-27 8:45 ` Duan, Zhenzhong
2025-04-02 13:51 ` Joao Martins
2025-03-26 7:50 ` [PATCH for-10.1 v2 11/37] vfio: Introduce new files for VFIORegion definitions and declarations Cédric Le Goater
2025-03-27 9:37 ` Duan, Zhenzhong
2025-04-07 6:52 ` Cédric Le Goater
2025-04-07 7:53 ` Duan, Zhenzhong
2025-04-07 8:26 ` Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 12/37] vfio: Introduce a new header file for VFIOcontainer declarations Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 13/37] vfio: Make vfio_group_list static Cédric Le Goater
2025-03-26 7:50 ` [PATCH for-10.1 v2 14/37] vfio: Move VFIOAddressSpace helpers into container-base.c Cédric Le Goater
2025-03-27 9:37 ` Duan, Zhenzhong
2025-03-27 10:37 ` John Levon
2025-03-26 7:51 ` [PATCH for-10.1 v2 15/37] vfio: Move Host IOMMU type declarations into their respective files Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 16/37] vfio: Introduce a new header file for helper services Cédric Le Goater
2025-03-27 9:42 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 17/37] vfio: Move vfio_get_info_dma_avail() into helpers.c Cédric Le Goater
2025-03-27 9:52 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 18/37] vfio: Move vfio_kvm_device_add/del_fd() to helpers.c Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 19/37] vfio: Move vfio_get_device_info() " Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 20/37] vfio: Introduce a new file for VFIODevice definitions Cédric Le Goater
2025-03-27 9:52 ` John Levon
2025-03-27 9:55 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 21/37] vfio: Introduce new files for CPR definitions and declarations Cédric Le Goater
2025-03-26 14:38 ` Steven Sistare
2025-03-26 16:13 ` Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 22/37] vfio: Move vfio_kvm_device_fd() into helpers.c Cédric Le Goater
2025-03-27 9:54 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 23/37] vfio: Move vfio_device_list into device.c Cédric Le Goater
2025-03-27 10:00 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 24/37] vfio: Move vfio_de/attach_device() " Cédric Le Goater
2025-03-27 10:01 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 25/37] vfio: Move vfio_reset_handler() " Cédric Le Goater
2025-03-27 9:33 ` John Levon
2025-03-27 10:02 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 26/37] vfio: Move dirty tracking related services into container-base.c Cédric Le Goater
2025-04-02 13:30 ` Joao Martins
[not found] ` <336d0630-6661-4e4b-8376-7288bf899d7e@nvidia.com>
2025-04-07 6:58 ` Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 27/37] vfio: Make vfio_devices_query_dirty_bitmap() static Cédric Le Goater
2025-03-27 9:48 ` John Levon
2025-04-02 13:46 ` Joao Martins
[not found] ` <74f8e5b1-a52e-49b3-b69e-52b3b59fcfd1@nvidia.com>
2025-04-07 7:03 ` Cédric Le Goater [this message]
2025-03-26 7:51 ` [PATCH for-10.1 v2 28/37] vfio: Make vfio_container_query_dirty_bitmap() static Cédric Le Goater
2025-03-27 9:52 ` John Levon
2025-04-02 13:34 ` Joao Martins
2025-04-07 7:11 ` Cédric Le Goater
[not found] ` <278abea8-5408-4261-8155-24f5f9c75e7d@nvidia.com>
2025-04-07 7:07 ` Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 29/37] vfio: Rename vfio_devices_all_dirty_tracking_started() Cédric Le Goater
2025-03-27 10:33 ` John Levon
2025-04-02 13:36 ` Joao Martins
[not found] ` <cbd73b82-2727-42bc-9ee6-a633dbed083d@nvidia.com>
2025-04-07 7:16 ` Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 30/37] vfio: Rename vfio_devices_all_device_dirty_tracking() Cédric Le Goater
2025-03-27 9:38 ` John Levon
[not found] ` <b2ad86a7-4ed6-4098-a9ad-d069c3fda882@nvidia.com>
2025-04-07 7:17 ` Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 31/37] vfio: Rename vfio_get_dirty_bitmap() Cédric Le Goater
2025-03-27 9:39 ` John Levon
2025-04-07 8:30 ` Cédric Le Goater
2025-04-02 13:44 ` Joao Martins
[not found] ` <a42633a4-b8b0-44e8-98cc-35585be8fbba@nvidia.com>
2025-04-07 8:32 ` Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 32/37] vfio: Introduce new files for VFIO MemoryListener Cédric Le Goater
2025-03-27 10:33 ` John Levon
2025-04-02 13:48 ` Joao Martins
[not found] ` <30af983e-c532-43f7-8b84-8bf2a5b0d53b@nvidia.com>
2025-04-07 8:33 ` Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 33/37] vfio: Rename RAM discard related services Cédric Le Goater
2025-03-27 9:51 ` John Levon
2025-04-07 8:36 ` Cédric Le Goater
[not found] ` <993a0f66-aa85-4d53-831a-0077c5c0f730@nvidia.com>
2025-04-07 8:36 ` Cédric Le Goater
2025-03-26 7:51 ` [PATCH for-10.1 v2 34/37] vfio: Introduce vfio_listener_un/register() routines Cédric Le Goater
2025-03-27 10:35 ` John Levon
2025-04-02 13:49 ` Joao Martins
2025-03-26 7:51 ` [PATCH for-10.1 v2 35/37] vfio: Rename vfio-common.h to vfio-device.h Cédric Le Goater
2025-03-27 9:54 ` John Levon
2025-04-07 8:42 ` Cédric Le Goater
2025-03-27 10:07 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 36/37] vfio: Rename VFIODevice related services Cédric Le Goater
2025-03-27 9:50 ` John Levon
2025-04-07 8:51 ` Cédric Le Goater
2025-03-27 10:08 ` Duan, Zhenzhong
2025-03-26 7:51 ` [PATCH for-10.1 v2 37/37] vfio: Rename VFIOContainer " Cédric Le Goater
2025-03-27 10:09 ` Duan, Zhenzhong
2025-04-07 9:00 ` [PATCH for-10.1 v2 00/37] vfio: Spring cleanup Cédric Le Goater
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=1a7f1d26-279d-4bb4-b5d4-6f28c24e2cc2@redhat.com \
--to=clg@redhat.com \
--cc=alex.williamson@redhat.com \
--cc=avihaih@nvidia.com \
--cc=eric.auger@redhat.com \
--cc=joao.m.martins@oracle.com \
--cc=john.levon@nutanix.com \
--cc=qemu-devel@nongnu.org \
--cc=zhenzhong.duan@intel.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).