[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <548d30a8-0a5a-4ada-5564-b61f88863afc@rasmusvillemoes.dk>
Date: Thu, 11 Jun 2020 13:26:05 +0200
From: Rasmus Villemoes <linux@...musvillemoes.dk>
To: jim.cromie@...il.com,
Stanimir Varbanov <stanimir.varbanov@...aro.org>,
Joe Perches <joe@...ches.com>
Cc: LKML <linux-kernel@...r.kernel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Jason Baron <jbaron@...mai.com>,
Randy Dunlap <rdunlap@...radead.org>
Subject: Re: WIP generic module->debug_flags and dynamic_debug
On 10/06/2020 20.32, jim.cromie@...il.com wrote:
> so Ive got a WIP / broken / partial approach to giving all modules a
> u32 flagset,
> and enabling pr_debug based upon it. I leave out the "pr_debug_typed(
> bitpos )" for now. For Stanimir, bits 1,2,3 could be high, middle,
> low.
>
> ATM its broken on my lack of container_of() skills.
>
> Im trying to use it to get a struct module* using its name value thats
> been copied
> into a ddebug_table member.
>
> Im relying on
> cdf6d006968 dynamic_debug: don't duplicate modname in ddebug_add_module
> to have the same value in both structs
>
> but Im clearly missing a few things
> besides the likely future trouble with .rodata builtin modules
> (after compile prob solved)
>
> It seems container_of wants me to use struct ddebug_table instead,
> but I dont want a *ddebug_table.
> Ive blindly guessed at adding & and * to 1st param, w/o understanding.
>
> can anyone diagnose my problem ?
Sorry, I have not the faintest idea of what you're trying to achieve.
Can you spell that out?
Rasmus
Powered by blists - more mailing lists