[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b04f3fdb-143b-ab34-b6e2-ac9fbb802ed0@broadcom.com>
Date: Tue, 17 Nov 2020 13:36:02 -0800
From: Scott Branden <scott.branden@...adcom.com>
To: Florian Fainelli <f.fainelli@...il.com>,
Arnd Bergmann <arnd@...db.de>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Desmond Yan <desmond.yan@...adcom.com>
Cc: Kees Cook <keescook@...omium.org>, linux-kernel@...r.kernel.org,
bcm-kernel-feedback-list@...adcom.com,
Olof Johansson <olof@...om.net>
Subject: Re: [PATCH v6 11/14] misc: bcm-vk: add BCM_VK_QSTATS
Hi Florian,
On 2020-10-02 6:39 p.m., Florian Fainelli wrote:
>
>
> On 10/2/2020 2:23 PM, Scott Branden wrote:
>> Add BCM_VK_QSTATS Kconfig option to allow for enabling debug VK
>> queue statistics.
>>
>> These statistics keep track of max, abs_max, and average for the
>> messages queues.
>>
>> Co-developed-by: Desmond Yan <desmond.yan@...adcom.com>
>> Signed-off-by: Desmond Yan <desmond.yan@...adcom.com>
>> Signed-off-by: Scott Branden <scott.branden@...adcom.com>
>
> would not it make more sense to have those debug prints be trace printks instead? Given what you explained in the previous patch version and the desire to correlate with other system wide activity, that might make more sense. Looking at the kernel's log for debugging performance or utilization or just to get a glimpse of what is going on is not quite suited past probe.
The debug prints have served our purpose up to this point.
But, in the interest of getting the other driver patches accepted I will drop this patch from the series and we will investigate further.
Thanks,
Scott
Download attachment "smime.p7s" of type "application/pkcs7-signature" (4169 bytes)
Powered by blists - more mailing lists