lkml.org 
[lkml]   [2021]   [Nov]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v10 04/10] i915/gvt: trim spaces from pr_debug "gvt: core:" prefixes
    Date
    Taking embedded spaces out of existing prefixes makes them more easily
    searchable; simplifying the extra quoting needed otherwise:

    $> echo format "^gvt: core:" +p >control

    Dropping the internal spaces means any trailing space in a query will
    more clearly terminate the prefix being searched for.

    Consider a generic drm-debug example:

    # turn off ATOMIC reports
    echo format "^drm:atomic: " -p > control

    # turn off all ATOMIC:* reports, including any sub-categories
    echo format "^drm:atomic:" -p > control

    # turn on ATOMIC:FAIL: reports
    echo format "^drm:atomic:fail: " +p > control

    Removing embedded spaces in the format prefixes simplifies the
    corresponding match string. This means that "quoted" match-prefixes
    are only needed when the trailing space is desired, in order to
    exclude explicitly sub-categorized pr-debugs; in this example,
    "drm:atomic:fail:".

    Signed-off-by: Jim Cromie <jim.cromie@gmail.com>
    ---
    drivers/gpu/drm/i915/gvt/debug.h | 18 +++++++++---------
    1 file changed, 9 insertions(+), 9 deletions(-)

    diff --git a/drivers/gpu/drm/i915/gvt/debug.h b/drivers/gpu/drm/i915/gvt/debug.h
    index c6027125c1ec..bbecc279e077 100644
    --- a/drivers/gpu/drm/i915/gvt/debug.h
    +++ b/drivers/gpu/drm/i915/gvt/debug.h
    @@ -36,30 +36,30 @@ do { \
    } while (0)

    #define gvt_dbg_core(fmt, args...) \
    - pr_debug("gvt: core: "fmt, ##args)
    + pr_debug("gvt:core: " fmt, ##args)

    #define gvt_dbg_irq(fmt, args...) \
    - pr_debug("gvt: irq: "fmt, ##args)
    + pr_debug("gvt:irq: " fmt, ##args)

    #define gvt_dbg_mm(fmt, args...) \
    - pr_debug("gvt: mm: "fmt, ##args)
    + pr_debug("gvt:mm: " fmt, ##args)

    #define gvt_dbg_mmio(fmt, args...) \
    - pr_debug("gvt: mmio: "fmt, ##args)
    + pr_debug("gvt:mmio: " fmt, ##args)

    #define gvt_dbg_dpy(fmt, args...) \
    - pr_debug("gvt: dpy: "fmt, ##args)
    + pr_debug("gvt:dpy: " fmt, ##args)

    #define gvt_dbg_el(fmt, args...) \
    - pr_debug("gvt: el: "fmt, ##args)
    + pr_debug("gvt:el: " fmt, ##args)

    #define gvt_dbg_sched(fmt, args...) \
    - pr_debug("gvt: sched: "fmt, ##args)
    + pr_debug("gvt:sched: " fmt, ##args)

    #define gvt_dbg_render(fmt, args...) \
    - pr_debug("gvt: render: "fmt, ##args)
    + pr_debug("gvt:render: " fmt, ##args)

    #define gvt_dbg_cmd(fmt, args...) \
    - pr_debug("gvt: cmd: "fmt, ##args)
    + pr_debug("gvt:cmd: " fmt, ##args)

    #endif
    --
    2.31.1
    \
     
     \ /
      Last update: 2021-11-11 23:03    [W:3.275 / U:0.372 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site