mbox series

[0/3] s390/vfio-ap: queue_configuration sysfs attribute for mdevctl automation

Message ID 20240126143533.14043-1-jjherne@linux.ibm.com
Headers
Series s390/vfio-ap: queue_configuration sysfs attribute for mdevctl automation |

Message

Jason J. Herne Jan. 26, 2024, 2:35 p.m. UTC 
  Mdevctl requires a way to atomically query and atomically update a vfio-ap
mdev's current state. This patch set creates the queue_configuration sysfs
attribute.  This new attribute allows reading and writing an mdev's entire
state in one go. If a newly written state is invalid for any reason the entire
state is rejected and the target mdev remains unchanged.

Jason J. Herne (3):
  s390/ap: Externalize AP bus specific bitmap reading function
  s390/vfio-ap: Add sysfs attr, queue_configuration, to export mdev
    state
  s390/vfio-ap: Add write support to sysfs attr ap_config

 Documentation/arch/s390/vfio-ap.rst   |  27 ++++
 drivers/s390/crypto/ap_bus.c          |  13 +-
 drivers/s390/crypto/ap_bus.h          |  22 +++
 drivers/s390/crypto/vfio_ap_ops.c     | 213 +++++++++++++++++++++++---
 drivers/s390/crypto/vfio_ap_private.h |   6 +-
 5 files changed, 248 insertions(+), 33 deletions(-)

Comments

Anthony Krowiak Jan. 29, 2024, 4:53 p.m. UTC | #1 
Isn't this v4 of this series?

On 1/26/24 9:35 AM, Jason J. Herne wrote:
> Mdevctl requires a way to atomically query and atomically update a vfio-ap
> mdev's current state. This patch set creates the queue_configuration sysfs
> attribute.  This new attribute allows reading and writing an mdev's entire
> state in one go. If a newly written state is invalid for any reason the entire
> state is rejected and the target mdev remains unchanged.
>
> Jason J. Herne (3):
>    s390/ap: Externalize AP bus specific bitmap reading function
>    s390/vfio-ap: Add sysfs attr, queue_configuration, to export mdev
>      state
>    s390/vfio-ap: Add write support to sysfs attr ap_config
>
>   Documentation/arch/s390/vfio-ap.rst   |  27 ++++
>   drivers/s390/crypto/ap_bus.c          |  13 +-
>   drivers/s390/crypto/ap_bus.h          |  22 +++
>   drivers/s390/crypto/vfio_ap_ops.c     | 213 +++++++++++++++++++++++---
>   drivers/s390/crypto/vfio_ap_private.h |   6 +-
>   5 files changed, 248 insertions(+), 33 deletions(-)
>
Matthew Rosato Jan. 29, 2024, 8:23 p.m. UTC | #2 
On 1/26/24 9:35 AM, Jason J. Herne wrote:
> Mdevctl requires a way to atomically query and atomically update a vfio-ap
> mdev's current state. This patch set creates the queue_configuration sysfs

s/queue_configuration/ap_config/

Same comment also for patch 1 commit description + patch 2 title.