[PATCH 0/2] Rework RPC message buffer

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

 



This patch set tries to fix corner cases where libvirt runs on huge
system, e.g. 4K CPU monster. In these cases, capabilities XML is
enormously big, as we are transferring info about each singe CPU core
(to which NUMA node it belongs, etc.). This XML is bigger than our
RPC limit, therefore users cannot get it as it is dropped on server,
leaving them with inability to connect. Therefore we need to increase
those limits (whole RPC message and RPC string). However, simple
lifting up will work, but increase mem usage.

Therefore I've reworked RPC buffer handling: changed it from
'statically' to dynamically allocated.
So in most cases - when small messages are sent - this will even
decrease our memory consumption. Leaving us flexible for corner cases
described above.

On the other hand, I realize we've had our history with RPC breakage.
So I think I'll require more than 1 ACK before pushing.

What's your opinion?

Michal Privoznik (2):
  rpc: Switch to dynamically allocated message buffer
  rpc: Size up RPC limits

 src/remote/remote_protocol.x |    2 +-
 src/rpc/virnetclient.c       |   16 ++-
 src/rpc/virnetmessage.c      |   12 ++-
 src/rpc/virnetmessage.h      |    6 +-
 src/rpc/virnetprotocol.x     |    6 +-
 src/rpc/virnetserverclient.c |   20 ++-
 tests/virnetmessagetest.c    |  393 +++++++++++++++++++++++-------------------
 7 files changed, 268 insertions(+), 187 deletions(-)

-- 
1.7.8.5

--
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]