| 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
| ||
|
Message-ID: <CACT4Y+bn1xR6YS-+mLsoneMriYEJRz37D-NDwp_M=4f3u0f_Zw@mail.gmail.com>
Date: Thu, 4 Jun 2020 14:05:47 +0200
From: Dmitry Vyukov <dvyukov@...gle.com>
To: Andrey Konovalov <andreyknvl@...gle.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Alan Stern <stern@...land.harvard.edu>,
Andrew Morton <akpm@...ux-foundation.org>,
USB list <linux-usb@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>,
Alexander Potapenko <glider@...gle.com>,
Marco Elver <elver@...gle.com>
Subject: Re: [PATCH v4 1/7] kcov: cleanup debug messages
On Thu, Mar 26, 2020 at 3:44 PM Andrey Konovalov <andreyknvl@...gle.com> wrote:
>
> Previous commit left a lot of excessive debug messages, clean them up.
>
> Signed-off-by: Andrey Konovalov <andreyknvl@...gle.com>
Reviewed-by: Dmitry Vyukov <dvyukov@...gle.com>
> ---
> kernel/kcov.c | 22 ++--------------------
> 1 file changed, 2 insertions(+), 20 deletions(-)
>
> diff --git a/kernel/kcov.c b/kernel/kcov.c
> index f50354202dbe..f6bd119c9419 100644
> --- a/kernel/kcov.c
> +++ b/kernel/kcov.c
> @@ -98,6 +98,7 @@ static struct kcov_remote *kcov_remote_find(u64 handle)
> return NULL;
> }
>
> +/* Must be called with kcov_remote_lock locked. */
> static struct kcov_remote *kcov_remote_add(struct kcov *kcov, u64 handle)
> {
> struct kcov_remote *remote;
> @@ -119,16 +120,13 @@ static struct kcov_remote_area *kcov_remote_area_get(unsigned int size)
> struct kcov_remote_area *area;
> struct list_head *pos;
>
> - kcov_debug("size = %u\n", size);
> list_for_each(pos, &kcov_remote_areas) {
> area = list_entry(pos, struct kcov_remote_area, list);
> if (area->size == size) {
> list_del(&area->list);
> - kcov_debug("rv = %px\n", area);
> return area;
> }
> }
> - kcov_debug("rv = NULL\n");
> return NULL;
> }
>
> @@ -136,7 +134,6 @@ static struct kcov_remote_area *kcov_remote_area_get(unsigned int size)
> static void kcov_remote_area_put(struct kcov_remote_area *area,
> unsigned int size)
> {
> - kcov_debug("area = %px, size = %u\n", area, size);
> INIT_LIST_HEAD(&area->list);
> area->size = size;
> list_add(&area->list, &kcov_remote_areas);
> @@ -366,7 +363,6 @@ static void kcov_remote_reset(struct kcov *kcov)
> hash_for_each_safe(kcov_remote_map, bkt, tmp, remote, hnode) {
> if (remote->kcov != kcov)
> continue;
> - kcov_debug("removing handle %llx\n", remote->handle);
> hash_del(&remote->hnode);
> kfree(remote);
> }
> @@ -553,7 +549,6 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
>
> switch (cmd) {
> case KCOV_INIT_TRACE:
> - kcov_debug("KCOV_INIT_TRACE\n");
> /*
> * Enable kcov in trace mode and setup buffer size.
> * Must happen before anything else.
> @@ -572,7 +567,6 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
> kcov->mode = KCOV_MODE_INIT;
> return 0;
> case KCOV_ENABLE:
> - kcov_debug("KCOV_ENABLE\n");
> /*
> * Enable coverage for the current task.
> * At this point user must have been enabled trace mode,
> @@ -598,7 +592,6 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
> kcov_get(kcov);
> return 0;
> case KCOV_DISABLE:
> - kcov_debug("KCOV_DISABLE\n");
> /* Disable coverage for the current task. */
> unused = arg;
> if (unused != 0 || current->kcov != kcov)
> @@ -610,7 +603,6 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
> kcov_put(kcov);
> return 0;
> case KCOV_REMOTE_ENABLE:
> - kcov_debug("KCOV_REMOTE_ENABLE\n");
> if (kcov->mode != KCOV_MODE_INIT || !kcov->area)
> return -EINVAL;
> t = current;
> @@ -629,7 +621,6 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
> kcov->remote_size = remote_arg->area_size;
> spin_lock(&kcov_remote_lock);
> for (i = 0; i < remote_arg->num_handles; i++) {
> - kcov_debug("handle %llx\n", remote_arg->handles[i]);
> if (!kcov_check_handle(remote_arg->handles[i],
> false, true, false)) {
> spin_unlock(&kcov_remote_lock);
> @@ -644,8 +635,6 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
> }
> }
> if (remote_arg->common_handle) {
> - kcov_debug("common handle %llx\n",
> - remote_arg->common_handle);
> if (!kcov_check_handle(remote_arg->common_handle,
> true, false, false)) {
> spin_unlock(&kcov_remote_lock);
> @@ -782,7 +771,6 @@ void kcov_remote_start(u64 handle)
> spin_lock(&kcov_remote_lock);
> remote = kcov_remote_find(handle);
> if (!remote) {
> - kcov_debug("no remote found");
> spin_unlock(&kcov_remote_lock);
> return;
> }
> @@ -810,8 +798,6 @@ void kcov_remote_start(u64 handle)
> /* Reset coverage size. */
> *(u64 *)area = 0;
>
> - kcov_debug("area = %px, size = %u", area, size);
> -
> kcov_start(t, size, area, mode, sequence);
>
> }
> @@ -881,10 +867,8 @@ void kcov_remote_stop(void)
> unsigned int size = t->kcov_size;
> int sequence = t->kcov_sequence;
>
> - if (!kcov) {
> - kcov_debug("no kcov found\n");
> + if (!kcov)
> return;
> - }
>
> kcov_stop(t);
> t->kcov = NULL;
> @@ -894,8 +878,6 @@ void kcov_remote_stop(void)
> * KCOV_DISABLE could have been called between kcov_remote_start()
> * and kcov_remote_stop(), hence the check.
> */
> - kcov_debug("move if: %d == %d && %d\n",
> - sequence, kcov->sequence, (int)kcov->remote);
> if (sequence == kcov->sequence && kcov->remote)
> kcov_move_area(kcov->mode, kcov->area, kcov->size, area);
> spin_unlock(&kcov->lock);
> --
> 2.26.0.rc2.310.g2932bb562d-goog
>
Powered by blists - more mailing lists