Re: [PATCH v2 07/32] perf/x86/intel/cqm: add helpers for per-package locking

2016-05-18 Thread Thomas Gleixner
On Wed, 18 May 2016, Thomas Gleixner wrote: > On Wed, 11 May 2016, David Carrillo-Cisneros wrote: > > + cqm_pkg_id_for_each_online(i) > > + mutex_lock_nested(_pkgs_data[i]->pkg_data_mutex, i); Peter just pointed out that this will fail when the number of nest levels exceeds 8. So any

Re: [PATCH v2 07/32] perf/x86/intel/cqm: add helpers for per-package locking

2016-05-18 Thread Thomas Gleixner
On Wed, 18 May 2016, Thomas Gleixner wrote: > On Wed, 11 May 2016, David Carrillo-Cisneros wrote: > > + cqm_pkg_id_for_each_online(i) > > + mutex_lock_nested(_pkgs_data[i]->pkg_data_mutex, i); Peter just pointed out that this will fail when the number of nest levels exceeds 8. So any

Re: [PATCH v2 07/32] perf/x86/intel/cqm: add helpers for per-package locking

2016-05-18 Thread Thomas Gleixner
On Wed, 11 May 2016, David Carrillo-Cisneros wrote: > Add helper macros and functions to acquire and release the locks > and mutexes in pkg_data. Why? Whatfor do we need nested locks? > Reviewed-by: Stephane Eranian > Signed-off-by: David Carrillo-Cisneros

Re: [PATCH v2 07/32] perf/x86/intel/cqm: add helpers for per-package locking

2016-05-18 Thread Thomas Gleixner
On Wed, 11 May 2016, David Carrillo-Cisneros wrote: > Add helper macros and functions to acquire and release the locks > and mutexes in pkg_data. Why? Whatfor do we need nested locks? > Reviewed-by: Stephane Eranian > Signed-off-by: David Carrillo-Cisneros > --- > arch/x86/events/intel/cqm.h

[PATCH v2 07/32] perf/x86/intel/cqm: add helpers for per-package locking

2016-05-11 Thread David Carrillo-Cisneros
Add helper macros and functions to acquire and release the locks and mutexes in pkg_data. Reviewed-by: Stephane Eranian Signed-off-by: David Carrillo-Cisneros --- arch/x86/events/intel/cqm.h | 78 + 1 file

[PATCH v2 07/32] perf/x86/intel/cqm: add helpers for per-package locking

2016-05-11 Thread David Carrillo-Cisneros
Add helper macros and functions to acquire and release the locks and mutexes in pkg_data. Reviewed-by: Stephane Eranian Signed-off-by: David Carrillo-Cisneros --- arch/x86/events/intel/cqm.h | 78 + 1 file changed, 78 insertions(+) diff --git