Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-22 Thread Jason Wang
On Thu, Jul 18, 2024 at 2:28 AM Steven Sistare wrote: > > On 7/16/2024 1:16 AM, Jason Wang wrote: > > On Mon, Jul 15, 2024 at 10:27 PM Steven Sistare > > wrote: > >> > >> On 7/14/2024 10:26 PM, Jason Wang wrote: > >>> On Fri, Jul 12, 2024 at 9:19 PM Steve Sistare > >>> wrote: > > Add

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-17 Thread Steven Sistare
On 7/16/2024 1:16 AM, Jason Wang wrote: On Mon, Jul 15, 2024 at 10:27 PM Steven Sistare wrote: On 7/14/2024 10:26 PM, Jason Wang wrote: On Fri, Jul 12, 2024 at 9:19 PM Steve Sistare wrote: Add an ioctl to transfer file descriptor ownership and pinned memory accounting from one process to a

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-15 Thread Jason Wang
On Mon, Jul 15, 2024 at 10:27 PM Steven Sistare wrote: > > On 7/14/2024 10:26 PM, Jason Wang wrote: > > On Fri, Jul 12, 2024 at 9:19 PM Steve Sistare > > wrote: > >> > >> Add an ioctl to transfer file descriptor ownership and pinned memory > >> accounting from one process to another. > >> > >> T

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-15 Thread Steven Sistare
On 7/15/2024 10:38 AM, Michael S. Tsirkin wrote: On Mon, Jul 15, 2024 at 10:29:26AM -0400, Steven Sistare wrote: On 7/15/2024 5:07 AM, Michael S. Tsirkin wrote: On Fri, Jul 12, 2024 at 06:18:49AM -0700, Steve Sistare wrote: Add an ioctl to transfer file descriptor ownership and pinned memory a

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-15 Thread Michael S. Tsirkin
On Mon, Jul 15, 2024 at 10:29:26AM -0400, Steven Sistare wrote: > On 7/15/2024 5:07 AM, Michael S. Tsirkin wrote: > > On Fri, Jul 12, 2024 at 06:18:49AM -0700, Steve Sistare wrote: > > > Add an ioctl to transfer file descriptor ownership and pinned memory > > > accounting from one process to anothe

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-15 Thread Steven Sistare
On 7/15/2024 5:07 AM, Michael S. Tsirkin wrote: On Fri, Jul 12, 2024 at 06:18:49AM -0700, Steve Sistare wrote: Add an ioctl to transfer file descriptor ownership and pinned memory accounting from one process to another. This is more efficient than VHOST_RESET_OWNER followed by VHOST_SET_OWNER,

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-15 Thread Steven Sistare
On 7/14/2024 10:26 PM, Jason Wang wrote: On Fri, Jul 12, 2024 at 9:19 PM Steve Sistare wrote: Add an ioctl to transfer file descriptor ownership and pinned memory accounting from one process to another. This is more efficient than VHOST_RESET_OWNER followed by VHOST_SET_OWNER, as that would u

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-15 Thread Michael S. Tsirkin
On Fri, Jul 12, 2024 at 06:18:49AM -0700, Steve Sistare wrote: > Add an ioctl to transfer file descriptor ownership and pinned memory > accounting from one process to another. > > This is more efficient than VHOST_RESET_OWNER followed by VHOST_SET_OWNER, > as that would unpin all physical pages, r

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-15 Thread Michael S. Tsirkin
On Mon, Jul 15, 2024 at 10:26:13AM +0800, Jason Wang wrote: > On Fri, Jul 12, 2024 at 9:19 PM Steve Sistare > wrote: > > > > Add an ioctl to transfer file descriptor ownership and pinned memory > > accounting from one process to another. > > > > This is more efficient than VHOST_RESET_OWNER follo

Re: [PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-14 Thread Jason Wang
On Fri, Jul 12, 2024 at 9:19 PM Steve Sistare wrote: > > Add an ioctl to transfer file descriptor ownership and pinned memory > accounting from one process to another. > > This is more efficient than VHOST_RESET_OWNER followed by VHOST_SET_OWNER, > as that would unpin all physical pages, requiring

[PATCH V2 3/7] vhost-vdpa: VHOST_NEW_OWNER

2024-07-12 Thread Steve Sistare
Add an ioctl to transfer file descriptor ownership and pinned memory accounting from one process to another. This is more efficient than VHOST_RESET_OWNER followed by VHOST_SET_OWNER, as that would unpin all physical pages, requiring them to be repinned in the new process. That would cost multipl