Re: [Xen-devel] [PATCH RFC 1/4] tmem: Move global stats in a tmem_statistics structure

2016-05-18 Thread Doug Goldstein
On 5/18/16 9:34 AM, Konrad Rzeszutek Wilk wrote: > On Tue, May 17, 2016 at 09:01:00PM -0500, Doug Goldstein wrote: >> On 5/17/16 8:57 PM, Doug Goldstein wrote: >>> On 5/16/16 10:58 AM, Konrad Rzeszutek Wilk wrote: And adjust the macro: atomic_inc_and_max to update the structure. Sadl

Re: [Xen-devel] [PATCH RFC 1/4] tmem: Move global stats in a tmem_statistics structure

2016-05-18 Thread Konrad Rzeszutek Wilk
On Tue, May 17, 2016 at 09:01:00PM -0500, Doug Goldstein wrote: > On 5/17/16 8:57 PM, Doug Goldstein wrote: > > On 5/16/16 10:58 AM, Konrad Rzeszutek Wilk wrote: > >> And adjust the macro: atomic_inc_and_max to update the structure. > >> > >> Sadly one entry: pool->pgp_count cannot use this macro a

Re: [Xen-devel] [PATCH RFC 1/4] tmem: Move global stats in a tmem_statistics structure

2016-05-17 Thread Doug Goldstein
On 5/17/16 8:57 PM, Doug Goldstein wrote: > On 5/16/16 10:58 AM, Konrad Rzeszutek Wilk wrote: >> And adjust the macro: atomic_inc_and_max to update the structure. >> >> Sadly one entry: pool->pgp_count cannot use this macro anymore >> so unroll the macro for this instance. >> >> No functional chang

Re: [Xen-devel] [PATCH RFC 1/4] tmem: Move global stats in a tmem_statistics structure

2016-05-17 Thread Doug Goldstein
On 5/16/16 10:58 AM, Konrad Rzeszutek Wilk wrote: > And adjust the macro: atomic_inc_and_max to update the structure. > > Sadly one entry: pool->pgp_count cannot use this macro anymore > so unroll the macro for this instance. > > No functional change. The name has the 'tmem_stats' as it will > be

[Xen-devel] [PATCH RFC 1/4] tmem: Move global stats in a tmem_statistics structure

2016-05-16 Thread Konrad Rzeszutek Wilk
And adjust the macro: atomic_inc_and_max to update the structure. Sadly one entry: pool->pgp_count cannot use this macro anymore so unroll the macro for this instance. No functional change. The name has the 'tmem_stats' as it will be eventually non-local. Signed-off-by: Konrad Rzeszutek Wilk --