Re: [PATCH 1/3] arm64: dump: Make ptdump debugfs a separate option

From: Laura Abbott
Date: Thu Sep 29 2016 - 21:11:57 EST


On 09/29/2016 05:48 PM, Mark Rutland wrote:
On Thu, Sep 29, 2016 at 05:31:09PM -0700, Laura Abbott wrote:
On 09/29/2016 05:13 PM, Mark Rutland wrote:
On Thu, Sep 29, 2016 at 02:32:55PM -0700, Laura Abbott wrote:
+int ptdump_register(struct ptdump_info *info, const char *name)
+{
+ ptdump_initialize(info);
+ return ptdump_debugfs_create(info, name);
}

It feels like a layering violation to have the core ptdump code call the
debugfs ptdump code. Is there some reason this has to live here?

Which 'this' are you referring to here? Are you suggesting moving
the ptdump_register elsewhere or moving the debugfs create elsewhere?

Sorry, I should have worded that better.

I meant moving ptdump_register into ptdump_debugfs.c, perhaps renamed to make it
clear it's debugfs-specific.

We could instead update existing users to call ptdump_debugfs_create()
directly, and have that call ptdump_initialize(), which could itself become a
staic inline in a header.

Ah okay, I see what you are suggesting. ptdump_initialize should still
happen regardless of debugfs status though so I guess ptdump_debugfs_create would just get turned into just ptdump_initialize
which seems a little unclear. I'll come up with some other shed colors^W^Wfunction names.

Thanks,
Laura