linux-kernel - Re: [PATCH v2] tools/power turbostat: Fix RAPL summary collection on AMD processors

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]

Message-ID: <4080ac25f6c8ca4088a950eb9d63da641c020941.camel@kepstin.ca>
Date:   Fri, 23 Apr 2021 10:04:10 -0400
From:   Calvin Walton <calvin.walton@...stin.ca>
To:     Chen Yu <yu.c.chen@...el.com>
Cc:     Borislav Petkov <bp@...e.de>, Terry Bowman <terry.bowman@....com>,
        lenb@...nel.org, linux-pm@...r.kernel.org,
        linux-kernel@...r.kernel.org, wei.huang2@....com, aros@....com,
        rui.zhang@...el.com
Subject: Re: [PATCH v2] tools/power turbostat: Fix RAPL summary collection
 on AMD processors

On Fri, 2021-04-23 at 20:16 +0800, Chen Yu wrote
> 
> Calvin, could you please take a look at the following version if it
> is suitible?

I assume this patch is adapted from my earlier submission of the
complete fix to be an incremental fix on top of the already applied
patch? If so, you are OK to add

Signed-off-by: Calvin Walton <calvin.walton@...stin.ca>

> From b2e63fe4f02e17289414b4f61237da822df115fb Mon Sep 17 00:00:00
> 2001
> From: Calvin Walton <calvin.walton@...stin.ca>
> Date: Fri, 23 Apr 2021 17:32:13 +0800
> Subject: [PATCH 3/5] tools/power turbostat: Fix offset overflow issue
> in index
>  converting
> 
> The idx_to_offset() function returns type int (32-bit signed), but
> MSR_PKG_ENERGY_STAT is greater than INT_MAX (or rather, would be
> interpreted as a negative number). The end result is that it hits
> the if (offset < 0) check in update_msr_sum() resulting in the timer
> callback for updating the stat in the background when long durations
> are used to not happen. The similar issue exists in offset_to_idx()
> and update_msr_sum().
> 
> This patch fixes this issue by converting the 'int' type to 'off_t'
> accordingly.

This patch covers all of the places I know about which had type issues
- it should be good. Thanks!

-- 
Calvin Walton <calvin.walton@...stin.ca>