From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ozlabs.org (ozlabs.org [103.22.144.67]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id C9CD41A030F for ; Thu, 28 Jan 2016 19:25:30 +1100 (AEDT) Received: from mail-pf0-x234.google.com (mail-pf0-x234.google.com [IPv6:2607:f8b0:400e:c00::234]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 4EF05140A98 for ; Thu, 28 Jan 2016 19:25:29 +1100 (AEDT) Received: by mail-pf0-x234.google.com with SMTP id x125so20150789pfb.0 for ; Thu, 28 Jan 2016 00:25:29 -0800 (PST) Date: Thu, 28 Jan 2016 13:55:25 +0530 From: Viresh Kumar To: Shilpasri G Bhat Cc: linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org, rjw@rjwysocki.net, linux-pm@vger.kernel.org, pc@us.ibm.com, anton@samba.org, ego@linux.vnet.ibm.com, shreyas@linux.vnet.ibm.com, bsingharora@gmail.com, mpe@ellerman.id.au Subject: Re: [PATCH v7 0/6] cpufreq: powernv: Redesign the presentation of throttle notification and solve bug-fixes in the driver Message-ID: <20160128082525.GG3935@vireshk> References: <1453965941-7363-1-git-send-email-shilpa.bhat@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1453965941-7363-1-git-send-email-shilpa.bhat@linux.vnet.ibm.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 28-01-16, 12:55, Shilpasri G Bhat wrote: > In POWER8, OCC(On-Chip-Controller) can throttle the frequency of the > CPU when the chip crosses its thermal and power limits. Currently, > powernv-cpufreq driver detects and reports this event as a console > message. Some machines may not sustain the max turbo frequency in all > conditions and can be throttled frequently. This can lead to the > flooding of console with throttle messages. So this patchset aims to > redesign the presentation of this event via sysfs counters and > tracepoints. And it also fixes couple of bugs reported in the driver. > > - Patch [1] fixes a memory leak bug > - Patch [2] fixes the cpu hot-plug bug in powernv_cpufreq_work_fn(). > - Patch [3] solves a bug in powernv_cpufreq_throttle_check(), which > calls in to cpu_to_chip_id() in hot path which reads DT every time > to find the chip id. > - Patches [4] to [6] will add a perf trace point > "power:powernv_throttle" and sysfs throttle counter stats in > /sys/devices/system/cpu/cpufreq/chipN. > > Changes from v6: > - Changes wrt comments from Balbir Singh and Viresh Kumar. Who cares about these names in version-log ?? You have completely missed what should have been present here. This is version log and that's what should be present here :) And because of that, I have to - search for your earlier version in my mailbox - Read all my comments - Haven't read what Balbir have said See .. -- viresh