search for: vhost_cross_endian_legacy

...there are two redeeming properties that > > made me merge this: > > > > 1. It's a trivial amount of code: since we wrap host/guest accesses > > anyway, almost all of it is well hidden from drivers. > > > > 2. Sane platforms would never set flags like VHOST_CROSS_ENDIAN_LEGACY - > > and when it's clear, there's zero overhead (as some point it was > > tested by compiling with and without the patches, got the same > > stripped binary). > > > > Maybe we could create a Kconfig symbol to enforce point (2): prevent > > p...

...there are two redeeming properties that > > made me merge this: > > > > 1. It's a trivial amount of code: since we wrap host/guest accesses > > anyway, almost all of it is well hidden from drivers. > > > > 2. Sane platforms would never set flags like VHOST_CROSS_ENDIAN_LEGACY - > > and when it's clear, there's zero overhead (as some point it was > > tested by compiling with and without the patches, got the same > > stripped binary). > > > > Maybe we could create a Kconfig symbol to enforce point (2): prevent > > p...

...s is exactly the motivation for this. I dislike it too, but there are two redeeming properties that made me merge this: 1. It's a trivial amount of code: since we wrap host/guest accesses anyway, almost all of it is well hidden from drivers. 2. Sane platforms would never set flags like VHOST_CROSS_ENDIAN_LEGACY - and when it's clear, there's zero overhead (as some point it was tested by compiling with and without the patches, got the same stripped binary). Maybe we could create a Kconfig symbol to enforce point (2): prevent people from enabling it e.g. on x86. I will look into this -...

...s is exactly the motivation for this. I dislike it too, but there are two redeeming properties that made me merge this: 1. It's a trivial amount of code: since we wrap host/guest accesses anyway, almost all of it is well hidden from drivers. 2. Sane platforms would never set flags like VHOST_CROSS_ENDIAN_LEGACY - and when it's clear, there's zero overhead (as some point it was tested by compiling with and without the patches, got the same stripped binary). Maybe we could create a Kconfig symbol to enforce point (2): prevent people from enabling it e.g. on x86. I will look into this -...

...made me merge this: > > > > > > > > 1. It's a trivial amount of code: since we wrap host/guest accesses > > > > anyway, almost all of it is well hidden from drivers. > > > > > > > > 2. Sane platforms would never set flags like VHOST_CROSS_ENDIAN_LEGACY - > > > > and when it's clear, there's zero overhead (as some point it was > > > > tested by compiling with and without the patches, got the same > > > > stripped binary). > > > > > > > > Maybe we could create a Kconfi...

...made me merge this: > > > > > > > > 1. It's a trivial amount of code: since we wrap host/guest accesses > > > > anyway, almost all of it is well hidden from drivers. > > > > > > > > 2. Sane platforms would never set flags like VHOST_CROSS_ENDIAN_LEGACY - > > > > and when it's clear, there's zero overhead (as some point it was > > > > tested by compiling with and without the patches, got the same > > > > stripped binary). > > > > > > > > Maybe we could create a Kconfi...

...rties that > > > made me merge this: > > > > > > 1. It's a trivial amount of code: since we wrap host/guest accesses > > > anyway, almost all of it is well hidden from drivers. > > > > > > 2. Sane platforms would never set flags like VHOST_CROSS_ENDIAN_LEGACY - > > > and when it's clear, there's zero overhead (as some point it was > > > tested by compiling with and without the patches, got the same > > > stripped binary). > > > > > > Maybe we could create a Kconfig symbol to enforce point...

...ost/Kconfig > +++ b/drivers/vhost/Kconfig > @@ -36,6 +36,7 @@ config VHOST_VSOCK > > config VHOST > tristate > + select VHOST_IOTLB > ---help--- > This option is selected by any driver which needs to access > the core of vhost. > @@ -54,3 +55,9 @@ config VHOST_CROSS_ENDIAN_LEGACY > adds some overhead, it is disabled by default. > > If unsure, say "N". > + > +config VHOST_IOTLB > + tristate > + default m "default m" should not be needed. Just make whatever needs it select it. > + help > + Generic IOTLB implementati...

...config VHOST_VSOCK > To compile this driver as a module, choose M here: the module will be called > vhost_vsock. > > -config VHOST > - tristate > - ---help--- > - This option is selected by any driver which needs to access > - the core of vhost. > - > config VHOST_CROSS_ENDIAN_LEGACY > bool "Cross-endian support for vhost" > default n > @@ -54,3 +58,4 @@ config VHOST_CROSS_ENDIAN_LEGACY > adds some overhead, it is disabled by default. > > If unsure, say "N". > +endif > diff --git a/drivers/vhost/Kconfig.vringh b/drivers/...

...config VHOST_VSOCK > To compile this driver as a module, choose M here: the module will be called > vhost_vsock. > > -config VHOST > - tristate > - ---help--- > - This option is selected by any driver which needs to access > - the core of vhost. > - > config VHOST_CROSS_ENDIAN_LEGACY > bool "Cross-endian support for vhost" > default n > @@ -54,3 +58,4 @@ config VHOST_CROSS_ENDIAN_LEGACY > adds some overhead, it is disabled by default. > > If unsure, say "N". > +endif > diff --git a/drivers/vhost/Kconfig.vringh b/drivers/...

...sed to override the default when activating the ring of a legacy > device. It has no effect on modern devices. > > Signed-off-by: Greg Kurz <gkurz at linux.vnet.ibm.com> > --- > > Changes since v4: > - rewrote patch title to mention cross-endian > - renamed config to VHOST_CROSS_ENDIAN_LEGACY > - rewrote config description and help > - moved ifdefery to top of vhost.c > - added a detailed comment about the lifecycle of vq->user_be in > vhost_init_is_le() > - renamed ioctls to VHOST_[GS]ET_VRING_ENDIAN > - added LE/BE defines to the ioctl API > - rewrote ioctl s...

...sed to override the default when activating the ring of a legacy > device. It has no effect on modern devices. > > Signed-off-by: Greg Kurz <gkurz at linux.vnet.ibm.com> > --- > > Changes since v4: > - rewrote patch title to mention cross-endian > - renamed config to VHOST_CROSS_ENDIAN_LEGACY > - rewrote config description and help > - moved ifdefery to top of vhost.c > - added a detailed comment about the lifecycle of vq->user_be in > vhost_init_is_le() > - renamed ioctls to VHOST_[GS]ET_VRING_ENDIAN > - added LE/BE defines to the ioctl API > - rewrote ioctl s...

...vers in a > little-endian kernel to parse vnet headers that come from a > diff --git a/drivers/vhost/Kconfig b/drivers/vhost/Kconfig > index 533eaf0..4d8ae6b 100644 > --- a/drivers/vhost/Kconfig > +++ b/drivers/vhost/Kconfig > @@ -35,6 +35,7 @@ config VHOST > > config VHOST_CROSS_ENDIAN_LEGACY > bool "Cross-endian support for vhost" > + depends on KVM_CROSS_ENDIAN_GUESTS > default n > ---help--- > This option allows vhost to support guests with a different byte > diff --git a/virt/kvm/Kconfig b/virt/kvm/Kconfig > index e2c876d..cc7b28a 100644 >...

...vers in a > little-endian kernel to parse vnet headers that come from a > diff --git a/drivers/vhost/Kconfig b/drivers/vhost/Kconfig > index 533eaf0..4d8ae6b 100644 > --- a/drivers/vhost/Kconfig > +++ b/drivers/vhost/Kconfig > @@ -35,6 +35,7 @@ config VHOST > > config VHOST_CROSS_ENDIAN_LEGACY > bool "Cross-endian support for vhost" > + depends on KVM_CROSS_ENDIAN_GUESTS > default n > ---help--- > This option allows vhost to support guests with a different byte > diff --git a/virt/kvm/Kconfig b/virt/kvm/Kconfig > index e2c876d..cc7b28a 100644 >...

...: the module will be called > > > vhost_vsock. > > > -config VHOST > > > - tristate > > > - ---help--- > > > - This option is selected by any driver which needs to access > > > - the core of vhost. > > > - > > > config VHOST_CROSS_ENDIAN_LEGACY > > > bool "Cross-endian support for vhost" > > > default n > > > @@ -54,3 +58,4 @@ config VHOST_CROSS_ENDIAN_LEGACY > > > adds some overhead, it is disabled by default. > > > If unsure, say "N". > > > +endif...

...: the module will be called > > > vhost_vsock. > > > -config VHOST > > > - tristate > > > - ---help--- > > > - This option is selected by any driver which needs to access > > > - the core of vhost. > > > - > > > config VHOST_CROSS_ENDIAN_LEGACY > > > bool "Cross-endian support for vhost" > > > default n > > > @@ -54,3 +58,4 @@ config VHOST_CROSS_ENDIAN_LEGACY > > > adds some overhead, it is disabled by default. > > > If unsure, say "N". > > > +endif...

...ing the ring of a legacy > > device. It has no effect on modern devices. > > > > Signed-off-by: Greg Kurz <gkurz at linux.vnet.ibm.com> > > --- > > > > Changes since v4: > > - rewrote patch title to mention cross-endian > > - renamed config to VHOST_CROSS_ENDIAN_LEGACY > > - rewrote config description and help > > - moved ifdefery to top of vhost.c > > - added a detailed comment about the lifecycle of vq->user_be in > > vhost_init_is_le() > > - renamed ioctls to VHOST_[GS]ET_VRING_ENDIAN > > - added LE/BE defines to the io...

...t a specific endianness. This field is used to override the default when activating the ring of a legacy device. It has no effect on modern devices. Signed-off-by: Greg Kurz <gkurz at linux.vnet.ibm.com> --- Changes since v4: - rewrote patch title to mention cross-endian - renamed config to VHOST_CROSS_ENDIAN_LEGACY - rewrote config description and help - moved ifdefery to top of vhost.c - added a detailed comment about the lifecycle of vq->user_be in vhost_init_is_le() - renamed ioctls to VHOST_[GS]ET_VRING_ENDIAN - added LE/BE defines to the ioctl API - rewrote ioctl sanity check with the LE/BE defines...

...tristate "Vhost driver for vDPA-based backend" > depends on EVENTFD > select VDPA > + select VHOST > help > This kernel module can be loaded in host kernel to accelerate > guest virtio devices with the vDPA-based backends. > @@ -78,5 +80,3 @@ config VHOST_CROSS_ENDIAN_LEGACY > adds some overhead, it is disabled by default. > > If unsure, say "N". > - > -endif > > > But now CONFIG_VHOST is always "y", never "m". > Which I think will make it a built-in. > Didn't figure out why yet. Is it becau...

...tristate "Vhost driver for vDPA-based backend" > depends on EVENTFD > select VDPA > + select VHOST > help > This kernel module can be loaded in host kernel to accelerate > guest virtio devices with the vDPA-based backends. > @@ -78,5 +80,3 @@ config VHOST_CROSS_ENDIAN_LEGACY > adds some overhead, it is disabled by default. > > If unsure, say "N". > - > -endif > > > But now CONFIG_VHOST is always "y", never "m". > Which I think will make it a built-in. > Didn't figure out why yet. Is it becau...