mbox series

[v2,vfio,0/6] vfio/pds: Clean-ups and multi-region support

Message ID 20231117001207.2793-1-brett.creeley@amd.com (mailing list archive)
Headers show
Series vfio/pds: Clean-ups and multi-region support | expand

Message

Brett Creeley Nov. 17, 2023, 12:12 a.m. UTC
This series contains various clean-ups, improvements, and support
for multiple dirty tracking regions. The majority of clean-up and
improvements are in preparation for the last patch in the series,
which adds support for multiple dirty tracking regions.

Changes:

v2:
- Make use of BITS_PER_BYTE #define
- Use C99 style for loops
- Fix subject line to use vfio/pds instead of pds-vfio-pci
- Separate out some calculation fixes into another patch
  so it can be backported to 6.6-stable
- Fix bounds check in pds_vfio_get_region()

v1:
https://lore.kernel.org/kvm/20231114210129.34318-1-brett.creeley@amd.com/T/

Brett Creeley (6):
  vfio/pds: Fix calculations in pds_vfio_dirty_sync
  vfio/pds: Only use a single SGL for both seq and ack
  vfio/pds: Move and rename region specific info
  vfio/pds: Pass region info to relevant functions
  vfio/pds: Move seq/ack bitmaps into region struct
  vfio/pds: Add multi-region support

 drivers/vfio/pci/pds/dirty.c | 309 ++++++++++++++++++++++-------------
 drivers/vfio/pci/pds/dirty.h |  18 +-
 2 files changed, 204 insertions(+), 123 deletions(-)

Comments

Jason Gunthorpe Nov. 29, 2023, 8:18 p.m. UTC | #1
On Thu, Nov 16, 2023 at 04:12:01PM -0800, Brett Creeley wrote:
> This series contains various clean-ups, improvements, and support
> for multiple dirty tracking regions. The majority of clean-up and
> improvements are in preparation for the last patch in the series,
> which adds support for multiple dirty tracking regions.

I did not look closely at every line but this looked Ok to me

Thanks,
Jason
Alex Williamson Dec. 5, 2023, midnight UTC | #2
On Thu, 16 Nov 2023 16:12:01 -0800
Brett Creeley <brett.creeley@amd.com> wrote:

> This series contains various clean-ups, improvements, and support
> for multiple dirty tracking regions. The majority of clean-up and
> improvements are in preparation for the last patch in the series,
> which adds support for multiple dirty tracking regions.
> 
> Changes:
> 
> v2:
> - Make use of BITS_PER_BYTE #define
> - Use C99 style for loops
> - Fix subject line to use vfio/pds instead of pds-vfio-pci
> - Separate out some calculation fixes into another patch
>   so it can be backported to 6.6-stable
> - Fix bounds check in pds_vfio_get_region()
> 
> v1:
> https://lore.kernel.org/kvm/20231114210129.34318-1-brett.creeley@amd.com/T/
> 
> Brett Creeley (6):
>   vfio/pds: Fix calculations in pds_vfio_dirty_sync
>   vfio/pds: Only use a single SGL for both seq and ack
>   vfio/pds: Move and rename region specific info
>   vfio/pds: Pass region info to relevant functions
>   vfio/pds: Move seq/ack bitmaps into region struct
>   vfio/pds: Add multi-region support
> 
>  drivers/vfio/pci/pds/dirty.c | 309 ++++++++++++++++++++++-------------
>  drivers/vfio/pci/pds/dirty.h |  18 +-
>  2 files changed, 204 insertions(+), 123 deletions(-)
> 

Applied to vfio next branch for v6.8.  Thanks,

Alex