From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f66.google.com (mail-lf1-f66.google.com [209.85.167.66]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8F36333BBB9 for ; Fri, 9 Jan 2026 11:00:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.66 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767956451; cv=none; b=JbVK0tmZQ5fNB0YvOfpFP7qa0bDttll1XM5jGII9SuGOkEYSG5dUWaVpLdDyN3ocsjaPzjlXNA9RB9vG8xZA16/58YG++uUMar4kJCEyZhJRXYDRL1HdAtJQP/HBCp5KbGO+JKr07P47k3HEoMjMrSCNnVeEpBJ7K1tu4ztWu1U= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1767956451; c=relaxed/simple; bh=hZXJGPMssGAfFbu+bAlt6gHtJ32mOvpL6uEOa1ztUIo=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=MyjlmJ63a5YqGg9mq5ZgIgcYayO111OZVoznSjfPr0UM1AemABpKC7uEC0duHr8qTZ+C/Z4olWJpOZemUguoJ+PM7FQnkmMp4R88cpJKB8IQ8aXNrFP+ClJ8MRTckrwHOcBCjwWfCwDREpwih+mMtf2SsnmhjJAsBA8EH1JFU2Q= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=yEUKQ9Nr; arc=none smtp.client-ip=209.85.167.66 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="yEUKQ9Nr" Received: by mail-lf1-f66.google.com with SMTP id 2adb3069b0e04-59a10df8027so4756165e87.0 for ; Fri, 09 Jan 2026 03:00:45 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1767956444; x=1768561244; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=bCAkSgcFGLWAnWdp4SQJtJGZ2mOqXTVgkdEMSVyfWUo=; b=yEUKQ9NrYWg/8MqEsPvrtT3Kx/sfUg3lBKkSTHfmNte2HfSRdYx2GmahD9EuuzflJs AV6i3ia8bK0dDQc2vqbUPank3zQivCP+vw+UQGTq5LhAbehOIIJo8j7AQMrL0oNyi2es t4Zq6vUxDlMkDUNCNXblFcQU8YfHmcbcJGtY1E/n9HyKuaVAkWBe2x+JWjCC2PlcGsLB 4d+2s0rbuZe/FmnE+a/Rf/ELkDiX24+4YMi5Ers9lCWtB6ZNX8Tjl+RSdjuhDcz4eAcW De7+iKvXGSvSps75XCFlVdoWCxh4vMuYGOTI93xCPpeZ6Nd0w+XWicmOfAmudo/hQME9 +m+A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1767956444; x=1768561244; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=bCAkSgcFGLWAnWdp4SQJtJGZ2mOqXTVgkdEMSVyfWUo=; b=UWJ9CxJ8J4ikaoz4HimQvyKABveGegpgD+uMQM9K8e234fDmTcgAPIv4cNCMGJHE5c +xHKtBlYpMZy7+CeoqVxO3QU5S+cfX/+wzjxRgEFXEUFR4oae01iB8Uv06FIcAfAh0HW mUrb/RQM8gtgjIomT3OHsMUpZV00uZ+oiHbGDVIkoOv+ZO376b9vRgKpqN8v6SQLwmUk FQVfM4HPT5UOtlEbfDLg6e6NYUF7BJYDNzFDkNqDasnx9P5Sz8jWLZQNVorba179SFi6 GFLt3h6jl/iRdKRAu2YNNJ74wJBUyd+l0UepfIdR1UU+MQhDc7Pj8b7h8m0qANvFP0gQ P2RA== X-Forwarded-Encrypted: i=1; AJvYcCUSM03Hv/2twuV4AtbF3EAzfACyAXVGOQB7hijYJssVZWY2g4nx1szuJK/eE2Kw9OVeUkdG42oSEX+XUpU=@vger.kernel.org X-Gm-Message-State: AOJu0YzFyFWDv7V9VDhPLODGROUWYnfVPvmHJlLs5o4T5puv3q4dMItm dqNRbVzDiqA6AYouajrA7ncdWNS5aAODtFYV3mrr+eId3UyE4SOAL4jhkYmv2B/eYE/rBZ3fniS VPW8nFy8= X-Gm-Gg: AY/fxX4rjDo9AfY+yeqSMMruuHD1D6GLPwxuL+H3nnnR4snIuYwTRL0l6hi1qyyubuw cDtRoWrsxY+Bjmf9PpZ46gFi9At+E7Rsae/cf24ELm0TahdnuV6P4p8uL/SfKosiN5n66Zwt841 u6mCNAJtivH3ceOCBYaELI8RiCwukeVuFkTpy9wOGYEM9x4LegPR8lbEne2Sq18DiyqxX1xv2uo 81gNMviAg4oeQ3/b76/IRsU0R4v+1r4oEH33s25qUJYRE1WG0vJlBcpSqfJRh0ToxcKBoDz6No3 +VCceDcyJQMxinrAId2iFkdAvYXzFHFIiyL1fiALhR3kUTPLqZsTdvtVp2ZwztGNj3PYtaePBX0 MuyY+JbVBklnWY9sGK7XKIPu62szc2lK6zu77fjiUFW8uZtouRmo7f5vBFY2v0HYWd3gwXqii47 tpe4GvJ/TK2Pyg7wTU X-Google-Smtp-Source: AGHT+IGXxI7r72eOkIy1KhlxJm58/JxPLQ5FF6CoxfgXHR2DOpaiFz27HbBI+CxtT+yMU4/t1/5Biw== X-Received: by 2002:ac2:4e04:0:b0:598:de38:9150 with SMTP id 2adb3069b0e04-59b6ebe3029mr3229097e87.0.1767956443522; Fri, 09 Jan 2026 03:00:43 -0800 (PST) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-59b728badebsm1800519e87.67.2026.01.09.03.00.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 09 Jan 2026 03:00:43 -0800 (PST) Date: Fri, 9 Jan 2026 14:00:39 +0300 From: Dan Carpenter To: Lorenzo Stoakes Cc: Steven Rostedt , Dave Hansen , Dave Hansen , James Bottomley , Dave Hansen , linux-kernel@vger.kernel.org, Shuah Khan , Kees Cook , Greg Kroah-Hartman , Miguel Ojeda , Luis Chamberlain , SeongJae Park , Dan Williams , NeilBrown , Theodore Ts'o , Sasha Levin , Jonathan Corbet , Vlastimil Babka , workflows@vger.kernel.org, ksummit@lists.linux.dev, Jens Axboe Subject: Re: [PATCH] [v3] Documentation: Provide guidelines for tool-generated content Message-ID: References: <611c4a95-cbf2-492c-a991-e54042cf226a@lucifer.local> <85614f7f-f217-47e5-a9f7-0a012f6e6ecd@lucifer.local> <6e9cab54-7b66-45e9-af96-e52b3eba1034@intel.com> <5a301272-31ea-44b8-9518-8151edca6c06@sr71.net> <20260108151437.3188cd53@gandalf.local.home> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Fri, Jan 09, 2026 at 07:48:35AM +0000, Lorenzo Stoakes wrote: > +cc Jens as reference him > > On Thu, Jan 08, 2026 at 03:14:37PM -0500, Steven Rostedt wrote: > > On Thu, 8 Jan 2026 11:50:29 -0800 > > Dave Hansen wrote: > > > > > On 1/8/26 11:23, Lorenzo Stoakes wrote: > > > > I'm also not sure why we're losing the scrutiny part? > > > > > > > > Something like: > > > > > > > > +If tools permit you to generate series entirely automatically, expect > > > > +additional scrutiny. > > > > > > The reason I resisted integrating this is it tries to draw too specific > > > a line in the sand. Someone could rightfully read that and say they > > > don't expect additional scrutiny because the entire series was not > > > automatically generated. > > I mean you are making an absolutely valid point, I'd say that'd be a rather > silly conclusion to take, but we have to be wary of 'lawyering' the doc > here. > > > > > > > What I want to say is: the more automation your tool provides, the more > > > scrutiny you get. Maybe: > > > > > > Expect increasing amounts of maintainer scrutiny on > > > contributions that were increasingly generated by tooling. > > > > Honestly that just sounds "grumpy" to me ;-) > > > > How about something like: > > > > All tooling is prone to make mistakes that differ from mistakes > > generated by humans. A maintainer may push back harder on > > submissions that were entirely or partially generated by tooling > > and expect the submitter to demonstrate that even the generated > > code was verified to be accurate. > > > > -- Steve > > I don't really read that as grumpy, I understand wanting to be agreeable > but sometimes it's appropriate to be emphatic, which is the entire purpose > of this amendment. > > Taking into account Jens's input too: > > +If tools permit you to generate series automatically, expect > +additional scrutiny in proportion to how much of it was generated. > + > +As with the output of any tooling, the result maybe incorrect or > +inappropriate, so you are expected to understand and to be able to defend > +everything you submit. If you are unable to do so, then don't submit the > +resulting changes. > + > +If you do so anyway, maintainers are entitled to reject your series without > +detailed review. This is too subtle. In real life if we suspect a patchset is AI Slop, then we're going to reject the whole thing immediately. No one is going to review all fifteen patches one by one as if we're searching through monkey poo for edible grains of corn. The AI slop patches I've seen were not bad actors. Someone saw a TODO in the file and thought that AI could solve it. The patch compiled, it was formatted correctly and the commit message sounded confident so they sent it. To me the audience for this is maybe a team working on AI and they don't have any kernel developers on staff so they assume they're being helpful sending unreviewed patches. The message should be that every patch needs to be reviewed carefully before it is sent upstream. I've been asked to review patches like this in the past. Get outside help if you need to, but every patch needs to be reviewed. regards, dan carpenter