Re: [PATCH v2] Tmem bug-fixes and cleanups.

[Andrew Cooper <andrew.cooper3@citrix.com>]

On 27/08/15 12:01, Konrad Rzeszutek Wilk wrote:
> Hey!
>
> At the Xenhackathon we spoke that the tmem code needs a bit of cleanups
> and simplification. One of the things that Andrew mentioned was that the
> TMEM_CONTROL should really be part of the sysctl hypercall. As I ventured
> this path I realized there were some other issues that need to be taken
> care of (like shared pools blowing up).
>
> This patchset has been tested with 32/64 guests, with a 64 hypervisor
> and 32 bit toolstack (and also with 64bit toolstack) with success.
>
> For fun I've also created an Linux module:
> http://xenbits.xen.org/gitweb/?p=xentesttools/bootstrap.git;a=blob;f=root_image/drivers/tmem_test/tmem_test.c
> that I will expand to cover in the future more interesting hypercall
> uses.
>
> Going forward the next step will be to move the 'tmem_control' function to
> its own file to simplify the code.
>
> The patches are also in my git tree:

Patches 1-4, 6-8 are definitely a net improvement, so Reviewed-by:
Andrew Cooper <andrew.cooper3@citrix.com>, with the proviso that I am
not knowlegable about tmem internals.

~Andrew