mbox series

[0/4] PCI devices passthrough pre-req patches

Message ID 20220201162508.417008-1-andr2000@gmail.com (mailing list archive)
Headers show
Series PCI devices passthrough pre-req patches | expand

Message

Oleksandr Andrushchenko Feb. 1, 2022, 4:25 p.m. UTC
From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>

Hi, all!

While working on vPCI series [1] I have created number of patches that
either add some useful helpers or serve as the ground for the upcoming
vPCI changes or both.

To ease the task of reviewing the bigger vPCI series I am sending these
now with the hope they can be accepted.

I would like to thank Roger and Jan for providing valuable comments and
ideas, some of which have materialized here.

Thank you,
Oleksandr

[1] https://patchwork.kernel.org/project/xen-devel/list/?series=585781

Oleksandr Andrushchenko (3):
  rangeset: add RANGESETF_no_print flag
  rangeset: add rangeset_reset helper function
  vpci: shrink critical section in vpci_{read/write}

Roger Pau Monne (1):
  vpci: move lock outside of struct vpci

 tools/tests/vpci/emul.h       |  5 ++-
 tools/tests/vpci/main.c       |  4 +--
 xen/arch/x86/hvm/vmsi.c       |  8 ++---
 xen/common/rangeset.c         | 11 +++++-
 xen/drivers/passthrough/pci.c |  1 +
 xen/drivers/vpci/header.c     | 21 ++++++++----
 xen/drivers/vpci/msi.c        | 11 ++++--
 xen/drivers/vpci/msix.c       |  8 ++---
 xen/drivers/vpci/vpci.c       | 64 ++++++++++++++++++++++-------------
 xen/include/xen/pci.h         |  1 +
 xen/include/xen/rangeset.h    |  8 +++--
 xen/include/xen/vpci.h        |  3 +-
 12 files changed, 94 insertions(+), 51 deletions(-)

Comments

Jan Beulich Feb. 2, 2022, 8:48 a.m. UTC | #1
On 01.02.2022 17:25, Oleksandr Andrushchenko wrote:
> Oleksandr Andrushchenko (3):
>   rangeset: add RANGESETF_no_print flag
>   rangeset: add rangeset_reset helper function
>   vpci: shrink critical section in vpci_{read/write}
> 
> Roger Pau Monne (1):
>   vpci: move lock outside of struct vpci

Btw, while I'll let Roger judge for the latter two, for the former
two while I appreciate you breaking this out from the larger series
I'm not convinced these will want committing without a user
appearing at least in close succession. Hence also why so far I
didn't commit patch 1, which as per its tags could have been put in
already.

Jan
Oleksandr Andrushchenko Feb. 2, 2022, 9:03 a.m. UTC | #2
Hi, Jan!

On 02.02.22 10:48, Jan Beulich wrote:
> On 01.02.2022 17:25, Oleksandr Andrushchenko wrote:
>> Oleksandr Andrushchenko (3):
>>    rangeset: add RANGESETF_no_print flag
>>    rangeset: add rangeset_reset helper function
>>    vpci: shrink critical section in vpci_{read/write}
>>
>> Roger Pau Monne (1):
>>    vpci: move lock outside of struct vpci
> Btw, while I'll let Roger judge for the latter two, for the former
> two while I appreciate you breaking this out from the larger series
> I'm not convinced these will want committing without a user
> appearing at least in close succession. Hence also why so far I
> didn't commit patch 1, which as per its tags could have been put in
> already.
This is fair
>
> Jan
>
Thank you,
Oleksandr