From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 67B7AC433EF for ; Fri, 6 May 2022 07:24:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B1A6E6B0071; Fri, 6 May 2022 03:24:40 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id AC94E6B0073; Fri, 6 May 2022 03:24:40 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9B8926B0074; Fri, 6 May 2022 03:24:40 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0011.hostedemail.com [216.40.44.11]) by kanga.kvack.org (Postfix) with ESMTP id 8CC8C6B0071 for ; Fri, 6 May 2022 03:24:40 -0400 (EDT) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay12.hostedemail.com (Postfix) with ESMTP id 53AA2120A8B for ; Fri, 6 May 2022 07:24:40 +0000 (UTC) X-FDA: 79434480720.23.23FCE92 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf03.hostedemail.com (Postfix) with ESMTP id 605C620002 for ; Fri, 6 May 2022 07:24:33 +0000 (UTC) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id E401C61FAC for ; Fri, 6 May 2022 07:24:38 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4E947C385AC for ; Fri, 6 May 2022 07:24:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1651821878; bh=StWfG862y0u2kQU0M4ZLAPlW/rb52wK1Pm4BbpZO+tE=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=FPuuz+QYQLJsxFXdf+bWPOaMtjdb7DlpsLJjGzqAmr4sijr00SCJZOQ958vkODYIG ecJxJo6UccWgEhFTEIDXXBAmSYRGQUfX95ch1ICCP4V6Q3rxF+eh8YagyL1VUS8FT+ VpMDAyAa2jPwihzq6+p24zAUuCHqeG/YXwd5xY7nKWdnnUtiM8cAfX2CFJfLkisUxl 5tK4bZY7vNa8Kchbwqf/jDMGqmn1ejSJ1lAGm01EbWRMmrPWIl2NmNMH0rn+LcKocY g0uNR5dFzbLr52Vf86Gn9dFHEsETNLuJYYUJ+3fkRi9OgZbkcDtSkR3yrCQGBrIHG4 Gj3/WkbwlaZQA== Received: by mail-ot1-f47.google.com with SMTP id z15-20020a9d65cf000000b00605f064482cso4430910oth.6 for ; Fri, 06 May 2022 00:24:38 -0700 (PDT) X-Gm-Message-State: AOAM5332pnKOthvbOUnpEugyMcFkJbANnuZhIpRjN37CmPDYw44iOOHy TvWFdjI0307LVk9k5GzKq3R5S5rrdgKueU2cwJ4= X-Google-Smtp-Source: ABdhPJwuFrRQYlf5wXREq8n4UegLU54tdjjmaysEtlWvqoHMsjkC01J7C/wN4KWbcr8LtHuSJmnoRa9yPqiRTKOk/YM= X-Received: by 2002:a05:6830:9c2:b0:606:1e0a:cc8d with SMTP id y2-20020a05683009c200b006061e0acc8dmr598659ott.265.1651821877389; Fri, 06 May 2022 00:24:37 -0700 (PDT) MIME-Version: 1.0 References: <202205031017.4TwMan3l-lkp@intel.com> <8704209d-d487-a297-b05a-5db99f5f808c@intel.com> In-Reply-To: From: Ard Biesheuvel Date: Fri, 6 May 2022 09:24:26 +0200 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH] ARM: dove: fix returnvar.cocci warnings To: Philip Li Cc: Dave Hansen , Arnd Bergmann , kernel test robot , kbuild-all@lists.01.org, Linux Memory Management List , Tony Lindgren , Russell King , linux-omap , Linux ARM , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" Authentication-Results: imf03.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=FPuuz+QY; spf=pass (imf03.hostedemail.com: domain of ardb@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=ardb@kernel.org; dmarc=pass (policy=none) header.from=kernel.org X-Rspam-User: X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: 605C620002 X-Stat-Signature: a9zpg7s16ekpghsj6isjeh34g66usk9b X-HE-Tag: 1651821873-489604 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Fri, 6 May 2022 at 03:12, Philip Li wrote: > > On Thu, May 05, 2022 at 09:31:37AM -0700, Dave Hansen wrote: > > On 5/3/22 00:21, Arnd Bergmann wrote: > > > On Tue, May 3, 2022 at 4:45 AM kernel test robot wrote: > > >> From: kernel test robot > > >> > > >> arch/arm/mach-omap2/dma.c:82:10-16: Unneeded variable: "errata". Return "0" on line 161 > > >> > > >> Remove unneeded variable used to store return value. > > >> > > >> Generated by: scripts/coccinelle/misc/returnvar.cocci > > >> > > >> Reported-by: kernel test robot > > >> Signed-off-by: kernel test robot > > > I checked the patch, and unfortunately it is wrong, the current code > > > needs to stay. > > > The problem is the SET_DMA_ERRATA() macro that accesses the > > > local 'errata' variable. > > > > 0day folks, do we have humans looking over these before they're going > > out to the list? If not, can we add some? If so, can the humans get a > > little more discerning? ;) > > Sorry all for the bad patch. So far, we pick up several cocci warnings that > we have confidence based on early result analysis and feedback, for these > warnings, 0day sends out patch automatically. > Could you please add a special header or something to such emails so I can filter them out? I am strongly opposed to such automatic spambot patch generation, as it wastes valuable reviewer bandwidth to save the bot operator some time, but it think it should be the other way around. We expect contributors to carefully prepare their patch submissions before sending them to the list, and automatically generated patches simply don't mesh with that. The fact that you use a bot does not mean you can ignore these rules.