lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Wed, 19 Jul 2017 19:08:25 +0530
From:   Rajneesh Bhardwaj <rajneesh.bhardwaj@...el.com>
To:     "Chakravarty, Souvik K" <souvik.k.chakravarty@...el.com>
Cc:     "platform-driver-x86@...r.kernel.org" 
        <platform-driver-x86@...r.kernel.org>,
        "dvhart@...radead.org" <dvhart@...radead.org>,
        "andy@...radead.org" <andy@...radead.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        "Murthy, Shanth" <shanth.murthy@...el.com>
Subject: Re: [PATCH 3/3] platform/x86: Add Audio domain PG status events

On Mon, Jul 17, 2017 at 10:48:30AM +0530, Chakravarty, Souvik K wrote:
> 
> 
> > -----Original Message-----
> > From: Bhardwaj, Rajneesh
> > Sent: Friday, July 14, 2017 5:24 PM
> > To: platform-driver-x86@...r.kernel.org
> > Cc: dvhart@...radead.org; andy@...radead.org; linux-
> > kernel@...r.kernel.org; Murthy, Shanth <shanth.murthy@...el.com>;
> > Chakravarty, Souvik K <souvik.k.chakravarty@...el.com>; Bhardwaj,
> > Rajneesh <rajneesh.bhardwaj@...el.com>
> > Subject: [PATCH 3/3] platform/x86: Add Audio domain PG status events
> > 
> > From: "Murthy, Shanth" <shanth.murthy@...el.com>
> > 
> > This patch adds events to ioss telemetry to read the power gating status for
> > the audio domain.
> > 
> > Signed-off-by: Shanth Murthy <shanth.murthy@...el.com>
> > Signed-off-by: Rajneesh Bhardwaj <rajneesh.bhardwaj@...el.com>
> > ---
> >  arch/x86/include/asm/intel_telemetry.h        | 2 +-
> >  drivers/platform/x86/intel_telemetry_pltdrv.c | 6 ++++++
> >  2 files changed, 7 insertions(+), 1 deletion(-)
> > 
> > diff --git a/arch/x86/include/asm/intel_telemetry.h
> > b/arch/x86/include/asm/intel_telemetry.h
> > index 85029b58d0cd..4eeae0a4f9a2 100644
> > --- a/arch/x86/include/asm/intel_telemetry.h
> > +++ b/arch/x86/include/asm/intel_telemetry.h
> > @@ -17,7 +17,7 @@
> >  #define INTEL_TELEMETRY_H
> > 
> >  #define TELEM_MAX_EVENTS_SRAM		28
> > -#define TELEM_MAX_OS_ALLOCATED_EVENTS	20
> > +#define TELEM_MAX_OS_ALLOCATED_EVENTS	25
> 
> This is something that should not be done without testing. There was an understanding to not use more than 20 counters in OS, so that tools like SoCWatch could use the rest 8 without total reconfiguration of the events. With this change we are not leaving much for tools and it may break SoCwatch/IMON or other tools dependent on using just 8. 
> The better way for this would be for the userspace tool to configure the extra 5 events via add_config() call and not increase the OS usage ceiling.
> 
> Or you could add some more sysfs apis for get/set events and do it via a daemon/script.
> 
> <snip>....

Thanks for the review and the feedback Souvik. Ideally user space tools
should not put a restriction such as this one. It would be good to document
such details.

Is there a plan to support more than 20 events in telemetry driver? If not
then we can probably think of removing few events and add audio related
events maintaining total count of 20?

For now i will respin the patches and drop this one. 

-- 
Best Regards,
Rajneesh

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ