RE: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-11-11 Thread Srujana Challa
> Subject: RE: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO- > IOMMU mode > > > Subject: Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for > > NO- IOMMU mode > > > > On Wed, Nov 06, 2024 at 12: 38: 02PM +, Srujana Challa wrote: >

RE: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-11-06 Thread Srujana Challa
> Subject: Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO- > IOMMU mode > > On Wed, Nov 06, 2024 at 12: 38: 02PM +, Srujana Challa wrote: > It is > going in circles, let me give the summary, > Issue: We need to address the > lack > of no-IOMM

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-11-06 Thread Michael S. Tsirkin
On Thu, Oct 24, 2024 at 02:05:43AM -0700, Christoph Hellwig wrote: > On Wed, Oct 23, 2024 at 04:19:02AM -0400, Michael S. Tsirkin wrote: > > On Tue, Oct 22, 2024 at 11:58:19PM -0700, Christoph Hellwig wrote: > > > On Sat, Oct 19, 2024 at 08:16:44PM -0400, Michael S. Tsirkin wrote: > > > > Because p

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-11-06 Thread Michael S. Tsirkin
On Wed, Nov 06, 2024 at 12:38:02PM +, Srujana Challa wrote: > > Subject: Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO- > > IOMMU mode > > > > On Wed, Oct 23, 2024 at 04: 19: 02AM -0400, Michael S. Tsirkin wrote: > On > > Tue, Oct 22, 2024

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-11-06 Thread Christoph Hellwig
On Wed, Nov 06, 2024 at 12:38:02PM +, Srujana Challa wrote: > It is going in circles, let me give the summary, > Issue: We need to address the lack of no-IOMMU support in the vhost vDPA > driver for better performance. > Measured Performance: On the machine "13th Gen Intel(R) Core(TM) i9-13900

RE: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-11-06 Thread Srujana Challa
> Subject: Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO- > IOMMU mode > > On Wed, Oct 23, 2024 at 04: 19: 02AM -0400, Michael S. Tsirkin wrote: > On > Tue, Oct 22, 2024 at 11: 58: 19PM -0700, Christoph Hellwig wrote: > > On Sat, > Oct 19, 2024 at 08

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-24 Thread Christoph Hellwig
On Wed, Oct 23, 2024 at 04:19:02AM -0400, Michael S. Tsirkin wrote: > On Tue, Oct 22, 2024 at 11:58:19PM -0700, Christoph Hellwig wrote: > > On Sat, Oct 19, 2024 at 08:16:44PM -0400, Michael S. Tsirkin wrote: > > > Because people want to move from some vendor specific solution with vfio > > > to a

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-23 Thread Michael S. Tsirkin
On Tue, Oct 22, 2024 at 11:58:19PM -0700, Christoph Hellwig wrote: > On Sat, Oct 19, 2024 at 08:16:44PM -0400, Michael S. Tsirkin wrote: > > Because people want to move from some vendor specific solution with vfio > > to a standard vdpa compatible one with vdpa. > > So now you have a want for new

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-23 Thread Christoph Hellwig
On Sat, Oct 19, 2024 at 08:16:44PM -0400, Michael S. Tsirkin wrote: > Because people want to move from some vendor specific solution with vfio > to a standard vdpa compatible one with vdpa. So now you have a want for new use cases and you turn that into a must for supporting completely insecure an

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-19 Thread Michael S. Tsirkin
On Wed, Oct 16, 2024 at 11:16:08PM -0700, Christoph Hellwig wrote: > On Wed, Oct 16, 2024 at 01:41:51PM -0400, Michael S. Tsirkin wrote: > > It's basically because vfio does, so we have to follow suit. > > That's a very bold argument, especially without any rationale of > > a) why you need to ma

RE: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-18 Thread Srujana Challa
> Subject: Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO- > IOMMU mode > > On Thu, Oct 17, 2024 at 08: 53: 08AM +, Srujana Challa wrote: > We > observed better performance with "intel_iommu=on" in high-end x86 > machines, > indicating th

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-17 Thread Christoph Hellwig
On Thu, Oct 17, 2024 at 08:53:08AM +, Srujana Challa wrote: > We observed better performance with "intel_iommu=on" in high-end x86 machines, > indicating that the performance limitations are specific to low-end x86 > hardware. What does "low-end" vs "high-end" mean? Atom vs other cores? > T

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-17 Thread Jason Wang
On Thu, Oct 17, 2024 at 4:53 PM Srujana Challa wrote: > > > Subject: Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO- > > IOMMU mode > > > > On Wed, Oct 16, 2024 at 05:28:27PM +, Srujana Challa wrote: > > > When using the DPDK virtio user

RE: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-17 Thread Srujana Challa
> Subject: Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO- > IOMMU mode > > On Wed, Oct 16, 2024 at 05:28:27PM +, Srujana Challa wrote: > > When using the DPDK virtio user PMD, we’ve noticed a significant 70% > > performance improvement when IOMMU is

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-16 Thread Christoph Hellwig
On Wed, Oct 16, 2024 at 05:28:27PM +, Srujana Challa wrote: > When using the DPDK virtio user PMD, we’ve noticed a significant 70% > performance improvement when IOMMU is disabled on specific low-end > x86 machines. This performance improvement can be particularly > advantageous for embedded pl

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-16 Thread Christoph Hellwig
On Wed, Oct 16, 2024 at 01:41:51PM -0400, Michael S. Tsirkin wrote: > It's basically because vfio does, so we have to follow suit. That's a very bold argument, especially without any rationale of a) why you need to match the feature set b) how even adding it to vfio was agood idea

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-16 Thread Michael S. Tsirkin
On Mon, Oct 14, 2024 at 08:48:27PM -0700, Christoph Hellwig wrote: > On Mon, Oct 14, 2024 at 01:18:01PM +, Srujana Challa wrote: > > > On Fri, Sep 20, 2024 at 07:35:28PM +0530, Srujana Challa wrote: > > > > This patchset introduces support for an UNSAFE, no-IOMMU mode in the > > > > vhost-vdpa

RE: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-16 Thread Srujana Challa
> Subject: Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO- > IOMMU mode > > On Mon, Oct 14, 2024 at 01: 18: 01PM +, Srujana Challa wrote: > > On Fri, > Sep 20, 2024 at 07: 35: 28PM +0530, Srujana Challa wrote: > > > This patchset > introduce

Re: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-14 Thread Christoph Hellwig
On Mon, Oct 14, 2024 at 01:18:01PM +, Srujana Challa wrote: > > On Fri, Sep 20, 2024 at 07:35:28PM +0530, Srujana Challa wrote: > > > This patchset introduces support for an UNSAFE, no-IOMMU mode in the > > > vhost-vdpa driver. When enabled, this mode provides no device > > > isolation, no DMA

RE: [EXTERNAL] Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-14 Thread Srujana Challa
> On Fri, Sep 20, 2024 at 07:35:28PM +0530, Srujana Challa wrote: > > This patchset introduces support for an UNSAFE, no-IOMMU mode in the > > vhost-vdpa driver. When enabled, this mode provides no device > > isolation, no DMA translation, no host kernel protection, and cannot > > be used for devic

Re: [PATCH v2 0/2] vhost-vdpa: Add support for NO-IOMMU mode

2024-10-01 Thread Christoph Hellwig
On Fri, Sep 20, 2024 at 07:35:28PM +0530, Srujana Challa wrote: > This patchset introduces support for an UNSAFE, no-IOMMU mode in the > vhost-vdpa driver. When enabled, this mode provides no device isolation, > no DMA translation, no host kernel protection, and cannot be used for > device assignme