linux-kernel - Re: [PATCH v2 0/4] tools/thermal: thermal library and tools

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] [day] [month] [year] [list]

Date:   Tue, 19 Apr 2022 20:16:40 -0700
From:   srinivas pandruvada <srinivas.pandruvada@...ux.intel.com>
To:     Daniel Lezcano <daniel.lezcano@...aro.org>,
        "Rafael J. Wysocki" <rafael@...nel.org>
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux PM <linux-pm@...r.kernel.org>
Subject: Re: [PATCH v2 0/4] tools/thermal: thermal library and tools

Hi Daniel,

On Tue, 2022-04-19 at 22:26 +0200, Daniel Lezcano wrote:
> 
> Hi Srinivas,
> 
> 
> On 18/04/2022 05:36, srinivas pandruvada wrote:
> > Hi Daniel,
> > 
> > The attached diff fixes the crash,
> > 
> > Also when you run in daemon mode, you need to use some pid lock
> > file.
> > Otherwise it launches multiple daemons.
> 
> I've been thinking about it and I don't think it is a problem to have
> multiple instances of the daemon. The netlink allows multicast and
> can 
> be run as non-root user.
> 
> If the finality of the thermal engine is to manage the system and has
> some code making actions on the system with the root privilege, the
> init 
> scripts can take care of the pid lock
> 
If there is use case, then this is fine otherwise wasting system
resources.

Thanks,
Srinivas





> 
>