Re: [PATCH] irqdomain: remove debugfs_file from struct irq_domain

Marc Zyngier
2021-02-17T20:03:10Z

On 2021-02-17 19:57, Michael Walle wrote:
> Hi Greg,
>
>> There's no need to keep around a dentry pointer to a simple file that
>> debugfs itself can look up when we need to remove it from the system.
>> So simplify the code by deleting the variable and cleaning up the
>> logic
>> around the debugfs file.
>
> This will generate the following oops on my board (arm64,
> freescale/fsl-ls1028a-kontron-sl28-var3-ads2.dts). In debugfs_lookup()
> debugfs_mount is NULL.

That's odd. I gave it a go yesterday, and nothing blew up.
Which makes me wonder whether I had the debug stuff enabled
the first place...

I've dropped the patch from -next for now until I figure it out
(probably tomorrow).

Thanks,

M.
--
Jazz is not dead. It just smells funny...