Re: [RFC v2 0/2] vireventpoll implimentation using epoll

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



22-Feb-17 17:35, Dmitry Derbyshev пишет:

Ping.
Also, noticed that I renamed patch series, old name was "Substitute poll by epoll in virEventPollRunOnce". My bad.


A few issues required to be fixed to make 'make syntax-check' pass. Otherwise looks good to me.

Maxim

/14/2017 9:04 PM, Derbyshev Dmitriy пишет:
From: Derbyshev Dmitry <dderbyshev@xxxxxxxxxxxxx>

Provides about 20% boost on local machine with 35 vms.
virEventPollDispatchHandles can also be split to pass cb via epoll
data field.

Should start sending as PATCH instead?

Changes since v1:
  * ifdef supstituded by 2 .c files with vireventpollinternal.h
    implementations
  * PROBE purged

Derbyshev Dmitry (2):
   vireventpoll: isolate common code
   vireventpoll implimentation using epoll

  configure.ac                                      |  28 +
  src/Makefile.am                                   |  12 +-
  src/util/vireventepoll.c                          | 201 +++++++
  src/util/vireventpoll.c                           | 700 ++--------------------
  src/util/{vireventpoll.c => vireventpollcommon.c} | 231 ++-----
  src/util/vireventpollinternal.h                   |  91 +++
  tests/commanddata/{test14.log => test3epoll.log}  |   2 +
  tests/commandtest.c                               |   4 +
  8 files changed, 451 insertions(+), 818 deletions(-)
  create mode 100644 src/util/vireventepoll.c
  copy src/util/{vireventpoll.c => vireventpollcommon.c} (78%)
  create mode 100644 src/util/vireventpollinternal.h
  copy tests/commanddata/{test14.log => test3epoll.log} (94%)


--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux