On Mon, Jul 22, 2019 at 11:14:34AM +0200, Stefano Garzarella wrote: > On Mon, Jul 22, 2019 at 10:08:35AM +0100, Stefan Hajnoczi wrote: > > On Wed, Jul 17, 2019 at 01:30:25PM +0200, Stefano Garzarella wrote: > > > This series tries to increase the throughput of virtio-vsock with slight > > > changes. > > > While I was testing the v2 of this series I discovered an huge use of > > > memory, > > > so I added patch 1 to mitigate this issue. I put it in this series in > > > order > > > to better track the performance trends. > > > > > > v4: > > > - rebased all patches on current master (conflicts is Patch 4) > > > - Patch 1: added Stefan's R-b > > > - Patch 3: removed lock when buf_alloc is written [David]; > > > moved this patch after "vsock/virtio: reduce credit update > > > messages" > > > to make it clearer > > > - Patch 4: vhost_exceeds_weight() is recently introduced, so I've solved > > > some > > > conflicts > > > > Stefano: Do you want to continue experimenting before we merge this > > patch series? The code looks functionally correct and the performance > > increases, so I'm happy for it to be merged. > > I think we can merge this series. > > I'll continue to do other experiments (e.g. removing TX workers, allocating > pages, etc.) but I think these changes are prerequisites for the other > patches, > so we can merge them. > > Thank you very much for the reviews!
All patches have been reviewed by here. Have an Ack for good measure: Acked-by: Stefan Hajnoczi <[email protected]> The topics discussed in sub-threads relate to longer-term optimization work that doesn't block this series. Please merge.
signature.asc
Description: PGP signature
