Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-26 Thread Stefan Hajnoczi
On Wed, Feb 26, 2025 at 10:53:01AM +0100, David Hildenbrand wrote: > > > As commented offline, maybe one would want the option to enable the > > > alternative mode, where such updates (in the SHM region) are not sent to > > > vhost-user devices. In such a configuration, the MEM_READ / MEM_WRITE > >

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-26 Thread David Hildenbrand
As commented offline, maybe one would want the option to enable the alternative mode, where such updates (in the SHM region) are not sent to vhost-user devices. In such a configuration, the MEM_READ / MEM_WRITE would be unavoidable. At first, I remember we discussed two options, having update me

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-24 Thread Albert Esteve
On Mon, Feb 24, 2025 at 2:57 PM David Hildenbrand wrote: > > On 24.02.25 14:41, Albert Esteve wrote: > > On Mon, Feb 24, 2025 at 10:49 AM David Hildenbrand wrote: > >> > >> On 24.02.25 10:35, Albert Esteve wrote: > >>> On Mon, Feb 24, 2025 at 10:16 AM David Hildenbrand > >>> wrote: > > >>>

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-24 Thread David Hildenbrand
On 24.02.25 14:41, Albert Esteve wrote: On Mon, Feb 24, 2025 at 10:49 AM David Hildenbrand wrote: On 24.02.25 10:35, Albert Esteve wrote: On Mon, Feb 24, 2025 at 10:16 AM David Hildenbrand wrote: On 24.02.25 09:54, Albert Esteve wrote: On Mon, Feb 17, 2025 at 9:01 PM David Hildenbrand wr

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-24 Thread Albert Esteve
On Mon, Feb 24, 2025 at 10:49 AM David Hildenbrand wrote: > > On 24.02.25 10:35, Albert Esteve wrote: > > On Mon, Feb 24, 2025 at 10:16 AM David Hildenbrand wrote: > >> > >> On 24.02.25 09:54, Albert Esteve wrote: > >>> On Mon, Feb 17, 2025 at 9:01 PM David Hildenbrand > >>> wrote: > > >>>

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-24 Thread David Hildenbrand
On 24.02.25 10:35, Albert Esteve wrote: On Mon, Feb 24, 2025 at 10:16 AM David Hildenbrand wrote: On 24.02.25 09:54, Albert Esteve wrote: On Mon, Feb 17, 2025 at 9:01 PM David Hildenbrand wrote: On 17.02.25 17:40, Albert Esteve wrote: Hi all, Hi, looks like our debugging session was s

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-24 Thread Albert Esteve
On Mon, Feb 24, 2025 at 10:16 AM David Hildenbrand wrote: > > On 24.02.25 09:54, Albert Esteve wrote: > > On Mon, Feb 17, 2025 at 9:01 PM David Hildenbrand wrote: > >> > >> On 17.02.25 17:40, Albert Esteve wrote: > >>> Hi all, > >>> > >> > >> Hi, > >> > >> looks like our debugging session was suc

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-24 Thread David Hildenbrand
On 24.02.25 09:54, Albert Esteve wrote: On Mon, Feb 17, 2025 at 9:01 PM David Hildenbrand wrote: On 17.02.25 17:40, Albert Esteve wrote: Hi all, Hi, looks like our debugging session was successfu :) One question below. v3->v4 - Change mmap strategy to use RAM blocks and subregions.

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-24 Thread Albert Esteve
On Mon, Feb 17, 2025 at 9:01 PM David Hildenbrand wrote: > > On 17.02.25 17:40, Albert Esteve wrote: > > Hi all, > > > > Hi, > > looks like our debugging session was successfu :) > > One question below. > > > v3->v4 > > - Change mmap strategy to use RAM blocks > >and subregions. > > - Add new

Re: [PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-17 Thread David Hildenbrand
On 17.02.25 17:40, Albert Esteve wrote: Hi all, Hi, looks like our debugging session was successfu :) One question below. v3->v4 - Change mmap strategy to use RAM blocks and subregions. - Add new bitfield to qmp feature map - Followed most review comments from last iteration. - Merge

[PATCH v4 0/9] vhost-user: Add SHMEM_MAP/UNMAP requests

2025-02-17 Thread Albert Esteve
Hi all, v3->v4 - Change mmap strategy to use RAM blocks and subregions. - Add new bitfield to qmp feature map - Followed most review comments from last iteration. - Merged documentation patch again with this one. Makes more sense to review them together after all. - Add documentation for M