Re: [PATCH v2 1/2] vhost: configure all host notifiers in a single MR transaction

2022-12-20 Thread Michael S. Tsirkin
On Wed, Dec 07, 2022 at 08:22:18AM +0800, Longpeng (Mike, Cloud Infrastructure Service Product Dept.) wrote: > > > And we must commit before invoking virtio_bus_cleanup_host_notifier. > > > > ... but with that info on top, finally your original patch is simpler. > > Yes, I'll try in next

Re: [PATCH v2 1/2] vhost: configure all host notifiers in a single MR transaction

2022-12-06 Thread longpeng2--- via
在 2022/12/6 18:45, Philippe Mathieu-Daudé 写道: On 6/12/22 11:28, Longpeng (Mike, Cloud Infrastructure Service Product Dept.) wrote: 在 2022/12/6 17:07, Philippe Mathieu-Daudé 写道: On 6/12/22 09:18, Longpeng(Mike) via wrote: From: Longpeng This allows the vhost device to batch the setup of

Re: [PATCH v2 1/2] vhost: configure all host notifiers in a single MR transaction

2022-12-06 Thread Philippe Mathieu-Daudé
On 6/12/22 11:28, Longpeng (Mike, Cloud Infrastructure Service Product Dept.) wrote: 在 2022/12/6 17:07, Philippe Mathieu-Daudé 写道: On 6/12/22 09:18, Longpeng(Mike) via wrote: From: Longpeng This allows the vhost device to batch the setup of all its host notifiers. This significantly

Re: [PATCH v2 1/2] vhost: configure all host notifiers in a single MR transaction

2022-12-06 Thread longpeng2--- via
在 2022/12/6 17:07, Philippe Mathieu-Daudé 写道: On 6/12/22 09:18, Longpeng(Mike) via wrote: From: Longpeng This allows the vhost device to batch the setup of all its host notifiers. This significantly reduces the device starting time, e.g. the time spend on enabling notifiers reduce from

Re: [PATCH v2 1/2] vhost: configure all host notifiers in a single MR transaction

2022-12-06 Thread Philippe Mathieu-Daudé
On 6/12/22 09:18, Longpeng(Mike) via wrote: From: Longpeng This allows the vhost device to batch the setup of all its host notifiers. This significantly reduces the device starting time, e.g. the time spend on enabling notifiers reduce from 376ms to 9.1ms for a VM with 64 vCPUs and 3

[PATCH v2 1/2] vhost: configure all host notifiers in a single MR transaction

2022-12-06 Thread Longpeng(Mike)
From: Longpeng This allows the vhost device to batch the setup of all its host notifiers. This significantly reduces the device starting time, e.g. the time spend on enabling notifiers reduce from 376ms to 9.1ms for a VM with 64 vCPUs and 3 vhost-vDPA generic devices[1] (64vq per device) [1]