Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <180b1a4fb9cf78c8905f275e0373e009224d4639.camel@linux.intel.com>
Subject: Re: BUG: bisected: thermald regression (MEMLEAK) in commit
 c7ff29763989bd09c433f73fae3c1e1c15d9cda4
From:   srinivas pandruvada <srinivas.pandruvada@linux.intel.com>
To:     Mirsad Goran Todorovac <mirsad.todorovac@alu.unizg.hr>,
        LKML <linux-kernel@vger.kernel.org>, rjw@rjwysocki.net
Cc:     regressions@lists.linux.dev, regressions@leemhuis.info,
        "linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
        Robert Moore <robert.moore@intel.com>, devel@acpica.org
Date:   Wed, 26 Oct 2022 15:48:35 -0700
In-Reply-To: <6c2f47c8-d6b5-074a-4c8f-e9cd04f62ef4@alu.unizg.hr>
References: <e0f06714-5a49-a4e6-24e6-c4103c820819@alu.unizg.hr>
         <9ef3674afd370050b86a68e44c97e4f0257f1adf.camel@linux.intel.com>
         <bd1f0d2a-d456-92cc-ecca-23e480aea4b1@alu.unizg.hr>
         <e5d3d561bb3a9c68bc903cfc35c27629a4a9225c.camel@linux.intel.com>
         <d034dbbc-613c-1a5e-df64-d0251453c8eb@alu.unizg.hr>
         <6c2f47c8-d6b5-074a-4c8f-e9cd04f62ef4@alu.unizg.hr>
Content-Type: text/plain; charset="UTF-8"
User-Agent: Evolution 3.42.4 (3.42.4-2.fc35) 
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

On Wed, 2022-10-26 at 19:52 +0200, Mirsad Goran Todorovac wrote:
> Dear all,
> 
> On 24. 10. 2022. 20:56, Mirsad Goran Todorovac wrote:
> > On 24. 10. 2022. 20:39, srinivas pandruvada wrote:
> > 
> > > > Thank you for the patch. Unfortunately, when applied to v6.0.3
> > > > it
> > > > didn't
> > > > fix the issue.
> > > Thanks for the test. I copied to acpi and acpica mailing list.
> > > Someone
> > > can tell us what is this call doing wrong here. 
> 
> I have worse news: after every
> 
> # systemctl stop thermald
> # systemctl start thermald
> 
> the number of leaks increases by one allocated block (apparently 80 
> bytes). The effect appears to be
> cummulative.
> 
> Please find the results of the MEMLEAK scan in the attachment.
> 
> In theory, motivated adversary could theoretically exhaust  i.e. 8
> GiB 
> in a loop of 10 million thermald stops/starts,
Of course it needs to be debugged. To start/stop systemctl service you
need root access. If you have root access, there are other worse things
can be done.

Thanks,
Srinivas

> on my laptop and 2 sec for stop+start, it would be approx. 230 days.
> 
> Hope this helps.
> 
> Mirsad
> 
> -- 
> 
> Mirsad Goran Todorovac
> Sistem inženjer
> Grafički fakultet | Akademija likovnih umjetnosti
> Sveučilište u Zagrebu