[PATCH v2 5/5] powerpc/configs: Disable SCOM_DEBUGFS in powernv_defconfig
ajd at linux.ibm.com
Wed Jul 31 11:45:50 AEST 2019
On 9/5/19 3:54 pm, Andrew Donnellan wrote:
> On 9/5/19 3:37 pm, Nicholas Piggin wrote:
>> Andrew Donnellan's on May 9, 2019 3:11 pm:
>>> SCOM_DEBUGFS is really not needed for anything other than low-level
>>> hardware debugging.
>>> opal-prd uses its own interface (/dev/prd) for SCOM access, so it
>>> need SCOM_DEBUGFS.
>>> At some point in the future we'll introduce a debug config fragment
>>> this can go instead.
>> That doesn't really explain why you want to disable it. It is useful
>> for low level hardware debugging, I added it.
> Mostly just a general feeling that it's not something we need to have by
> default. Security-wise, PRD still provides SCOM access, though we are
> going to look at how we can further lock that down. Shrinks the build by
> only a few kilobytes...
> mpe said he's planning on adding a debug.config where we can shift stuff
> like this, and if/when we do that I would like to see this moved there,
> but perhaps this patch can wait until then. I'll let mpe decide.
mpe do you have thoughts on this? I would like to see at least the rest
of this series merged.
>>> Signed-off-by: Andrew Donnellan <ajd at linux.ibm.com>
>>> - new patch
>>> arch/powerpc/configs/powernv_defconfig | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>> diff --git a/arch/powerpc/configs/powernv_defconfig
>>> index ef2ef98d3f28..d5a6608cb2e0 100644
>>> --- a/arch/powerpc/configs/powernv_defconfig
>>> +++ b/arch/powerpc/configs/powernv_defconfig
>>> @@ -38,7 +38,7 @@ CONFIG_MODULE_UNLOAD=y
>>> +# CONFIG_SCOM_DEBUGFS is not set
>>> # CONFIG_PPC_PSERIES is not set
Andrew Donnellan OzLabs, ADL Canberra
ajd at linux.ibm.com IBM Australia Limited
More information about the Linuxppc-dev