Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH] coccinelle: api: remove kobj_to_dev.cocci script
To:     Keith Busch <kbusch@kernel.org>, linux-kernel@vger.kernel.org
Cc:     Christoph Hellwig <hch@lst.de>, Jens Axboe <axboe@kernel.dk>,
        Julia Lawall <Julia.Lawall@inria.fr>
References: <20210621174808.1489111-1-kbusch@kernel.org>
From:   Denis Efremov <efremov@linux.com>
Message-ID: <cc3b2db6-70fd-0bb2-d083-f87f9a35e9a7@linux.com>
Date:   Mon, 21 Jun 2021 21:32:15 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.11.0
MIME-Version: 1.0
In-Reply-To: <20210621174808.1489111-1-kbusch@kernel.org>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 6/21/21 8:48 PM, Keith Busch wrote:
> Using kobj_to_dev() instead of container_of() is not universally
> accepted among maintainers as an improvement. The warning leads to
> repeated patch submissions that won't be accepted. Remove the script.
> 
> Cc: Christoph Hellwig <hch@lst.de>
> Cc: Jens Axboe <axboe@kernel.dk>
> Cc: Denis Efremov <efremov@linux.com>
> Cc: Julia Lawall <Julia.Lawall@inria.fr>
> Signed-off-by: Keith Busch <kbusch@kernel.org>

Acked-by: Denis Efremov <efremov@linux.com>

> ---
>  scripts/coccinelle/api/kobj_to_dev.cocci | 45 ------------------------
>  1 file changed, 45 deletions(-)
>  delete mode 100644 scripts/coccinelle/api/kobj_to_dev.cocci
> 
> diff --git a/scripts/coccinelle/api/kobj_to_dev.cocci b/scripts/coccinelle/api/kobj_to_dev.cocci
> deleted file mode 100644
> index cd5d31c6fe76..000000000000
> --- a/scripts/coccinelle/api/kobj_to_dev.cocci
> +++ /dev/null
> @@ -1,45 +0,0 @@
> -// SPDX-License-Identifier: GPL-2.0-only
> -///
> -/// Use kobj_to_dev() instead of container_of()
> -///
> -// Confidence: High
> -// Copyright: (C) 2020 Denis Efremov ISPRAS
> -// Options: --no-includes --include-headers
> -//
> -// Keywords: kobj_to_dev, container_of
> -//
> -
> -virtual context
> -virtual report
> -virtual org
> -virtual patch
> -
> -
> -@r depends on !patch@
> -expression ptr;
> -symbol kobj;
> -position p;
> -@@
> -
> -* container_of(ptr, struct device, kobj)@p
> -
> -
> -@depends on patch@
> -expression ptr;
> -@@
> -
> -- container_of(ptr, struct device, kobj)
> -+ kobj_to_dev(ptr)
> -
> -
> -@script:python depends on report@
> -p << r.p;
> -@@
> -
> -coccilib.report.print_report(p[0], "WARNING opportunity for kobj_to_dev()")
> -
> -@script:python depends on org@
> -p << r.p;
> -@@
> -
> -coccilib.org.print_todo(p[0], "WARNING opportunity for kobj_to_dev()")
>