Received: by 2002:a05:6358:3188:b0:123:57c1:9b43 with SMTP id q8csp418618rwd; Wed, 7 Jun 2023 01:34:48 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ5hDisJuybha4U7ivJUbwcmriMPDhV/2I3y98d8t29cjP+KkFwvXDuE0dWbDC/ECHXbg3pT X-Received: by 2002:a05:6a00:98c:b0:653:ac91:89a with SMTP id u12-20020a056a00098c00b00653ac91089amr7467533pfg.9.1686126888167; Wed, 07 Jun 2023 01:34:48 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686126888; cv=none; d=google.com; s=arc-20160816; b=nJznNDwHxQ+IWyW6dlKRbgWoADCIfBAPo+3FF01xDamH5C+dLcPXvPbvS8yqwRBEfj s7rrVTN6TFLdL+8JGpK4jDAKUzrb+y5bP3ysBruPUpsfPGbkANSRcsn1BNLbF7iNu6i6 Z5k/DWY9LhcBBLfeqfezfwLo35+mmqriszpqODWeh9Zx2SGQt5X61t7ZtSZcmOLY9nGf G8vhjVgIzipdkk5J0ERx21SZ3Kd4nnH3uzJ2i4/GGf19CzeFk3Wy8MtSl9tHy4GDXQ+S xpyWK8oEt4G0ifNMH5D2CNWHpHVR+UZxmVBl3RYL/661bjLtZXUpEMmGRBuQ7mLtENqY S92w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:user-agent :content-transfer-encoding:references:in-reply-to:date:cc:to:from :subject:message-id:dkim-signature; bh=Hg9joOitzyrlswtLr/eslIOSFjCc+Bk2gMqM/S9yDPw=; b=y05/At9Fx3ZUxWygAJKnGHtWd9vxDn0bfxPQArgW0HHWkrKh2eRidbEpcawV8oNWDW qf6BDgGdUGtG0Z4lpE0xGT2VmcQwBz2RDR781HcUeLHiZOP+op7SqzbQ0CDPj7fFukF+ NK9VZdhZBe46G+pe8u/AeZyVt7hs3v8t8YwuRFp++/ginV6mpPli+nqUN8nVuFcCGny0 c/mWTGiqjp52QZ2FertpBhejLoh0icWEdcwFL+XXIK2yvFni24iTrOBnbpSkrJjfq/WF DRE5YD6SUSHOwR52sjuMpDdHtyVVeqLHY+Qd8vYXJzscobej8Bg/Vw61R4vAn28N20V6 Zl3Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sipsolutions.net header.s=mail header.b=dAo85xRu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=REJECT dis=NONE) header.from=sipsolutions.net Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id v13-20020aa799cd000000b0065b4e2b52c5si4498203pfi.196.2023.06.07.01.34.33; Wed, 07 Jun 2023 01:34:48 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@sipsolutions.net header.s=mail header.b=dAo85xRu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=REJECT dis=NONE) header.from=sipsolutions.net Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239472AbjFGI20 (ORCPT + 99 others); Wed, 7 Jun 2023 04:28:26 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37466 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238263AbjFGI2Y (ORCPT ); Wed, 7 Jun 2023 04:28:24 -0400 Received: from sipsolutions.net (s3.sipsolutions.net [IPv6:2a01:4f8:191:4433::2]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 37B3F1B5; Wed, 7 Jun 2023 01:28:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sipsolutions.net; s=mail; h=MIME-Version:Content-Transfer-Encoding: Content-Type:References:In-Reply-To:Date:Cc:To:From:Subject:Message-ID:Sender :Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-To: Resent-Cc:Resent-Message-ID; bh=Hg9joOitzyrlswtLr/eslIOSFjCc+Bk2gMqM/S9yDPw=; t=1686126503; x=1687336103; b=dAo85xRu7FJnjyGyxWa0Iu4I6MEDjHQ/AMvRXNeRu7y108g E6zNGVDjqwvWlbWijx7F1X96RZIVmTs/z9s7x872E9pARg5i/gBw5Baos9Xe3UtIbrNbRBDiQ11bF h4sz+I3yY3TkbDmtyNpNe4Ig55PEAixgjuMyVum8euWy56u7y6fZ5/VBbNZ0kffYAumL9Lu0YZ0Sg Irx69vknRHHF933HiVjvCKUXPXsuTKkbBVGwLrg2t18XmmXBNsoZ+CcIV/7VwrF+3fnMOUqgQ9Pp0 Ukccr0TaSsJqHZjSJ8+OQbrYlqR9s09WYkYUyglBb8+aZuRubyBZ+k1WqTq/WYHg==; Received: by sipsolutions.net with esmtpsa (TLS1.3:ECDHE_X25519__RSA_PSS_RSAE_SHA256__AES_256_GCM:256) (Exim 4.96) (envelope-from ) id 1q6oWa-00GTQu-27; Wed, 07 Jun 2023 10:28:12 +0200 Message-ID: <57443d420183b7a0bcab8041c2dfe3ba44e054f4.camel@sipsolutions.net> Subject: Reported-by/Closes tag for uncommitted issues (was: Re: [PATCH v2] uml: Replace strlcpy with strscpy) From: Johannes Berg To: Eric Biggers , Azeem Shaikh Cc: Richard Weinberger , Maxim Krasnyansky , anton ivanov , linux-hardening , linux-kernel , linux-um , "Jason A. Donenfeld" , kernel test robot Date: Wed, 07 Jun 2023 10:28:11 +0200 In-Reply-To: <20230607042340.GA941@sol.localdomain> References: <20230606182410.3976487-1-azeemshaikh38@gmail.com> <1833651082.3690424.1686084717406.JavaMail.zimbra@nod.at> <20230607042340.GA941@sol.localdomain> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.48.2 (3.48.2-1.fc38) MIME-Version: 1.0 X-malware-bazaar: not-scanned X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS, T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2023-06-06 at 21:23 -0700, Eric Biggers wrote: >=20 > > > > Reported-by: kernel test robot > > > > Closes: > > > > https://lore.kernel.org/oe-kbuild-all/202305311135.zGMT1gYR-lkp@int= el.com/ > > >=20 > > > Are you sure Reported-by and Closes make sense? > > > AFAIK the report was only on your first patch and nothing against ups= tream. > > > So stating this in the updated patch is in vain. > >=20 > > I left the metadata in only for the sake of posterity. If it's not > > helpful, I'm ok with removing it. > >=20 >=20 > IMO using Reported-by in cases like this is harmful, as it makes commits = seem > like bug fixes when they are not. I've yet to see anyone disagree with that, and yet, the robot actively asks for such tags to be included in patch resubmissions. On the one hand, I can understand their desire to be recognised for their efforts. At this point then we might suggest that we introduce a different tag, say "Improved-by:" or "Issues-found-by:" or something. On the other hand, I don't feel like we should give a robot more recognition than we give _people_ reviewing, and we currently really only recognise them by a Reviewed-by tag. Then again, that doesn't work with the robot - it is pretty much looking at each patch and only comments on a small fraction. We also really don't want it to comment on each and every patch ... So it seems we should ask the robot maintainers to just stop suggesting those tags? johannes