Re: [PATCH] dma-debug: Kconfig for PREALLOC_DMA_DEBUG_ENTRIES
From: Christoph Hellwig
Date: Sat Dec 01 2018 - 11:37:07 EST
On Fri, Nov 30, 2018 at 07:39:50PM +0000, Robin Murphy wrote:
> I was assuming the point was to also add something like
>
> default 131072 if HNS_ENET
>
> so that DMA debug doesn't require too much thought from the user. If they
> still have to notice the overflow message and empirically figure out a
> value that does work, rebuilding the kernel each time is far less
> convenient than simply adding "dma_debug_entries=..." to their kernel
> command line and rebooting, which they can do today. If they do already
> know up-front that the default will need overriding and what the
> appropriate value is, then the command line still seems seems just as
> convenient.
I'm not so fond of random drivers changing the defaults. My idea
was rather to have the config option so that the defconfig files for
the Hisilicon SOCs with this hardware could select a larger number
without making a total mess of the kernel configuration.
If we really have to we could do different defaults, but I'd still
much rather do this on a arch/platform basis than specific drivers.