kcov used to obey clang-format style, but somehow diverged over time.
This patch applies clang-format to kernel/kcov.c and
include/linux/kcov.h, no functional change.
Signed-off-by: Alexander Potapenko <glider@google.com>
---
include/linux/kcov.h | 54 +++++++++++------
kernel/kcov.c | 134 ++++++++++++++++++++++---------------------
2 files changed, 105 insertions(+), 83 deletions(-)
diff --git a/include/linux/kcov.h b/include/linux/kcov.h
index 75a2fb8b16c32..932b4face1005 100644
--- a/include/linux/kcov.h
+++ b/include/linux/kcov.h
@@ -25,20 +25,20 @@ enum kcov_mode {
KCOV_MODE_REMOTE = 4,
};
-#define KCOV_IN_CTXSW (1 << 30)
+#define KCOV_IN_CTXSW (1 << 30)
void kcov_task_init(struct task_struct *t);
void kcov_task_exit(struct task_struct *t);
-#define kcov_prepare_switch(t) \
-do { \
- (t)->kcov_mode |= KCOV_IN_CTXSW; \
-} while (0)
+#define kcov_prepare_switch(t) \
+ do { \
+ (t)->kcov_mode |= KCOV_IN_CTXSW; \
+ } while (0)
-#define kcov_finish_switch(t) \
-do { \
- (t)->kcov_mode &= ~KCOV_IN_CTXSW; \
-} while (0)
+#define kcov_finish_switch(t) \
+ do { \
+ (t)->kcov_mode &= ~KCOV_IN_CTXSW; \
+ } while (0)
/* See Documentation/dev-tools/kcov.rst for usage details. */
void kcov_remote_start(u64 handle);
@@ -119,23 +119,41 @@ void __sanitizer_cov_trace_switch(kcov_u64 val, void *cases);
#else
-static inline void kcov_task_init(struct task_struct *t) {}
-static inline void kcov_task_exit(struct task_struct *t) {}
-static inline void kcov_prepare_switch(struct task_struct *t) {}
-static inline void kcov_finish_switch(struct task_struct *t) {}
-static inline void kcov_remote_start(u64 handle) {}
-static inline void kcov_remote_stop(void) {}
+static inline void kcov_task_init(struct task_struct *t)
+{
+}
+static inline void kcov_task_exit(struct task_struct *t)
+{
+}
+static inline void kcov_prepare_switch(struct task_struct *t)
+{
+}
+static inline void kcov_finish_switch(struct task_struct *t)
+{
+}
+static inline void kcov_remote_start(u64 handle)
+{
+}
+static inline void kcov_remote_stop(void)
+{
+}
static inline u64 kcov_common_handle(void)
{
return 0;
}
-static inline void kcov_remote_start_common(u64 id) {}
-static inline void kcov_remote_start_usb(u64 id) {}
+static inline void kcov_remote_start_common(u64 id)
+{
+}
+static inline void kcov_remote_start_usb(u64 id)
+{
+}
static inline unsigned long kcov_remote_start_usb_softirq(u64 id)
{
return 0;
}
-static inline void kcov_remote_stop_softirq(unsigned long flags) {}
+static inline void kcov_remote_stop_softirq(unsigned long flags)
+{
+}
#endif /* CONFIG_KCOV */
#endif /* _LINUX_KCOV_H */
diff --git a/kernel/kcov.c b/kernel/kcov.c
index 187ba1b80bda1..7cc6123c2baa4 100644
--- a/kernel/kcov.c
+++ b/kernel/kcov.c
@@ -4,27 +4,28 @@
#define DISABLE_BRANCH_PROFILING
#include <linux/atomic.h>
#include <linux/compiler.h>
+#include <linux/debugfs.h>
#include <linux/errno.h>
#include <linux/export.h>
-#include <linux/types.h>
#include <linux/file.h>
#include <linux/fs.h>
#include <linux/hashtable.h>
#include <linux/init.h>
#include <linux/jiffies.h>
+#include <linux/kcov.h>
#include <linux/kmsan-checks.h>
+#include <linux/log2.h>
#include <linux/mm.h>
#include <linux/preempt.h>
#include <linux/printk.h>
+#include <linux/refcount.h>
#include <linux/sched.h>
#include <linux/slab.h>
#include <linux/spinlock.h>
-#include <linux/vmalloc.h>
-#include <linux/debugfs.h>
+#include <linux/types.h>
#include <linux/uaccess.h>
-#include <linux/kcov.h>
-#include <linux/refcount.h>
-#include <linux/log2.h>
+#include <linux/vmalloc.h>
+
#include <asm/setup.h>
#define kcov_debug(fmt, ...) pr_debug("%s: " fmt, __func__, ##__VA_ARGS__)
@@ -52,36 +53,36 @@ struct kcov {
* - task with enabled coverage (we can't unwire it from another task)
* - each code section for remote coverage collection
*/
- refcount_t refcount;
+ refcount_t refcount;
/* The lock protects mode, size, area and t. */
- spinlock_t lock;
- enum kcov_mode mode;
+ spinlock_t lock;
+ enum kcov_mode mode;
/* Size of arena (in long's). */
- unsigned int size;
+ unsigned int size;
/* Coverage buffer shared with user space. */
- void *area;
+ void *area;
/* Task for which we collect coverage, or NULL. */
- struct task_struct *t;
+ struct task_struct *t;
/* Collecting coverage from remote (background) threads. */
- bool remote;
+ bool remote;
/* Size of remote area (in long's). */
- unsigned int remote_size;
+ unsigned int remote_size;
/*
* Sequence is incremented each time kcov is reenabled, used by
* kcov_remote_stop(), see the comment there.
*/
- int sequence;
+ int sequence;
};
struct kcov_remote_area {
- struct list_head list;
- unsigned int size;
+ struct list_head list;
+ unsigned int size;
};
struct kcov_remote {
- u64 handle;
- struct kcov *kcov;
- struct hlist_node hnode;
+ u64 handle;
+ struct kcov *kcov;
+ struct hlist_node hnode;
};
static DEFINE_SPINLOCK(kcov_remote_lock);
@@ -89,14 +90,14 @@ static DEFINE_HASHTABLE(kcov_remote_map, 4);
static struct list_head kcov_remote_areas = LIST_HEAD_INIT(kcov_remote_areas);
struct kcov_percpu_data {
- void *irq_area;
- local_lock_t lock;
-
- unsigned int saved_mode;
- unsigned int saved_size;
- void *saved_area;
- struct kcov *saved_kcov;
- int saved_sequence;
+ void *irq_area;
+ local_lock_t lock;
+
+ unsigned int saved_mode;
+ unsigned int saved_size;
+ void *saved_area;
+ struct kcov *saved_kcov;
+ int saved_sequence;
};
static DEFINE_PER_CPU(struct kcov_percpu_data, kcov_percpu_data) = {
@@ -149,7 +150,7 @@ static struct kcov_remote_area *kcov_remote_area_get(unsigned int size)
/* Must be called with kcov_remote_lock locked. */
static void kcov_remote_area_put(struct kcov_remote_area *area,
- unsigned int size)
+ unsigned int size)
{
INIT_LIST_HEAD(&area->list);
area->size = size;
@@ -171,7 +172,8 @@ static __always_inline bool in_softirq_really(void)
return in_serving_softirq() && !in_hardirq() && !in_nmi();
}
-static notrace bool check_kcov_mode(enum kcov_mode needed_mode, struct task_struct *t)
+static notrace bool check_kcov_mode(enum kcov_mode needed_mode,
+ struct task_struct *t)
{
unsigned int mode;
@@ -354,8 +356,8 @@ EXPORT_SYMBOL(__sanitizer_cov_trace_switch);
#endif /* ifdef CONFIG_KCOV_ENABLE_COMPARISONS */
static void kcov_start(struct task_struct *t, struct kcov *kcov,
- unsigned int size, void *area, enum kcov_mode mode,
- int sequence)
+ unsigned int size, void *area, enum kcov_mode mode,
+ int sequence)
{
kcov_debug("t = %px, size = %u, area = %px\n", t, size, area);
t->kcov = kcov;
@@ -566,14 +568,14 @@ static void kcov_fault_in_area(struct kcov *kcov)
}
static inline bool kcov_check_handle(u64 handle, bool common_valid,
- bool uncommon_valid, bool zero_valid)
+ bool uncommon_valid, bool zero_valid)
{
if (handle & ~(KCOV_SUBSYSTEM_MASK | KCOV_INSTANCE_MASK))
return false;
switch (handle & KCOV_SUBSYSTEM_MASK) {
case KCOV_SUBSYSTEM_COMMON:
- return (handle & KCOV_INSTANCE_MASK) ?
- common_valid : zero_valid;
+ return (handle & KCOV_INSTANCE_MASK) ? common_valid :
+ zero_valid;
case KCOV_SUBSYSTEM_USB:
return uncommon_valid;
default:
@@ -611,7 +613,7 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
kcov_fault_in_area(kcov);
kcov->mode = mode;
kcov_start(t, kcov, kcov->size, kcov->area, kcov->mode,
- kcov->sequence);
+ kcov->sequence);
kcov->t = t;
/* Put either in kcov_task_exit() or in KCOV_DISABLE. */
kcov_get(kcov);
@@ -642,40 +644,40 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
return -EINVAL;
kcov->mode = mode;
t->kcov = kcov;
- t->kcov_mode = KCOV_MODE_REMOTE;
+ t->kcov_mode = KCOV_MODE_REMOTE;
kcov->t = t;
kcov->remote = true;
kcov->remote_size = remote_arg->area_size;
spin_lock_irqsave(&kcov_remote_lock, flags);
for (i = 0; i < remote_arg->num_handles; i++) {
- if (!kcov_check_handle(remote_arg->handles[i],
- false, true, false)) {
+ if (!kcov_check_handle(remote_arg->handles[i], false,
+ true, false)) {
spin_unlock_irqrestore(&kcov_remote_lock,
- flags);
+ flags);
kcov_disable(t, kcov);
return -EINVAL;
}
remote = kcov_remote_add(kcov, remote_arg->handles[i]);
if (IS_ERR(remote)) {
spin_unlock_irqrestore(&kcov_remote_lock,
- flags);
+ flags);
kcov_disable(t, kcov);
return PTR_ERR(remote);
}
}
if (remote_arg->common_handle) {
- if (!kcov_check_handle(remote_arg->common_handle,
- true, false, false)) {
+ if (!kcov_check_handle(remote_arg->common_handle, true,
+ false, false)) {
spin_unlock_irqrestore(&kcov_remote_lock,
- flags);
+ flags);
kcov_disable(t, kcov);
return -EINVAL;
}
remote = kcov_remote_add(kcov,
- remote_arg->common_handle);
+ remote_arg->common_handle);
if (IS_ERR(remote)) {
spin_unlock_irqrestore(&kcov_remote_lock,
- flags);
+ flags);
kcov_disable(t, kcov);
return PTR_ERR(remote);
}
@@ -728,13 +730,15 @@ static long kcov_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
spin_unlock_irqrestore(&kcov->lock, flags);
return 0;
case KCOV_REMOTE_ENABLE:
- if (get_user(remote_num_handles, (unsigned __user *)(arg +
- offsetof(struct kcov_remote_arg, num_handles))))
+ if (get_user(remote_num_handles,
+ (unsigned __user *)(arg +
+ offsetof(struct kcov_remote_arg,
+ num_handles))))
return -EFAULT;
if (remote_num_handles > KCOV_REMOTE_MAX_HANDLES)
return -EINVAL;
- remote_arg_size = struct_size(remote_arg, handles,
- remote_num_handles);
+ remote_arg_size =
+ struct_size(remote_arg, handles, remote_num_handles);
remote_arg = memdup_user((void __user *)arg, remote_arg_size);
if (IS_ERR(remote_arg))
return PTR_ERR(remote_arg);
@@ -758,11 +762,11 @@ static long kcov_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
}
static const struct file_operations kcov_fops = {
- .open = kcov_open,
- .unlocked_ioctl = kcov_ioctl,
- .compat_ioctl = kcov_ioctl,
- .mmap = kcov_mmap,
- .release = kcov_close,
+ .open = kcov_open,
+ .unlocked_ioctl = kcov_ioctl,
+ .compat_ioctl = kcov_ioctl,
+ .mmap = kcov_mmap,
+ .release = kcov_close,
};
/*
@@ -836,8 +840,8 @@ static void kcov_remote_softirq_stop(struct task_struct *t)
if (data->saved_kcov) {
kcov_start(t, data->saved_kcov, data->saved_size,
- data->saved_area, data->saved_mode,
- data->saved_sequence);
+ data->saved_area, data->saved_mode,
+ data->saved_sequence);
data->saved_mode = 0;
data->saved_size = 0;
data->saved_area = NULL;
@@ -891,7 +895,7 @@ void kcov_remote_start(u64 handle)
return;
}
kcov_debug("handle = %llx, context: %s\n", handle,
- in_task() ? "task" : "softirq");
+ in_task() ? "task" : "softirq");
kcov = remote->kcov;
/* Put in kcov_remote_stop(). */
kcov_get(kcov);
@@ -931,12 +935,11 @@ void kcov_remote_start(u64 handle)
kcov_start(t, kcov, size, area, mode, sequence);
local_unlock_irqrestore(&kcov_percpu_data.lock, flags);
-
}
EXPORT_SYMBOL(kcov_remote_start);
static void kcov_move_area(enum kcov_mode mode, void *dst_area,
- unsigned int dst_area_size, void *src_area)
+ unsigned int dst_area_size, void *src_area)
{
u64 word_size = sizeof(unsigned long);
u64 count_size, entry_size_log;
@@ -944,8 +947,8 @@ static void kcov_move_area(enum kcov_mode mode, void *dst_area,
void *dst_entries, *src_entries;
u64 dst_occupied, dst_free, bytes_to_move, entries_moved;
- kcov_debug("%px %u <= %px %lu\n",
- dst_area, dst_area_size, src_area, *(unsigned long *)src_area);
+ kcov_debug("%px %u <= %px %lu\n", dst_area, dst_area_size, src_area,
+ *(unsigned long *)src_area);
switch (mode) {
case KCOV_MODE_TRACE_PC:
@@ -967,8 +970,8 @@ static void kcov_move_area(enum kcov_mode mode, void *dst_area,
}
/* As arm can't divide u64 integers use log of entry size. */
- if (dst_len > ((dst_area_size * word_size - count_size) >>
- entry_size_log))
+ if (dst_len >
+ ((dst_area_size * word_size - count_size) >> entry_size_log))
return;
dst_occupied = count_size + (dst_len << entry_size_log);
dst_free = dst_area_size * word_size - dst_occupied;
@@ -1100,7 +1103,8 @@ static int __init kcov_init(void)
for_each_possible_cpu(cpu) {
void *area = vmalloc_node(CONFIG_KCOV_IRQ_AREA_SIZE *
- sizeof(unsigned long), cpu_to_node(cpu));
+ sizeof(unsigned long),
+ cpu_to_node(cpu));
if (!area)
return -ENOMEM;
per_cpu_ptr(&kcov_percpu_data, cpu)->irq_area = area;
--
2.49.0.604.gff1f9ca942-googOn Wed, 16 Apr 2025 at 10:55, Alexander Potapenko <glider@google.com> wrote:
>
> kcov used to obey clang-format style, but somehow diverged over time.
> This patch applies clang-format to kernel/kcov.c and
> include/linux/kcov.h, no functional change.
>
> Signed-off-by: Alexander Potapenko <glider@google.com>
> ---
> include/linux/kcov.h | 54 +++++++++++------
> kernel/kcov.c | 134 ++++++++++++++++++++++---------------------
> 2 files changed, 105 insertions(+), 83 deletions(-)
>
> diff --git a/include/linux/kcov.h b/include/linux/kcov.h
> index 75a2fb8b16c32..932b4face1005 100644
> --- a/include/linux/kcov.h
> +++ b/include/linux/kcov.h
> @@ -25,20 +25,20 @@ enum kcov_mode {
> KCOV_MODE_REMOTE = 4,
> };
>
> -#define KCOV_IN_CTXSW (1 << 30)
> +#define KCOV_IN_CTXSW (1 << 30)
>
> void kcov_task_init(struct task_struct *t);
> void kcov_task_exit(struct task_struct *t);
>
> -#define kcov_prepare_switch(t) \
> -do { \
> - (t)->kcov_mode |= KCOV_IN_CTXSW; \
> -} while (0)
> +#define kcov_prepare_switch(t) \
> + do { \
> + (t)->kcov_mode |= KCOV_IN_CTXSW; \
> + } while (0)
>
> -#define kcov_finish_switch(t) \
> -do { \
> - (t)->kcov_mode &= ~KCOV_IN_CTXSW; \
> -} while (0)
> +#define kcov_finish_switch(t) \
> + do { \
> + (t)->kcov_mode &= ~KCOV_IN_CTXSW; \
> + } while (0)
>
> /* See Documentation/dev-tools/kcov.rst for usage details. */
> void kcov_remote_start(u64 handle);
> @@ -119,23 +119,41 @@ void __sanitizer_cov_trace_switch(kcov_u64 val, void *cases);
>
> #else
>
> -static inline void kcov_task_init(struct task_struct *t) {}
> -static inline void kcov_task_exit(struct task_struct *t) {}
> -static inline void kcov_prepare_switch(struct task_struct *t) {}
> -static inline void kcov_finish_switch(struct task_struct *t) {}
> -static inline void kcov_remote_start(u64 handle) {}
> -static inline void kcov_remote_stop(void) {}
> +static inline void kcov_task_init(struct task_struct *t)
> +{
> +}
> +static inline void kcov_task_exit(struct task_struct *t)
> +{
> +}
> +static inline void kcov_prepare_switch(struct task_struct *t)
> +{
> +}
> +static inline void kcov_finish_switch(struct task_struct *t)
> +{
> +}
> +static inline void kcov_remote_start(u64 handle)
> +{
> +}
> +static inline void kcov_remote_stop(void)
> +{
> +}
This excessive-new-line style is not an improvement over previously.
But nothing we can do about I guess...
> static inline u64 kcov_common_handle(void)
> {
> return 0;
> }
> -static inline void kcov_remote_start_common(u64 id) {}
> -static inline void kcov_remote_start_usb(u64 id) {}
> +static inline void kcov_remote_start_common(u64 id)
> +{
> +}
> +static inline void kcov_remote_start_usb(u64 id)
> +{
> +}
> static inline unsigned long kcov_remote_start_usb_softirq(u64 id)
> {
> return 0;
> }
> -static inline void kcov_remote_stop_softirq(unsigned long flags) {}
> +static inline void kcov_remote_stop_softirq(unsigned long flags)
> +{
> +}
>
> #endif /* CONFIG_KCOV */
> #endif /* _LINUX_KCOV_H */
> diff --git a/kernel/kcov.c b/kernel/kcov.c
> index 187ba1b80bda1..7cc6123c2baa4 100644
> --- a/kernel/kcov.c
> +++ b/kernel/kcov.c
> @@ -4,27 +4,28 @@
> #define DISABLE_BRANCH_PROFILING
> #include <linux/atomic.h>
> #include <linux/compiler.h>
> +#include <linux/debugfs.h>
> #include <linux/errno.h>
> #include <linux/export.h>
> -#include <linux/types.h>
> #include <linux/file.h>
> #include <linux/fs.h>
> #include <linux/hashtable.h>
> #include <linux/init.h>
> #include <linux/jiffies.h>
> +#include <linux/kcov.h>
> #include <linux/kmsan-checks.h>
> +#include <linux/log2.h>
> #include <linux/mm.h>
> #include <linux/preempt.h>
> #include <linux/printk.h>
> +#include <linux/refcount.h>
> #include <linux/sched.h>
> #include <linux/slab.h>
> #include <linux/spinlock.h>
> -#include <linux/vmalloc.h>
> -#include <linux/debugfs.h>
> +#include <linux/types.h>
> #include <linux/uaccess.h>
> -#include <linux/kcov.h>
> -#include <linux/refcount.h>
> -#include <linux/log2.h>
> +#include <linux/vmalloc.h>
> +
> #include <asm/setup.h>
>
> #define kcov_debug(fmt, ...) pr_debug("%s: " fmt, __func__, ##__VA_ARGS__)
> @@ -52,36 +53,36 @@ struct kcov {
> * - task with enabled coverage (we can't unwire it from another task)
> * - each code section for remote coverage collection
> */
> - refcount_t refcount;
> + refcount_t refcount;
> /* The lock protects mode, size, area and t. */
> - spinlock_t lock;
> - enum kcov_mode mode;
> + spinlock_t lock;
> + enum kcov_mode mode;
> /* Size of arena (in long's). */
> - unsigned int size;
> + unsigned int size;
> /* Coverage buffer shared with user space. */
> - void *area;
> + void *area;
> /* Task for which we collect coverage, or NULL. */
> - struct task_struct *t;
> + struct task_struct *t;
> /* Collecting coverage from remote (background) threads. */
> - bool remote;
> + bool remote;
> /* Size of remote area (in long's). */
> - unsigned int remote_size;
> + unsigned int remote_size;
> /*
> * Sequence is incremented each time kcov is reenabled, used by
> * kcov_remote_stop(), see the comment there.
> */
> - int sequence;
> + int sequence;
> };
>
> struct kcov_remote_area {
> - struct list_head list;
> - unsigned int size;
> + struct list_head list;
> + unsigned int size;
> };
>
> struct kcov_remote {
> - u64 handle;
> - struct kcov *kcov;
> - struct hlist_node hnode;
> + u64 handle;
> + struct kcov *kcov;
> + struct hlist_node hnode;
> };
>
> static DEFINE_SPINLOCK(kcov_remote_lock);
> @@ -89,14 +90,14 @@ static DEFINE_HASHTABLE(kcov_remote_map, 4);
> static struct list_head kcov_remote_areas = LIST_HEAD_INIT(kcov_remote_areas);
>
> struct kcov_percpu_data {
> - void *irq_area;
> - local_lock_t lock;
> -
> - unsigned int saved_mode;
> - unsigned int saved_size;
> - void *saved_area;
> - struct kcov *saved_kcov;
> - int saved_sequence;
> + void *irq_area;
> + local_lock_t lock;
> +
> + unsigned int saved_mode;
> + unsigned int saved_size;
> + void *saved_area;
> + struct kcov *saved_kcov;
> + int saved_sequence;
> };
>
> static DEFINE_PER_CPU(struct kcov_percpu_data, kcov_percpu_data) = {
> @@ -149,7 +150,7 @@ static struct kcov_remote_area *kcov_remote_area_get(unsigned int size)
>
> /* Must be called with kcov_remote_lock locked. */
> static void kcov_remote_area_put(struct kcov_remote_area *area,
> - unsigned int size)
> + unsigned int size)
> {
> INIT_LIST_HEAD(&area->list);
> area->size = size;
> @@ -171,7 +172,8 @@ static __always_inline bool in_softirq_really(void)
> return in_serving_softirq() && !in_hardirq() && !in_nmi();
> }
>
> -static notrace bool check_kcov_mode(enum kcov_mode needed_mode, struct task_struct *t)
> +static notrace bool check_kcov_mode(enum kcov_mode needed_mode,
> + struct task_struct *t)
> {
> unsigned int mode;
>
> @@ -354,8 +356,8 @@ EXPORT_SYMBOL(__sanitizer_cov_trace_switch);
> #endif /* ifdef CONFIG_KCOV_ENABLE_COMPARISONS */
>
> static void kcov_start(struct task_struct *t, struct kcov *kcov,
> - unsigned int size, void *area, enum kcov_mode mode,
> - int sequence)
> + unsigned int size, void *area, enum kcov_mode mode,
> + int sequence)
> {
> kcov_debug("t = %px, size = %u, area = %px\n", t, size, area);
> t->kcov = kcov;
> @@ -566,14 +568,14 @@ static void kcov_fault_in_area(struct kcov *kcov)
> }
>
> static inline bool kcov_check_handle(u64 handle, bool common_valid,
> - bool uncommon_valid, bool zero_valid)
> + bool uncommon_valid, bool zero_valid)
> {
> if (handle & ~(KCOV_SUBSYSTEM_MASK | KCOV_INSTANCE_MASK))
> return false;
> switch (handle & KCOV_SUBSYSTEM_MASK) {
> case KCOV_SUBSYSTEM_COMMON:
> - return (handle & KCOV_INSTANCE_MASK) ?
> - common_valid : zero_valid;
> + return (handle & KCOV_INSTANCE_MASK) ? common_valid :
> + zero_valid;
> case KCOV_SUBSYSTEM_USB:
> return uncommon_valid;
> default:
> @@ -611,7 +613,7 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
> kcov_fault_in_area(kcov);
> kcov->mode = mode;
> kcov_start(t, kcov, kcov->size, kcov->area, kcov->mode,
> - kcov->sequence);
> + kcov->sequence);
> kcov->t = t;
> /* Put either in kcov_task_exit() or in KCOV_DISABLE. */
> kcov_get(kcov);
> @@ -642,40 +644,40 @@ static int kcov_ioctl_locked(struct kcov *kcov, unsigned int cmd,
> return -EINVAL;
> kcov->mode = mode;
> t->kcov = kcov;
> - t->kcov_mode = KCOV_MODE_REMOTE;
> + t->kcov_mode = KCOV_MODE_REMOTE;
> kcov->t = t;
> kcov->remote = true;
> kcov->remote_size = remote_arg->area_size;
> spin_lock_irqsave(&kcov_remote_lock, flags);
> for (i = 0; i < remote_arg->num_handles; i++) {
> - if (!kcov_check_handle(remote_arg->handles[i],
> - false, true, false)) {
> + if (!kcov_check_handle(remote_arg->handles[i], false,
> + true, false)) {
> spin_unlock_irqrestore(&kcov_remote_lock,
> - flags);
> + flags);
> kcov_disable(t, kcov);
> return -EINVAL;
> }
> remote = kcov_remote_add(kcov, remote_arg->handles[i]);
> if (IS_ERR(remote)) {
> spin_unlock_irqrestore(&kcov_remote_lock,
> - flags);
> + flags);
> kcov_disable(t, kcov);
> return PTR_ERR(remote);
> }
> }
> if (remote_arg->common_handle) {
> - if (!kcov_check_handle(remote_arg->common_handle,
> - true, false, false)) {
> + if (!kcov_check_handle(remote_arg->common_handle, true,
> + false, false)) {
> spin_unlock_irqrestore(&kcov_remote_lock,
> - flags);
> + flags);
> kcov_disable(t, kcov);
> return -EINVAL;
> }
> remote = kcov_remote_add(kcov,
> - remote_arg->common_handle);
> + remote_arg->common_handle);
> if (IS_ERR(remote)) {
> spin_unlock_irqrestore(&kcov_remote_lock,
> - flags);
> + flags);
> kcov_disable(t, kcov);
> return PTR_ERR(remote);
> }
> @@ -728,13 +730,15 @@ static long kcov_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
> spin_unlock_irqrestore(&kcov->lock, flags);
> return 0;
> case KCOV_REMOTE_ENABLE:
> - if (get_user(remote_num_handles, (unsigned __user *)(arg +
> - offsetof(struct kcov_remote_arg, num_handles))))
> + if (get_user(remote_num_handles,
> + (unsigned __user *)(arg +
> + offsetof(struct kcov_remote_arg,
> + num_handles))))
Ouch. Maybe move the address calculation before and assign to
temporary to avoid this mess?
> return -EFAULT;
> if (remote_num_handles > KCOV_REMOTE_MAX_HANDLES)
> return -EINVAL;
> - remote_arg_size = struct_size(remote_arg, handles,
> - remote_num_handles);
> + remote_arg_size =
> + struct_size(remote_arg, handles, remote_num_handles);
> remote_arg = memdup_user((void __user *)arg, remote_arg_size);
> if (IS_ERR(remote_arg))
> return PTR_ERR(remote_arg);
> @@ -758,11 +762,11 @@ static long kcov_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
> }
>
> static const struct file_operations kcov_fops = {
> - .open = kcov_open,
> - .unlocked_ioctl = kcov_ioctl,
> - .compat_ioctl = kcov_ioctl,
> - .mmap = kcov_mmap,
> - .release = kcov_close,
> + .open = kcov_open,
> + .unlocked_ioctl = kcov_ioctl,
> + .compat_ioctl = kcov_ioctl,
> + .mmap = kcov_mmap,
> + .release = kcov_close,
> };
>
> /*
> @@ -836,8 +840,8 @@ static void kcov_remote_softirq_stop(struct task_struct *t)
>
> if (data->saved_kcov) {
> kcov_start(t, data->saved_kcov, data->saved_size,
> - data->saved_area, data->saved_mode,
> - data->saved_sequence);
> + data->saved_area, data->saved_mode,
> + data->saved_sequence);
> data->saved_mode = 0;
> data->saved_size = 0;
> data->saved_area = NULL;
> @@ -891,7 +895,7 @@ void kcov_remote_start(u64 handle)
> return;
> }
> kcov_debug("handle = %llx, context: %s\n", handle,
> - in_task() ? "task" : "softirq");
> + in_task() ? "task" : "softirq");
> kcov = remote->kcov;
> /* Put in kcov_remote_stop(). */
> kcov_get(kcov);
> @@ -931,12 +935,11 @@ void kcov_remote_start(u64 handle)
> kcov_start(t, kcov, size, area, mode, sequence);
>
> local_unlock_irqrestore(&kcov_percpu_data.lock, flags);
> -
> }
> EXPORT_SYMBOL(kcov_remote_start);
>
> static void kcov_move_area(enum kcov_mode mode, void *dst_area,
> - unsigned int dst_area_size, void *src_area)
> + unsigned int dst_area_size, void *src_area)
> {
> u64 word_size = sizeof(unsigned long);
> u64 count_size, entry_size_log;
> @@ -944,8 +947,8 @@ static void kcov_move_area(enum kcov_mode mode, void *dst_area,
> void *dst_entries, *src_entries;
> u64 dst_occupied, dst_free, bytes_to_move, entries_moved;
>
> - kcov_debug("%px %u <= %px %lu\n",
> - dst_area, dst_area_size, src_area, *(unsigned long *)src_area);
> + kcov_debug("%px %u <= %px %lu\n", dst_area, dst_area_size, src_area,
> + *(unsigned long *)src_area);
>
> switch (mode) {
> case KCOV_MODE_TRACE_PC:
> @@ -967,8 +970,8 @@ static void kcov_move_area(enum kcov_mode mode, void *dst_area,
> }
>
> /* As arm can't divide u64 integers use log of entry size. */
> - if (dst_len > ((dst_area_size * word_size - count_size) >>
> - entry_size_log))
> + if (dst_len >
> + ((dst_area_size * word_size - count_size) >> entry_size_log))
> return;
> dst_occupied = count_size + (dst_len << entry_size_log);
> dst_free = dst_area_size * word_size - dst_occupied;
> @@ -1100,7 +1103,8 @@ static int __init kcov_init(void)
>
> for_each_possible_cpu(cpu) {
> void *area = vmalloc_node(CONFIG_KCOV_IRQ_AREA_SIZE *
> - sizeof(unsigned long), cpu_to_node(cpu));
> + sizeof(unsigned long),
> + cpu_to_node(cpu));
Ouch.
> > +static inline void kcov_remote_start(u64 handle)
> > +{
> > +}
> > +static inline void kcov_remote_stop(void)
> > +{
> > +}
>
> This excessive-new-line style is not an improvement over previously.
> But nothing we can do about I guess...
I think we'd better stick with whatever clang-format gives us.
> > @@ -728,13 +730,15 @@ static long kcov_ioctl(struct file *filep, unsigned int cmd, unsigned long arg)
> > spin_unlock_irqrestore(&kcov->lock, flags);
> > return 0;
> > case KCOV_REMOTE_ENABLE:
> > - if (get_user(remote_num_handles, (unsigned __user *)(arg +
> > - offsetof(struct kcov_remote_arg, num_handles))))
> > + if (get_user(remote_num_handles,
> > + (unsigned __user *)(arg +
> > + offsetof(struct kcov_remote_arg,
> > + num_handles))))
>
> Ouch. Maybe move the address calculation before and assign to
> temporary to avoid this mess?
I factored out offsetof(), because the address calculation looked all
the same after formatting.
> > for_each_possible_cpu(cpu) {
> > void *area = vmalloc_node(CONFIG_KCOV_IRQ_AREA_SIZE *
> > - sizeof(unsigned long), cpu_to_node(cpu));
> > + sizeof(unsigned long),
> > + cpu_to_node(cpu));
>
> Ouch.
Someday we'll probably switch clang-format to 100 columns
© 2016 - 2025 Red Hat, Inc.