linux-kernel - Re: [-next RESEND] fault-injection: using debugfs_create_xul() instead of debugfs_create

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: <CAC5umygFSZ1ho4nDn_8sjU4_tsi8Hu4bkw0m4g=n6UO0+=cz4A@mail.gmail.com>
Date:   Thu, 6 Oct 2022 23:41:12 +0900
From:   Akinobu Mita <akinobu.mita@...il.com>
To:     Wang Yufen <wangyufen@...wei.com>,
        Andrew Morton <akpm@...ux-foundation.org>
Cc:     linux-kernel@...r.kernel.org
Subject: Re: [-next RESEND] fault-injection: using debugfs_create_xul()
 instead of debugfs_create_xl()

Andrew, could you consider picking up this patch?

2022年9月28日(水) 10:54 Wang Yufen <wangyufen@...wei.com>:
>
> Using debugfs_create_xul() instead of debugfs_create_xl(), and del the
> local define debugfs_create_xl().
>
> Suggested-by: Akinobu Mita <akinobu.mita@...il.com>
> Signed-off-by: Wang Yufen <wangyufen@...wei.com>
> Signed-off-by: Wei Yongjun <weiyongjun1@...wei.com>

Reviewed-by: Akinobu Mita <akinobu.mita@...il.com>

> ---
>  lib/fault-inject.c | 16 ++++------------
>  1 file changed, 4 insertions(+), 12 deletions(-)
>
> diff --git a/lib/fault-inject.c b/lib/fault-inject.c
> index 9dd1dd1..dbb5409 100644
> --- a/lib/fault-inject.c
> +++ b/lib/fault-inject.c
> @@ -185,14 +185,6 @@ static void debugfs_create_ul(const char *name, umode_t mode,
>
>  #ifdef CONFIG_FAULT_INJECTION_STACKTRACE_FILTER
>
> -DEFINE_SIMPLE_ATTRIBUTE(fops_xl, debugfs_ul_get, debugfs_ul_set, "0x%llx\n");
> -
> -static void debugfs_create_xl(const char *name, umode_t mode,
> -                             struct dentry *parent, unsigned long *value)
> -{
> -       debugfs_create_file(name, mode, parent, value, &fops_xl);
> -}
> -
>  static int debugfs_stacktrace_depth_set(void *data, u64 val)
>  {
>         *(unsigned long *)data =
> @@ -237,10 +229,10 @@ struct dentry *fault_create_debugfs_attr(const char *name,
>  #ifdef CONFIG_FAULT_INJECTION_STACKTRACE_FILTER
>         debugfs_create_stacktrace_depth("stacktrace-depth", mode, dir,
>                                         &attr->stacktrace_depth);
> -       debugfs_create_xl("require-start", mode, dir, &attr->require_start);
> -       debugfs_create_xl("require-end", mode, dir, &attr->require_end);
> -       debugfs_create_xl("reject-start", mode, dir, &attr->reject_start);
> -       debugfs_create_xl("reject-end", mode, dir, &attr->reject_end);
> +       debugfs_create_xul("require-start", mode, dir, &attr->require_start);
> +       debugfs_create_xul("require-end", mode, dir, &attr->require_end);
> +       debugfs_create_xul("reject-start", mode, dir, &attr->reject_start);
> +       debugfs_create_xul("reject-end", mode, dir, &attr->reject_end);
>  #endif /* CONFIG_FAULT_INJECTION_STACKTRACE_FILTER */
>
>         attr->dname = dget(dir);
> --
> 1.8.3.1
>