linux-kernel - Re: [PATCH] Lower the log level of debug outputs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAPBb6MXQ1EXA2uZn0VoLKjh=V_y1Jhx-U-nmpTX9gfKwLagKjg@mail.gmail.com>
Date:   Tue, 30 May 2017 16:39:54 +0900
From:   Alexandre Courbot <acourbot@...omium.org>
To:     Hirokazu Honda <hiroh@...omium.org>
Cc:     Pawel Osciak <pawel@...iak.com>,
        Marek Szyprowski <m.szyprowski@...sung.com>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Lower the log level of debug outputs

On Tue, May 30, 2017 at 3:54 PM, Hirokazu Honda <hiroh@...omium.org> wrote:
> Some debug output whose log level is set 1 flooded the log.
> Their log level is lowered to find the important log easily.
>
> Signed-off-by: Hirokazu Honda <hiroh@...omium.org>

Your patch title should specify the subsystem of your patch. Something
like "[media] vb2: core: Lower the log level of debug outputs"

Otherwise,

Acked-by: Alexandre Courbot <acourbot@...omium.org>


> ---
>  drivers/media/v4l2-core/videobuf2-core.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/media/v4l2-core/videobuf2-core.c b/drivers/media/v4l2-core/videobuf2-core.c
> index 94afbbf92807..25257f92bbcf 100644
> --- a/drivers/media/v4l2-core/videobuf2-core.c
> +++ b/drivers/media/v4l2-core/videobuf2-core.c
> @@ -1139,7 +1139,7 @@ static int __qbuf_dmabuf(struct vb2_buffer *vb, const void *pb)
>                         continue;
>                 }
>
> -               dprintk(1, "buffer for plane %d changed\n", plane);
> +               dprintk(3, "buffer for plane %d changed\n", plane);
>
>                 if (!reacquired) {
>                         reacquired = true;
> @@ -1294,7 +1294,7 @@ int vb2_core_prepare_buf(struct vb2_queue *q, unsigned int index, void *pb)
>         /* Fill buffer information for the userspace */
>         call_void_bufop(q, fill_user_buffer, vb, pb);
>
> -       dprintk(1, "prepare of buffer %d succeeded\n", vb->index);
> +       dprintk(2, "prepare of buffer %d succeeded\n", vb->index);
>
>         return ret;
>  }
> @@ -1424,7 +1424,7 @@ int vb2_core_qbuf(struct vb2_queue *q, unsigned int index, void *pb)
>                         return ret;
>         }
>
> -       dprintk(1, "qbuf of buffer %d succeeded\n", vb->index);
> +       dprintk(2, "qbuf of buffer %d succeeded\n", vb->index);
>         return 0;
>  }
>  EXPORT_SYMBOL_GPL(vb2_core_qbuf);
> @@ -1472,7 +1472,7 @@ static int __vb2_wait_for_done_vb(struct vb2_queue *q, int nonblocking)
>                 }
>
>                 if (nonblocking) {
> -                       dprintk(1, "nonblocking and no buffers to dequeue, will not wait\n");
> +                       dprintk(3, "nonblocking and no buffers to dequeue, will not wait\n");
>                         return -EAGAIN;
>                 }
>
> @@ -1619,7 +1619,7 @@ int vb2_core_dqbuf(struct vb2_queue *q, unsigned int *pindex, void *pb,
>         /* go back to dequeued state */
>         __vb2_dqbuf(vb);
>
> -       dprintk(1, "dqbuf of buffer %d, with state %d\n",
> +       dprintk(2, "dqbuf of buffer %d, with state %d\n",
>                         vb->index, vb->state);
>
>         return 0;
> --
> 2.13.0.219.gdb65acc882-goog
>