* [PATCH v2 0/4] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG
@ 2022-01-27 14:29 Jean-Philippe Brucker
2022-01-27 14:29 ` [PATCH v2 1/4] linux-headers: update to v5.17-rc1 Jean-Philippe Brucker
` (3 more replies)
0 siblings, 4 replies; 20+ messages in thread
From: Jean-Philippe Brucker @ 2022-01-27 14:29 UTC (permalink / raw)
To: eric.auger
Cc: lvivier, Jean-Philippe Brucker, thuth, mst, cohuck, qemu-devel,
pasic, pbonzini
Replace the VIRTIO_IOMMU_F_BYPASS feature with
VIRTIO_IOMMU_F_BYPASS_CONFIG, which enables a config space bit to switch
global bypass on and off.
Add a boot-bypass option, which defaults to 'on' to be in line with
other vIOMMUs and to allow running firmware/bootloader that are unaware
of the IOMMU. x86 doesn't need a workaround to boot with virtio-iommu
anymore.
Since v1 [1]:
* The specification [2] and Linux header [3] changes have been merged
* Patch 1 pulls in the whole Linux header changes, in order to
get the virtio_iommu.h update
* Reset bypass on system reset, and other fixes suggested by Eric
* Added a simple test, patch 4, that checks the boot bypass value
[1] https://lore.kernel.org/qemu-devel/20210930185050.262759-1-jean-philippe@linaro.org/
[2] https://github.com/oasis-tcs/virtio-spec/issues/119
[3] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=063ebb19d962
Jean-Philippe Brucker (4):
linux-headers: update to v5.17-rc1
virtio-iommu: Default to bypass during boot
virtio-iommu: Support bypass domain
tests/qtest/virtio-iommu-test: Check bypass config
include/hw/virtio/virtio-iommu.h | 1 +
include/standard-headers/asm-x86/kvm_para.h | 1 +
include/standard-headers/drm/drm_fourcc.h | 11 ++
include/standard-headers/linux/ethtool.h | 1 +
include/standard-headers/linux/fuse.h | 60 +++++++-
include/standard-headers/linux/pci_regs.h | 142 +++++++++---------
include/standard-headers/linux/virtio_gpio.h | 72 +++++++++
include/standard-headers/linux/virtio_i2c.h | 47 ++++++
include/standard-headers/linux/virtio_iommu.h | 8 +-
.../standard-headers/linux/virtio_pcidev.h | 65 ++++++++
include/standard-headers/linux/virtio_scmi.h | 24 +++
linux-headers/asm-generic/unistd.h | 5 +-
linux-headers/asm-mips/unistd_n32.h | 2 +
linux-headers/asm-mips/unistd_n64.h | 2 +
linux-headers/asm-mips/unistd_o32.h | 2 +
linux-headers/asm-powerpc/unistd_32.h | 2 +
linux-headers/asm-powerpc/unistd_64.h | 2 +
linux-headers/asm-riscv/bitsperlong.h | 14 ++
linux-headers/asm-riscv/mman.h | 1 +
linux-headers/asm-riscv/unistd.h | 44 ++++++
linux-headers/asm-s390/unistd_32.h | 2 +
linux-headers/asm-s390/unistd_64.h | 2 +
linux-headers/asm-x86/kvm.h | 16 +-
linux-headers/asm-x86/unistd_32.h | 1 +
linux-headers/asm-x86/unistd_64.h | 1 +
linux-headers/asm-x86/unistd_x32.h | 1 +
linux-headers/linux/kvm.h | 17 +++
hw/virtio/virtio-iommu.c | 83 +++++++++-
tests/qtest/virtio-iommu-test.c | 2 +
hw/virtio/trace-events | 4 +-
30 files changed, 552 insertions(+), 83 deletions(-)
create mode 100644 include/standard-headers/linux/virtio_gpio.h
create mode 100644 include/standard-headers/linux/virtio_i2c.h
create mode 100644 include/standard-headers/linux/virtio_pcidev.h
create mode 100644 include/standard-headers/linux/virtio_scmi.h
create mode 100644 linux-headers/asm-riscv/bitsperlong.h
create mode 100644 linux-headers/asm-riscv/mman.h
create mode 100644 linux-headers/asm-riscv/unistd.h
--
2.34.1
^ permalink raw reply [flat|nested] 20+ messages in thread* [PATCH v2 1/4] linux-headers: update to v5.17-rc1 2022-01-27 14:29 [PATCH v2 0/4] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG Jean-Philippe Brucker @ 2022-01-27 14:29 ` Jean-Philippe Brucker 2022-01-31 9:14 ` Eric Auger 2022-01-27 14:29 ` [PATCH v2 2/4] virtio-iommu: Default to bypass during boot Jean-Philippe Brucker ` (2 subsequent siblings) 3 siblings, 1 reply; 20+ messages in thread From: Jean-Philippe Brucker @ 2022-01-27 14:29 UTC (permalink / raw) To: eric.auger Cc: lvivier, Jean-Philippe Brucker, thuth, mst, cohuck, qemu-devel, pasic, pbonzini Update Linux headers to v5.17-rc1 Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> --- include/standard-headers/asm-x86/kvm_para.h | 1 + include/standard-headers/drm/drm_fourcc.h | 11 ++ include/standard-headers/linux/ethtool.h | 1 + include/standard-headers/linux/fuse.h | 60 +++++++- include/standard-headers/linux/pci_regs.h | 142 +++++++++--------- include/standard-headers/linux/virtio_gpio.h | 72 +++++++++ include/standard-headers/linux/virtio_i2c.h | 47 ++++++ include/standard-headers/linux/virtio_iommu.h | 8 +- .../standard-headers/linux/virtio_pcidev.h | 65 ++++++++ include/standard-headers/linux/virtio_scmi.h | 24 +++ linux-headers/asm-generic/unistd.h | 5 +- linux-headers/asm-mips/unistd_n32.h | 2 + linux-headers/asm-mips/unistd_n64.h | 2 + linux-headers/asm-mips/unistd_o32.h | 2 + linux-headers/asm-powerpc/unistd_32.h | 2 + linux-headers/asm-powerpc/unistd_64.h | 2 + linux-headers/asm-riscv/bitsperlong.h | 14 ++ linux-headers/asm-riscv/mman.h | 1 + linux-headers/asm-riscv/unistd.h | 44 ++++++ linux-headers/asm-s390/unistd_32.h | 2 + linux-headers/asm-s390/unistd_64.h | 2 + linux-headers/asm-x86/kvm.h | 16 +- linux-headers/asm-x86/unistd_32.h | 1 + linux-headers/asm-x86/unistd_64.h | 1 + linux-headers/asm-x86/unistd_x32.h | 1 + linux-headers/linux/kvm.h | 17 +++ 26 files changed, 469 insertions(+), 76 deletions(-) create mode 100644 include/standard-headers/linux/virtio_gpio.h create mode 100644 include/standard-headers/linux/virtio_i2c.h create mode 100644 include/standard-headers/linux/virtio_pcidev.h create mode 100644 include/standard-headers/linux/virtio_scmi.h create mode 100644 linux-headers/asm-riscv/bitsperlong.h create mode 100644 linux-headers/asm-riscv/mman.h create mode 100644 linux-headers/asm-riscv/unistd.h diff --git a/include/standard-headers/asm-x86/kvm_para.h b/include/standard-headers/asm-x86/kvm_para.h index 204cfb8640..f0235e58a1 100644 --- a/include/standard-headers/asm-x86/kvm_para.h +++ b/include/standard-headers/asm-x86/kvm_para.h @@ -8,6 +8,7 @@ * should be used to determine that a VM is running under KVM. */ #define KVM_CPUID_SIGNATURE 0x40000000 +#define KVM_SIGNATURE "KVMKVMKVM\0\0\0" /* This CPUID returns two feature bitmaps in eax, edx. Before enabling * a particular paravirtualization, the appropriate feature bit should diff --git a/include/standard-headers/drm/drm_fourcc.h b/include/standard-headers/drm/drm_fourcc.h index 2c025cb4fe..4888f85f69 100644 --- a/include/standard-headers/drm/drm_fourcc.h +++ b/include/standard-headers/drm/drm_fourcc.h @@ -313,6 +313,13 @@ extern "C" { */ #define DRM_FORMAT_P016 fourcc_code('P', '0', '1', '6') /* 2x2 subsampled Cr:Cb plane 16 bits per channel */ +/* 2 plane YCbCr420. + * 3 10 bit components and 2 padding bits packed into 4 bytes. + * index 0 = Y plane, [31:0] x:Y2:Y1:Y0 2:10:10:10 little endian + * index 1 = Cr:Cb plane, [63:0] x:Cr2:Cb2:Cr1:x:Cb1:Cr0:Cb0 [2:10:10:10:2:10:10:10] little endian + */ +#define DRM_FORMAT_P030 fourcc_code('P', '0', '3', '0') /* 2x2 subsampled Cr:Cb plane 10 bits per channel packed */ + /* 3 plane non-subsampled (444) YCbCr * 16 bits per component, but only 10 bits are used and 6 bits are padded * index 0: Y plane, [15:0] Y:x [10:6] little endian @@ -853,6 +860,10 @@ drm_fourcc_canonicalize_nvidia_format_mod(uint64_t modifier) * and UV. Some SAND-using hardware stores UV in a separate tiled * image from Y to reduce the column height, which is not supported * with these modifiers. + * + * The DRM_FORMAT_MOD_BROADCOM_SAND128_COL_HEIGHT modifier is also + * supported for DRM_FORMAT_P030 where the columns remain as 128 bytes + * wide, but as this is a 10 bpp format that translates to 96 pixels. */ #define DRM_FORMAT_MOD_BROADCOM_SAND32_COL_HEIGHT(v) \ diff --git a/include/standard-headers/linux/ethtool.h b/include/standard-headers/linux/ethtool.h index 688eb8dc39..38d5a4cd6e 100644 --- a/include/standard-headers/linux/ethtool.h +++ b/include/standard-headers/linux/ethtool.h @@ -231,6 +231,7 @@ enum tunable_id { ETHTOOL_RX_COPYBREAK, ETHTOOL_TX_COPYBREAK, ETHTOOL_PFC_PREVENTION_TOUT, /* timeout in msecs */ + ETHTOOL_TX_COPYBREAK_BUF_SIZE, /* * Add your fresh new tunable attribute above and remember to update * tunable_strings[] in net/ethtool/common.c diff --git a/include/standard-headers/linux/fuse.h b/include/standard-headers/linux/fuse.h index 23ea31708b..bda06258be 100644 --- a/include/standard-headers/linux/fuse.h +++ b/include/standard-headers/linux/fuse.h @@ -184,6 +184,16 @@ * * 7.34 * - add FUSE_SYNCFS + * + * 7.35 + * - add FOPEN_NOFLUSH + * + * 7.36 + * - extend fuse_init_in with reserved fields, add FUSE_INIT_EXT init flag + * - add flags2 to fuse_init_in and fuse_init_out + * - add FUSE_SECURITY_CTX init flag + * - add security context to create, mkdir, symlink, and mknod requests + * - add FUSE_HAS_INODE_DAX, FUSE_ATTR_DAX */ #ifndef _LINUX_FUSE_H @@ -215,7 +225,7 @@ #define FUSE_KERNEL_VERSION 7 /** Minor version number of this interface */ -#define FUSE_KERNEL_MINOR_VERSION 34 +#define FUSE_KERNEL_MINOR_VERSION 36 /** The node ID of the root inode */ #define FUSE_ROOT_ID 1 @@ -286,12 +296,14 @@ struct fuse_file_lock { * FOPEN_NONSEEKABLE: the file is not seekable * FOPEN_CACHE_DIR: allow caching this directory * FOPEN_STREAM: the file is stream-like (no file position at all) + * FOPEN_NOFLUSH: don't flush data cache on close (unless FUSE_WRITEBACK_CACHE) */ #define FOPEN_DIRECT_IO (1 << 0) #define FOPEN_KEEP_CACHE (1 << 1) #define FOPEN_NONSEEKABLE (1 << 2) #define FOPEN_CACHE_DIR (1 << 3) #define FOPEN_STREAM (1 << 4) +#define FOPEN_NOFLUSH (1 << 5) /** * INIT request/reply flags @@ -332,6 +344,11 @@ struct fuse_file_lock { * write/truncate sgid is killed only if file has group * execute permission. (Same as Linux VFS behavior). * FUSE_SETXATTR_EXT: Server supports extended struct fuse_setxattr_in + * FUSE_INIT_EXT: extended fuse_init_in request + * FUSE_INIT_RESERVED: reserved, do not use + * FUSE_SECURITY_CTX: add security context to create, mkdir, symlink, and + * mknod + * FUSE_HAS_INODE_DAX: use per inode DAX */ #define FUSE_ASYNC_READ (1 << 0) #define FUSE_POSIX_LOCKS (1 << 1) @@ -363,6 +380,11 @@ struct fuse_file_lock { #define FUSE_SUBMOUNTS (1 << 27) #define FUSE_HANDLE_KILLPRIV_V2 (1 << 28) #define FUSE_SETXATTR_EXT (1 << 29) +#define FUSE_INIT_EXT (1 << 30) +#define FUSE_INIT_RESERVED (1 << 31) +/* bits 32..63 get shifted down 32 bits into the flags2 field */ +#define FUSE_SECURITY_CTX (1ULL << 32) +#define FUSE_HAS_INODE_DAX (1ULL << 33) /** * CUSE INIT request/reply flags @@ -445,8 +467,10 @@ struct fuse_file_lock { * fuse_attr flags * * FUSE_ATTR_SUBMOUNT: Object is a submount root + * FUSE_ATTR_DAX: Enable DAX for this file in per inode DAX mode */ #define FUSE_ATTR_SUBMOUNT (1 << 0) +#define FUSE_ATTR_DAX (1 << 1) /** * Open flags @@ -732,6 +756,8 @@ struct fuse_init_in { uint32_t minor; uint32_t max_readahead; uint32_t flags; + uint32_t flags2; + uint32_t unused[11]; }; #define FUSE_COMPAT_INIT_OUT_SIZE 8 @@ -748,7 +774,8 @@ struct fuse_init_out { uint32_t time_gran; uint16_t max_pages; uint16_t map_alignment; - uint32_t unused[8]; + uint32_t flags2; + uint32_t unused[7]; }; #define CUSE_INIT_INFO_MAX 4096 @@ -856,9 +883,12 @@ struct fuse_dirent { char name[]; }; -#define FUSE_NAME_OFFSET offsetof(struct fuse_dirent, name) -#define FUSE_DIRENT_ALIGN(x) \ +/* Align variable length records to 64bit boundary */ +#define FUSE_REC_ALIGN(x) \ (((x) + sizeof(uint64_t) - 1) & ~(sizeof(uint64_t) - 1)) + +#define FUSE_NAME_OFFSET offsetof(struct fuse_dirent, name) +#define FUSE_DIRENT_ALIGN(x) FUSE_REC_ALIGN(x) #define FUSE_DIRENT_SIZE(d) \ FUSE_DIRENT_ALIGN(FUSE_NAME_OFFSET + (d)->namelen) @@ -975,4 +1005,26 @@ struct fuse_syncfs_in { uint64_t padding; }; +/* + * For each security context, send fuse_secctx with size of security context + * fuse_secctx will be followed by security context name and this in turn + * will be followed by actual context label. + * fuse_secctx, name, context + */ +struct fuse_secctx { + uint32_t size; + uint32_t padding; +}; + +/* + * Contains the information about how many fuse_secctx structures are being + * sent and what's the total size of all security contexts (including + * size of fuse_secctx_header). + * + */ +struct fuse_secctx_header { + uint32_t size; + uint32_t nr_secctx; +}; + #endif /* _LINUX_FUSE_H */ diff --git a/include/standard-headers/linux/pci_regs.h b/include/standard-headers/linux/pci_regs.h index ff6ccbc6ef..bee1a9ed6e 100644 --- a/include/standard-headers/linux/pci_regs.h +++ b/include/standard-headers/linux/pci_regs.h @@ -301,23 +301,23 @@ #define PCI_SID_ESR_FIC 0x20 /* First In Chassis Flag */ #define PCI_SID_CHASSIS_NR 3 /* Chassis Number */ -/* Message Signalled Interrupt registers */ +/* Message Signaled Interrupt registers */ -#define PCI_MSI_FLAGS 2 /* Message Control */ +#define PCI_MSI_FLAGS 0x02 /* Message Control */ #define PCI_MSI_FLAGS_ENABLE 0x0001 /* MSI feature enabled */ #define PCI_MSI_FLAGS_QMASK 0x000e /* Maximum queue size available */ #define PCI_MSI_FLAGS_QSIZE 0x0070 /* Message queue size configured */ #define PCI_MSI_FLAGS_64BIT 0x0080 /* 64-bit addresses allowed */ #define PCI_MSI_FLAGS_MASKBIT 0x0100 /* Per-vector masking capable */ #define PCI_MSI_RFU 3 /* Rest of capability flags */ -#define PCI_MSI_ADDRESS_LO 4 /* Lower 32 bits */ -#define PCI_MSI_ADDRESS_HI 8 /* Upper 32 bits (if PCI_MSI_FLAGS_64BIT set) */ -#define PCI_MSI_DATA_32 8 /* 16 bits of data for 32-bit devices */ -#define PCI_MSI_MASK_32 12 /* Mask bits register for 32-bit devices */ -#define PCI_MSI_PENDING_32 16 /* Pending intrs for 32-bit devices */ -#define PCI_MSI_DATA_64 12 /* 16 bits of data for 64-bit devices */ -#define PCI_MSI_MASK_64 16 /* Mask bits register for 64-bit devices */ -#define PCI_MSI_PENDING_64 20 /* Pending intrs for 64-bit devices */ +#define PCI_MSI_ADDRESS_LO 0x04 /* Lower 32 bits */ +#define PCI_MSI_ADDRESS_HI 0x08 /* Upper 32 bits (if PCI_MSI_FLAGS_64BIT set) */ +#define PCI_MSI_DATA_32 0x08 /* 16 bits of data for 32-bit devices */ +#define PCI_MSI_MASK_32 0x0c /* Mask bits register for 32-bit devices */ +#define PCI_MSI_PENDING_32 0x10 /* Pending intrs for 32-bit devices */ +#define PCI_MSI_DATA_64 0x0c /* 16 bits of data for 64-bit devices */ +#define PCI_MSI_MASK_64 0x10 /* Mask bits register for 64-bit devices */ +#define PCI_MSI_PENDING_64 0x14 /* Pending intrs for 64-bit devices */ /* MSI-X registers (in MSI-X capability) */ #define PCI_MSIX_FLAGS 2 /* Message Control */ @@ -335,10 +335,10 @@ /* MSI-X Table entry format (in memory mapped by a BAR) */ #define PCI_MSIX_ENTRY_SIZE 16 -#define PCI_MSIX_ENTRY_LOWER_ADDR 0 /* Message Address */ -#define PCI_MSIX_ENTRY_UPPER_ADDR 4 /* Message Upper Address */ -#define PCI_MSIX_ENTRY_DATA 8 /* Message Data */ -#define PCI_MSIX_ENTRY_VECTOR_CTRL 12 /* Vector Control */ +#define PCI_MSIX_ENTRY_LOWER_ADDR 0x0 /* Message Address */ +#define PCI_MSIX_ENTRY_UPPER_ADDR 0x4 /* Message Upper Address */ +#define PCI_MSIX_ENTRY_DATA 0x8 /* Message Data */ +#define PCI_MSIX_ENTRY_VECTOR_CTRL 0xc /* Vector Control */ #define PCI_MSIX_ENTRY_CTRL_MASKBIT 0x00000001 /* CompactPCI Hotswap Register */ @@ -470,7 +470,7 @@ /* PCI Express capability registers */ -#define PCI_EXP_FLAGS 2 /* Capabilities register */ +#define PCI_EXP_FLAGS 0x02 /* Capabilities register */ #define PCI_EXP_FLAGS_VERS 0x000f /* Capability version */ #define PCI_EXP_FLAGS_TYPE 0x00f0 /* Device/Port type */ #define PCI_EXP_TYPE_ENDPOINT 0x0 /* Express Endpoint */ @@ -484,7 +484,7 @@ #define PCI_EXP_TYPE_RC_EC 0xa /* Root Complex Event Collector */ #define PCI_EXP_FLAGS_SLOT 0x0100 /* Slot implemented */ #define PCI_EXP_FLAGS_IRQ 0x3e00 /* Interrupt message number */ -#define PCI_EXP_DEVCAP 4 /* Device capabilities */ +#define PCI_EXP_DEVCAP 0x04 /* Device capabilities */ #define PCI_EXP_DEVCAP_PAYLOAD 0x00000007 /* Max_Payload_Size */ #define PCI_EXP_DEVCAP_PHANTOM 0x00000018 /* Phantom functions */ #define PCI_EXP_DEVCAP_EXT_TAG 0x00000020 /* Extended tags */ @@ -497,7 +497,7 @@ #define PCI_EXP_DEVCAP_PWR_VAL 0x03fc0000 /* Slot Power Limit Value */ #define PCI_EXP_DEVCAP_PWR_SCL 0x0c000000 /* Slot Power Limit Scale */ #define PCI_EXP_DEVCAP_FLR 0x10000000 /* Function Level Reset */ -#define PCI_EXP_DEVCTL 8 /* Device Control */ +#define PCI_EXP_DEVCTL 0x08 /* Device Control */ #define PCI_EXP_DEVCTL_CERE 0x0001 /* Correctable Error Reporting En. */ #define PCI_EXP_DEVCTL_NFERE 0x0002 /* Non-Fatal Error Reporting Enable */ #define PCI_EXP_DEVCTL_FERE 0x0004 /* Fatal Error Reporting Enable */ @@ -522,7 +522,7 @@ #define PCI_EXP_DEVCTL_READRQ_2048B 0x4000 /* 2048 Bytes */ #define PCI_EXP_DEVCTL_READRQ_4096B 0x5000 /* 4096 Bytes */ #define PCI_EXP_DEVCTL_BCR_FLR 0x8000 /* Bridge Configuration Retry / FLR */ -#define PCI_EXP_DEVSTA 10 /* Device Status */ +#define PCI_EXP_DEVSTA 0x0a /* Device Status */ #define PCI_EXP_DEVSTA_CED 0x0001 /* Correctable Error Detected */ #define PCI_EXP_DEVSTA_NFED 0x0002 /* Non-Fatal Error Detected */ #define PCI_EXP_DEVSTA_FED 0x0004 /* Fatal Error Detected */ @@ -530,7 +530,7 @@ #define PCI_EXP_DEVSTA_AUXPD 0x0010 /* AUX Power Detected */ #define PCI_EXP_DEVSTA_TRPND 0x0020 /* Transactions Pending */ #define PCI_CAP_EXP_RC_ENDPOINT_SIZEOF_V1 12 /* v1 endpoints without link end here */ -#define PCI_EXP_LNKCAP 12 /* Link Capabilities */ +#define PCI_EXP_LNKCAP 0x0c /* Link Capabilities */ #define PCI_EXP_LNKCAP_SLS 0x0000000f /* Supported Link Speeds */ #define PCI_EXP_LNKCAP_SLS_2_5GB 0x00000001 /* LNKCAP2 SLS Vector bit 0 */ #define PCI_EXP_LNKCAP_SLS_5_0GB 0x00000002 /* LNKCAP2 SLS Vector bit 1 */ @@ -549,7 +549,7 @@ #define PCI_EXP_LNKCAP_DLLLARC 0x00100000 /* Data Link Layer Link Active Reporting Capable */ #define PCI_EXP_LNKCAP_LBNC 0x00200000 /* Link Bandwidth Notification Capability */ #define PCI_EXP_LNKCAP_PN 0xff000000 /* Port Number */ -#define PCI_EXP_LNKCTL 16 /* Link Control */ +#define PCI_EXP_LNKCTL 0x10 /* Link Control */ #define PCI_EXP_LNKCTL_ASPMC 0x0003 /* ASPM Control */ #define PCI_EXP_LNKCTL_ASPM_L0S 0x0001 /* L0s Enable */ #define PCI_EXP_LNKCTL_ASPM_L1 0x0002 /* L1 Enable */ @@ -562,7 +562,7 @@ #define PCI_EXP_LNKCTL_HAWD 0x0200 /* Hardware Autonomous Width Disable */ #define PCI_EXP_LNKCTL_LBMIE 0x0400 /* Link Bandwidth Management Interrupt Enable */ #define PCI_EXP_LNKCTL_LABIE 0x0800 /* Link Autonomous Bandwidth Interrupt Enable */ -#define PCI_EXP_LNKSTA 18 /* Link Status */ +#define PCI_EXP_LNKSTA 0x12 /* Link Status */ #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */ #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */ #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */ @@ -582,7 +582,7 @@ #define PCI_EXP_LNKSTA_LBMS 0x4000 /* Link Bandwidth Management Status */ #define PCI_EXP_LNKSTA_LABS 0x8000 /* Link Autonomous Bandwidth Status */ #define PCI_CAP_EXP_ENDPOINT_SIZEOF_V1 20 /* v1 endpoints with link end here */ -#define PCI_EXP_SLTCAP 20 /* Slot Capabilities */ +#define PCI_EXP_SLTCAP 0x14 /* Slot Capabilities */ #define PCI_EXP_SLTCAP_ABP 0x00000001 /* Attention Button Present */ #define PCI_EXP_SLTCAP_PCP 0x00000002 /* Power Controller Present */ #define PCI_EXP_SLTCAP_MRLSP 0x00000004 /* MRL Sensor Present */ @@ -595,7 +595,7 @@ #define PCI_EXP_SLTCAP_EIP 0x00020000 /* Electromechanical Interlock Present */ #define PCI_EXP_SLTCAP_NCCS 0x00040000 /* No Command Completed Support */ #define PCI_EXP_SLTCAP_PSN 0xfff80000 /* Physical Slot Number */ -#define PCI_EXP_SLTCTL 24 /* Slot Control */ +#define PCI_EXP_SLTCTL 0x18 /* Slot Control */ #define PCI_EXP_SLTCTL_ABPE 0x0001 /* Attention Button Pressed Enable */ #define PCI_EXP_SLTCTL_PFDE 0x0002 /* Power Fault Detected Enable */ #define PCI_EXP_SLTCTL_MRLSCE 0x0004 /* MRL Sensor Changed Enable */ @@ -617,7 +617,7 @@ #define PCI_EXP_SLTCTL_EIC 0x0800 /* Electromechanical Interlock Control */ #define PCI_EXP_SLTCTL_DLLSCE 0x1000 /* Data Link Layer State Changed Enable */ #define PCI_EXP_SLTCTL_IBPD_DISABLE 0x4000 /* In-band PD disable */ -#define PCI_EXP_SLTSTA 26 /* Slot Status */ +#define PCI_EXP_SLTSTA 0x1a /* Slot Status */ #define PCI_EXP_SLTSTA_ABP 0x0001 /* Attention Button Pressed */ #define PCI_EXP_SLTSTA_PFD 0x0002 /* Power Fault Detected */ #define PCI_EXP_SLTSTA_MRLSC 0x0004 /* MRL Sensor Changed */ @@ -627,15 +627,15 @@ #define PCI_EXP_SLTSTA_PDS 0x0040 /* Presence Detect State */ #define PCI_EXP_SLTSTA_EIS 0x0080 /* Electromechanical Interlock Status */ #define PCI_EXP_SLTSTA_DLLSC 0x0100 /* Data Link Layer State Changed */ -#define PCI_EXP_RTCTL 28 /* Root Control */ +#define PCI_EXP_RTCTL 0x1c /* Root Control */ #define PCI_EXP_RTCTL_SECEE 0x0001 /* System Error on Correctable Error */ #define PCI_EXP_RTCTL_SENFEE 0x0002 /* System Error on Non-Fatal Error */ #define PCI_EXP_RTCTL_SEFEE 0x0004 /* System Error on Fatal Error */ #define PCI_EXP_RTCTL_PMEIE 0x0008 /* PME Interrupt Enable */ #define PCI_EXP_RTCTL_CRSSVE 0x0010 /* CRS Software Visibility Enable */ -#define PCI_EXP_RTCAP 30 /* Root Capabilities */ +#define PCI_EXP_RTCAP 0x1e /* Root Capabilities */ #define PCI_EXP_RTCAP_CRSVIS 0x0001 /* CRS Software Visibility capability */ -#define PCI_EXP_RTSTA 32 /* Root Status */ +#define PCI_EXP_RTSTA 0x20 /* Root Status */ #define PCI_EXP_RTSTA_PME 0x00010000 /* PME status */ #define PCI_EXP_RTSTA_PENDING 0x00020000 /* PME pending */ /* @@ -646,7 +646,7 @@ * Use pcie_capability_read_word() and similar interfaces to use them * safely. */ -#define PCI_EXP_DEVCAP2 36 /* Device Capabilities 2 */ +#define PCI_EXP_DEVCAP2 0x24 /* Device Capabilities 2 */ #define PCI_EXP_DEVCAP2_COMP_TMOUT_DIS 0x00000010 /* Completion Timeout Disable supported */ #define PCI_EXP_DEVCAP2_ARI 0x00000020 /* Alternative Routing-ID */ #define PCI_EXP_DEVCAP2_ATOMIC_ROUTE 0x00000040 /* Atomic Op routing */ @@ -658,7 +658,7 @@ #define PCI_EXP_DEVCAP2_OBFF_MSG 0x00040000 /* New message signaling */ #define PCI_EXP_DEVCAP2_OBFF_WAKE 0x00080000 /* Re-use WAKE# for OBFF */ #define PCI_EXP_DEVCAP2_EE_PREFIX 0x00200000 /* End-End TLP Prefix */ -#define PCI_EXP_DEVCTL2 40 /* Device Control 2 */ +#define PCI_EXP_DEVCTL2 0x28 /* Device Control 2 */ #define PCI_EXP_DEVCTL2_COMP_TIMEOUT 0x000f /* Completion Timeout Value */ #define PCI_EXP_DEVCTL2_COMP_TMOUT_DIS 0x0010 /* Completion Timeout Disable */ #define PCI_EXP_DEVCTL2_ARI 0x0020 /* Alternative Routing-ID */ @@ -670,9 +670,9 @@ #define PCI_EXP_DEVCTL2_OBFF_MSGA_EN 0x2000 /* Enable OBFF Message type A */ #define PCI_EXP_DEVCTL2_OBFF_MSGB_EN 0x4000 /* Enable OBFF Message type B */ #define PCI_EXP_DEVCTL2_OBFF_WAKE_EN 0x6000 /* OBFF using WAKE# signaling */ -#define PCI_EXP_DEVSTA2 42 /* Device Status 2 */ -#define PCI_CAP_EXP_RC_ENDPOINT_SIZEOF_V2 44 /* v2 endpoints without link end here */ -#define PCI_EXP_LNKCAP2 44 /* Link Capabilities 2 */ +#define PCI_EXP_DEVSTA2 0x2a /* Device Status 2 */ +#define PCI_CAP_EXP_RC_ENDPOINT_SIZEOF_V2 0x2c /* end of v2 EPs w/o link */ +#define PCI_EXP_LNKCAP2 0x2c /* Link Capabilities 2 */ #define PCI_EXP_LNKCAP2_SLS_2_5GB 0x00000002 /* Supported Speed 2.5GT/s */ #define PCI_EXP_LNKCAP2_SLS_5_0GB 0x00000004 /* Supported Speed 5GT/s */ #define PCI_EXP_LNKCAP2_SLS_8_0GB 0x00000008 /* Supported Speed 8GT/s */ @@ -680,7 +680,7 @@ #define PCI_EXP_LNKCAP2_SLS_32_0GB 0x00000020 /* Supported Speed 32GT/s */ #define PCI_EXP_LNKCAP2_SLS_64_0GB 0x00000040 /* Supported Speed 64GT/s */ #define PCI_EXP_LNKCAP2_CROSSLINK 0x00000100 /* Crosslink supported */ -#define PCI_EXP_LNKCTL2 48 /* Link Control 2 */ +#define PCI_EXP_LNKCTL2 0x30 /* Link Control 2 */ #define PCI_EXP_LNKCTL2_TLS 0x000f #define PCI_EXP_LNKCTL2_TLS_2_5GT 0x0001 /* Supported Speed 2.5GT/s */ #define PCI_EXP_LNKCTL2_TLS_5_0GT 0x0002 /* Supported Speed 5GT/s */ @@ -691,12 +691,12 @@ #define PCI_EXP_LNKCTL2_ENTER_COMP 0x0010 /* Enter Compliance */ #define PCI_EXP_LNKCTL2_TX_MARGIN 0x0380 /* Transmit Margin */ #define PCI_EXP_LNKCTL2_HASD 0x0020 /* HW Autonomous Speed Disable */ -#define PCI_EXP_LNKSTA2 50 /* Link Status 2 */ -#define PCI_CAP_EXP_ENDPOINT_SIZEOF_V2 52 /* v2 endpoints with link end here */ -#define PCI_EXP_SLTCAP2 52 /* Slot Capabilities 2 */ +#define PCI_EXP_LNKSTA2 0x32 /* Link Status 2 */ +#define PCI_CAP_EXP_ENDPOINT_SIZEOF_V2 0x32 /* end of v2 EPs w/ link */ +#define PCI_EXP_SLTCAP2 0x34 /* Slot Capabilities 2 */ #define PCI_EXP_SLTCAP2_IBPD 0x00000001 /* In-band PD Disable Supported */ -#define PCI_EXP_SLTCTL2 56 /* Slot Control 2 */ -#define PCI_EXP_SLTSTA2 58 /* Slot Status 2 */ +#define PCI_EXP_SLTCTL2 0x38 /* Slot Control 2 */ +#define PCI_EXP_SLTSTA2 0x3a /* Slot Status 2 */ /* Extended Capabilities (PCI-X 2.0 and Express) */ #define PCI_EXT_CAP_ID(header) (header & 0x0000ffff) @@ -742,7 +742,7 @@ #define PCI_EXT_CAP_MCAST_ENDPOINT_SIZEOF 40 /* Advanced Error Reporting */ -#define PCI_ERR_UNCOR_STATUS 4 /* Uncorrectable Error Status */ +#define PCI_ERR_UNCOR_STATUS 0x04 /* Uncorrectable Error Status */ #define PCI_ERR_UNC_UND 0x00000001 /* Undefined */ #define PCI_ERR_UNC_DLP 0x00000010 /* Data Link Protocol */ #define PCI_ERR_UNC_SURPDN 0x00000020 /* Surprise Down */ @@ -760,11 +760,11 @@ #define PCI_ERR_UNC_MCBTLP 0x00800000 /* MC blocked TLP */ #define PCI_ERR_UNC_ATOMEG 0x01000000 /* Atomic egress blocked */ #define PCI_ERR_UNC_TLPPRE 0x02000000 /* TLP prefix blocked */ -#define PCI_ERR_UNCOR_MASK 8 /* Uncorrectable Error Mask */ +#define PCI_ERR_UNCOR_MASK 0x08 /* Uncorrectable Error Mask */ /* Same bits as above */ -#define PCI_ERR_UNCOR_SEVER 12 /* Uncorrectable Error Severity */ +#define PCI_ERR_UNCOR_SEVER 0x0c /* Uncorrectable Error Severity */ /* Same bits as above */ -#define PCI_ERR_COR_STATUS 16 /* Correctable Error Status */ +#define PCI_ERR_COR_STATUS 0x10 /* Correctable Error Status */ #define PCI_ERR_COR_RCVR 0x00000001 /* Receiver Error Status */ #define PCI_ERR_COR_BAD_TLP 0x00000040 /* Bad TLP Status */ #define PCI_ERR_COR_BAD_DLLP 0x00000080 /* Bad DLLP Status */ @@ -773,20 +773,20 @@ #define PCI_ERR_COR_ADV_NFAT 0x00002000 /* Advisory Non-Fatal */ #define PCI_ERR_COR_INTERNAL 0x00004000 /* Corrected Internal */ #define PCI_ERR_COR_LOG_OVER 0x00008000 /* Header Log Overflow */ -#define PCI_ERR_COR_MASK 20 /* Correctable Error Mask */ +#define PCI_ERR_COR_MASK 0x14 /* Correctable Error Mask */ /* Same bits as above */ -#define PCI_ERR_CAP 24 /* Advanced Error Capabilities */ -#define PCI_ERR_CAP_FEP(x) ((x) & 31) /* First Error Pointer */ +#define PCI_ERR_CAP 0x18 /* Advanced Error Capabilities & Ctrl*/ +#define PCI_ERR_CAP_FEP(x) ((x) & 0x1f) /* First Error Pointer */ #define PCI_ERR_CAP_ECRC_GENC 0x00000020 /* ECRC Generation Capable */ #define PCI_ERR_CAP_ECRC_GENE 0x00000040 /* ECRC Generation Enable */ #define PCI_ERR_CAP_ECRC_CHKC 0x00000080 /* ECRC Check Capable */ #define PCI_ERR_CAP_ECRC_CHKE 0x00000100 /* ECRC Check Enable */ -#define PCI_ERR_HEADER_LOG 28 /* Header Log Register (16 bytes) */ -#define PCI_ERR_ROOT_COMMAND 44 /* Root Error Command */ +#define PCI_ERR_HEADER_LOG 0x1c /* Header Log Register (16 bytes) */ +#define PCI_ERR_ROOT_COMMAND 0x2c /* Root Error Command */ #define PCI_ERR_ROOT_CMD_COR_EN 0x00000001 /* Correctable Err Reporting Enable */ #define PCI_ERR_ROOT_CMD_NONFATAL_EN 0x00000002 /* Non-Fatal Err Reporting Enable */ #define PCI_ERR_ROOT_CMD_FATAL_EN 0x00000004 /* Fatal Err Reporting Enable */ -#define PCI_ERR_ROOT_STATUS 48 +#define PCI_ERR_ROOT_STATUS 0x30 #define PCI_ERR_ROOT_COR_RCV 0x00000001 /* ERR_COR Received */ #define PCI_ERR_ROOT_MULTI_COR_RCV 0x00000002 /* Multiple ERR_COR */ #define PCI_ERR_ROOT_UNCOR_RCV 0x00000004 /* ERR_FATAL/NONFATAL */ @@ -795,52 +795,52 @@ #define PCI_ERR_ROOT_NONFATAL_RCV 0x00000020 /* Non-Fatal Received */ #define PCI_ERR_ROOT_FATAL_RCV 0x00000040 /* Fatal Received */ #define PCI_ERR_ROOT_AER_IRQ 0xf8000000 /* Advanced Error Interrupt Message Number */ -#define PCI_ERR_ROOT_ERR_SRC 52 /* Error Source Identification */ +#define PCI_ERR_ROOT_ERR_SRC 0x34 /* Error Source Identification */ /* Virtual Channel */ -#define PCI_VC_PORT_CAP1 4 +#define PCI_VC_PORT_CAP1 0x04 #define PCI_VC_CAP1_EVCC 0x00000007 /* extended VC count */ #define PCI_VC_CAP1_LPEVCC 0x00000070 /* low prio extended VC count */ #define PCI_VC_CAP1_ARB_SIZE 0x00000c00 -#define PCI_VC_PORT_CAP2 8 +#define PCI_VC_PORT_CAP2 0x08 #define PCI_VC_CAP2_32_PHASE 0x00000002 #define PCI_VC_CAP2_64_PHASE 0x00000004 #define PCI_VC_CAP2_128_PHASE 0x00000008 #define PCI_VC_CAP2_ARB_OFF 0xff000000 -#define PCI_VC_PORT_CTRL 12 +#define PCI_VC_PORT_CTRL 0x0c #define PCI_VC_PORT_CTRL_LOAD_TABLE 0x00000001 -#define PCI_VC_PORT_STATUS 14 +#define PCI_VC_PORT_STATUS 0x0e #define PCI_VC_PORT_STATUS_TABLE 0x00000001 -#define PCI_VC_RES_CAP 16 +#define PCI_VC_RES_CAP 0x10 #define PCI_VC_RES_CAP_32_PHASE 0x00000002 #define PCI_VC_RES_CAP_64_PHASE 0x00000004 #define PCI_VC_RES_CAP_128_PHASE 0x00000008 #define PCI_VC_RES_CAP_128_PHASE_TB 0x00000010 #define PCI_VC_RES_CAP_256_PHASE 0x00000020 #define PCI_VC_RES_CAP_ARB_OFF 0xff000000 -#define PCI_VC_RES_CTRL 20 +#define PCI_VC_RES_CTRL 0x14 #define PCI_VC_RES_CTRL_LOAD_TABLE 0x00010000 #define PCI_VC_RES_CTRL_ARB_SELECT 0x000e0000 #define PCI_VC_RES_CTRL_ID 0x07000000 #define PCI_VC_RES_CTRL_ENABLE 0x80000000 -#define PCI_VC_RES_STATUS 26 +#define PCI_VC_RES_STATUS 0x1a #define PCI_VC_RES_STATUS_TABLE 0x00000001 #define PCI_VC_RES_STATUS_NEGO 0x00000002 #define PCI_CAP_VC_BASE_SIZEOF 0x10 -#define PCI_CAP_VC_PER_VC_SIZEOF 0x0C +#define PCI_CAP_VC_PER_VC_SIZEOF 0x0c /* Power Budgeting */ -#define PCI_PWR_DSR 4 /* Data Select Register */ -#define PCI_PWR_DATA 8 /* Data Register */ +#define PCI_PWR_DSR 0x04 /* Data Select Register */ +#define PCI_PWR_DATA 0x08 /* Data Register */ #define PCI_PWR_DATA_BASE(x) ((x) & 0xff) /* Base Power */ #define PCI_PWR_DATA_SCALE(x) (((x) >> 8) & 3) /* Data Scale */ #define PCI_PWR_DATA_PM_SUB(x) (((x) >> 10) & 7) /* PM Sub State */ #define PCI_PWR_DATA_PM_STATE(x) (((x) >> 13) & 3) /* PM State */ #define PCI_PWR_DATA_TYPE(x) (((x) >> 15) & 7) /* Type */ #define PCI_PWR_DATA_RAIL(x) (((x) >> 18) & 7) /* Power Rail */ -#define PCI_PWR_CAP 12 /* Capability */ +#define PCI_PWR_CAP 0x0c /* Capability */ #define PCI_PWR_CAP_BUDGET(x) ((x) & 1) /* Included in system budget */ -#define PCI_EXT_CAP_PWR_SIZEOF 16 +#define PCI_EXT_CAP_PWR_SIZEOF 0x10 /* Root Complex Event Collector Endpoint Association */ #define PCI_RCEC_RCIEP_BITMAP 4 /* Associated Bitmap for RCiEPs */ @@ -964,7 +964,7 @@ #define PCI_SRIOV_VFM_MI 0x1 /* Dormant.MigrateIn */ #define PCI_SRIOV_VFM_MO 0x2 /* Active.MigrateOut */ #define PCI_SRIOV_VFM_AV 0x3 /* Active.Available */ -#define PCI_EXT_CAP_SRIOV_SIZEOF 64 +#define PCI_EXT_CAP_SRIOV_SIZEOF 0x40 #define PCI_LTR_MAX_SNOOP_LAT 0x4 #define PCI_LTR_MAX_NOSNOOP_LAT 0x6 @@ -1017,12 +1017,12 @@ #define PCI_TPH_LOC_NONE 0x000 /* no location */ #define PCI_TPH_LOC_CAP 0x200 /* in capability */ #define PCI_TPH_LOC_MSIX 0x400 /* in MSI-X */ -#define PCI_TPH_CAP_ST_MASK 0x07FF0000 /* st table mask */ -#define PCI_TPH_CAP_ST_SHIFT 16 /* st table shift */ -#define PCI_TPH_BASE_SIZEOF 12 /* size with no st table */ +#define PCI_TPH_CAP_ST_MASK 0x07FF0000 /* ST table mask */ +#define PCI_TPH_CAP_ST_SHIFT 16 /* ST table shift */ +#define PCI_TPH_BASE_SIZEOF 0xc /* size with no ST table */ /* Downstream Port Containment */ -#define PCI_EXP_DPC_CAP 4 /* DPC Capability */ +#define PCI_EXP_DPC_CAP 0x04 /* DPC Capability */ #define PCI_EXP_DPC_IRQ 0x001F /* Interrupt Message Number */ #define PCI_EXP_DPC_CAP_RP_EXT 0x0020 /* Root Port Extensions */ #define PCI_EXP_DPC_CAP_POISONED_TLP 0x0040 /* Poisoned TLP Egress Blocking Supported */ @@ -1030,19 +1030,19 @@ #define PCI_EXP_DPC_RP_PIO_LOG_SIZE 0x0F00 /* RP PIO Log Size */ #define PCI_EXP_DPC_CAP_DL_ACTIVE 0x1000 /* ERR_COR signal on DL_Active supported */ -#define PCI_EXP_DPC_CTL 6 /* DPC control */ +#define PCI_EXP_DPC_CTL 0x06 /* DPC control */ #define PCI_EXP_DPC_CTL_EN_FATAL 0x0001 /* Enable trigger on ERR_FATAL message */ #define PCI_EXP_DPC_CTL_EN_NONFATAL 0x0002 /* Enable trigger on ERR_NONFATAL message */ #define PCI_EXP_DPC_CTL_INT_EN 0x0008 /* DPC Interrupt Enable */ -#define PCI_EXP_DPC_STATUS 8 /* DPC Status */ +#define PCI_EXP_DPC_STATUS 0x08 /* DPC Status */ #define PCI_EXP_DPC_STATUS_TRIGGER 0x0001 /* Trigger Status */ #define PCI_EXP_DPC_STATUS_TRIGGER_RSN 0x0006 /* Trigger Reason */ #define PCI_EXP_DPC_STATUS_INTERRUPT 0x0008 /* Interrupt Status */ #define PCI_EXP_DPC_RP_BUSY 0x0010 /* Root Port Busy */ #define PCI_EXP_DPC_STATUS_TRIGGER_RSN_EXT 0x0060 /* Trig Reason Extension */ -#define PCI_EXP_DPC_SOURCE_ID 10 /* DPC Source Identifier */ +#define PCI_EXP_DPC_SOURCE_ID 0x0A /* DPC Source Identifier */ #define PCI_EXP_DPC_RP_PIO_STATUS 0x0C /* RP PIO Status */ #define PCI_EXP_DPC_RP_PIO_MASK 0x10 /* RP PIO Mask */ @@ -1086,7 +1086,11 @@ /* Designated Vendor-Specific (DVSEC, PCI_EXT_CAP_ID_DVSEC) */ #define PCI_DVSEC_HEADER1 0x4 /* Designated Vendor-Specific Header1 */ +#define PCI_DVSEC_HEADER1_VID(x) ((x) & 0xffff) +#define PCI_DVSEC_HEADER1_REV(x) (((x) >> 16) & 0xf) +#define PCI_DVSEC_HEADER1_LEN(x) (((x) >> 20) & 0xfff) #define PCI_DVSEC_HEADER2 0x8 /* Designated Vendor-Specific Header2 */ +#define PCI_DVSEC_HEADER2_ID(x) ((x) & 0xffff) /* Data Link Feature */ #define PCI_DLF_CAP 0x04 /* Capabilities Register */ diff --git a/include/standard-headers/linux/virtio_gpio.h b/include/standard-headers/linux/virtio_gpio.h new file mode 100644 index 0000000000..2b5cf06349 --- /dev/null +++ b/include/standard-headers/linux/virtio_gpio.h @@ -0,0 +1,72 @@ +/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ + +#ifndef _LINUX_VIRTIO_GPIO_H +#define _LINUX_VIRTIO_GPIO_H + +#include "standard-headers/linux/types.h" + +/* Virtio GPIO Feature bits */ +#define VIRTIO_GPIO_F_IRQ 0 + +/* Virtio GPIO request types */ +#define VIRTIO_GPIO_MSG_GET_NAMES 0x0001 +#define VIRTIO_GPIO_MSG_GET_DIRECTION 0x0002 +#define VIRTIO_GPIO_MSG_SET_DIRECTION 0x0003 +#define VIRTIO_GPIO_MSG_GET_VALUE 0x0004 +#define VIRTIO_GPIO_MSG_SET_VALUE 0x0005 +#define VIRTIO_GPIO_MSG_IRQ_TYPE 0x0006 + +/* Possible values of the status field */ +#define VIRTIO_GPIO_STATUS_OK 0x0 +#define VIRTIO_GPIO_STATUS_ERR 0x1 + +/* Direction types */ +#define VIRTIO_GPIO_DIRECTION_NONE 0x00 +#define VIRTIO_GPIO_DIRECTION_OUT 0x01 +#define VIRTIO_GPIO_DIRECTION_IN 0x02 + +/* Virtio GPIO IRQ types */ +#define VIRTIO_GPIO_IRQ_TYPE_NONE 0x00 +#define VIRTIO_GPIO_IRQ_TYPE_EDGE_RISING 0x01 +#define VIRTIO_GPIO_IRQ_TYPE_EDGE_FALLING 0x02 +#define VIRTIO_GPIO_IRQ_TYPE_EDGE_BOTH 0x03 +#define VIRTIO_GPIO_IRQ_TYPE_LEVEL_HIGH 0x04 +#define VIRTIO_GPIO_IRQ_TYPE_LEVEL_LOW 0x08 + +struct virtio_gpio_config { + uint16_t ngpio; + uint8_t padding[2]; + uint32_t gpio_names_size; +}; + +/* Virtio GPIO Request / Response */ +struct virtio_gpio_request { + uint16_t type; + uint16_t gpio; + uint32_t value; +}; + +struct virtio_gpio_response { + uint8_t status; + uint8_t value; +}; + +struct virtio_gpio_response_get_names { + uint8_t status; + uint8_t value[]; +}; + +/* Virtio GPIO IRQ Request / Response */ +struct virtio_gpio_irq_request { + uint16_t gpio; +}; + +struct virtio_gpio_irq_response { + uint8_t status; +}; + +/* Possible values of the interrupt status field */ +#define VIRTIO_GPIO_IRQ_STATUS_INVALID 0x0 +#define VIRTIO_GPIO_IRQ_STATUS_VALID 0x1 + +#endif /* _LINUX_VIRTIO_GPIO_H */ diff --git a/include/standard-headers/linux/virtio_i2c.h b/include/standard-headers/linux/virtio_i2c.h new file mode 100644 index 0000000000..09fa907793 --- /dev/null +++ b/include/standard-headers/linux/virtio_i2c.h @@ -0,0 +1,47 @@ +/* SPDX-License-Identifier: GPL-2.0-or-later WITH Linux-syscall-note */ +/* + * Definitions for virtio I2C Adpter + * + * Copyright (c) 2021 Intel Corporation. All rights reserved. + */ + +#ifndef _LINUX_VIRTIO_I2C_H +#define _LINUX_VIRTIO_I2C_H + +#include "standard-headers/linux/const.h" +#include "standard-headers/linux/types.h" + +/* Virtio I2C Feature bits */ +#define VIRTIO_I2C_F_ZERO_LENGTH_REQUEST 0 + +/* The bit 0 of the @virtio_i2c_out_hdr.@flags, used to group the requests */ +#define VIRTIO_I2C_FLAGS_FAIL_NEXT _BITUL(0) + +/* The bit 1 of the @virtio_i2c_out_hdr.@flags, used to mark a buffer as read */ +#define VIRTIO_I2C_FLAGS_M_RD _BITUL(1) + +/** + * struct virtio_i2c_out_hdr - the virtio I2C message OUT header + * @addr: the controlled device address + * @padding: used to pad to full dword + * @flags: used for feature extensibility + */ +struct virtio_i2c_out_hdr { + uint16_t addr; + uint16_t padding; + uint32_t flags; +}; + +/** + * struct virtio_i2c_in_hdr - the virtio I2C message IN header + * @status: the processing result from the backend + */ +struct virtio_i2c_in_hdr { + uint8_t status; +}; + +/* The final status written by the device */ +#define VIRTIO_I2C_MSG_OK 0 +#define VIRTIO_I2C_MSG_ERR 1 + +#endif /* _LINUX_VIRTIO_I2C_H */ diff --git a/include/standard-headers/linux/virtio_iommu.h b/include/standard-headers/linux/virtio_iommu.h index b9443b83a1..366379c2f0 100644 --- a/include/standard-headers/linux/virtio_iommu.h +++ b/include/standard-headers/linux/virtio_iommu.h @@ -16,6 +16,7 @@ #define VIRTIO_IOMMU_F_BYPASS 3 #define VIRTIO_IOMMU_F_PROBE 4 #define VIRTIO_IOMMU_F_MMIO 5 +#define VIRTIO_IOMMU_F_BYPASS_CONFIG 6 struct virtio_iommu_range_64 { uint64_t start; @@ -36,6 +37,8 @@ struct virtio_iommu_config { struct virtio_iommu_range_32 domain_range; /* Probe buffer size */ uint32_t probe_size; + uint8_t bypass; + uint8_t reserved[3]; }; /* Request types */ @@ -66,11 +69,14 @@ struct virtio_iommu_req_tail { uint8_t reserved[3]; }; +#define VIRTIO_IOMMU_ATTACH_F_BYPASS (1 << 0) + struct virtio_iommu_req_attach { struct virtio_iommu_req_head head; uint32_t domain; uint32_t endpoint; - uint8_t reserved[8]; + uint32_t flags; + uint8_t reserved[4]; struct virtio_iommu_req_tail tail; }; diff --git a/include/standard-headers/linux/virtio_pcidev.h b/include/standard-headers/linux/virtio_pcidev.h new file mode 100644 index 0000000000..bdf1d062da --- /dev/null +++ b/include/standard-headers/linux/virtio_pcidev.h @@ -0,0 +1,65 @@ +/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */ +/* + * Copyright (C) 2021 Intel Corporation + * Author: Johannes Berg <johannes@sipsolutions.net> + */ +#ifndef _LINUX_VIRTIO_PCIDEV_H +#define _LINUX_VIRTIO_PCIDEV_H +#include "standard-headers/linux/types.h" + +/** + * enum virtio_pcidev_ops - virtual PCI device operations + * @VIRTIO_PCIDEV_OP_RESERVED: reserved to catch errors + * @VIRTIO_PCIDEV_OP_CFG_READ: read config space, size is 1, 2, 4 or 8; + * the @data field should be filled in by the device (in little endian). + * @VIRTIO_PCIDEV_OP_CFG_WRITE: write config space, size is 1, 2, 4 or 8; + * the @data field contains the data to write (in little endian). + * @VIRTIO_PCIDEV_OP_MMIO_READ: read BAR mem/pio, size can be variable; + * the @data field should be filled in by the device (in little endian). + * @VIRTIO_PCIDEV_OP_MMIO_WRITE: write BAR mem/pio, size can be variable; + * the @data field contains the data to write (in little endian). + * @VIRTIO_PCIDEV_OP_MMIO_MEMSET: memset MMIO, size is variable but + * the @data field only has one byte (unlike @VIRTIO_PCIDEV_OP_MMIO_WRITE) + * @VIRTIO_PCIDEV_OP_INT: legacy INTx# pin interrupt, the addr field is 1-4 for + * the number + * @VIRTIO_PCIDEV_OP_MSI: MSI(-X) interrupt, this message basically transports + * the 16- or 32-bit write that would otherwise be done into memory, + * analogous to the write messages (@VIRTIO_PCIDEV_OP_MMIO_WRITE) above + * @VIRTIO_PCIDEV_OP_PME: Dummy message whose content is ignored (and should be + * all zeroes) to signal the PME# pin. + */ +enum virtio_pcidev_ops { + VIRTIO_PCIDEV_OP_RESERVED = 0, + VIRTIO_PCIDEV_OP_CFG_READ, + VIRTIO_PCIDEV_OP_CFG_WRITE, + VIRTIO_PCIDEV_OP_MMIO_READ, + VIRTIO_PCIDEV_OP_MMIO_WRITE, + VIRTIO_PCIDEV_OP_MMIO_MEMSET, + VIRTIO_PCIDEV_OP_INT, + VIRTIO_PCIDEV_OP_MSI, + VIRTIO_PCIDEV_OP_PME, +}; + +/** + * struct virtio_pcidev_msg - virtio PCI device operation + * @op: the operation to do + * @bar: the bar (only with BAR read/write messages) + * @reserved: reserved + * @size: the size of the read/write (in bytes) + * @addr: the address to read/write + * @data: the data, normally @size long, but just one byte for + * %VIRTIO_PCIDEV_OP_MMIO_MEMSET + * + * Note: the fields are all in native (CPU) endian, however, the + * @data values will often be in little endian (see the ops above.) + */ +struct virtio_pcidev_msg { + uint8_t op; + uint8_t bar; + uint16_t reserved; + uint32_t size; + uint64_t addr; + uint8_t data[]; +}; + +#endif /* _LINUX_VIRTIO_PCIDEV_H */ diff --git a/include/standard-headers/linux/virtio_scmi.h b/include/standard-headers/linux/virtio_scmi.h new file mode 100644 index 0000000000..8f2c305aea --- /dev/null +++ b/include/standard-headers/linux/virtio_scmi.h @@ -0,0 +1,24 @@ +/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */ +/* + * Copyright (C) 2020-2021 OpenSynergy GmbH + * Copyright (C) 2021 ARM Ltd. + */ + +#ifndef _LINUX_VIRTIO_SCMI_H +#define _LINUX_VIRTIO_SCMI_H + +#include "standard-headers/linux/virtio_types.h" + +/* Device implements some SCMI notifications, or delayed responses. */ +#define VIRTIO_SCMI_F_P2A_CHANNELS 0 + +/* Device implements any SCMI statistics shared memory region */ +#define VIRTIO_SCMI_F_SHARED_MEMORY 1 + +/* Virtqueues */ + +#define VIRTIO_SCMI_VQ_TX 0 /* cmdq */ +#define VIRTIO_SCMI_VQ_RX 1 /* eventq */ +#define VIRTIO_SCMI_VQ_MAX_CNT 2 + +#endif /* _LINUX_VIRTIO_SCMI_H */ diff --git a/linux-headers/asm-generic/unistd.h b/linux-headers/asm-generic/unistd.h index 4557a8b608..1c48b0ae3b 100644 --- a/linux-headers/asm-generic/unistd.h +++ b/linux-headers/asm-generic/unistd.h @@ -883,8 +883,11 @@ __SYSCALL(__NR_process_mrelease, sys_process_mrelease) #define __NR_futex_waitv 449 __SYSCALL(__NR_futex_waitv, sys_futex_waitv) +#define __NR_set_mempolicy_home_node 450 +__SYSCALL(__NR_set_mempolicy_home_node, sys_set_mempolicy_home_node) + #undef __NR_syscalls -#define __NR_syscalls 450 +#define __NR_syscalls 451 /* * 32 bit systems traditionally used different diff --git a/linux-headers/asm-mips/unistd_n32.h b/linux-headers/asm-mips/unistd_n32.h index 4b3e7ad1ec..1f14a6fad3 100644 --- a/linux-headers/asm-mips/unistd_n32.h +++ b/linux-headers/asm-mips/unistd_n32.h @@ -377,5 +377,7 @@ #define __NR_landlock_add_rule (__NR_Linux + 445) #define __NR_landlock_restrict_self (__NR_Linux + 446) #define __NR_process_mrelease (__NR_Linux + 448) +#define __NR_futex_waitv (__NR_Linux + 449) +#define __NR_set_mempolicy_home_node (__NR_Linux + 450) #endif /* _ASM_UNISTD_N32_H */ diff --git a/linux-headers/asm-mips/unistd_n64.h b/linux-headers/asm-mips/unistd_n64.h index 488d9298d9..e5a8ebec78 100644 --- a/linux-headers/asm-mips/unistd_n64.h +++ b/linux-headers/asm-mips/unistd_n64.h @@ -353,5 +353,7 @@ #define __NR_landlock_add_rule (__NR_Linux + 445) #define __NR_landlock_restrict_self (__NR_Linux + 446) #define __NR_process_mrelease (__NR_Linux + 448) +#define __NR_futex_waitv (__NR_Linux + 449) +#define __NR_set_mempolicy_home_node (__NR_Linux + 450) #endif /* _ASM_UNISTD_N64_H */ diff --git a/linux-headers/asm-mips/unistd_o32.h b/linux-headers/asm-mips/unistd_o32.h index f47399870a..871d57168f 100644 --- a/linux-headers/asm-mips/unistd_o32.h +++ b/linux-headers/asm-mips/unistd_o32.h @@ -423,5 +423,7 @@ #define __NR_landlock_add_rule (__NR_Linux + 445) #define __NR_landlock_restrict_self (__NR_Linux + 446) #define __NR_process_mrelease (__NR_Linux + 448) +#define __NR_futex_waitv (__NR_Linux + 449) +#define __NR_set_mempolicy_home_node (__NR_Linux + 450) #endif /* _ASM_UNISTD_O32_H */ diff --git a/linux-headers/asm-powerpc/unistd_32.h b/linux-headers/asm-powerpc/unistd_32.h index 11d54696dc..585c7fefbc 100644 --- a/linux-headers/asm-powerpc/unistd_32.h +++ b/linux-headers/asm-powerpc/unistd_32.h @@ -430,6 +430,8 @@ #define __NR_landlock_add_rule 445 #define __NR_landlock_restrict_self 446 #define __NR_process_mrelease 448 +#define __NR_futex_waitv 449 +#define __NR_set_mempolicy_home_node 450 #endif /* _ASM_UNISTD_32_H */ diff --git a/linux-headers/asm-powerpc/unistd_64.h b/linux-headers/asm-powerpc/unistd_64.h index cf740bab13..350f7ec0ac 100644 --- a/linux-headers/asm-powerpc/unistd_64.h +++ b/linux-headers/asm-powerpc/unistd_64.h @@ -402,6 +402,8 @@ #define __NR_landlock_add_rule 445 #define __NR_landlock_restrict_self 446 #define __NR_process_mrelease 448 +#define __NR_futex_waitv 449 +#define __NR_set_mempolicy_home_node 450 #endif /* _ASM_UNISTD_64_H */ diff --git a/linux-headers/asm-riscv/bitsperlong.h b/linux-headers/asm-riscv/bitsperlong.h new file mode 100644 index 0000000000..cc5c45a9ce --- /dev/null +++ b/linux-headers/asm-riscv/bitsperlong.h @@ -0,0 +1,14 @@ +/* SPDX-License-Identifier: GPL-2.0-only WITH Linux-syscall-note */ +/* + * Copyright (C) 2012 ARM Ltd. + * Copyright (C) 2015 Regents of the University of California + */ + +#ifndef _ASM_RISCV_BITSPERLONG_H +#define _ASM_RISCV_BITSPERLONG_H + +#define __BITS_PER_LONG (__SIZEOF_POINTER__ * 8) + +#include <asm-generic/bitsperlong.h> + +#endif /* _ASM_RISCV_BITSPERLONG_H */ diff --git a/linux-headers/asm-riscv/mman.h b/linux-headers/asm-riscv/mman.h new file mode 100644 index 0000000000..8eebf89f5a --- /dev/null +++ b/linux-headers/asm-riscv/mman.h @@ -0,0 +1 @@ +#include <asm-generic/mman.h> diff --git a/linux-headers/asm-riscv/unistd.h b/linux-headers/asm-riscv/unistd.h new file mode 100644 index 0000000000..8062996c2d --- /dev/null +++ b/linux-headers/asm-riscv/unistd.h @@ -0,0 +1,44 @@ +/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ +/* + * Copyright (C) 2018 David Abdurachmanov <david.abdurachmanov@gmail.com> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see <https://www.gnu.org/licenses/>. + */ + +#ifdef __LP64__ +#define __ARCH_WANT_NEW_STAT +#define __ARCH_WANT_SET_GET_RLIMIT +#endif /* __LP64__ */ + +#define __ARCH_WANT_SYS_CLONE3 + +#include <asm-generic/unistd.h> + +/* + * Allows the instruction cache to be flushed from userspace. Despite RISC-V + * having a direct 'fence.i' instruction available to userspace (which we + * can't trap!), that's not actually viable when running on Linux because the + * kernel might schedule a process on another hart. There is no way for + * userspace to handle this without invoking the kernel (as it doesn't know the + * thread->hart mappings), so we've defined a RISC-V specific system call to + * flush the instruction cache. + * + * __NR_riscv_flush_icache is defined to flush the instruction cache over an + * address range, with the flush applying to either all threads or just the + * caller. We don't currently do anything with the address range, that's just + * in there for forwards compatibility. + */ +#ifndef __NR_riscv_flush_icache +#define __NR_riscv_flush_icache (__NR_arch_specific_syscall + 15) +#endif +__SYSCALL(__NR_riscv_flush_icache, sys_riscv_flush_icache) diff --git a/linux-headers/asm-s390/unistd_32.h b/linux-headers/asm-s390/unistd_32.h index 8f97d98128..8e644d65f5 100644 --- a/linux-headers/asm-s390/unistd_32.h +++ b/linux-headers/asm-s390/unistd_32.h @@ -420,5 +420,7 @@ #define __NR_landlock_add_rule 445 #define __NR_landlock_restrict_self 446 #define __NR_process_mrelease 448 +#define __NR_futex_waitv 449 +#define __NR_set_mempolicy_home_node 450 #endif /* _ASM_S390_UNISTD_32_H */ diff --git a/linux-headers/asm-s390/unistd_64.h b/linux-headers/asm-s390/unistd_64.h index 021ffc30e6..51da542fec 100644 --- a/linux-headers/asm-s390/unistd_64.h +++ b/linux-headers/asm-s390/unistd_64.h @@ -368,5 +368,7 @@ #define __NR_landlock_add_rule 445 #define __NR_landlock_restrict_self 446 #define __NR_process_mrelease 448 +#define __NR_futex_waitv 449 +#define __NR_set_mempolicy_home_node 450 #endif /* _ASM_S390_UNISTD_64_H */ diff --git a/linux-headers/asm-x86/kvm.h b/linux-headers/asm-x86/kvm.h index 5a776a08f7..2da3316bb5 100644 --- a/linux-headers/asm-x86/kvm.h +++ b/linux-headers/asm-x86/kvm.h @@ -373,9 +373,23 @@ struct kvm_debugregs { __u64 reserved[9]; }; -/* for KVM_CAP_XSAVE */ +/* for KVM_CAP_XSAVE and KVM_CAP_XSAVE2 */ struct kvm_xsave { + /* + * KVM_GET_XSAVE2 and KVM_SET_XSAVE write and read as many bytes + * as are returned by KVM_CHECK_EXTENSION(KVM_CAP_XSAVE2) + * respectively, when invoked on the vm file descriptor. + * + * The size value returned by KVM_CHECK_EXTENSION(KVM_CAP_XSAVE2) + * will always be at least 4096. Currently, it is only greater + * than 4096 if a dynamic feature has been enabled with + * ``arch_prctl()``, but this may change in the future. + * + * The offsets of the state save areas in struct kvm_xsave follow + * the contents of CPUID leaf 0xD on the host. + */ __u32 region[1024]; + __u32 extra[0]; }; #define KVM_MAX_XCRS 16 diff --git a/linux-headers/asm-x86/unistd_32.h b/linux-headers/asm-x86/unistd_32.h index 9c9ffe312b..87e1e977af 100644 --- a/linux-headers/asm-x86/unistd_32.h +++ b/linux-headers/asm-x86/unistd_32.h @@ -440,6 +440,7 @@ #define __NR_memfd_secret 447 #define __NR_process_mrelease 448 #define __NR_futex_waitv 449 +#define __NR_set_mempolicy_home_node 450 #endif /* _ASM_UNISTD_32_H */ diff --git a/linux-headers/asm-x86/unistd_64.h b/linux-headers/asm-x86/unistd_64.h index 084f1eef9c..147a78d623 100644 --- a/linux-headers/asm-x86/unistd_64.h +++ b/linux-headers/asm-x86/unistd_64.h @@ -362,6 +362,7 @@ #define __NR_memfd_secret 447 #define __NR_process_mrelease 448 #define __NR_futex_waitv 449 +#define __NR_set_mempolicy_home_node 450 #endif /* _ASM_UNISTD_64_H */ diff --git a/linux-headers/asm-x86/unistd_x32.h b/linux-headers/asm-x86/unistd_x32.h index a2441affc2..27098db7fb 100644 --- a/linux-headers/asm-x86/unistd_x32.h +++ b/linux-headers/asm-x86/unistd_x32.h @@ -315,6 +315,7 @@ #define __NR_memfd_secret (__X32_SYSCALL_BIT + 447) #define __NR_process_mrelease (__X32_SYSCALL_BIT + 448) #define __NR_futex_waitv (__X32_SYSCALL_BIT + 449) +#define __NR_set_mempolicy_home_node (__X32_SYSCALL_BIT + 450) #define __NR_rt_sigaction (__X32_SYSCALL_BIT + 512) #define __NR_rt_sigreturn (__X32_SYSCALL_BIT + 513) #define __NR_ioctl (__X32_SYSCALL_BIT + 514) diff --git a/linux-headers/linux/kvm.h b/linux-headers/linux/kvm.h index 02c5e7b7bb..00af3bc333 100644 --- a/linux-headers/linux/kvm.h +++ b/linux-headers/linux/kvm.h @@ -1130,6 +1130,9 @@ struct kvm_ppc_resize_hpt { #define KVM_CAP_BINARY_STATS_FD 203 #define KVM_CAP_EXIT_ON_EMULATION_FAILURE 204 #define KVM_CAP_ARM_MTE 205 +#define KVM_CAP_VM_MOVE_ENC_CONTEXT_FROM 206 +#define KVM_CAP_VM_GPA_BITS 207 +#define KVM_CAP_XSAVE2 208 #ifdef KVM_CAP_IRQ_ROUTING @@ -1161,11 +1164,20 @@ struct kvm_irq_routing_hv_sint { __u32 sint; }; +struct kvm_irq_routing_xen_evtchn { + __u32 port; + __u32 vcpu; + __u32 priority; +}; + +#define KVM_IRQ_ROUTING_XEN_EVTCHN_PRIO_2LEVEL ((__u32)(-1)) + /* gsi routing entry types */ #define KVM_IRQ_ROUTING_IRQCHIP 1 #define KVM_IRQ_ROUTING_MSI 2 #define KVM_IRQ_ROUTING_S390_ADAPTER 3 #define KVM_IRQ_ROUTING_HV_SINT 4 +#define KVM_IRQ_ROUTING_XEN_EVTCHN 5 struct kvm_irq_routing_entry { __u32 gsi; @@ -1177,6 +1189,7 @@ struct kvm_irq_routing_entry { struct kvm_irq_routing_msi msi; struct kvm_irq_routing_s390_adapter adapter; struct kvm_irq_routing_hv_sint hv_sint; + struct kvm_irq_routing_xen_evtchn xen_evtchn; __u32 pad[8]; } u; }; @@ -1207,6 +1220,7 @@ struct kvm_x86_mce { #define KVM_XEN_HVM_CONFIG_INTERCEPT_HCALL (1 << 1) #define KVM_XEN_HVM_CONFIG_SHARED_INFO (1 << 2) #define KVM_XEN_HVM_CONFIG_RUNSTATE (1 << 3) +#define KVM_XEN_HVM_CONFIG_EVTCHN_2LEVEL (1 << 4) struct kvm_xen_hvm_config { __u32 flags; @@ -1609,6 +1623,9 @@ struct kvm_enc_region { #define KVM_S390_NORMAL_RESET _IO(KVMIO, 0xc3) #define KVM_S390_CLEAR_RESET _IO(KVMIO, 0xc4) +/* Available with KVM_CAP_XSAVE2 */ +#define KVM_GET_XSAVE2 _IOR(KVMIO, 0xcf, struct kvm_xsave) + struct kvm_s390_pv_sec_parm { __u64 origin; __u64 length; -- 2.34.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH v2 1/4] linux-headers: update to v5.17-rc1 2022-01-27 14:29 ` [PATCH v2 1/4] linux-headers: update to v5.17-rc1 Jean-Philippe Brucker @ 2022-01-31 9:14 ` Eric Auger 0 siblings, 0 replies; 20+ messages in thread From: Eric Auger @ 2022-01-31 9:14 UTC (permalink / raw) To: Jean-Philippe Brucker Cc: lvivier, thuth, mst, cohuck, qemu-devel, pasic, pbonzini Hi Jean, On 1/27/22 3:29 PM, Jean-Philippe Brucker wrote: > Update Linux headers to v5.17-rc1 > > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> Reviewed-by: Eric Auger <eric.auger@redhat.com> Eric > --- > include/standard-headers/asm-x86/kvm_para.h | 1 + > include/standard-headers/drm/drm_fourcc.h | 11 ++ > include/standard-headers/linux/ethtool.h | 1 + > include/standard-headers/linux/fuse.h | 60 +++++++- > include/standard-headers/linux/pci_regs.h | 142 +++++++++--------- > include/standard-headers/linux/virtio_gpio.h | 72 +++++++++ > include/standard-headers/linux/virtio_i2c.h | 47 ++++++ > include/standard-headers/linux/virtio_iommu.h | 8 +- > .../standard-headers/linux/virtio_pcidev.h | 65 ++++++++ > include/standard-headers/linux/virtio_scmi.h | 24 +++ > linux-headers/asm-generic/unistd.h | 5 +- > linux-headers/asm-mips/unistd_n32.h | 2 + > linux-headers/asm-mips/unistd_n64.h | 2 + > linux-headers/asm-mips/unistd_o32.h | 2 + > linux-headers/asm-powerpc/unistd_32.h | 2 + > linux-headers/asm-powerpc/unistd_64.h | 2 + > linux-headers/asm-riscv/bitsperlong.h | 14 ++ > linux-headers/asm-riscv/mman.h | 1 + > linux-headers/asm-riscv/unistd.h | 44 ++++++ > linux-headers/asm-s390/unistd_32.h | 2 + > linux-headers/asm-s390/unistd_64.h | 2 + > linux-headers/asm-x86/kvm.h | 16 +- > linux-headers/asm-x86/unistd_32.h | 1 + > linux-headers/asm-x86/unistd_64.h | 1 + > linux-headers/asm-x86/unistd_x32.h | 1 + > linux-headers/linux/kvm.h | 17 +++ > 26 files changed, 469 insertions(+), 76 deletions(-) > create mode 100644 include/standard-headers/linux/virtio_gpio.h > create mode 100644 include/standard-headers/linux/virtio_i2c.h > create mode 100644 include/standard-headers/linux/virtio_pcidev.h > create mode 100644 include/standard-headers/linux/virtio_scmi.h > create mode 100644 linux-headers/asm-riscv/bitsperlong.h > create mode 100644 linux-headers/asm-riscv/mman.h > create mode 100644 linux-headers/asm-riscv/unistd.h > > diff --git a/include/standard-headers/asm-x86/kvm_para.h b/include/standard-headers/asm-x86/kvm_para.h > index 204cfb8640..f0235e58a1 100644 > --- a/include/standard-headers/asm-x86/kvm_para.h > +++ b/include/standard-headers/asm-x86/kvm_para.h > @@ -8,6 +8,7 @@ > * should be used to determine that a VM is running under KVM. > */ > #define KVM_CPUID_SIGNATURE 0x40000000 > +#define KVM_SIGNATURE "KVMKVMKVM\0\0\0" > > /* This CPUID returns two feature bitmaps in eax, edx. Before enabling > * a particular paravirtualization, the appropriate feature bit should > diff --git a/include/standard-headers/drm/drm_fourcc.h b/include/standard-headers/drm/drm_fourcc.h > index 2c025cb4fe..4888f85f69 100644 > --- a/include/standard-headers/drm/drm_fourcc.h > +++ b/include/standard-headers/drm/drm_fourcc.h > @@ -313,6 +313,13 @@ extern "C" { > */ > #define DRM_FORMAT_P016 fourcc_code('P', '0', '1', '6') /* 2x2 subsampled Cr:Cb plane 16 bits per channel */ > > +/* 2 plane YCbCr420. > + * 3 10 bit components and 2 padding bits packed into 4 bytes. > + * index 0 = Y plane, [31:0] x:Y2:Y1:Y0 2:10:10:10 little endian > + * index 1 = Cr:Cb plane, [63:0] x:Cr2:Cb2:Cr1:x:Cb1:Cr0:Cb0 [2:10:10:10:2:10:10:10] little endian > + */ > +#define DRM_FORMAT_P030 fourcc_code('P', '0', '3', '0') /* 2x2 subsampled Cr:Cb plane 10 bits per channel packed */ > + > /* 3 plane non-subsampled (444) YCbCr > * 16 bits per component, but only 10 bits are used and 6 bits are padded > * index 0: Y plane, [15:0] Y:x [10:6] little endian > @@ -853,6 +860,10 @@ drm_fourcc_canonicalize_nvidia_format_mod(uint64_t modifier) > * and UV. Some SAND-using hardware stores UV in a separate tiled > * image from Y to reduce the column height, which is not supported > * with these modifiers. > + * > + * The DRM_FORMAT_MOD_BROADCOM_SAND128_COL_HEIGHT modifier is also > + * supported for DRM_FORMAT_P030 where the columns remain as 128 bytes > + * wide, but as this is a 10 bpp format that translates to 96 pixels. > */ > > #define DRM_FORMAT_MOD_BROADCOM_SAND32_COL_HEIGHT(v) \ > diff --git a/include/standard-headers/linux/ethtool.h b/include/standard-headers/linux/ethtool.h > index 688eb8dc39..38d5a4cd6e 100644 > --- a/include/standard-headers/linux/ethtool.h > +++ b/include/standard-headers/linux/ethtool.h > @@ -231,6 +231,7 @@ enum tunable_id { > ETHTOOL_RX_COPYBREAK, > ETHTOOL_TX_COPYBREAK, > ETHTOOL_PFC_PREVENTION_TOUT, /* timeout in msecs */ > + ETHTOOL_TX_COPYBREAK_BUF_SIZE, > /* > * Add your fresh new tunable attribute above and remember to update > * tunable_strings[] in net/ethtool/common.c > diff --git a/include/standard-headers/linux/fuse.h b/include/standard-headers/linux/fuse.h > index 23ea31708b..bda06258be 100644 > --- a/include/standard-headers/linux/fuse.h > +++ b/include/standard-headers/linux/fuse.h > @@ -184,6 +184,16 @@ > * > * 7.34 > * - add FUSE_SYNCFS > + * > + * 7.35 > + * - add FOPEN_NOFLUSH > + * > + * 7.36 > + * - extend fuse_init_in with reserved fields, add FUSE_INIT_EXT init flag > + * - add flags2 to fuse_init_in and fuse_init_out > + * - add FUSE_SECURITY_CTX init flag > + * - add security context to create, mkdir, symlink, and mknod requests > + * - add FUSE_HAS_INODE_DAX, FUSE_ATTR_DAX > */ > > #ifndef _LINUX_FUSE_H > @@ -215,7 +225,7 @@ > #define FUSE_KERNEL_VERSION 7 > > /** Minor version number of this interface */ > -#define FUSE_KERNEL_MINOR_VERSION 34 > +#define FUSE_KERNEL_MINOR_VERSION 36 > > /** The node ID of the root inode */ > #define FUSE_ROOT_ID 1 > @@ -286,12 +296,14 @@ struct fuse_file_lock { > * FOPEN_NONSEEKABLE: the file is not seekable > * FOPEN_CACHE_DIR: allow caching this directory > * FOPEN_STREAM: the file is stream-like (no file position at all) > + * FOPEN_NOFLUSH: don't flush data cache on close (unless FUSE_WRITEBACK_CACHE) > */ > #define FOPEN_DIRECT_IO (1 << 0) > #define FOPEN_KEEP_CACHE (1 << 1) > #define FOPEN_NONSEEKABLE (1 << 2) > #define FOPEN_CACHE_DIR (1 << 3) > #define FOPEN_STREAM (1 << 4) > +#define FOPEN_NOFLUSH (1 << 5) > > /** > * INIT request/reply flags > @@ -332,6 +344,11 @@ struct fuse_file_lock { > * write/truncate sgid is killed only if file has group > * execute permission. (Same as Linux VFS behavior). > * FUSE_SETXATTR_EXT: Server supports extended struct fuse_setxattr_in > + * FUSE_INIT_EXT: extended fuse_init_in request > + * FUSE_INIT_RESERVED: reserved, do not use > + * FUSE_SECURITY_CTX: add security context to create, mkdir, symlink, and > + * mknod > + * FUSE_HAS_INODE_DAX: use per inode DAX > */ > #define FUSE_ASYNC_READ (1 << 0) > #define FUSE_POSIX_LOCKS (1 << 1) > @@ -363,6 +380,11 @@ struct fuse_file_lock { > #define FUSE_SUBMOUNTS (1 << 27) > #define FUSE_HANDLE_KILLPRIV_V2 (1 << 28) > #define FUSE_SETXATTR_EXT (1 << 29) > +#define FUSE_INIT_EXT (1 << 30) > +#define FUSE_INIT_RESERVED (1 << 31) > +/* bits 32..63 get shifted down 32 bits into the flags2 field */ > +#define FUSE_SECURITY_CTX (1ULL << 32) > +#define FUSE_HAS_INODE_DAX (1ULL << 33) > > /** > * CUSE INIT request/reply flags > @@ -445,8 +467,10 @@ struct fuse_file_lock { > * fuse_attr flags > * > * FUSE_ATTR_SUBMOUNT: Object is a submount root > + * FUSE_ATTR_DAX: Enable DAX for this file in per inode DAX mode > */ > #define FUSE_ATTR_SUBMOUNT (1 << 0) > +#define FUSE_ATTR_DAX (1 << 1) > > /** > * Open flags > @@ -732,6 +756,8 @@ struct fuse_init_in { > uint32_t minor; > uint32_t max_readahead; > uint32_t flags; > + uint32_t flags2; > + uint32_t unused[11]; > }; > > #define FUSE_COMPAT_INIT_OUT_SIZE 8 > @@ -748,7 +774,8 @@ struct fuse_init_out { > uint32_t time_gran; > uint16_t max_pages; > uint16_t map_alignment; > - uint32_t unused[8]; > + uint32_t flags2; > + uint32_t unused[7]; > }; > > #define CUSE_INIT_INFO_MAX 4096 > @@ -856,9 +883,12 @@ struct fuse_dirent { > char name[]; > }; > > -#define FUSE_NAME_OFFSET offsetof(struct fuse_dirent, name) > -#define FUSE_DIRENT_ALIGN(x) \ > +/* Align variable length records to 64bit boundary */ > +#define FUSE_REC_ALIGN(x) \ > (((x) + sizeof(uint64_t) - 1) & ~(sizeof(uint64_t) - 1)) > + > +#define FUSE_NAME_OFFSET offsetof(struct fuse_dirent, name) > +#define FUSE_DIRENT_ALIGN(x) FUSE_REC_ALIGN(x) > #define FUSE_DIRENT_SIZE(d) \ > FUSE_DIRENT_ALIGN(FUSE_NAME_OFFSET + (d)->namelen) > > @@ -975,4 +1005,26 @@ struct fuse_syncfs_in { > uint64_t padding; > }; > > +/* > + * For each security context, send fuse_secctx with size of security context > + * fuse_secctx will be followed by security context name and this in turn > + * will be followed by actual context label. > + * fuse_secctx, name, context > + */ > +struct fuse_secctx { > + uint32_t size; > + uint32_t padding; > +}; > + > +/* > + * Contains the information about how many fuse_secctx structures are being > + * sent and what's the total size of all security contexts (including > + * size of fuse_secctx_header). > + * > + */ > +struct fuse_secctx_header { > + uint32_t size; > + uint32_t nr_secctx; > +}; > + > #endif /* _LINUX_FUSE_H */ > diff --git a/include/standard-headers/linux/pci_regs.h b/include/standard-headers/linux/pci_regs.h > index ff6ccbc6ef..bee1a9ed6e 100644 > --- a/include/standard-headers/linux/pci_regs.h > +++ b/include/standard-headers/linux/pci_regs.h > @@ -301,23 +301,23 @@ > #define PCI_SID_ESR_FIC 0x20 /* First In Chassis Flag */ > #define PCI_SID_CHASSIS_NR 3 /* Chassis Number */ > > -/* Message Signalled Interrupt registers */ > +/* Message Signaled Interrupt registers */ > > -#define PCI_MSI_FLAGS 2 /* Message Control */ > +#define PCI_MSI_FLAGS 0x02 /* Message Control */ > #define PCI_MSI_FLAGS_ENABLE 0x0001 /* MSI feature enabled */ > #define PCI_MSI_FLAGS_QMASK 0x000e /* Maximum queue size available */ > #define PCI_MSI_FLAGS_QSIZE 0x0070 /* Message queue size configured */ > #define PCI_MSI_FLAGS_64BIT 0x0080 /* 64-bit addresses allowed */ > #define PCI_MSI_FLAGS_MASKBIT 0x0100 /* Per-vector masking capable */ > #define PCI_MSI_RFU 3 /* Rest of capability flags */ > -#define PCI_MSI_ADDRESS_LO 4 /* Lower 32 bits */ > -#define PCI_MSI_ADDRESS_HI 8 /* Upper 32 bits (if PCI_MSI_FLAGS_64BIT set) */ > -#define PCI_MSI_DATA_32 8 /* 16 bits of data for 32-bit devices */ > -#define PCI_MSI_MASK_32 12 /* Mask bits register for 32-bit devices */ > -#define PCI_MSI_PENDING_32 16 /* Pending intrs for 32-bit devices */ > -#define PCI_MSI_DATA_64 12 /* 16 bits of data for 64-bit devices */ > -#define PCI_MSI_MASK_64 16 /* Mask bits register for 64-bit devices */ > -#define PCI_MSI_PENDING_64 20 /* Pending intrs for 64-bit devices */ > +#define PCI_MSI_ADDRESS_LO 0x04 /* Lower 32 bits */ > +#define PCI_MSI_ADDRESS_HI 0x08 /* Upper 32 bits (if PCI_MSI_FLAGS_64BIT set) */ > +#define PCI_MSI_DATA_32 0x08 /* 16 bits of data for 32-bit devices */ > +#define PCI_MSI_MASK_32 0x0c /* Mask bits register for 32-bit devices */ > +#define PCI_MSI_PENDING_32 0x10 /* Pending intrs for 32-bit devices */ > +#define PCI_MSI_DATA_64 0x0c /* 16 bits of data for 64-bit devices */ > +#define PCI_MSI_MASK_64 0x10 /* Mask bits register for 64-bit devices */ > +#define PCI_MSI_PENDING_64 0x14 /* Pending intrs for 64-bit devices */ > > /* MSI-X registers (in MSI-X capability) */ > #define PCI_MSIX_FLAGS 2 /* Message Control */ > @@ -335,10 +335,10 @@ > > /* MSI-X Table entry format (in memory mapped by a BAR) */ > #define PCI_MSIX_ENTRY_SIZE 16 > -#define PCI_MSIX_ENTRY_LOWER_ADDR 0 /* Message Address */ > -#define PCI_MSIX_ENTRY_UPPER_ADDR 4 /* Message Upper Address */ > -#define PCI_MSIX_ENTRY_DATA 8 /* Message Data */ > -#define PCI_MSIX_ENTRY_VECTOR_CTRL 12 /* Vector Control */ > +#define PCI_MSIX_ENTRY_LOWER_ADDR 0x0 /* Message Address */ > +#define PCI_MSIX_ENTRY_UPPER_ADDR 0x4 /* Message Upper Address */ > +#define PCI_MSIX_ENTRY_DATA 0x8 /* Message Data */ > +#define PCI_MSIX_ENTRY_VECTOR_CTRL 0xc /* Vector Control */ > #define PCI_MSIX_ENTRY_CTRL_MASKBIT 0x00000001 > > /* CompactPCI Hotswap Register */ > @@ -470,7 +470,7 @@ > > /* PCI Express capability registers */ > > -#define PCI_EXP_FLAGS 2 /* Capabilities register */ > +#define PCI_EXP_FLAGS 0x02 /* Capabilities register */ > #define PCI_EXP_FLAGS_VERS 0x000f /* Capability version */ > #define PCI_EXP_FLAGS_TYPE 0x00f0 /* Device/Port type */ > #define PCI_EXP_TYPE_ENDPOINT 0x0 /* Express Endpoint */ > @@ -484,7 +484,7 @@ > #define PCI_EXP_TYPE_RC_EC 0xa /* Root Complex Event Collector */ > #define PCI_EXP_FLAGS_SLOT 0x0100 /* Slot implemented */ > #define PCI_EXP_FLAGS_IRQ 0x3e00 /* Interrupt message number */ > -#define PCI_EXP_DEVCAP 4 /* Device capabilities */ > +#define PCI_EXP_DEVCAP 0x04 /* Device capabilities */ > #define PCI_EXP_DEVCAP_PAYLOAD 0x00000007 /* Max_Payload_Size */ > #define PCI_EXP_DEVCAP_PHANTOM 0x00000018 /* Phantom functions */ > #define PCI_EXP_DEVCAP_EXT_TAG 0x00000020 /* Extended tags */ > @@ -497,7 +497,7 @@ > #define PCI_EXP_DEVCAP_PWR_VAL 0x03fc0000 /* Slot Power Limit Value */ > #define PCI_EXP_DEVCAP_PWR_SCL 0x0c000000 /* Slot Power Limit Scale */ > #define PCI_EXP_DEVCAP_FLR 0x10000000 /* Function Level Reset */ > -#define PCI_EXP_DEVCTL 8 /* Device Control */ > +#define PCI_EXP_DEVCTL 0x08 /* Device Control */ > #define PCI_EXP_DEVCTL_CERE 0x0001 /* Correctable Error Reporting En. */ > #define PCI_EXP_DEVCTL_NFERE 0x0002 /* Non-Fatal Error Reporting Enable */ > #define PCI_EXP_DEVCTL_FERE 0x0004 /* Fatal Error Reporting Enable */ > @@ -522,7 +522,7 @@ > #define PCI_EXP_DEVCTL_READRQ_2048B 0x4000 /* 2048 Bytes */ > #define PCI_EXP_DEVCTL_READRQ_4096B 0x5000 /* 4096 Bytes */ > #define PCI_EXP_DEVCTL_BCR_FLR 0x8000 /* Bridge Configuration Retry / FLR */ > -#define PCI_EXP_DEVSTA 10 /* Device Status */ > +#define PCI_EXP_DEVSTA 0x0a /* Device Status */ > #define PCI_EXP_DEVSTA_CED 0x0001 /* Correctable Error Detected */ > #define PCI_EXP_DEVSTA_NFED 0x0002 /* Non-Fatal Error Detected */ > #define PCI_EXP_DEVSTA_FED 0x0004 /* Fatal Error Detected */ > @@ -530,7 +530,7 @@ > #define PCI_EXP_DEVSTA_AUXPD 0x0010 /* AUX Power Detected */ > #define PCI_EXP_DEVSTA_TRPND 0x0020 /* Transactions Pending */ > #define PCI_CAP_EXP_RC_ENDPOINT_SIZEOF_V1 12 /* v1 endpoints without link end here */ > -#define PCI_EXP_LNKCAP 12 /* Link Capabilities */ > +#define PCI_EXP_LNKCAP 0x0c /* Link Capabilities */ > #define PCI_EXP_LNKCAP_SLS 0x0000000f /* Supported Link Speeds */ > #define PCI_EXP_LNKCAP_SLS_2_5GB 0x00000001 /* LNKCAP2 SLS Vector bit 0 */ > #define PCI_EXP_LNKCAP_SLS_5_0GB 0x00000002 /* LNKCAP2 SLS Vector bit 1 */ > @@ -549,7 +549,7 @@ > #define PCI_EXP_LNKCAP_DLLLARC 0x00100000 /* Data Link Layer Link Active Reporting Capable */ > #define PCI_EXP_LNKCAP_LBNC 0x00200000 /* Link Bandwidth Notification Capability */ > #define PCI_EXP_LNKCAP_PN 0xff000000 /* Port Number */ > -#define PCI_EXP_LNKCTL 16 /* Link Control */ > +#define PCI_EXP_LNKCTL 0x10 /* Link Control */ > #define PCI_EXP_LNKCTL_ASPMC 0x0003 /* ASPM Control */ > #define PCI_EXP_LNKCTL_ASPM_L0S 0x0001 /* L0s Enable */ > #define PCI_EXP_LNKCTL_ASPM_L1 0x0002 /* L1 Enable */ > @@ -562,7 +562,7 @@ > #define PCI_EXP_LNKCTL_HAWD 0x0200 /* Hardware Autonomous Width Disable */ > #define PCI_EXP_LNKCTL_LBMIE 0x0400 /* Link Bandwidth Management Interrupt Enable */ > #define PCI_EXP_LNKCTL_LABIE 0x0800 /* Link Autonomous Bandwidth Interrupt Enable */ > -#define PCI_EXP_LNKSTA 18 /* Link Status */ > +#define PCI_EXP_LNKSTA 0x12 /* Link Status */ > #define PCI_EXP_LNKSTA_CLS 0x000f /* Current Link Speed */ > #define PCI_EXP_LNKSTA_CLS_2_5GB 0x0001 /* Current Link Speed 2.5GT/s */ > #define PCI_EXP_LNKSTA_CLS_5_0GB 0x0002 /* Current Link Speed 5.0GT/s */ > @@ -582,7 +582,7 @@ > #define PCI_EXP_LNKSTA_LBMS 0x4000 /* Link Bandwidth Management Status */ > #define PCI_EXP_LNKSTA_LABS 0x8000 /* Link Autonomous Bandwidth Status */ > #define PCI_CAP_EXP_ENDPOINT_SIZEOF_V1 20 /* v1 endpoints with link end here */ > -#define PCI_EXP_SLTCAP 20 /* Slot Capabilities */ > +#define PCI_EXP_SLTCAP 0x14 /* Slot Capabilities */ > #define PCI_EXP_SLTCAP_ABP 0x00000001 /* Attention Button Present */ > #define PCI_EXP_SLTCAP_PCP 0x00000002 /* Power Controller Present */ > #define PCI_EXP_SLTCAP_MRLSP 0x00000004 /* MRL Sensor Present */ > @@ -595,7 +595,7 @@ > #define PCI_EXP_SLTCAP_EIP 0x00020000 /* Electromechanical Interlock Present */ > #define PCI_EXP_SLTCAP_NCCS 0x00040000 /* No Command Completed Support */ > #define PCI_EXP_SLTCAP_PSN 0xfff80000 /* Physical Slot Number */ > -#define PCI_EXP_SLTCTL 24 /* Slot Control */ > +#define PCI_EXP_SLTCTL 0x18 /* Slot Control */ > #define PCI_EXP_SLTCTL_ABPE 0x0001 /* Attention Button Pressed Enable */ > #define PCI_EXP_SLTCTL_PFDE 0x0002 /* Power Fault Detected Enable */ > #define PCI_EXP_SLTCTL_MRLSCE 0x0004 /* MRL Sensor Changed Enable */ > @@ -617,7 +617,7 @@ > #define PCI_EXP_SLTCTL_EIC 0x0800 /* Electromechanical Interlock Control */ > #define PCI_EXP_SLTCTL_DLLSCE 0x1000 /* Data Link Layer State Changed Enable */ > #define PCI_EXP_SLTCTL_IBPD_DISABLE 0x4000 /* In-band PD disable */ > -#define PCI_EXP_SLTSTA 26 /* Slot Status */ > +#define PCI_EXP_SLTSTA 0x1a /* Slot Status */ > #define PCI_EXP_SLTSTA_ABP 0x0001 /* Attention Button Pressed */ > #define PCI_EXP_SLTSTA_PFD 0x0002 /* Power Fault Detected */ > #define PCI_EXP_SLTSTA_MRLSC 0x0004 /* MRL Sensor Changed */ > @@ -627,15 +627,15 @@ > #define PCI_EXP_SLTSTA_PDS 0x0040 /* Presence Detect State */ > #define PCI_EXP_SLTSTA_EIS 0x0080 /* Electromechanical Interlock Status */ > #define PCI_EXP_SLTSTA_DLLSC 0x0100 /* Data Link Layer State Changed */ > -#define PCI_EXP_RTCTL 28 /* Root Control */ > +#define PCI_EXP_RTCTL 0x1c /* Root Control */ > #define PCI_EXP_RTCTL_SECEE 0x0001 /* System Error on Correctable Error */ > #define PCI_EXP_RTCTL_SENFEE 0x0002 /* System Error on Non-Fatal Error */ > #define PCI_EXP_RTCTL_SEFEE 0x0004 /* System Error on Fatal Error */ > #define PCI_EXP_RTCTL_PMEIE 0x0008 /* PME Interrupt Enable */ > #define PCI_EXP_RTCTL_CRSSVE 0x0010 /* CRS Software Visibility Enable */ > -#define PCI_EXP_RTCAP 30 /* Root Capabilities */ > +#define PCI_EXP_RTCAP 0x1e /* Root Capabilities */ > #define PCI_EXP_RTCAP_CRSVIS 0x0001 /* CRS Software Visibility capability */ > -#define PCI_EXP_RTSTA 32 /* Root Status */ > +#define PCI_EXP_RTSTA 0x20 /* Root Status */ > #define PCI_EXP_RTSTA_PME 0x00010000 /* PME status */ > #define PCI_EXP_RTSTA_PENDING 0x00020000 /* PME pending */ > /* > @@ -646,7 +646,7 @@ > * Use pcie_capability_read_word() and similar interfaces to use them > * safely. > */ > -#define PCI_EXP_DEVCAP2 36 /* Device Capabilities 2 */ > +#define PCI_EXP_DEVCAP2 0x24 /* Device Capabilities 2 */ > #define PCI_EXP_DEVCAP2_COMP_TMOUT_DIS 0x00000010 /* Completion Timeout Disable supported */ > #define PCI_EXP_DEVCAP2_ARI 0x00000020 /* Alternative Routing-ID */ > #define PCI_EXP_DEVCAP2_ATOMIC_ROUTE 0x00000040 /* Atomic Op routing */ > @@ -658,7 +658,7 @@ > #define PCI_EXP_DEVCAP2_OBFF_MSG 0x00040000 /* New message signaling */ > #define PCI_EXP_DEVCAP2_OBFF_WAKE 0x00080000 /* Re-use WAKE# for OBFF */ > #define PCI_EXP_DEVCAP2_EE_PREFIX 0x00200000 /* End-End TLP Prefix */ > -#define PCI_EXP_DEVCTL2 40 /* Device Control 2 */ > +#define PCI_EXP_DEVCTL2 0x28 /* Device Control 2 */ > #define PCI_EXP_DEVCTL2_COMP_TIMEOUT 0x000f /* Completion Timeout Value */ > #define PCI_EXP_DEVCTL2_COMP_TMOUT_DIS 0x0010 /* Completion Timeout Disable */ > #define PCI_EXP_DEVCTL2_ARI 0x0020 /* Alternative Routing-ID */ > @@ -670,9 +670,9 @@ > #define PCI_EXP_DEVCTL2_OBFF_MSGA_EN 0x2000 /* Enable OBFF Message type A */ > #define PCI_EXP_DEVCTL2_OBFF_MSGB_EN 0x4000 /* Enable OBFF Message type B */ > #define PCI_EXP_DEVCTL2_OBFF_WAKE_EN 0x6000 /* OBFF using WAKE# signaling */ > -#define PCI_EXP_DEVSTA2 42 /* Device Status 2 */ > -#define PCI_CAP_EXP_RC_ENDPOINT_SIZEOF_V2 44 /* v2 endpoints without link end here */ > -#define PCI_EXP_LNKCAP2 44 /* Link Capabilities 2 */ > +#define PCI_EXP_DEVSTA2 0x2a /* Device Status 2 */ > +#define PCI_CAP_EXP_RC_ENDPOINT_SIZEOF_V2 0x2c /* end of v2 EPs w/o link */ > +#define PCI_EXP_LNKCAP2 0x2c /* Link Capabilities 2 */ > #define PCI_EXP_LNKCAP2_SLS_2_5GB 0x00000002 /* Supported Speed 2.5GT/s */ > #define PCI_EXP_LNKCAP2_SLS_5_0GB 0x00000004 /* Supported Speed 5GT/s */ > #define PCI_EXP_LNKCAP2_SLS_8_0GB 0x00000008 /* Supported Speed 8GT/s */ > @@ -680,7 +680,7 @@ > #define PCI_EXP_LNKCAP2_SLS_32_0GB 0x00000020 /* Supported Speed 32GT/s */ > #define PCI_EXP_LNKCAP2_SLS_64_0GB 0x00000040 /* Supported Speed 64GT/s */ > #define PCI_EXP_LNKCAP2_CROSSLINK 0x00000100 /* Crosslink supported */ > -#define PCI_EXP_LNKCTL2 48 /* Link Control 2 */ > +#define PCI_EXP_LNKCTL2 0x30 /* Link Control 2 */ > #define PCI_EXP_LNKCTL2_TLS 0x000f > #define PCI_EXP_LNKCTL2_TLS_2_5GT 0x0001 /* Supported Speed 2.5GT/s */ > #define PCI_EXP_LNKCTL2_TLS_5_0GT 0x0002 /* Supported Speed 5GT/s */ > @@ -691,12 +691,12 @@ > #define PCI_EXP_LNKCTL2_ENTER_COMP 0x0010 /* Enter Compliance */ > #define PCI_EXP_LNKCTL2_TX_MARGIN 0x0380 /* Transmit Margin */ > #define PCI_EXP_LNKCTL2_HASD 0x0020 /* HW Autonomous Speed Disable */ > -#define PCI_EXP_LNKSTA2 50 /* Link Status 2 */ > -#define PCI_CAP_EXP_ENDPOINT_SIZEOF_V2 52 /* v2 endpoints with link end here */ > -#define PCI_EXP_SLTCAP2 52 /* Slot Capabilities 2 */ > +#define PCI_EXP_LNKSTA2 0x32 /* Link Status 2 */ > +#define PCI_CAP_EXP_ENDPOINT_SIZEOF_V2 0x32 /* end of v2 EPs w/ link */ > +#define PCI_EXP_SLTCAP2 0x34 /* Slot Capabilities 2 */ > #define PCI_EXP_SLTCAP2_IBPD 0x00000001 /* In-band PD Disable Supported */ > -#define PCI_EXP_SLTCTL2 56 /* Slot Control 2 */ > -#define PCI_EXP_SLTSTA2 58 /* Slot Status 2 */ > +#define PCI_EXP_SLTCTL2 0x38 /* Slot Control 2 */ > +#define PCI_EXP_SLTSTA2 0x3a /* Slot Status 2 */ > > /* Extended Capabilities (PCI-X 2.0 and Express) */ > #define PCI_EXT_CAP_ID(header) (header & 0x0000ffff) > @@ -742,7 +742,7 @@ > #define PCI_EXT_CAP_MCAST_ENDPOINT_SIZEOF 40 > > /* Advanced Error Reporting */ > -#define PCI_ERR_UNCOR_STATUS 4 /* Uncorrectable Error Status */ > +#define PCI_ERR_UNCOR_STATUS 0x04 /* Uncorrectable Error Status */ > #define PCI_ERR_UNC_UND 0x00000001 /* Undefined */ > #define PCI_ERR_UNC_DLP 0x00000010 /* Data Link Protocol */ > #define PCI_ERR_UNC_SURPDN 0x00000020 /* Surprise Down */ > @@ -760,11 +760,11 @@ > #define PCI_ERR_UNC_MCBTLP 0x00800000 /* MC blocked TLP */ > #define PCI_ERR_UNC_ATOMEG 0x01000000 /* Atomic egress blocked */ > #define PCI_ERR_UNC_TLPPRE 0x02000000 /* TLP prefix blocked */ > -#define PCI_ERR_UNCOR_MASK 8 /* Uncorrectable Error Mask */ > +#define PCI_ERR_UNCOR_MASK 0x08 /* Uncorrectable Error Mask */ > /* Same bits as above */ > -#define PCI_ERR_UNCOR_SEVER 12 /* Uncorrectable Error Severity */ > +#define PCI_ERR_UNCOR_SEVER 0x0c /* Uncorrectable Error Severity */ > /* Same bits as above */ > -#define PCI_ERR_COR_STATUS 16 /* Correctable Error Status */ > +#define PCI_ERR_COR_STATUS 0x10 /* Correctable Error Status */ > #define PCI_ERR_COR_RCVR 0x00000001 /* Receiver Error Status */ > #define PCI_ERR_COR_BAD_TLP 0x00000040 /* Bad TLP Status */ > #define PCI_ERR_COR_BAD_DLLP 0x00000080 /* Bad DLLP Status */ > @@ -773,20 +773,20 @@ > #define PCI_ERR_COR_ADV_NFAT 0x00002000 /* Advisory Non-Fatal */ > #define PCI_ERR_COR_INTERNAL 0x00004000 /* Corrected Internal */ > #define PCI_ERR_COR_LOG_OVER 0x00008000 /* Header Log Overflow */ > -#define PCI_ERR_COR_MASK 20 /* Correctable Error Mask */ > +#define PCI_ERR_COR_MASK 0x14 /* Correctable Error Mask */ > /* Same bits as above */ > -#define PCI_ERR_CAP 24 /* Advanced Error Capabilities */ > -#define PCI_ERR_CAP_FEP(x) ((x) & 31) /* First Error Pointer */ > +#define PCI_ERR_CAP 0x18 /* Advanced Error Capabilities & Ctrl*/ > +#define PCI_ERR_CAP_FEP(x) ((x) & 0x1f) /* First Error Pointer */ > #define PCI_ERR_CAP_ECRC_GENC 0x00000020 /* ECRC Generation Capable */ > #define PCI_ERR_CAP_ECRC_GENE 0x00000040 /* ECRC Generation Enable */ > #define PCI_ERR_CAP_ECRC_CHKC 0x00000080 /* ECRC Check Capable */ > #define PCI_ERR_CAP_ECRC_CHKE 0x00000100 /* ECRC Check Enable */ > -#define PCI_ERR_HEADER_LOG 28 /* Header Log Register (16 bytes) */ > -#define PCI_ERR_ROOT_COMMAND 44 /* Root Error Command */ > +#define PCI_ERR_HEADER_LOG 0x1c /* Header Log Register (16 bytes) */ > +#define PCI_ERR_ROOT_COMMAND 0x2c /* Root Error Command */ > #define PCI_ERR_ROOT_CMD_COR_EN 0x00000001 /* Correctable Err Reporting Enable */ > #define PCI_ERR_ROOT_CMD_NONFATAL_EN 0x00000002 /* Non-Fatal Err Reporting Enable */ > #define PCI_ERR_ROOT_CMD_FATAL_EN 0x00000004 /* Fatal Err Reporting Enable */ > -#define PCI_ERR_ROOT_STATUS 48 > +#define PCI_ERR_ROOT_STATUS 0x30 > #define PCI_ERR_ROOT_COR_RCV 0x00000001 /* ERR_COR Received */ > #define PCI_ERR_ROOT_MULTI_COR_RCV 0x00000002 /* Multiple ERR_COR */ > #define PCI_ERR_ROOT_UNCOR_RCV 0x00000004 /* ERR_FATAL/NONFATAL */ > @@ -795,52 +795,52 @@ > #define PCI_ERR_ROOT_NONFATAL_RCV 0x00000020 /* Non-Fatal Received */ > #define PCI_ERR_ROOT_FATAL_RCV 0x00000040 /* Fatal Received */ > #define PCI_ERR_ROOT_AER_IRQ 0xf8000000 /* Advanced Error Interrupt Message Number */ > -#define PCI_ERR_ROOT_ERR_SRC 52 /* Error Source Identification */ > +#define PCI_ERR_ROOT_ERR_SRC 0x34 /* Error Source Identification */ > > /* Virtual Channel */ > -#define PCI_VC_PORT_CAP1 4 > +#define PCI_VC_PORT_CAP1 0x04 > #define PCI_VC_CAP1_EVCC 0x00000007 /* extended VC count */ > #define PCI_VC_CAP1_LPEVCC 0x00000070 /* low prio extended VC count */ > #define PCI_VC_CAP1_ARB_SIZE 0x00000c00 > -#define PCI_VC_PORT_CAP2 8 > +#define PCI_VC_PORT_CAP2 0x08 > #define PCI_VC_CAP2_32_PHASE 0x00000002 > #define PCI_VC_CAP2_64_PHASE 0x00000004 > #define PCI_VC_CAP2_128_PHASE 0x00000008 > #define PCI_VC_CAP2_ARB_OFF 0xff000000 > -#define PCI_VC_PORT_CTRL 12 > +#define PCI_VC_PORT_CTRL 0x0c > #define PCI_VC_PORT_CTRL_LOAD_TABLE 0x00000001 > -#define PCI_VC_PORT_STATUS 14 > +#define PCI_VC_PORT_STATUS 0x0e > #define PCI_VC_PORT_STATUS_TABLE 0x00000001 > -#define PCI_VC_RES_CAP 16 > +#define PCI_VC_RES_CAP 0x10 > #define PCI_VC_RES_CAP_32_PHASE 0x00000002 > #define PCI_VC_RES_CAP_64_PHASE 0x00000004 > #define PCI_VC_RES_CAP_128_PHASE 0x00000008 > #define PCI_VC_RES_CAP_128_PHASE_TB 0x00000010 > #define PCI_VC_RES_CAP_256_PHASE 0x00000020 > #define PCI_VC_RES_CAP_ARB_OFF 0xff000000 > -#define PCI_VC_RES_CTRL 20 > +#define PCI_VC_RES_CTRL 0x14 > #define PCI_VC_RES_CTRL_LOAD_TABLE 0x00010000 > #define PCI_VC_RES_CTRL_ARB_SELECT 0x000e0000 > #define PCI_VC_RES_CTRL_ID 0x07000000 > #define PCI_VC_RES_CTRL_ENABLE 0x80000000 > -#define PCI_VC_RES_STATUS 26 > +#define PCI_VC_RES_STATUS 0x1a > #define PCI_VC_RES_STATUS_TABLE 0x00000001 > #define PCI_VC_RES_STATUS_NEGO 0x00000002 > #define PCI_CAP_VC_BASE_SIZEOF 0x10 > -#define PCI_CAP_VC_PER_VC_SIZEOF 0x0C > +#define PCI_CAP_VC_PER_VC_SIZEOF 0x0c > > /* Power Budgeting */ > -#define PCI_PWR_DSR 4 /* Data Select Register */ > -#define PCI_PWR_DATA 8 /* Data Register */ > +#define PCI_PWR_DSR 0x04 /* Data Select Register */ > +#define PCI_PWR_DATA 0x08 /* Data Register */ > #define PCI_PWR_DATA_BASE(x) ((x) & 0xff) /* Base Power */ > #define PCI_PWR_DATA_SCALE(x) (((x) >> 8) & 3) /* Data Scale */ > #define PCI_PWR_DATA_PM_SUB(x) (((x) >> 10) & 7) /* PM Sub State */ > #define PCI_PWR_DATA_PM_STATE(x) (((x) >> 13) & 3) /* PM State */ > #define PCI_PWR_DATA_TYPE(x) (((x) >> 15) & 7) /* Type */ > #define PCI_PWR_DATA_RAIL(x) (((x) >> 18) & 7) /* Power Rail */ > -#define PCI_PWR_CAP 12 /* Capability */ > +#define PCI_PWR_CAP 0x0c /* Capability */ > #define PCI_PWR_CAP_BUDGET(x) ((x) & 1) /* Included in system budget */ > -#define PCI_EXT_CAP_PWR_SIZEOF 16 > +#define PCI_EXT_CAP_PWR_SIZEOF 0x10 > > /* Root Complex Event Collector Endpoint Association */ > #define PCI_RCEC_RCIEP_BITMAP 4 /* Associated Bitmap for RCiEPs */ > @@ -964,7 +964,7 @@ > #define PCI_SRIOV_VFM_MI 0x1 /* Dormant.MigrateIn */ > #define PCI_SRIOV_VFM_MO 0x2 /* Active.MigrateOut */ > #define PCI_SRIOV_VFM_AV 0x3 /* Active.Available */ > -#define PCI_EXT_CAP_SRIOV_SIZEOF 64 > +#define PCI_EXT_CAP_SRIOV_SIZEOF 0x40 > > #define PCI_LTR_MAX_SNOOP_LAT 0x4 > #define PCI_LTR_MAX_NOSNOOP_LAT 0x6 > @@ -1017,12 +1017,12 @@ > #define PCI_TPH_LOC_NONE 0x000 /* no location */ > #define PCI_TPH_LOC_CAP 0x200 /* in capability */ > #define PCI_TPH_LOC_MSIX 0x400 /* in MSI-X */ > -#define PCI_TPH_CAP_ST_MASK 0x07FF0000 /* st table mask */ > -#define PCI_TPH_CAP_ST_SHIFT 16 /* st table shift */ > -#define PCI_TPH_BASE_SIZEOF 12 /* size with no st table */ > +#define PCI_TPH_CAP_ST_MASK 0x07FF0000 /* ST table mask */ > +#define PCI_TPH_CAP_ST_SHIFT 16 /* ST table shift */ > +#define PCI_TPH_BASE_SIZEOF 0xc /* size with no ST table */ > > /* Downstream Port Containment */ > -#define PCI_EXP_DPC_CAP 4 /* DPC Capability */ > +#define PCI_EXP_DPC_CAP 0x04 /* DPC Capability */ > #define PCI_EXP_DPC_IRQ 0x001F /* Interrupt Message Number */ > #define PCI_EXP_DPC_CAP_RP_EXT 0x0020 /* Root Port Extensions */ > #define PCI_EXP_DPC_CAP_POISONED_TLP 0x0040 /* Poisoned TLP Egress Blocking Supported */ > @@ -1030,19 +1030,19 @@ > #define PCI_EXP_DPC_RP_PIO_LOG_SIZE 0x0F00 /* RP PIO Log Size */ > #define PCI_EXP_DPC_CAP_DL_ACTIVE 0x1000 /* ERR_COR signal on DL_Active supported */ > > -#define PCI_EXP_DPC_CTL 6 /* DPC control */ > +#define PCI_EXP_DPC_CTL 0x06 /* DPC control */ > #define PCI_EXP_DPC_CTL_EN_FATAL 0x0001 /* Enable trigger on ERR_FATAL message */ > #define PCI_EXP_DPC_CTL_EN_NONFATAL 0x0002 /* Enable trigger on ERR_NONFATAL message */ > #define PCI_EXP_DPC_CTL_INT_EN 0x0008 /* DPC Interrupt Enable */ > > -#define PCI_EXP_DPC_STATUS 8 /* DPC Status */ > +#define PCI_EXP_DPC_STATUS 0x08 /* DPC Status */ > #define PCI_EXP_DPC_STATUS_TRIGGER 0x0001 /* Trigger Status */ > #define PCI_EXP_DPC_STATUS_TRIGGER_RSN 0x0006 /* Trigger Reason */ > #define PCI_EXP_DPC_STATUS_INTERRUPT 0x0008 /* Interrupt Status */ > #define PCI_EXP_DPC_RP_BUSY 0x0010 /* Root Port Busy */ > #define PCI_EXP_DPC_STATUS_TRIGGER_RSN_EXT 0x0060 /* Trig Reason Extension */ > > -#define PCI_EXP_DPC_SOURCE_ID 10 /* DPC Source Identifier */ > +#define PCI_EXP_DPC_SOURCE_ID 0x0A /* DPC Source Identifier */ > > #define PCI_EXP_DPC_RP_PIO_STATUS 0x0C /* RP PIO Status */ > #define PCI_EXP_DPC_RP_PIO_MASK 0x10 /* RP PIO Mask */ > @@ -1086,7 +1086,11 @@ > > /* Designated Vendor-Specific (DVSEC, PCI_EXT_CAP_ID_DVSEC) */ > #define PCI_DVSEC_HEADER1 0x4 /* Designated Vendor-Specific Header1 */ > +#define PCI_DVSEC_HEADER1_VID(x) ((x) & 0xffff) > +#define PCI_DVSEC_HEADER1_REV(x) (((x) >> 16) & 0xf) > +#define PCI_DVSEC_HEADER1_LEN(x) (((x) >> 20) & 0xfff) > #define PCI_DVSEC_HEADER2 0x8 /* Designated Vendor-Specific Header2 */ > +#define PCI_DVSEC_HEADER2_ID(x) ((x) & 0xffff) > > /* Data Link Feature */ > #define PCI_DLF_CAP 0x04 /* Capabilities Register */ > diff --git a/include/standard-headers/linux/virtio_gpio.h b/include/standard-headers/linux/virtio_gpio.h > new file mode 100644 > index 0000000000..2b5cf06349 > --- /dev/null > +++ b/include/standard-headers/linux/virtio_gpio.h > @@ -0,0 +1,72 @@ > +/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ > + > +#ifndef _LINUX_VIRTIO_GPIO_H > +#define _LINUX_VIRTIO_GPIO_H > + > +#include "standard-headers/linux/types.h" > + > +/* Virtio GPIO Feature bits */ > +#define VIRTIO_GPIO_F_IRQ 0 > + > +/* Virtio GPIO request types */ > +#define VIRTIO_GPIO_MSG_GET_NAMES 0x0001 > +#define VIRTIO_GPIO_MSG_GET_DIRECTION 0x0002 > +#define VIRTIO_GPIO_MSG_SET_DIRECTION 0x0003 > +#define VIRTIO_GPIO_MSG_GET_VALUE 0x0004 > +#define VIRTIO_GPIO_MSG_SET_VALUE 0x0005 > +#define VIRTIO_GPIO_MSG_IRQ_TYPE 0x0006 > + > +/* Possible values of the status field */ > +#define VIRTIO_GPIO_STATUS_OK 0x0 > +#define VIRTIO_GPIO_STATUS_ERR 0x1 > + > +/* Direction types */ > +#define VIRTIO_GPIO_DIRECTION_NONE 0x00 > +#define VIRTIO_GPIO_DIRECTION_OUT 0x01 > +#define VIRTIO_GPIO_DIRECTION_IN 0x02 > + > +/* Virtio GPIO IRQ types */ > +#define VIRTIO_GPIO_IRQ_TYPE_NONE 0x00 > +#define VIRTIO_GPIO_IRQ_TYPE_EDGE_RISING 0x01 > +#define VIRTIO_GPIO_IRQ_TYPE_EDGE_FALLING 0x02 > +#define VIRTIO_GPIO_IRQ_TYPE_EDGE_BOTH 0x03 > +#define VIRTIO_GPIO_IRQ_TYPE_LEVEL_HIGH 0x04 > +#define VIRTIO_GPIO_IRQ_TYPE_LEVEL_LOW 0x08 > + > +struct virtio_gpio_config { > + uint16_t ngpio; > + uint8_t padding[2]; > + uint32_t gpio_names_size; > +}; > + > +/* Virtio GPIO Request / Response */ > +struct virtio_gpio_request { > + uint16_t type; > + uint16_t gpio; > + uint32_t value; > +}; > + > +struct virtio_gpio_response { > + uint8_t status; > + uint8_t value; > +}; > + > +struct virtio_gpio_response_get_names { > + uint8_t status; > + uint8_t value[]; > +}; > + > +/* Virtio GPIO IRQ Request / Response */ > +struct virtio_gpio_irq_request { > + uint16_t gpio; > +}; > + > +struct virtio_gpio_irq_response { > + uint8_t status; > +}; > + > +/* Possible values of the interrupt status field */ > +#define VIRTIO_GPIO_IRQ_STATUS_INVALID 0x0 > +#define VIRTIO_GPIO_IRQ_STATUS_VALID 0x1 > + > +#endif /* _LINUX_VIRTIO_GPIO_H */ > diff --git a/include/standard-headers/linux/virtio_i2c.h b/include/standard-headers/linux/virtio_i2c.h > new file mode 100644 > index 0000000000..09fa907793 > --- /dev/null > +++ b/include/standard-headers/linux/virtio_i2c.h > @@ -0,0 +1,47 @@ > +/* SPDX-License-Identifier: GPL-2.0-or-later WITH Linux-syscall-note */ > +/* > + * Definitions for virtio I2C Adpter > + * > + * Copyright (c) 2021 Intel Corporation. All rights reserved. > + */ > + > +#ifndef _LINUX_VIRTIO_I2C_H > +#define _LINUX_VIRTIO_I2C_H > + > +#include "standard-headers/linux/const.h" > +#include "standard-headers/linux/types.h" > + > +/* Virtio I2C Feature bits */ > +#define VIRTIO_I2C_F_ZERO_LENGTH_REQUEST 0 > + > +/* The bit 0 of the @virtio_i2c_out_hdr.@flags, used to group the requests */ > +#define VIRTIO_I2C_FLAGS_FAIL_NEXT _BITUL(0) > + > +/* The bit 1 of the @virtio_i2c_out_hdr.@flags, used to mark a buffer as read */ > +#define VIRTIO_I2C_FLAGS_M_RD _BITUL(1) > + > +/** > + * struct virtio_i2c_out_hdr - the virtio I2C message OUT header > + * @addr: the controlled device address > + * @padding: used to pad to full dword > + * @flags: used for feature extensibility > + */ > +struct virtio_i2c_out_hdr { > + uint16_t addr; > + uint16_t padding; > + uint32_t flags; > +}; > + > +/** > + * struct virtio_i2c_in_hdr - the virtio I2C message IN header > + * @status: the processing result from the backend > + */ > +struct virtio_i2c_in_hdr { > + uint8_t status; > +}; > + > +/* The final status written by the device */ > +#define VIRTIO_I2C_MSG_OK 0 > +#define VIRTIO_I2C_MSG_ERR 1 > + > +#endif /* _LINUX_VIRTIO_I2C_H */ > diff --git a/include/standard-headers/linux/virtio_iommu.h b/include/standard-headers/linux/virtio_iommu.h > index b9443b83a1..366379c2f0 100644 > --- a/include/standard-headers/linux/virtio_iommu.h > +++ b/include/standard-headers/linux/virtio_iommu.h > @@ -16,6 +16,7 @@ > #define VIRTIO_IOMMU_F_BYPASS 3 > #define VIRTIO_IOMMU_F_PROBE 4 > #define VIRTIO_IOMMU_F_MMIO 5 > +#define VIRTIO_IOMMU_F_BYPASS_CONFIG 6 > > struct virtio_iommu_range_64 { > uint64_t start; > @@ -36,6 +37,8 @@ struct virtio_iommu_config { > struct virtio_iommu_range_32 domain_range; > /* Probe buffer size */ > uint32_t probe_size; > + uint8_t bypass; > + uint8_t reserved[3]; > }; > > /* Request types */ > @@ -66,11 +69,14 @@ struct virtio_iommu_req_tail { > uint8_t reserved[3]; > }; > > +#define VIRTIO_IOMMU_ATTACH_F_BYPASS (1 << 0) > + > struct virtio_iommu_req_attach { > struct virtio_iommu_req_head head; > uint32_t domain; > uint32_t endpoint; > - uint8_t reserved[8]; > + uint32_t flags; > + uint8_t reserved[4]; > struct virtio_iommu_req_tail tail; > }; > > diff --git a/include/standard-headers/linux/virtio_pcidev.h b/include/standard-headers/linux/virtio_pcidev.h > new file mode 100644 > index 0000000000..bdf1d062da > --- /dev/null > +++ b/include/standard-headers/linux/virtio_pcidev.h > @@ -0,0 +1,65 @@ > +/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */ > +/* > + * Copyright (C) 2021 Intel Corporation > + * Author: Johannes Berg <johannes@sipsolutions.net> > + */ > +#ifndef _LINUX_VIRTIO_PCIDEV_H > +#define _LINUX_VIRTIO_PCIDEV_H > +#include "standard-headers/linux/types.h" > + > +/** > + * enum virtio_pcidev_ops - virtual PCI device operations > + * @VIRTIO_PCIDEV_OP_RESERVED: reserved to catch errors > + * @VIRTIO_PCIDEV_OP_CFG_READ: read config space, size is 1, 2, 4 or 8; > + * the @data field should be filled in by the device (in little endian). > + * @VIRTIO_PCIDEV_OP_CFG_WRITE: write config space, size is 1, 2, 4 or 8; > + * the @data field contains the data to write (in little endian). > + * @VIRTIO_PCIDEV_OP_MMIO_READ: read BAR mem/pio, size can be variable; > + * the @data field should be filled in by the device (in little endian). > + * @VIRTIO_PCIDEV_OP_MMIO_WRITE: write BAR mem/pio, size can be variable; > + * the @data field contains the data to write (in little endian). > + * @VIRTIO_PCIDEV_OP_MMIO_MEMSET: memset MMIO, size is variable but > + * the @data field only has one byte (unlike @VIRTIO_PCIDEV_OP_MMIO_WRITE) > + * @VIRTIO_PCIDEV_OP_INT: legacy INTx# pin interrupt, the addr field is 1-4 for > + * the number > + * @VIRTIO_PCIDEV_OP_MSI: MSI(-X) interrupt, this message basically transports > + * the 16- or 32-bit write that would otherwise be done into memory, > + * analogous to the write messages (@VIRTIO_PCIDEV_OP_MMIO_WRITE) above > + * @VIRTIO_PCIDEV_OP_PME: Dummy message whose content is ignored (and should be > + * all zeroes) to signal the PME# pin. > + */ > +enum virtio_pcidev_ops { > + VIRTIO_PCIDEV_OP_RESERVED = 0, > + VIRTIO_PCIDEV_OP_CFG_READ, > + VIRTIO_PCIDEV_OP_CFG_WRITE, > + VIRTIO_PCIDEV_OP_MMIO_READ, > + VIRTIO_PCIDEV_OP_MMIO_WRITE, > + VIRTIO_PCIDEV_OP_MMIO_MEMSET, > + VIRTIO_PCIDEV_OP_INT, > + VIRTIO_PCIDEV_OP_MSI, > + VIRTIO_PCIDEV_OP_PME, > +}; > + > +/** > + * struct virtio_pcidev_msg - virtio PCI device operation > + * @op: the operation to do > + * @bar: the bar (only with BAR read/write messages) > + * @reserved: reserved > + * @size: the size of the read/write (in bytes) > + * @addr: the address to read/write > + * @data: the data, normally @size long, but just one byte for > + * %VIRTIO_PCIDEV_OP_MMIO_MEMSET > + * > + * Note: the fields are all in native (CPU) endian, however, the > + * @data values will often be in little endian (see the ops above.) > + */ > +struct virtio_pcidev_msg { > + uint8_t op; > + uint8_t bar; > + uint16_t reserved; > + uint32_t size; > + uint64_t addr; > + uint8_t data[]; > +}; > + > +#endif /* _LINUX_VIRTIO_PCIDEV_H */ > diff --git a/include/standard-headers/linux/virtio_scmi.h b/include/standard-headers/linux/virtio_scmi.h > new file mode 100644 > index 0000000000..8f2c305aea > --- /dev/null > +++ b/include/standard-headers/linux/virtio_scmi.h > @@ -0,0 +1,24 @@ > +/* SPDX-License-Identifier: ((GPL-2.0 WITH Linux-syscall-note) OR BSD-3-Clause) */ > +/* > + * Copyright (C) 2020-2021 OpenSynergy GmbH > + * Copyright (C) 2021 ARM Ltd. > + */ > + > +#ifndef _LINUX_VIRTIO_SCMI_H > +#define _LINUX_VIRTIO_SCMI_H > + > +#include "standard-headers/linux/virtio_types.h" > + > +/* Device implements some SCMI notifications, or delayed responses. */ > +#define VIRTIO_SCMI_F_P2A_CHANNELS 0 > + > +/* Device implements any SCMI statistics shared memory region */ > +#define VIRTIO_SCMI_F_SHARED_MEMORY 1 > + > +/* Virtqueues */ > + > +#define VIRTIO_SCMI_VQ_TX 0 /* cmdq */ > +#define VIRTIO_SCMI_VQ_RX 1 /* eventq */ > +#define VIRTIO_SCMI_VQ_MAX_CNT 2 > + > +#endif /* _LINUX_VIRTIO_SCMI_H */ > diff --git a/linux-headers/asm-generic/unistd.h b/linux-headers/asm-generic/unistd.h > index 4557a8b608..1c48b0ae3b 100644 > --- a/linux-headers/asm-generic/unistd.h > +++ b/linux-headers/asm-generic/unistd.h > @@ -883,8 +883,11 @@ __SYSCALL(__NR_process_mrelease, sys_process_mrelease) > #define __NR_futex_waitv 449 > __SYSCALL(__NR_futex_waitv, sys_futex_waitv) > > +#define __NR_set_mempolicy_home_node 450 > +__SYSCALL(__NR_set_mempolicy_home_node, sys_set_mempolicy_home_node) > + > #undef __NR_syscalls > -#define __NR_syscalls 450 > +#define __NR_syscalls 451 > > /* > * 32 bit systems traditionally used different > diff --git a/linux-headers/asm-mips/unistd_n32.h b/linux-headers/asm-mips/unistd_n32.h > index 4b3e7ad1ec..1f14a6fad3 100644 > --- a/linux-headers/asm-mips/unistd_n32.h > +++ b/linux-headers/asm-mips/unistd_n32.h > @@ -377,5 +377,7 @@ > #define __NR_landlock_add_rule (__NR_Linux + 445) > #define __NR_landlock_restrict_self (__NR_Linux + 446) > #define __NR_process_mrelease (__NR_Linux + 448) > +#define __NR_futex_waitv (__NR_Linux + 449) > +#define __NR_set_mempolicy_home_node (__NR_Linux + 450) > > #endif /* _ASM_UNISTD_N32_H */ > diff --git a/linux-headers/asm-mips/unistd_n64.h b/linux-headers/asm-mips/unistd_n64.h > index 488d9298d9..e5a8ebec78 100644 > --- a/linux-headers/asm-mips/unistd_n64.h > +++ b/linux-headers/asm-mips/unistd_n64.h > @@ -353,5 +353,7 @@ > #define __NR_landlock_add_rule (__NR_Linux + 445) > #define __NR_landlock_restrict_self (__NR_Linux + 446) > #define __NR_process_mrelease (__NR_Linux + 448) > +#define __NR_futex_waitv (__NR_Linux + 449) > +#define __NR_set_mempolicy_home_node (__NR_Linux + 450) > > #endif /* _ASM_UNISTD_N64_H */ > diff --git a/linux-headers/asm-mips/unistd_o32.h b/linux-headers/asm-mips/unistd_o32.h > index f47399870a..871d57168f 100644 > --- a/linux-headers/asm-mips/unistd_o32.h > +++ b/linux-headers/asm-mips/unistd_o32.h > @@ -423,5 +423,7 @@ > #define __NR_landlock_add_rule (__NR_Linux + 445) > #define __NR_landlock_restrict_self (__NR_Linux + 446) > #define __NR_process_mrelease (__NR_Linux + 448) > +#define __NR_futex_waitv (__NR_Linux + 449) > +#define __NR_set_mempolicy_home_node (__NR_Linux + 450) > > #endif /* _ASM_UNISTD_O32_H */ > diff --git a/linux-headers/asm-powerpc/unistd_32.h b/linux-headers/asm-powerpc/unistd_32.h > index 11d54696dc..585c7fefbc 100644 > --- a/linux-headers/asm-powerpc/unistd_32.h > +++ b/linux-headers/asm-powerpc/unistd_32.h > @@ -430,6 +430,8 @@ > #define __NR_landlock_add_rule 445 > #define __NR_landlock_restrict_self 446 > #define __NR_process_mrelease 448 > +#define __NR_futex_waitv 449 > +#define __NR_set_mempolicy_home_node 450 > > > #endif /* _ASM_UNISTD_32_H */ > diff --git a/linux-headers/asm-powerpc/unistd_64.h b/linux-headers/asm-powerpc/unistd_64.h > index cf740bab13..350f7ec0ac 100644 > --- a/linux-headers/asm-powerpc/unistd_64.h > +++ b/linux-headers/asm-powerpc/unistd_64.h > @@ -402,6 +402,8 @@ > #define __NR_landlock_add_rule 445 > #define __NR_landlock_restrict_self 446 > #define __NR_process_mrelease 448 > +#define __NR_futex_waitv 449 > +#define __NR_set_mempolicy_home_node 450 > > > #endif /* _ASM_UNISTD_64_H */ > diff --git a/linux-headers/asm-riscv/bitsperlong.h b/linux-headers/asm-riscv/bitsperlong.h > new file mode 100644 > index 0000000000..cc5c45a9ce > --- /dev/null > +++ b/linux-headers/asm-riscv/bitsperlong.h > @@ -0,0 +1,14 @@ > +/* SPDX-License-Identifier: GPL-2.0-only WITH Linux-syscall-note */ > +/* > + * Copyright (C) 2012 ARM Ltd. > + * Copyright (C) 2015 Regents of the University of California > + */ > + > +#ifndef _ASM_RISCV_BITSPERLONG_H > +#define _ASM_RISCV_BITSPERLONG_H > + > +#define __BITS_PER_LONG (__SIZEOF_POINTER__ * 8) > + > +#include <asm-generic/bitsperlong.h> > + > +#endif /* _ASM_RISCV_BITSPERLONG_H */ > diff --git a/linux-headers/asm-riscv/mman.h b/linux-headers/asm-riscv/mman.h > new file mode 100644 > index 0000000000..8eebf89f5a > --- /dev/null > +++ b/linux-headers/asm-riscv/mman.h > @@ -0,0 +1 @@ > +#include <asm-generic/mman.h> > diff --git a/linux-headers/asm-riscv/unistd.h b/linux-headers/asm-riscv/unistd.h > new file mode 100644 > index 0000000000..8062996c2d > --- /dev/null > +++ b/linux-headers/asm-riscv/unistd.h > @@ -0,0 +1,44 @@ > +/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */ > +/* > + * Copyright (C) 2018 David Abdurachmanov <david.abdurachmanov@gmail.com> > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 as > + * published by the Free Software Foundation. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program. If not, see <https://www.gnu.org/licenses/>. > + */ > + > +#ifdef __LP64__ > +#define __ARCH_WANT_NEW_STAT > +#define __ARCH_WANT_SET_GET_RLIMIT > +#endif /* __LP64__ */ > + > +#define __ARCH_WANT_SYS_CLONE3 > + > +#include <asm-generic/unistd.h> > + > +/* > + * Allows the instruction cache to be flushed from userspace. Despite RISC-V > + * having a direct 'fence.i' instruction available to userspace (which we > + * can't trap!), that's not actually viable when running on Linux because the > + * kernel might schedule a process on another hart. There is no way for > + * userspace to handle this without invoking the kernel (as it doesn't know the > + * thread->hart mappings), so we've defined a RISC-V specific system call to > + * flush the instruction cache. > + * > + * __NR_riscv_flush_icache is defined to flush the instruction cache over an > + * address range, with the flush applying to either all threads or just the > + * caller. We don't currently do anything with the address range, that's just > + * in there for forwards compatibility. > + */ > +#ifndef __NR_riscv_flush_icache > +#define __NR_riscv_flush_icache (__NR_arch_specific_syscall + 15) > +#endif > +__SYSCALL(__NR_riscv_flush_icache, sys_riscv_flush_icache) > diff --git a/linux-headers/asm-s390/unistd_32.h b/linux-headers/asm-s390/unistd_32.h > index 8f97d98128..8e644d65f5 100644 > --- a/linux-headers/asm-s390/unistd_32.h > +++ b/linux-headers/asm-s390/unistd_32.h > @@ -420,5 +420,7 @@ > #define __NR_landlock_add_rule 445 > #define __NR_landlock_restrict_self 446 > #define __NR_process_mrelease 448 > +#define __NR_futex_waitv 449 > +#define __NR_set_mempolicy_home_node 450 > > #endif /* _ASM_S390_UNISTD_32_H */ > diff --git a/linux-headers/asm-s390/unistd_64.h b/linux-headers/asm-s390/unistd_64.h > index 021ffc30e6..51da542fec 100644 > --- a/linux-headers/asm-s390/unistd_64.h > +++ b/linux-headers/asm-s390/unistd_64.h > @@ -368,5 +368,7 @@ > #define __NR_landlock_add_rule 445 > #define __NR_landlock_restrict_self 446 > #define __NR_process_mrelease 448 > +#define __NR_futex_waitv 449 > +#define __NR_set_mempolicy_home_node 450 > > #endif /* _ASM_S390_UNISTD_64_H */ > diff --git a/linux-headers/asm-x86/kvm.h b/linux-headers/asm-x86/kvm.h > index 5a776a08f7..2da3316bb5 100644 > --- a/linux-headers/asm-x86/kvm.h > +++ b/linux-headers/asm-x86/kvm.h > @@ -373,9 +373,23 @@ struct kvm_debugregs { > __u64 reserved[9]; > }; > > -/* for KVM_CAP_XSAVE */ > +/* for KVM_CAP_XSAVE and KVM_CAP_XSAVE2 */ > struct kvm_xsave { > + /* > + * KVM_GET_XSAVE2 and KVM_SET_XSAVE write and read as many bytes > + * as are returned by KVM_CHECK_EXTENSION(KVM_CAP_XSAVE2) > + * respectively, when invoked on the vm file descriptor. > + * > + * The size value returned by KVM_CHECK_EXTENSION(KVM_CAP_XSAVE2) > + * will always be at least 4096. Currently, it is only greater > + * than 4096 if a dynamic feature has been enabled with > + * ``arch_prctl()``, but this may change in the future. > + * > + * The offsets of the state save areas in struct kvm_xsave follow > + * the contents of CPUID leaf 0xD on the host. > + */ > __u32 region[1024]; > + __u32 extra[0]; > }; > > #define KVM_MAX_XCRS 16 > diff --git a/linux-headers/asm-x86/unistd_32.h b/linux-headers/asm-x86/unistd_32.h > index 9c9ffe312b..87e1e977af 100644 > --- a/linux-headers/asm-x86/unistd_32.h > +++ b/linux-headers/asm-x86/unistd_32.h > @@ -440,6 +440,7 @@ > #define __NR_memfd_secret 447 > #define __NR_process_mrelease 448 > #define __NR_futex_waitv 449 > +#define __NR_set_mempolicy_home_node 450 > > > #endif /* _ASM_UNISTD_32_H */ > diff --git a/linux-headers/asm-x86/unistd_64.h b/linux-headers/asm-x86/unistd_64.h > index 084f1eef9c..147a78d623 100644 > --- a/linux-headers/asm-x86/unistd_64.h > +++ b/linux-headers/asm-x86/unistd_64.h > @@ -362,6 +362,7 @@ > #define __NR_memfd_secret 447 > #define __NR_process_mrelease 448 > #define __NR_futex_waitv 449 > +#define __NR_set_mempolicy_home_node 450 > > > #endif /* _ASM_UNISTD_64_H */ > diff --git a/linux-headers/asm-x86/unistd_x32.h b/linux-headers/asm-x86/unistd_x32.h > index a2441affc2..27098db7fb 100644 > --- a/linux-headers/asm-x86/unistd_x32.h > +++ b/linux-headers/asm-x86/unistd_x32.h > @@ -315,6 +315,7 @@ > #define __NR_memfd_secret (__X32_SYSCALL_BIT + 447) > #define __NR_process_mrelease (__X32_SYSCALL_BIT + 448) > #define __NR_futex_waitv (__X32_SYSCALL_BIT + 449) > +#define __NR_set_mempolicy_home_node (__X32_SYSCALL_BIT + 450) > #define __NR_rt_sigaction (__X32_SYSCALL_BIT + 512) > #define __NR_rt_sigreturn (__X32_SYSCALL_BIT + 513) > #define __NR_ioctl (__X32_SYSCALL_BIT + 514) > diff --git a/linux-headers/linux/kvm.h b/linux-headers/linux/kvm.h > index 02c5e7b7bb..00af3bc333 100644 > --- a/linux-headers/linux/kvm.h > +++ b/linux-headers/linux/kvm.h > @@ -1130,6 +1130,9 @@ struct kvm_ppc_resize_hpt { > #define KVM_CAP_BINARY_STATS_FD 203 > #define KVM_CAP_EXIT_ON_EMULATION_FAILURE 204 > #define KVM_CAP_ARM_MTE 205 > +#define KVM_CAP_VM_MOVE_ENC_CONTEXT_FROM 206 > +#define KVM_CAP_VM_GPA_BITS 207 > +#define KVM_CAP_XSAVE2 208 > > #ifdef KVM_CAP_IRQ_ROUTING > > @@ -1161,11 +1164,20 @@ struct kvm_irq_routing_hv_sint { > __u32 sint; > }; > > +struct kvm_irq_routing_xen_evtchn { > + __u32 port; > + __u32 vcpu; > + __u32 priority; > +}; > + > +#define KVM_IRQ_ROUTING_XEN_EVTCHN_PRIO_2LEVEL ((__u32)(-1)) > + > /* gsi routing entry types */ > #define KVM_IRQ_ROUTING_IRQCHIP 1 > #define KVM_IRQ_ROUTING_MSI 2 > #define KVM_IRQ_ROUTING_S390_ADAPTER 3 > #define KVM_IRQ_ROUTING_HV_SINT 4 > +#define KVM_IRQ_ROUTING_XEN_EVTCHN 5 > > struct kvm_irq_routing_entry { > __u32 gsi; > @@ -1177,6 +1189,7 @@ struct kvm_irq_routing_entry { > struct kvm_irq_routing_msi msi; > struct kvm_irq_routing_s390_adapter adapter; > struct kvm_irq_routing_hv_sint hv_sint; > + struct kvm_irq_routing_xen_evtchn xen_evtchn; > __u32 pad[8]; > } u; > }; > @@ -1207,6 +1220,7 @@ struct kvm_x86_mce { > #define KVM_XEN_HVM_CONFIG_INTERCEPT_HCALL (1 << 1) > #define KVM_XEN_HVM_CONFIG_SHARED_INFO (1 << 2) > #define KVM_XEN_HVM_CONFIG_RUNSTATE (1 << 3) > +#define KVM_XEN_HVM_CONFIG_EVTCHN_2LEVEL (1 << 4) > > struct kvm_xen_hvm_config { > __u32 flags; > @@ -1609,6 +1623,9 @@ struct kvm_enc_region { > #define KVM_S390_NORMAL_RESET _IO(KVMIO, 0xc3) > #define KVM_S390_CLEAR_RESET _IO(KVMIO, 0xc4) > > +/* Available with KVM_CAP_XSAVE2 */ > +#define KVM_GET_XSAVE2 _IOR(KVMIO, 0xcf, struct kvm_xsave) > + > struct kvm_s390_pv_sec_parm { > __u64 origin; > __u64 length; ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 2/4] virtio-iommu: Default to bypass during boot 2022-01-27 14:29 [PATCH v2 0/4] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG Jean-Philippe Brucker 2022-01-27 14:29 ` [PATCH v2 1/4] linux-headers: update to v5.17-rc1 Jean-Philippe Brucker @ 2022-01-27 14:29 ` Jean-Philippe Brucker 2022-01-31 9:14 ` Eric Auger 2022-02-08 17:42 ` Cornelia Huck 2022-01-27 14:29 ` [PATCH v2 3/4] virtio-iommu: Support bypass domain Jean-Philippe Brucker 2022-01-27 14:29 ` [PATCH v2 4/4] tests/qtest/virtio-iommu-test: Check bypass config Jean-Philippe Brucker 3 siblings, 2 replies; 20+ messages in thread From: Jean-Philippe Brucker @ 2022-01-27 14:29 UTC (permalink / raw) To: eric.auger Cc: lvivier, Jean-Philippe Brucker, thuth, mst, cohuck, qemu-devel, pasic, pbonzini Currently the virtio-iommu device must be programmed before it allows DMA from any PCI device. This can make the VM entirely unusable when a virtio-iommu driver isn't present, for example in a bootloader that loads the OS from storage. Similarly to the other vIOMMU implementations, default to DMA bypassing the IOMMU during boot. Add a "boot-bypass" property, defaulting to true, that lets users change this behavior. Replace the VIRTIO_IOMMU_F_BYPASS feature, which didn't support bypass before feature negotiation, with VIRTIO_IOMMU_F_BYPASS_CONFIG. Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> --- include/hw/virtio/virtio-iommu.h | 1 + hw/virtio/virtio-iommu.c | 51 +++++++++++++++++++++++++++++--- hw/virtio/trace-events | 4 ++- 3 files changed, 51 insertions(+), 5 deletions(-) diff --git a/include/hw/virtio/virtio-iommu.h b/include/hw/virtio/virtio-iommu.h index e2339e5b72..84391f8448 100644 --- a/include/hw/virtio/virtio-iommu.h +++ b/include/hw/virtio/virtio-iommu.h @@ -58,6 +58,7 @@ struct VirtIOIOMMU { GTree *domains; QemuMutex mutex; GTree *endpoints; + bool boot_bypass; }; #endif diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c index aa9c16a17b..ec02029bb6 100644 --- a/hw/virtio/virtio-iommu.c +++ b/hw/virtio/virtio-iommu.c @@ -24,6 +24,7 @@ #include "hw/qdev-properties.h" #include "hw/virtio/virtio.h" #include "sysemu/kvm.h" +#include "sysemu/reset.h" #include "qapi/error.h" #include "qemu/error-report.h" #include "trace.h" @@ -728,8 +729,7 @@ static IOMMUTLBEntry virtio_iommu_translate(IOMMUMemoryRegion *mr, hwaddr addr, .perm = IOMMU_NONE, }; - bypass_allowed = virtio_vdev_has_feature(&s->parent_obj, - VIRTIO_IOMMU_F_BYPASS); + bypass_allowed = s->config.bypass; sid = virtio_iommu_get_bdf(sdev); @@ -831,13 +831,37 @@ static void virtio_iommu_get_config(VirtIODevice *vdev, uint8_t *config_data) out_config->domain_range.start = cpu_to_le32(dev_config->domain_range.start); out_config->domain_range.end = cpu_to_le32(dev_config->domain_range.end); out_config->probe_size = cpu_to_le32(dev_config->probe_size); + out_config->bypass = dev_config->bypass; trace_virtio_iommu_get_config(dev_config->page_size_mask, dev_config->input_range.start, dev_config->input_range.end, dev_config->domain_range.start, dev_config->domain_range.end, - dev_config->probe_size); + dev_config->probe_size, + dev_config->bypass); +} + +static void virtio_iommu_set_config(VirtIODevice *vdev, + const uint8_t *config_data) +{ + VirtIOIOMMU *dev = VIRTIO_IOMMU(vdev); + struct virtio_iommu_config *dev_config = &dev->config; + const struct virtio_iommu_config *in_config = (void *)config_data; + + if (in_config->bypass != dev_config->bypass) { + if (!virtio_vdev_has_feature(vdev, VIRTIO_IOMMU_F_BYPASS_CONFIG)) { + virtio_error(vdev, "cannot set config.bypass"); + return; + } else if (in_config->bypass != 0 && in_config->bypass != 1) { + virtio_error(vdev, "invalid config.bypass value '%u'", + in_config->bypass); + return; + } + dev_config->bypass = in_config->bypass; + } + + trace_virtio_iommu_set_config(in_config->bypass); } static uint64_t virtio_iommu_get_features(VirtIODevice *vdev, uint64_t f, @@ -963,6 +987,19 @@ static int virtio_iommu_set_page_size_mask(IOMMUMemoryRegion *mr, return 0; } +static void virtio_iommu_system_reset(void *opaque) +{ + VirtIOIOMMU *s = opaque; + + trace_virtio_iommu_system_reset(); + + /* + * config.bypass is sticky across device reset, but should be restored on + * system reset + */ + s->config.bypass = s->boot_bypass; +} + static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) { VirtIODevice *vdev = VIRTIO_DEVICE(dev); @@ -988,9 +1025,9 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) virtio_add_feature(&s->features, VIRTIO_IOMMU_F_INPUT_RANGE); virtio_add_feature(&s->features, VIRTIO_IOMMU_F_DOMAIN_RANGE); virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MAP_UNMAP); - virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS); virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MMIO); virtio_add_feature(&s->features, VIRTIO_IOMMU_F_PROBE); + virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS_CONFIG); qemu_mutex_init(&s->mutex); @@ -1001,6 +1038,8 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) } else { error_setg(errp, "VIRTIO-IOMMU is not attached to any PCI bus!"); } + + qemu_register_reset(virtio_iommu_system_reset, s); } static void virtio_iommu_device_unrealize(DeviceState *dev) @@ -1008,6 +1047,8 @@ static void virtio_iommu_device_unrealize(DeviceState *dev) VirtIODevice *vdev = VIRTIO_DEVICE(dev); VirtIOIOMMU *s = VIRTIO_IOMMU(dev); + qemu_unregister_reset(virtio_iommu_system_reset, s); + g_hash_table_destroy(s->as_by_busptr); if (s->domains) { g_tree_destroy(s->domains); @@ -1164,6 +1205,7 @@ static const VMStateDescription vmstate_virtio_iommu = { static Property virtio_iommu_properties[] = { DEFINE_PROP_LINK("primary-bus", VirtIOIOMMU, primary_bus, "PCI", PCIBus *), + DEFINE_PROP_BOOL("boot-bypass", VirtIOIOMMU, boot_bypass, true), DEFINE_PROP_END_OF_LIST(), }; @@ -1180,6 +1222,7 @@ static void virtio_iommu_class_init(ObjectClass *klass, void *data) vdc->unrealize = virtio_iommu_device_unrealize; vdc->reset = virtio_iommu_device_reset; vdc->get_config = virtio_iommu_get_config; + vdc->set_config = virtio_iommu_set_config; vdc->get_features = virtio_iommu_get_features; vdc->set_status = virtio_iommu_set_status; vdc->vmsd = &vmstate_virtio_iommu_device; diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events index f7ad6be5fb..a5102eac9e 100644 --- a/hw/virtio/trace-events +++ b/hw/virtio/trace-events @@ -89,9 +89,11 @@ virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d" # virtio-iommu.c virtio_iommu_device_reset(void) "reset!" +virtio_iommu_system_reset(void) "system reset!" virtio_iommu_get_features(uint64_t features) "device supports features=0x%"PRIx64 virtio_iommu_device_status(uint8_t status) "driver status = %d" -virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start, uint64_t end, uint32_t domain_start, uint32_t domain_end, uint32_t probe_size) "page_size_mask=0x%"PRIx64" input range start=0x%"PRIx64" input range end=0x%"PRIx64" domain range start=%d domain range end=%d probe_size=0x%x" +virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start, uint64_t end, uint32_t domain_start, uint32_t domain_end, uint32_t probe_size, uint8_t bypass) "page_size_mask=0x%"PRIx64" input range start=0x%"PRIx64" input range end=0x%"PRIx64" domain range start=%d domain range end=%d probe_size=0x%x bypass=0x%x" +virtio_iommu_set_config(uint8_t bypass) "bypass=0x%x" virtio_iommu_attach(uint32_t domain_id, uint32_t ep_id) "domain=%d endpoint=%d" virtio_iommu_detach(uint32_t domain_id, uint32_t ep_id) "domain=%d endpoint=%d" virtio_iommu_map(uint32_t domain_id, uint64_t virt_start, uint64_t virt_end, uint64_t phys_start, uint32_t flags) "domain=%d virt_start=0x%"PRIx64" virt_end=0x%"PRIx64 " phys_start=0x%"PRIx64" flags=%d" -- 2.34.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/4] virtio-iommu: Default to bypass during boot 2022-01-27 14:29 ` [PATCH v2 2/4] virtio-iommu: Default to bypass during boot Jean-Philippe Brucker @ 2022-01-31 9:14 ` Eric Auger 2022-02-08 17:42 ` Cornelia Huck 1 sibling, 0 replies; 20+ messages in thread From: Eric Auger @ 2022-01-31 9:14 UTC (permalink / raw) To: Jean-Philippe Brucker Cc: lvivier, thuth, mst, cohuck, qemu-devel, pasic, pbonzini On 1/27/22 3:29 PM, Jean-Philippe Brucker wrote: > Currently the virtio-iommu device must be programmed before it allows > DMA from any PCI device. This can make the VM entirely unusable when a > virtio-iommu driver isn't present, for example in a bootloader that > loads the OS from storage. > > Similarly to the other vIOMMU implementations, default to DMA bypassing > the IOMMU during boot. Add a "boot-bypass" property, defaulting to true, > that lets users change this behavior. > > Replace the VIRTIO_IOMMU_F_BYPASS feature, which didn't support bypass > before feature negotiation, with VIRTIO_IOMMU_F_BYPASS_CONFIG. > > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> Reviewed-by: Eric Auger <eric.auger@redhat.com> Eric > --- > include/hw/virtio/virtio-iommu.h | 1 + > hw/virtio/virtio-iommu.c | 51 +++++++++++++++++++++++++++++--- > hw/virtio/trace-events | 4 ++- > 3 files changed, 51 insertions(+), 5 deletions(-) > > diff --git a/include/hw/virtio/virtio-iommu.h b/include/hw/virtio/virtio-iommu.h > index e2339e5b72..84391f8448 100644 > --- a/include/hw/virtio/virtio-iommu.h > +++ b/include/hw/virtio/virtio-iommu.h > @@ -58,6 +58,7 @@ struct VirtIOIOMMU { > GTree *domains; > QemuMutex mutex; > GTree *endpoints; > + bool boot_bypass; > }; > > #endif > diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c > index aa9c16a17b..ec02029bb6 100644 > --- a/hw/virtio/virtio-iommu.c > +++ b/hw/virtio/virtio-iommu.c > @@ -24,6 +24,7 @@ > #include "hw/qdev-properties.h" > #include "hw/virtio/virtio.h" > #include "sysemu/kvm.h" > +#include "sysemu/reset.h" > #include "qapi/error.h" > #include "qemu/error-report.h" > #include "trace.h" > @@ -728,8 +729,7 @@ static IOMMUTLBEntry virtio_iommu_translate(IOMMUMemoryRegion *mr, hwaddr addr, > .perm = IOMMU_NONE, > }; > > - bypass_allowed = virtio_vdev_has_feature(&s->parent_obj, > - VIRTIO_IOMMU_F_BYPASS); > + bypass_allowed = s->config.bypass; > > sid = virtio_iommu_get_bdf(sdev); > > @@ -831,13 +831,37 @@ static void virtio_iommu_get_config(VirtIODevice *vdev, uint8_t *config_data) > out_config->domain_range.start = cpu_to_le32(dev_config->domain_range.start); > out_config->domain_range.end = cpu_to_le32(dev_config->domain_range.end); > out_config->probe_size = cpu_to_le32(dev_config->probe_size); > + out_config->bypass = dev_config->bypass; > > trace_virtio_iommu_get_config(dev_config->page_size_mask, > dev_config->input_range.start, > dev_config->input_range.end, > dev_config->domain_range.start, > dev_config->domain_range.end, > - dev_config->probe_size); > + dev_config->probe_size, > + dev_config->bypass); > +} > + > +static void virtio_iommu_set_config(VirtIODevice *vdev, > + const uint8_t *config_data) > +{ > + VirtIOIOMMU *dev = VIRTIO_IOMMU(vdev); > + struct virtio_iommu_config *dev_config = &dev->config; > + const struct virtio_iommu_config *in_config = (void *)config_data; > + > + if (in_config->bypass != dev_config->bypass) { > + if (!virtio_vdev_has_feature(vdev, VIRTIO_IOMMU_F_BYPASS_CONFIG)) { > + virtio_error(vdev, "cannot set config.bypass"); > + return; > + } else if (in_config->bypass != 0 && in_config->bypass != 1) { > + virtio_error(vdev, "invalid config.bypass value '%u'", > + in_config->bypass); > + return; > + } > + dev_config->bypass = in_config->bypass; > + } > + > + trace_virtio_iommu_set_config(in_config->bypass); > } > > static uint64_t virtio_iommu_get_features(VirtIODevice *vdev, uint64_t f, > @@ -963,6 +987,19 @@ static int virtio_iommu_set_page_size_mask(IOMMUMemoryRegion *mr, > return 0; > } > > +static void virtio_iommu_system_reset(void *opaque) > +{ > + VirtIOIOMMU *s = opaque; > + > + trace_virtio_iommu_system_reset(); > + > + /* > + * config.bypass is sticky across device reset, but should be restored on > + * system reset > + */ > + s->config.bypass = s->boot_bypass; > +} > + > static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) > { > VirtIODevice *vdev = VIRTIO_DEVICE(dev); > @@ -988,9 +1025,9 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_INPUT_RANGE); > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_DOMAIN_RANGE); > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MAP_UNMAP); > - virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS); > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MMIO); > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_PROBE); > + virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS_CONFIG); > > qemu_mutex_init(&s->mutex); > > @@ -1001,6 +1038,8 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) > } else { > error_setg(errp, "VIRTIO-IOMMU is not attached to any PCI bus!"); > } > + > + qemu_register_reset(virtio_iommu_system_reset, s); > } > > static void virtio_iommu_device_unrealize(DeviceState *dev) > @@ -1008,6 +1047,8 @@ static void virtio_iommu_device_unrealize(DeviceState *dev) > VirtIODevice *vdev = VIRTIO_DEVICE(dev); > VirtIOIOMMU *s = VIRTIO_IOMMU(dev); > > + qemu_unregister_reset(virtio_iommu_system_reset, s); > + > g_hash_table_destroy(s->as_by_busptr); > if (s->domains) { > g_tree_destroy(s->domains); > @@ -1164,6 +1205,7 @@ static const VMStateDescription vmstate_virtio_iommu = { > > static Property virtio_iommu_properties[] = { > DEFINE_PROP_LINK("primary-bus", VirtIOIOMMU, primary_bus, "PCI", PCIBus *), > + DEFINE_PROP_BOOL("boot-bypass", VirtIOIOMMU, boot_bypass, true), > DEFINE_PROP_END_OF_LIST(), > }; > > @@ -1180,6 +1222,7 @@ static void virtio_iommu_class_init(ObjectClass *klass, void *data) > vdc->unrealize = virtio_iommu_device_unrealize; > vdc->reset = virtio_iommu_device_reset; > vdc->get_config = virtio_iommu_get_config; > + vdc->set_config = virtio_iommu_set_config; > vdc->get_features = virtio_iommu_get_features; > vdc->set_status = virtio_iommu_set_status; > vdc->vmsd = &vmstate_virtio_iommu_device; > diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events > index f7ad6be5fb..a5102eac9e 100644 > --- a/hw/virtio/trace-events > +++ b/hw/virtio/trace-events > @@ -89,9 +89,11 @@ virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d" > > # virtio-iommu.c > virtio_iommu_device_reset(void) "reset!" > +virtio_iommu_system_reset(void) "system reset!" > virtio_iommu_get_features(uint64_t features) "device supports features=0x%"PRIx64 > virtio_iommu_device_status(uint8_t status) "driver status = %d" > -virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start, uint64_t end, uint32_t domain_start, uint32_t domain_end, uint32_t probe_size) "page_size_mask=0x%"PRIx64" input range start=0x%"PRIx64" input range end=0x%"PRIx64" domain range start=%d domain range end=%d probe_size=0x%x" > +virtio_iommu_get_config(uint64_t page_size_mask, uint64_t start, uint64_t end, uint32_t domain_start, uint32_t domain_end, uint32_t probe_size, uint8_t bypass) "page_size_mask=0x%"PRIx64" input range start=0x%"PRIx64" input range end=0x%"PRIx64" domain range start=%d domain range end=%d probe_size=0x%x bypass=0x%x" > +virtio_iommu_set_config(uint8_t bypass) "bypass=0x%x" > virtio_iommu_attach(uint32_t domain_id, uint32_t ep_id) "domain=%d endpoint=%d" > virtio_iommu_detach(uint32_t domain_id, uint32_t ep_id) "domain=%d endpoint=%d" > virtio_iommu_map(uint32_t domain_id, uint64_t virt_start, uint64_t virt_end, uint64_t phys_start, uint32_t flags) "domain=%d virt_start=0x%"PRIx64" virt_end=0x%"PRIx64 " phys_start=0x%"PRIx64" flags=%d" ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/4] virtio-iommu: Default to bypass during boot 2022-01-27 14:29 ` [PATCH v2 2/4] virtio-iommu: Default to bypass during boot Jean-Philippe Brucker 2022-01-31 9:14 ` Eric Auger @ 2022-02-08 17:42 ` Cornelia Huck 2022-02-08 21:07 ` Michael S. Tsirkin 1 sibling, 1 reply; 20+ messages in thread From: Cornelia Huck @ 2022-02-08 17:42 UTC (permalink / raw) To: Jean-Philippe Brucker, eric.auger Cc: lvivier, Jean-Philippe Brucker, thuth, mst, qemu-devel, pasic, pbonzini On Thu, Jan 27 2022, Jean-Philippe Brucker <jean-philippe@linaro.org> wrote: > @@ -988,9 +1025,9 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_INPUT_RANGE); > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_DOMAIN_RANGE); > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MAP_UNMAP); > - virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS); > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MMIO); > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_PROBE); > + virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS_CONFIG); Hm. In the other patch, you say that you don't support cross-version migration (I assume across QEMU versions?) Because changing the feature set will be guest-visible, and would need some compat handling if you plan to support this. ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/4] virtio-iommu: Default to bypass during boot 2022-02-08 17:42 ` Cornelia Huck @ 2022-02-08 21:07 ` Michael S. Tsirkin 2022-02-09 11:10 ` Cornelia Huck 0 siblings, 1 reply; 20+ messages in thread From: Michael S. Tsirkin @ 2022-02-08 21:07 UTC (permalink / raw) To: Cornelia Huck Cc: lvivier, Jean-Philippe Brucker, thuth, qemu-devel, pasic, eric.auger, pbonzini On Tue, Feb 08, 2022 at 06:42:57PM +0100, Cornelia Huck wrote: > On Thu, Jan 27 2022, Jean-Philippe Brucker <jean-philippe@linaro.org> wrote: > > > @@ -988,9 +1025,9 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) > > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_INPUT_RANGE); > > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_DOMAIN_RANGE); > > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MAP_UNMAP); > > - virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS); > > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MMIO); > > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_PROBE); > > + virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS_CONFIG); > > Hm. In the other patch, you say that you don't support cross-version > migration (I assume across QEMU versions?) I missed that ... where does it say this? > Because changing the feature > set will be guest-visible, and would need some compat handling if you > plan to support this. ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/4] virtio-iommu: Default to bypass during boot 2022-02-08 21:07 ` Michael S. Tsirkin @ 2022-02-09 11:10 ` Cornelia Huck 2022-02-09 11:32 ` Eric Auger 0 siblings, 1 reply; 20+ messages in thread From: Cornelia Huck @ 2022-02-09 11:10 UTC (permalink / raw) To: Michael S. Tsirkin Cc: lvivier, Jean-Philippe Brucker, thuth, qemu-devel, pasic, eric.auger, pbonzini On Tue, Feb 08 2022, "Michael S. Tsirkin" <mst@redhat.com> wrote: > On Tue, Feb 08, 2022 at 06:42:57PM +0100, Cornelia Huck wrote: >> On Thu, Jan 27 2022, Jean-Philippe Brucker <jean-philippe@linaro.org> wrote: >> >> > @@ -988,9 +1025,9 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) >> > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_INPUT_RANGE); >> > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_DOMAIN_RANGE); >> > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MAP_UNMAP); >> > - virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS); >> > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MMIO); >> > virtio_add_feature(&s->features, VIRTIO_IOMMU_F_PROBE); >> > + virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS_CONFIG); >> >> Hm. In the other patch, you say that you don't support cross-version >> migration (I assume across QEMU versions?) > > I missed that ... where does it say this? In bf447d9b-c039-ccdc-f24f-ab8b56c1b196@redhat.com and follow-ups (unless I misread that; maybe it's more about this concrete boundary and not generally?) > >> Because changing the feature >> set will be guest-visible, and would need some compat handling if you >> plan to support this. ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/4] virtio-iommu: Default to bypass during boot 2022-02-09 11:10 ` Cornelia Huck @ 2022-02-09 11:32 ` Eric Auger 2022-02-09 12:48 ` Cornelia Huck 0 siblings, 1 reply; 20+ messages in thread From: Eric Auger @ 2022-02-09 11:32 UTC (permalink / raw) To: Cornelia Huck, Michael S. Tsirkin Cc: lvivier, Jean-Philippe Brucker, thuth, qemu-devel, pasic, pbonzini Hi, On 2/9/22 12:10 PM, Cornelia Huck wrote: > On Tue, Feb 08 2022, "Michael S. Tsirkin" <mst@redhat.com> wrote: > >> On Tue, Feb 08, 2022 at 06:42:57PM +0100, Cornelia Huck wrote: >>> On Thu, Jan 27 2022, Jean-Philippe Brucker <jean-philippe@linaro.org> wrote: >>> >>>> @@ -988,9 +1025,9 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) >>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_INPUT_RANGE); >>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_DOMAIN_RANGE); >>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MAP_UNMAP); >>>> - virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS); >>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MMIO); >>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_PROBE); >>>> + virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS_CONFIG); >>> Hm. In the other patch, you say that you don't support cross-version >>> migration (I assume across QEMU versions?) >> I missed that ... where does it say this? > In bf447d9b-c039-ccdc-f24f-ab8b56c1b196@redhat.com and follow-ups > (unless I misread that; maybe it's more about this concrete boundary and > not generally?) We were considering the virtio-iommu QEMU device currently isn't used for production yet, as far as we know, because we were missing the ACPI integration. So we envisionned to not care about mig subsections and just add the new field in the VMState. would that make sense? Thanks Eric > >>> Because changing the feature >>> set will be guest-visible, and would need some compat handling if you >>> plan to support this. ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/4] virtio-iommu: Default to bypass during boot 2022-02-09 11:32 ` Eric Auger @ 2022-02-09 12:48 ` Cornelia Huck 0 siblings, 0 replies; 20+ messages in thread From: Cornelia Huck @ 2022-02-09 12:48 UTC (permalink / raw) To: eric.auger, Michael S. Tsirkin Cc: lvivier, Jean-Philippe Brucker, thuth, qemu-devel, pasic, pbonzini On Wed, Feb 09 2022, Eric Auger <eric.auger@redhat.com> wrote: > Hi, > > On 2/9/22 12:10 PM, Cornelia Huck wrote: >> On Tue, Feb 08 2022, "Michael S. Tsirkin" <mst@redhat.com> wrote: >> >>> On Tue, Feb 08, 2022 at 06:42:57PM +0100, Cornelia Huck wrote: >>>> On Thu, Jan 27 2022, Jean-Philippe Brucker <jean-philippe@linaro.org> wrote: >>>> >>>>> @@ -988,9 +1025,9 @@ static void virtio_iommu_device_realize(DeviceState *dev, Error **errp) >>>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_INPUT_RANGE); >>>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_DOMAIN_RANGE); >>>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MAP_UNMAP); >>>>> - virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS); >>>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_MMIO); >>>>> virtio_add_feature(&s->features, VIRTIO_IOMMU_F_PROBE); >>>>> + virtio_add_feature(&s->features, VIRTIO_IOMMU_F_BYPASS_CONFIG); >>>> Hm. In the other patch, you say that you don't support cross-version >>>> migration (I assume across QEMU versions?) >>> I missed that ... where does it say this? >> In bf447d9b-c039-ccdc-f24f-ab8b56c1b196@redhat.com and follow-ups >> (unless I misread that; maybe it's more about this concrete boundary and >> not generally?) > > We were considering the virtio-iommu QEMU device currently isn't used > for production yet, as far as we know, because we were missing the ACPI > integration. > So we envisionned to not care about mig subsections and just add the new > field in the VMState. > > would that make sense? If people are currently mostly playing with this device, it would not hurt too much, I guess. (Should we document expectations somewhere?) Adding a compat prop for this feature would be easy enough, but juggling subsections is a bit more involved, so I see the argument for not caring about them. > > Thanks > > Eric > >> >>>> Because changing the feature >>>> set will be guest-visible, and would need some compat handling if you >>>> plan to support this. ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 3/4] virtio-iommu: Support bypass domain 2022-01-27 14:29 [PATCH v2 0/4] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG Jean-Philippe Brucker 2022-01-27 14:29 ` [PATCH v2 1/4] linux-headers: update to v5.17-rc1 Jean-Philippe Brucker 2022-01-27 14:29 ` [PATCH v2 2/4] virtio-iommu: Default to bypass during boot Jean-Philippe Brucker @ 2022-01-27 14:29 ` Jean-Philippe Brucker 2022-01-31 9:22 ` Eric Auger 2022-01-27 14:29 ` [PATCH v2 4/4] tests/qtest/virtio-iommu-test: Check bypass config Jean-Philippe Brucker 3 siblings, 1 reply; 20+ messages in thread From: Jean-Philippe Brucker @ 2022-01-27 14:29 UTC (permalink / raw) To: eric.auger Cc: lvivier, Jean-Philippe Brucker, thuth, mst, cohuck, qemu-devel, pasic, pbonzini The driver can create a bypass domain by passing the VIRTIO_IOMMU_ATTACH_F_BYPASS flag on the ATTACH request. Bypass domains perform slightly better than domains with identity mappings since they skip translation. Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> --- hw/virtio/virtio-iommu.c | 32 ++++++++++++++++++++++++++++++-- 1 file changed, 30 insertions(+), 2 deletions(-) diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c index ec02029bb6..a112428c65 100644 --- a/hw/virtio/virtio-iommu.c +++ b/hw/virtio/virtio-iommu.c @@ -43,6 +43,7 @@ typedef struct VirtIOIOMMUDomain { uint32_t id; + bool bypass; GTree *mappings; QLIST_HEAD(, VirtIOIOMMUEndpoint) endpoint_list; } VirtIOIOMMUDomain; @@ -258,12 +259,16 @@ static void virtio_iommu_put_endpoint(gpointer data) } static VirtIOIOMMUDomain *virtio_iommu_get_domain(VirtIOIOMMU *s, - uint32_t domain_id) + uint32_t domain_id, + bool bypass) { VirtIOIOMMUDomain *domain; domain = g_tree_lookup(s->domains, GUINT_TO_POINTER(domain_id)); if (domain) { + if (domain->bypass != bypass) { + return NULL; + } return domain; } domain = g_malloc0(sizeof(*domain)); @@ -271,6 +276,7 @@ static VirtIOIOMMUDomain *virtio_iommu_get_domain(VirtIOIOMMU *s, domain->mappings = g_tree_new_full((GCompareDataFunc)interval_cmp, NULL, (GDestroyNotify)g_free, (GDestroyNotify)g_free); + domain->bypass = bypass; g_tree_insert(s->domains, GUINT_TO_POINTER(domain_id), domain); QLIST_INIT(&domain->endpoint_list); trace_virtio_iommu_get_domain(domain_id); @@ -334,11 +340,16 @@ static int virtio_iommu_attach(VirtIOIOMMU *s, { uint32_t domain_id = le32_to_cpu(req->domain); uint32_t ep_id = le32_to_cpu(req->endpoint); + uint32_t flags = le32_to_cpu(req->flags); VirtIOIOMMUDomain *domain; VirtIOIOMMUEndpoint *ep; trace_virtio_iommu_attach(domain_id, ep_id); + if (flags & ~VIRTIO_IOMMU_ATTACH_F_BYPASS) { + return VIRTIO_IOMMU_S_INVAL; + } + ep = virtio_iommu_get_endpoint(s, ep_id); if (!ep) { return VIRTIO_IOMMU_S_NOENT; @@ -356,7 +367,12 @@ static int virtio_iommu_attach(VirtIOIOMMU *s, } } - domain = virtio_iommu_get_domain(s, domain_id); + domain = virtio_iommu_get_domain(s, domain_id, + flags & VIRTIO_IOMMU_ATTACH_F_BYPASS); + if (!domain) { + /* Incompatible bypass flag */ + return VIRTIO_IOMMU_S_INVAL; + } QLIST_INSERT_HEAD(&domain->endpoint_list, ep, next); ep->domain = domain; @@ -419,6 +435,10 @@ static int virtio_iommu_map(VirtIOIOMMU *s, return VIRTIO_IOMMU_S_NOENT; } + if (domain->bypass) { + return VIRTIO_IOMMU_S_INVAL; + } + interval = g_malloc0(sizeof(*interval)); interval->low = virt_start; @@ -464,6 +484,11 @@ static int virtio_iommu_unmap(VirtIOIOMMU *s, if (!domain) { return VIRTIO_IOMMU_S_NOENT; } + + if (domain->bypass) { + return VIRTIO_IOMMU_S_INVAL; + } + interval.low = virt_start; interval.high = virt_end; @@ -780,6 +805,9 @@ static IOMMUTLBEntry virtio_iommu_translate(IOMMUMemoryRegion *mr, hwaddr addr, entry.perm = flag; } goto unlock; + } else if (ep->domain->bypass) { + entry.perm = flag; + goto unlock; } found = g_tree_lookup_extended(ep->domain->mappings, (gpointer)(&interval), -- 2.34.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH v2 3/4] virtio-iommu: Support bypass domain 2022-01-27 14:29 ` [PATCH v2 3/4] virtio-iommu: Support bypass domain Jean-Philippe Brucker @ 2022-01-31 9:22 ` Eric Auger 2022-01-31 13:07 ` Dr. David Alan Gilbert 0 siblings, 1 reply; 20+ messages in thread From: Eric Auger @ 2022-01-31 9:22 UTC (permalink / raw) To: Jean-Philippe Brucker, Juan Quintela, Dr. David Alan Gilbert, Peter Maydell Cc: lvivier, thuth, mst, cohuck, qemu-devel, pasic, pbonzini Hi Jean, On 1/27/22 3:29 PM, Jean-Philippe Brucker wrote: > The driver can create a bypass domain by passing the > VIRTIO_IOMMU_ATTACH_F_BYPASS flag on the ATTACH request. Bypass domains > perform slightly better than domains with identity mappings since they > skip translation. > > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> > --- > hw/virtio/virtio-iommu.c | 32 ++++++++++++++++++++++++++++++-- > 1 file changed, 30 insertions(+), 2 deletions(-) > > diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c > index ec02029bb6..a112428c65 100644 > --- a/hw/virtio/virtio-iommu.c > +++ b/hw/virtio/virtio-iommu.c > @@ -43,6 +43,7 @@ > > typedef struct VirtIOIOMMUDomain { > uint32_t id; > + bool bypass; I am afraid this will break the migration if you don't change vmstate_domain. See static const VMStateDescription vmstate_domain. Also you need to migrate the new bypass field. Logically we should handle this with a vmstate subsection I think to handle migration of older devices. However I doubt the device has been used in production environment supporting migration so my guess is we may skip that burden and just add the missing field. Adding Juan, Dave & Peter for advices. Thanks Eric > GTree *mappings; > QLIST_HEAD(, VirtIOIOMMUEndpoint) endpoint_list; > } VirtIOIOMMUDomain; > @@ -258,12 +259,16 @@ static void virtio_iommu_put_endpoint(gpointer data) > } > > static VirtIOIOMMUDomain *virtio_iommu_get_domain(VirtIOIOMMU *s, > - uint32_t domain_id) > + uint32_t domain_id, > + bool bypass) > { > VirtIOIOMMUDomain *domain; > > domain = g_tree_lookup(s->domains, GUINT_TO_POINTER(domain_id)); > if (domain) { > + if (domain->bypass != bypass) { > + return NULL; > + } > return domain; > } > domain = g_malloc0(sizeof(*domain)); > @@ -271,6 +276,7 @@ static VirtIOIOMMUDomain *virtio_iommu_get_domain(VirtIOIOMMU *s, > domain->mappings = g_tree_new_full((GCompareDataFunc)interval_cmp, > NULL, (GDestroyNotify)g_free, > (GDestroyNotify)g_free); > + domain->bypass = bypass; > g_tree_insert(s->domains, GUINT_TO_POINTER(domain_id), domain); > QLIST_INIT(&domain->endpoint_list); > trace_virtio_iommu_get_domain(domain_id); > @@ -334,11 +340,16 @@ static int virtio_iommu_attach(VirtIOIOMMU *s, > { > uint32_t domain_id = le32_to_cpu(req->domain); > uint32_t ep_id = le32_to_cpu(req->endpoint); > + uint32_t flags = le32_to_cpu(req->flags); > VirtIOIOMMUDomain *domain; > VirtIOIOMMUEndpoint *ep; > > trace_virtio_iommu_attach(domain_id, ep_id); > > + if (flags & ~VIRTIO_IOMMU_ATTACH_F_BYPASS) { > + return VIRTIO_IOMMU_S_INVAL; > + } > + > ep = virtio_iommu_get_endpoint(s, ep_id); > if (!ep) { > return VIRTIO_IOMMU_S_NOENT; > @@ -356,7 +367,12 @@ static int virtio_iommu_attach(VirtIOIOMMU *s, > } > } > > - domain = virtio_iommu_get_domain(s, domain_id); > + domain = virtio_iommu_get_domain(s, domain_id, > + flags & VIRTIO_IOMMU_ATTACH_F_BYPASS); > + if (!domain) { > + /* Incompatible bypass flag */ > + return VIRTIO_IOMMU_S_INVAL; > + } > QLIST_INSERT_HEAD(&domain->endpoint_list, ep, next); > > ep->domain = domain; > @@ -419,6 +435,10 @@ static int virtio_iommu_map(VirtIOIOMMU *s, > return VIRTIO_IOMMU_S_NOENT; > } > > + if (domain->bypass) { > + return VIRTIO_IOMMU_S_INVAL; > + } > + > interval = g_malloc0(sizeof(*interval)); > > interval->low = virt_start; > @@ -464,6 +484,11 @@ static int virtio_iommu_unmap(VirtIOIOMMU *s, > if (!domain) { > return VIRTIO_IOMMU_S_NOENT; > } > + > + if (domain->bypass) { > + return VIRTIO_IOMMU_S_INVAL; > + } > + > interval.low = virt_start; > interval.high = virt_end; > > @@ -780,6 +805,9 @@ static IOMMUTLBEntry virtio_iommu_translate(IOMMUMemoryRegion *mr, hwaddr addr, > entry.perm = flag; > } > goto unlock; > + } else if (ep->domain->bypass) { > + entry.perm = flag; > + goto unlock; > } > > found = g_tree_lookup_extended(ep->domain->mappings, (gpointer)(&interval), ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 3/4] virtio-iommu: Support bypass domain 2022-01-31 9:22 ` Eric Auger @ 2022-01-31 13:07 ` Dr. David Alan Gilbert 2022-02-02 13:21 ` Eric Auger 0 siblings, 1 reply; 20+ messages in thread From: Dr. David Alan Gilbert @ 2022-01-31 13:07 UTC (permalink / raw) To: Eric Auger Cc: lvivier, Peter Maydell, thuth, Jean-Philippe Brucker, cohuck, qemu-devel, pasic, Juan Quintela, mst, pbonzini * Eric Auger (eric.auger@redhat.com) wrote: > Hi Jean, > > On 1/27/22 3:29 PM, Jean-Philippe Brucker wrote: > > The driver can create a bypass domain by passing the > > VIRTIO_IOMMU_ATTACH_F_BYPASS flag on the ATTACH request. Bypass domains > > perform slightly better than domains with identity mappings since they > > skip translation. > > > > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> > > --- > > hw/virtio/virtio-iommu.c | 32 ++++++++++++++++++++++++++++++-- > > 1 file changed, 30 insertions(+), 2 deletions(-) > > > > diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c > > index ec02029bb6..a112428c65 100644 > > --- a/hw/virtio/virtio-iommu.c > > +++ b/hw/virtio/virtio-iommu.c > > @@ -43,6 +43,7 @@ > > > > typedef struct VirtIOIOMMUDomain { > > uint32_t id; > > + bool bypass; > I am afraid this will break the migration if you don't change > vmstate_domain. > > See static const VMStateDescription vmstate_domain. > Also you need to migrate the new bypass field. > > Logically we should handle this with a vmstate subsection I think to > handle migration of older devices. However I doubt the device has been > used in production environment supporting migration so my guess is we > may skip that burden and just add the missing field. Adding Juan, Dave & > Peter for advices. I'm not sure about users of this; if no one has used it then yeh; you could bump up the version_id to make it a bit clearer. Dave > Thanks > > Eric > > > GTree *mappings; > > QLIST_HEAD(, VirtIOIOMMUEndpoint) endpoint_list; > > } VirtIOIOMMUDomain; > > @@ -258,12 +259,16 @@ static void virtio_iommu_put_endpoint(gpointer data) > > } > > > > static VirtIOIOMMUDomain *virtio_iommu_get_domain(VirtIOIOMMU *s, > > - uint32_t domain_id) > > + uint32_t domain_id, > > + bool bypass) > > { > > VirtIOIOMMUDomain *domain; > > > > domain = g_tree_lookup(s->domains, GUINT_TO_POINTER(domain_id)); > > if (domain) { > > + if (domain->bypass != bypass) { > > + return NULL; > > + } > > return domain; > > } > > domain = g_malloc0(sizeof(*domain)); > > @@ -271,6 +276,7 @@ static VirtIOIOMMUDomain *virtio_iommu_get_domain(VirtIOIOMMU *s, > > domain->mappings = g_tree_new_full((GCompareDataFunc)interval_cmp, > > NULL, (GDestroyNotify)g_free, > > (GDestroyNotify)g_free); > > + domain->bypass = bypass; > > g_tree_insert(s->domains, GUINT_TO_POINTER(domain_id), domain); > > QLIST_INIT(&domain->endpoint_list); > > trace_virtio_iommu_get_domain(domain_id); > > @@ -334,11 +340,16 @@ static int virtio_iommu_attach(VirtIOIOMMU *s, > > { > > uint32_t domain_id = le32_to_cpu(req->domain); > > uint32_t ep_id = le32_to_cpu(req->endpoint); > > + uint32_t flags = le32_to_cpu(req->flags); > > VirtIOIOMMUDomain *domain; > > VirtIOIOMMUEndpoint *ep; > > > > trace_virtio_iommu_attach(domain_id, ep_id); > > > > + if (flags & ~VIRTIO_IOMMU_ATTACH_F_BYPASS) { > > + return VIRTIO_IOMMU_S_INVAL; > > + } > > + > > ep = virtio_iommu_get_endpoint(s, ep_id); > > if (!ep) { > > return VIRTIO_IOMMU_S_NOENT; > > @@ -356,7 +367,12 @@ static int virtio_iommu_attach(VirtIOIOMMU *s, > > } > > } > > > > - domain = virtio_iommu_get_domain(s, domain_id); > > + domain = virtio_iommu_get_domain(s, domain_id, > > + flags & VIRTIO_IOMMU_ATTACH_F_BYPASS); > > + if (!domain) { > > + /* Incompatible bypass flag */ > > + return VIRTIO_IOMMU_S_INVAL; > > + } > > QLIST_INSERT_HEAD(&domain->endpoint_list, ep, next); > > > > ep->domain = domain; > > @@ -419,6 +435,10 @@ static int virtio_iommu_map(VirtIOIOMMU *s, > > return VIRTIO_IOMMU_S_NOENT; > > } > > > > + if (domain->bypass) { > > + return VIRTIO_IOMMU_S_INVAL; > > + } > > + > > interval = g_malloc0(sizeof(*interval)); > > > > interval->low = virt_start; > > @@ -464,6 +484,11 @@ static int virtio_iommu_unmap(VirtIOIOMMU *s, > > if (!domain) { > > return VIRTIO_IOMMU_S_NOENT; > > } > > + > > + if (domain->bypass) { > > + return VIRTIO_IOMMU_S_INVAL; > > + } > > + > > interval.low = virt_start; > > interval.high = virt_end; > > > > @@ -780,6 +805,9 @@ static IOMMUTLBEntry virtio_iommu_translate(IOMMUMemoryRegion *mr, hwaddr addr, > > entry.perm = flag; > > } > > goto unlock; > > + } else if (ep->domain->bypass) { > > + entry.perm = flag; > > + goto unlock; > > } > > > > found = g_tree_lookup_extended(ep->domain->mappings, (gpointer)(&interval), > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 3/4] virtio-iommu: Support bypass domain 2022-01-31 13:07 ` Dr. David Alan Gilbert @ 2022-02-02 13:21 ` Eric Auger 2022-02-08 12:30 ` Jean-Philippe Brucker 0 siblings, 1 reply; 20+ messages in thread From: Eric Auger @ 2022-02-02 13:21 UTC (permalink / raw) To: Dr. David Alan Gilbert Cc: lvivier, Peter Maydell, thuth, Jean-Philippe Brucker, cohuck, qemu-devel, pasic, Juan Quintela, mst, pbonzini Hi Dave, On 1/31/22 2:07 PM, Dr. David Alan Gilbert wrote: > * Eric Auger (eric.auger@redhat.com) wrote: >> Hi Jean, >> >> On 1/27/22 3:29 PM, Jean-Philippe Brucker wrote: >>> The driver can create a bypass domain by passing the >>> VIRTIO_IOMMU_ATTACH_F_BYPASS flag on the ATTACH request. Bypass domains >>> perform slightly better than domains with identity mappings since they >>> skip translation. >>> >>> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> >>> --- >>> hw/virtio/virtio-iommu.c | 32 ++++++++++++++++++++++++++++++-- >>> 1 file changed, 30 insertions(+), 2 deletions(-) >>> >>> diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c >>> index ec02029bb6..a112428c65 100644 >>> --- a/hw/virtio/virtio-iommu.c >>> +++ b/hw/virtio/virtio-iommu.c >>> @@ -43,6 +43,7 @@ >>> >>> typedef struct VirtIOIOMMUDomain { >>> uint32_t id; >>> + bool bypass; >> I am afraid this will break the migration if you don't change >> vmstate_domain. >> >> See static const VMStateDescription vmstate_domain. >> Also you need to migrate the new bypass field. >> >> Logically we should handle this with a vmstate subsection I think to >> handle migration of older devices. However I doubt the device has been >> used in production environment supporting migration so my guess is we >> may skip that burden and just add the missing field. Adding Juan, Dave & >> Peter for advices. > I'm not sure about users of this; if no one has used it then yeh; you > could bump up the version_id to make it a bit clearer. Thank you for your input. Yes to me it sounds OK to only bump the version_id while adding the new field. Eric > > Dave > >> Thanks >> >> Eric >> >>> GTree *mappings; >>> QLIST_HEAD(, VirtIOIOMMUEndpoint) endpoint_list; >>> } VirtIOIOMMUDomain; >>> @@ -258,12 +259,16 @@ static void virtio_iommu_put_endpoint(gpointer data) >>> } >>> >>> static VirtIOIOMMUDomain *virtio_iommu_get_domain(VirtIOIOMMU *s, >>> - uint32_t domain_id) >>> + uint32_t domain_id, >>> + bool bypass) >>> { >>> VirtIOIOMMUDomain *domain; >>> >>> domain = g_tree_lookup(s->domains, GUINT_TO_POINTER(domain_id)); >>> if (domain) { >>> + if (domain->bypass != bypass) { >>> + return NULL; >>> + } >>> return domain; >>> } >>> domain = g_malloc0(sizeof(*domain)); >>> @@ -271,6 +276,7 @@ static VirtIOIOMMUDomain *virtio_iommu_get_domain(VirtIOIOMMU *s, >>> domain->mappings = g_tree_new_full((GCompareDataFunc)interval_cmp, >>> NULL, (GDestroyNotify)g_free, >>> (GDestroyNotify)g_free); >>> + domain->bypass = bypass; >>> g_tree_insert(s->domains, GUINT_TO_POINTER(domain_id), domain); >>> QLIST_INIT(&domain->endpoint_list); >>> trace_virtio_iommu_get_domain(domain_id); >>> @@ -334,11 +340,16 @@ static int virtio_iommu_attach(VirtIOIOMMU *s, >>> { >>> uint32_t domain_id = le32_to_cpu(req->domain); >>> uint32_t ep_id = le32_to_cpu(req->endpoint); >>> + uint32_t flags = le32_to_cpu(req->flags); >>> VirtIOIOMMUDomain *domain; >>> VirtIOIOMMUEndpoint *ep; >>> >>> trace_virtio_iommu_attach(domain_id, ep_id); >>> >>> + if (flags & ~VIRTIO_IOMMU_ATTACH_F_BYPASS) { >>> + return VIRTIO_IOMMU_S_INVAL; >>> + } >>> + >>> ep = virtio_iommu_get_endpoint(s, ep_id); >>> if (!ep) { >>> return VIRTIO_IOMMU_S_NOENT; >>> @@ -356,7 +367,12 @@ static int virtio_iommu_attach(VirtIOIOMMU *s, >>> } >>> } >>> >>> - domain = virtio_iommu_get_domain(s, domain_id); >>> + domain = virtio_iommu_get_domain(s, domain_id, >>> + flags & VIRTIO_IOMMU_ATTACH_F_BYPASS); >>> + if (!domain) { >>> + /* Incompatible bypass flag */ >>> + return VIRTIO_IOMMU_S_INVAL; >>> + } >>> QLIST_INSERT_HEAD(&domain->endpoint_list, ep, next); >>> >>> ep->domain = domain; >>> @@ -419,6 +435,10 @@ static int virtio_iommu_map(VirtIOIOMMU *s, >>> return VIRTIO_IOMMU_S_NOENT; >>> } >>> >>> + if (domain->bypass) { >>> + return VIRTIO_IOMMU_S_INVAL; >>> + } >>> + >>> interval = g_malloc0(sizeof(*interval)); >>> >>> interval->low = virt_start; >>> @@ -464,6 +484,11 @@ static int virtio_iommu_unmap(VirtIOIOMMU *s, >>> if (!domain) { >>> return VIRTIO_IOMMU_S_NOENT; >>> } >>> + >>> + if (domain->bypass) { >>> + return VIRTIO_IOMMU_S_INVAL; >>> + } >>> + >>> interval.low = virt_start; >>> interval.high = virt_end; >>> >>> @@ -780,6 +805,9 @@ static IOMMUTLBEntry virtio_iommu_translate(IOMMUMemoryRegion *mr, hwaddr addr, >>> entry.perm = flag; >>> } >>> goto unlock; >>> + } else if (ep->domain->bypass) { >>> + entry.perm = flag; >>> + goto unlock; >>> } >>> >>> found = g_tree_lookup_extended(ep->domain->mappings, (gpointer)(&interval), ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 3/4] virtio-iommu: Support bypass domain 2022-02-02 13:21 ` Eric Auger @ 2022-02-08 12:30 ` Jean-Philippe Brucker 2022-02-08 13:09 ` Dr. David Alan Gilbert 0 siblings, 1 reply; 20+ messages in thread From: Jean-Philippe Brucker @ 2022-02-08 12:30 UTC (permalink / raw) To: Eric Auger Cc: lvivier, Peter Maydell, thuth, mst, cohuck, Dr. David Alan Gilbert, qemu-devel, pasic, Juan Quintela, pbonzini On Wed, Feb 02, 2022 at 02:21:37PM +0100, Eric Auger wrote: > >>> diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c > >>> index ec02029bb6..a112428c65 100644 > >>> --- a/hw/virtio/virtio-iommu.c > >>> +++ b/hw/virtio/virtio-iommu.c > >>> @@ -43,6 +43,7 @@ > >>> > >>> typedef struct VirtIOIOMMUDomain { > >>> uint32_t id; > >>> + bool bypass; > >> I am afraid this will break the migration if you don't change > >> vmstate_domain. > >> > >> See static const VMStateDescription vmstate_domain. > >> Also you need to migrate the new bypass field. > >> > >> Logically we should handle this with a vmstate subsection I think to > >> handle migration of older devices. However I doubt the device has been > >> used in production environment supporting migration so my guess is we > >> may skip that burden and just add the missing field. Adding Juan, Dave & > >> Peter for advices. > > I'm not sure about users of this; if no one has used it then yeh; you > > could bump up the version_id to make it a bit clearer. > > Thank you for your input. Yes to me it sounds OK to only bump the > version_id while adding the new field. Ok. Just to make sure we're on the same page, this means we don't support migration from new->old or old->new instances, since the migration stream doesn't carry a version ID for the virtio-iommu-device and domain vmstates, as far as I understand. I also believe backward-incompatible changes are fine this time around, though I don't have much visibility in what's being used. Thanks, Jean ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 3/4] virtio-iommu: Support bypass domain 2022-02-08 12:30 ` Jean-Philippe Brucker @ 2022-02-08 13:09 ` Dr. David Alan Gilbert 2022-02-08 13:29 ` Eric Auger 0 siblings, 1 reply; 20+ messages in thread From: Dr. David Alan Gilbert @ 2022-02-08 13:09 UTC (permalink / raw) To: Jean-Philippe Brucker Cc: lvivier, Peter Maydell, thuth, Juan Quintela, mst, cohuck, qemu-devel, pasic, Eric Auger, pbonzini * Jean-Philippe Brucker (jean-philippe@linaro.org) wrote: > On Wed, Feb 02, 2022 at 02:21:37PM +0100, Eric Auger wrote: > > >>> diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c > > >>> index ec02029bb6..a112428c65 100644 > > >>> --- a/hw/virtio/virtio-iommu.c > > >>> +++ b/hw/virtio/virtio-iommu.c > > >>> @@ -43,6 +43,7 @@ > > >>> > > >>> typedef struct VirtIOIOMMUDomain { > > >>> uint32_t id; > > >>> + bool bypass; > > >> I am afraid this will break the migration if you don't change > > >> vmstate_domain. > > >> > > >> See static const VMStateDescription vmstate_domain. > > >> Also you need to migrate the new bypass field. > > >> > > >> Logically we should handle this with a vmstate subsection I think to > > >> handle migration of older devices. However I doubt the device has been > > >> used in production environment supporting migration so my guess is we > > >> may skip that burden and just add the missing field. Adding Juan, Dave & > > >> Peter for advices. > > > I'm not sure about users of this; if no one has used it then yeh; you > > > could bump up the version_id to make it a bit clearer. > > > > Thank you for your input. Yes to me it sounds OK to only bump the > > version_id while adding the new field. > > Ok. Just to make sure we're on the same page, this means we don't support > migration from new->old or old->new instances, since the migration stream > doesn't carry a version ID for the virtio-iommu-device and domain > vmstates, as far as I understand. I also believe backward-incompatible > changes are fine this time around, though I don't have much visibility in > what's being used. I think the stream only has it for top level devices; I've not dug into this device. Dave > Thanks, > Jean > -- Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 3/4] virtio-iommu: Support bypass domain 2022-02-08 13:09 ` Dr. David Alan Gilbert @ 2022-02-08 13:29 ` Eric Auger 2022-02-08 17:02 ` Jean-Philippe Brucker 0 siblings, 1 reply; 20+ messages in thread From: Eric Auger @ 2022-02-08 13:29 UTC (permalink / raw) To: Dr. David Alan Gilbert, Jean-Philippe Brucker Cc: lvivier, Peter Maydell, thuth, mst, cohuck, qemu-devel, pasic, Juan Quintela, pbonzini Hi Jean, On 2/8/22 2:09 PM, Dr. David Alan Gilbert wrote: > * Jean-Philippe Brucker (jean-philippe@linaro.org) wrote: >> On Wed, Feb 02, 2022 at 02:21:37PM +0100, Eric Auger wrote: >>>>>> diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c >>>>>> index ec02029bb6..a112428c65 100644 >>>>>> --- a/hw/virtio/virtio-iommu.c >>>>>> +++ b/hw/virtio/virtio-iommu.c >>>>>> @@ -43,6 +43,7 @@ >>>>>> >>>>>> typedef struct VirtIOIOMMUDomain { >>>>>> uint32_t id; >>>>>> + bool bypass; >>>>> I am afraid this will break the migration if you don't change >>>>> vmstate_domain. >>>>> >>>>> See static const VMStateDescription vmstate_domain. >>>>> Also you need to migrate the new bypass field. >>>>> >>>>> Logically we should handle this with a vmstate subsection I think to >>>>> handle migration of older devices. However I doubt the device has been >>>>> used in production environment supporting migration so my guess is we >>>>> may skip that burden and just add the missing field. Adding Juan, Dave & >>>>> Peter for advices. >>>> I'm not sure about users of this; if no one has used it then yeh; you >>>> could bump up the version_id to make it a bit clearer. >>> Thank you for your input. Yes to me it sounds OK to only bump the >>> version_id while adding the new field. >> Ok. Just to make sure we're on the same page, this means we don't support >> migration from new->old or old->new instances, since the migration stream >> doesn't carry a version ID for the virtio-iommu-device and domain >> vmstates, as far as I understand. I also believe backward-incompatible >> changes are fine this time around, though I don't have much visibility in >> what's being used. > I think the stream only has it for top level devices; I've not dug into > this device. Not sure I get what you meant: vmstate_virtio_iommu has a version_id. Also vmstate_domain has one. Thanks Eric > > Dave > >> Thanks, >> Jean >> ^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 3/4] virtio-iommu: Support bypass domain 2022-02-08 13:29 ` Eric Auger @ 2022-02-08 17:02 ` Jean-Philippe Brucker 0 siblings, 0 replies; 20+ messages in thread From: Jean-Philippe Brucker @ 2022-02-08 17:02 UTC (permalink / raw) To: Eric Auger Cc: lvivier, Peter Maydell, thuth, mst, cohuck, Dr. David Alan Gilbert, qemu-devel, pasic, Juan Quintela, pbonzini On Tue, Feb 08, 2022 at 02:29:21PM +0100, Eric Auger wrote: > Hi Jean, > > On 2/8/22 2:09 PM, Dr. David Alan Gilbert wrote: > > * Jean-Philippe Brucker (jean-philippe@linaro.org) wrote: > >> On Wed, Feb 02, 2022 at 02:21:37PM +0100, Eric Auger wrote: > >>>>>> diff --git a/hw/virtio/virtio-iommu.c b/hw/virtio/virtio-iommu.c > >>>>>> index ec02029bb6..a112428c65 100644 > >>>>>> --- a/hw/virtio/virtio-iommu.c > >>>>>> +++ b/hw/virtio/virtio-iommu.c > >>>>>> @@ -43,6 +43,7 @@ > >>>>>> > >>>>>> typedef struct VirtIOIOMMUDomain { > >>>>>> uint32_t id; > >>>>>> + bool bypass; > >>>>> I am afraid this will break the migration if you don't change > >>>>> vmstate_domain. > >>>>> > >>>>> See static const VMStateDescription vmstate_domain. > >>>>> Also you need to migrate the new bypass field. > >>>>> > >>>>> Logically we should handle this with a vmstate subsection I think to > >>>>> handle migration of older devices. However I doubt the device has been > >>>>> used in production environment supporting migration so my guess is we > >>>>> may skip that burden and just add the missing field. Adding Juan, Dave & > >>>>> Peter for advices. > >>>> I'm not sure about users of this; if no one has used it then yeh; you > >>>> could bump up the version_id to make it a bit clearer. > >>> Thank you for your input. Yes to me it sounds OK to only bump the > >>> version_id while adding the new field. > >> Ok. Just to make sure we're on the same page, this means we don't support > >> migration from new->old or old->new instances, since the migration stream > >> doesn't carry a version ID for the virtio-iommu-device and domain > >> vmstates, as far as I understand. I also believe backward-incompatible > >> changes are fine this time around, though I don't have much visibility in > >> what's being used. > > I think the stream only has it for top level devices; I've not dug into > > this device. > Not sure I get what you meant: > > vmstate_virtio_iommu has a version_id. Also vmstate_domain has one. These version numbers are not sent as part of the stream, unless I missed it. On the incoming side, virtio_load() calls vmstate_load_state() for vmstate_virtio_iommu_device and only looks at the internal version number (dc->vmsd->version_id), it doesn't check version numbers from the stream. So if the sender is old and the receiver is new, the stream doesn't have a bypass field but the receiver will assume it does, and will consider the stream corrupted. Since we're not concerned about compatibility for the moment, I think we could just add the bypass field without bumping the version number, the behavior will be the same. Thanks, Jean ^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 4/4] tests/qtest/virtio-iommu-test: Check bypass config 2022-01-27 14:29 [PATCH v2 0/4] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG Jean-Philippe Brucker ` (2 preceding siblings ...) 2022-01-27 14:29 ` [PATCH v2 3/4] virtio-iommu: Support bypass domain Jean-Philippe Brucker @ 2022-01-27 14:29 ` Jean-Philippe Brucker 2022-01-31 9:14 ` Eric Auger 3 siblings, 1 reply; 20+ messages in thread From: Jean-Philippe Brucker @ 2022-01-27 14:29 UTC (permalink / raw) To: eric.auger Cc: lvivier, Jean-Philippe Brucker, thuth, mst, cohuck, qemu-devel, pasic, pbonzini The bypass config field should be initialized to 1 by default. Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> --- tests/qtest/virtio-iommu-test.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qtest/virtio-iommu-test.c b/tests/qtest/virtio-iommu-test.c index 47e68388a0..068e7a9e6c 100644 --- a/tests/qtest/virtio-iommu-test.c +++ b/tests/qtest/virtio-iommu-test.c @@ -31,11 +31,13 @@ static void pci_config(void *obj, void *data, QGuestAllocator *t_alloc) uint64_t input_range_end = qvirtio_config_readq(dev, 16); uint32_t domain_range_start = qvirtio_config_readl(dev, 24); uint32_t domain_range_end = qvirtio_config_readl(dev, 28); + uint8_t bypass = qvirtio_config_readb(dev, 36); g_assert_cmpint(input_range_start, ==, 0); g_assert_cmphex(input_range_end, ==, UINT64_MAX); g_assert_cmpint(domain_range_start, ==, 0); g_assert_cmpint(domain_range_end, ==, UINT32_MAX); + g_assert_cmpint(bypass, ==, 1); } static int read_tail_status(struct virtio_iommu_req_tail *buffer) -- 2.34.1 ^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH v2 4/4] tests/qtest/virtio-iommu-test: Check bypass config 2022-01-27 14:29 ` [PATCH v2 4/4] tests/qtest/virtio-iommu-test: Check bypass config Jean-Philippe Brucker @ 2022-01-31 9:14 ` Eric Auger 0 siblings, 0 replies; 20+ messages in thread From: Eric Auger @ 2022-01-31 9:14 UTC (permalink / raw) To: Jean-Philippe Brucker Cc: lvivier, thuth, mst, cohuck, qemu-devel, pasic, pbonzini On 1/27/22 3:29 PM, Jean-Philippe Brucker wrote: > The bypass config field should be initialized to 1 by default. > > Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org> Reviewed-by: Eric Auger <eric.auger@redhat.com> Eric > --- > tests/qtest/virtio-iommu-test.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/tests/qtest/virtio-iommu-test.c b/tests/qtest/virtio-iommu-test.c > index 47e68388a0..068e7a9e6c 100644 > --- a/tests/qtest/virtio-iommu-test.c > +++ b/tests/qtest/virtio-iommu-test.c > @@ -31,11 +31,13 @@ static void pci_config(void *obj, void *data, QGuestAllocator *t_alloc) > uint64_t input_range_end = qvirtio_config_readq(dev, 16); > uint32_t domain_range_start = qvirtio_config_readl(dev, 24); > uint32_t domain_range_end = qvirtio_config_readl(dev, 28); > + uint8_t bypass = qvirtio_config_readb(dev, 36); > > g_assert_cmpint(input_range_start, ==, 0); > g_assert_cmphex(input_range_end, ==, UINT64_MAX); > g_assert_cmpint(domain_range_start, ==, 0); > g_assert_cmpint(domain_range_end, ==, UINT32_MAX); > + g_assert_cmpint(bypass, ==, 1); > } > > static int read_tail_status(struct virtio_iommu_req_tail *buffer) ^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2022-02-09 13:22 UTC | newest] Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-01-27 14:29 [PATCH v2 0/4] virtio-iommu: Support VIRTIO_IOMMU_F_BYPASS_CONFIG Jean-Philippe Brucker 2022-01-27 14:29 ` [PATCH v2 1/4] linux-headers: update to v5.17-rc1 Jean-Philippe Brucker 2022-01-31 9:14 ` Eric Auger 2022-01-27 14:29 ` [PATCH v2 2/4] virtio-iommu: Default to bypass during boot Jean-Philippe Brucker 2022-01-31 9:14 ` Eric Auger 2022-02-08 17:42 ` Cornelia Huck 2022-02-08 21:07 ` Michael S. Tsirkin 2022-02-09 11:10 ` Cornelia Huck 2022-02-09 11:32 ` Eric Auger 2022-02-09 12:48 ` Cornelia Huck 2022-01-27 14:29 ` [PATCH v2 3/4] virtio-iommu: Support bypass domain Jean-Philippe Brucker 2022-01-31 9:22 ` Eric Auger 2022-01-31 13:07 ` Dr. David Alan Gilbert 2022-02-02 13:21 ` Eric Auger 2022-02-08 12:30 ` Jean-Philippe Brucker 2022-02-08 13:09 ` Dr. David Alan Gilbert 2022-02-08 13:29 ` Eric Auger 2022-02-08 17:02 ` Jean-Philippe Brucker 2022-01-27 14:29 ` [PATCH v2 4/4] tests/qtest/virtio-iommu-test: Check bypass config Jean-Philippe Brucker 2022-01-31 9:14 ` Eric Auger
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.