Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v5 00/18] dynamic_debug fixes, cleanups, features, export
To:     Jim Cromie <jim.cromie@gmail.com>, linux-kernel@vger.kernel.org,
        akpm@linuxfoundation.org, gregkh@linuxfoundation.org
Cc:     linux@rasmusvillemoes.dk
References: <20200719231058.1586423-1-jim.cromie@gmail.com>
From:   Jason Baron <jbaron@akamai.com>
Message-ID: <4d7244b1-fa4d-f1bd-42e4-7123116e708f@akamai.com>
Date:   Fri, 24 Jul 2020 09:48:54 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
MIME-Version: 1.0
In-Reply-To: <20200719231058.1586423-1-jim.cromie@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


On 7/19/20 7:10 PM, Jim Cromie wrote:
> this is v5, changes from previous:
>  - moved a chunk from patch 13 to 12, per Jason
>  - shorten logging prefix to "dyndbg", drop __func__
>  - now with more commit-log advocacy
>  - shuffle EXPORT_GPL(ddebug_exec_queries) last.
>  - v4+ series Acked-by: jbaron@akamai.com

Yes, series looks good to me:
Acked-by: <jbaron@akamai.com>

Greg, what do you think about pulling in this series?

Thanks,

-Jason

> 
> v4: https://urldefense.proofpoint.com/v2/url?u=https-3A__lore.kernel.org_lkml_20200620180643.887546-2D1-2Djim.cromie-40gmail.com_&d=DwIDAg&c=96ZbZZcaMF4w0F4jpN6LZg&r=1fLh1mlLqbfetnnGsbwXfpwmGlG4m83mXgtV4vZ1B1A&m=Fx_Nr_SBO_RxVITjMD8U8davIy4lmiMx9iQ0RiHncbw&s=Yk6jd9kp4ASId49a5PYXxFrYDDDw8eD595yeyH8JOrU&e= 
> v3: https://urldefense.proofpoint.com/v2/url?u=https-3A__lore.kernel.org_lkml_20200617162536.611386-2D1-2Djim.cromie-40gmail.com_&d=DwIDAg&c=96ZbZZcaMF4w0F4jpN6LZg&r=1fLh1mlLqbfetnnGsbwXfpwmGlG4m83mXgtV4vZ1B1A&m=Fx_Nr_SBO_RxVITjMD8U8davIy4lmiMx9iQ0RiHncbw&s=fwZUOLHfC4rKPBAanu69vs_StJyGBUag2iMQm7l9k5s&e= 
> v2: https://urldefense.proofpoint.com/v2/url?u=https-3A__lore.kernel.org_lkml_20200613155738.2249399-2D1-2Djim.cromie-40gmail.com_&d=DwIDAg&c=96ZbZZcaMF4w0F4jpN6LZg&r=1fLh1mlLqbfetnnGsbwXfpwmGlG4m83mXgtV4vZ1B1A&m=Fx_Nr_SBO_RxVITjMD8U8davIy4lmiMx9iQ0RiHncbw&s=Xwb7nTFjQ7rmEnEQXARt_J5EV8Jy2lRMq4PiKKBfWeE&e= 
> v1: https://urldefense.proofpoint.com/v2/url?u=https-3A__lore.kernel.org_lkml_20200605162645.289174-2D1-2Djim.cromie-40gmail.com_&d=DwIDAg&c=96ZbZZcaMF4w0F4jpN6LZg&r=1fLh1mlLqbfetnnGsbwXfpwmGlG4m83mXgtV4vZ1B1A&m=Fx_Nr_SBO_RxVITjMD8U8davIy4lmiMx9iQ0RiHncbw&s=09pJ7MRxLOyDjzN2yYYEOrqP0HtPJZPWLL9rVhKj1EI&e= 
> 
> 
> Patchset starts with cleanups;
>  - change section name from vague "__verbose" to specific "__dyndbg"
>  - cleaner docs, drop obsolete comment & useless debug prints,
>  - refine verbose/debug logging
>  - fix a BUG_ON, ram reporting miscounts. etc..
> 
> Then adds query parsing conveniences
>  - allow "file inode.c:100-200"		# combined file & line-range
>  - allow "file inode.c:start_*"		# file & function
>  - accept "module=foo" query form
> 
> internal improvement
>  - combine flags & mask in a struct, clean 3 func interfaces with it.
>  
> make precise format queries easier
>  - accept "format=^ClassString" anchored query
> 
> finally, EXPORT_GPL(ddebug_exec_queries)
> 
> This gives module authors complete run-time control over all their
> *pr_debug* callsites (when CONFIG_DYNAMIC_DEBUG=y).
> 
> Following the drm.debug UI model, drm.debug_chan2 could be wired to a
> callback which invokes ddebug_exec_queries to toggle arbitary groups
> of pr_debug callsites.
> 
> Useful callsite groups would exploit existing message-prefix
> classifcation schemes:
> 
>   "format=^[IF_TRACE]: +p; format=^[SURFACE]: +p" >control
> 
> 
> Jim Cromie (18):
>   dyndbg-docs: eschew file /full/path query in docs
>   dyndbg-docs: initialization is done early, not arch
>   dyndbg: drop obsolete comment on ddebug_proc_open
>   dyndbg: refine debug verbosity; 1 is basic, 2 more chatty
>   dyndbg: rename __verbose section to __dyndbg
>   dyndbg: fix overcounting of ram used by dyndbg
>   dyndbg: fix a BUG_ON in ddebug_describe_flags
>   dyndbg: fix pr_err with empty string
>   dyndbg: prefer declarative init in caller, to memset in callee
>   dyndbg: make ddebug_tables list LIFO for add/remove_module
>   dyndbg: use gcc ?: to reduce word count
>   dyndbg: refactor parse_linerange out of ddebug_parse_query
>   dyndbg: accept 'file foo.c:func1' and 'file foo.c:10-100'
>   dyndbg: accept query terms like file=bar and module=foo
>   dyndbg: combine flags & mask into a struct, simplify with it
>   dyndbg: allow anchored match on format query term
>   dyndbg: shorten our logging prefix, drop __func__
>   dyndbg: export ddebug_exec_queries
> 
>  .../admin-guide/dynamic-debug-howto.rst       |  29 +-
>  include/asm-generic/vmlinux.lds.h             |   6 +-
>  include/linux/dynamic_debug.h                 |   4 +-
>  kernel/module.c                               |   2 +-
>  lib/dynamic_debug.c                           | 269 ++++++++++--------
>  5 files changed, 173 insertions(+), 137 deletions(-)
>