On Mon, Sep 10, 2018 at 01:46:13PM +0300, Alexey Skidanov wrote:
> 
> 
> On 09/10/2018 11:27 AM, Greg KH wrote:
> > On Sun, Sep 09, 2018 at 01:44:31AM +0300, Alexey Skidanov wrote:
> >> The heap statistics have been removed and currently even basics statistics
> >> are missing.
> >>
> >> This patch creates per heap debugfs directory /sys/kernel/debug/<heap_name>
> >> and adds two counters - the number of allocated buffers and number of
> >> allocated bytes.
> > 
> > Why?  What can we do with this information?
> Are you in doubt that we need these two specific counters or debugfs at all?

These specific counters.  What are you going to do with them?

And you also moved debugfs files (you didn't comment on that comment),
which will probably break tools that expect the files to be in one
place, right?  How did you test this patch?

thanks,

greg k-h
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to