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;
From:   Nick Alcock <nick.alcock@oracle.com>
To:     Luis Chamberlain <mcgrof@kernel.org>
Cc:     masahiroy@kernel.org, linux-modules@vger.kernel.org,
        linux-kernel@vger.kernel.org, arnd@arndb.de,
        akpm@linux-foundation.org, eugene.loh@oracle.com,
        kris.van.hees@oracle.com, thunder.leizhen@huawei.com,
        Steven Rostedt <rostedt@goodmis.org>
Subject: Re: [PATCH v9 3/8] kbuild: generate an address ranges map at
 vmlinux link time
References: <20221109134132.9052-1-nick.alcock@oracle.com>
        <20221109134132.9052-4-nick.alcock@oracle.com>
        <Y3BeTbHxj9OmmmIY@bombadil.infradead.org>
        <87v8nhv5vq.fsf@esperi.org.uk>
        <Y3QDIVy0c01Tie3L@bombadil.infradead.org>
Emacs:  why choose between a word processor and a Lisp interpreter when you
 could have neither instead?
Date:   Wed, 16 Nov 2022 16:06:06 +0000
In-Reply-To: <Y3QDIVy0c01Tie3L@bombadil.infradead.org> (Luis Chamberlain's
        message of "Tue, 15 Nov 2022 13:22:41 -0800")
Message-ID: <871qq2vq8h.fsf@esperi.org.uk>
User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.1.91 (gnu/linux)
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?wUBMnzjB7eDRottIIPisBmvqFlBqUL7h9/s5eK0ibtg4IjUiRTXj7Ys+u8nV?=
 =?us-ascii?Q?W/jqdnkAVXXXxrv5TsbAw+h9Aro7Xod+AmCl33faGT2QYIvNoE2EMAmPKLcM?=
 =?us-ascii?Q?+nlyB2gF2+GHB1KyL6szRLTTpIpDlPiJnd7h4b1J1S6EqMJIqiVCKTMoBb5K?=
 =?us-ascii?Q?iwQarw/gZKYqBC9+tZ870q7ZlXlVhWlblt6h7Rka0+0YNnc5UNkiv4qHa9Vu?=
 =?us-ascii?Q?YUmNztCHA6/jGdRXAZQ9pMm9XflhIh9CNDGVvmFEIhOWD8U8NVn2wX91YHr8?=
 =?us-ascii?Q?sxLeHmOCMlCz3VSnzHP9kIxri2gVtFhJIq/HJPrQUFzE+hySrzEm5jMPWj+K?=
 =?us-ascii?Q?CYi05pJPMmnWfFSQcq2+PgrHC/QwcJDcQncpMsb+NdrmU3h5ukqduwdYqx2u?=
 =?us-ascii?Q?UYNmwPKYFVLF7DZVQGfSF8qwDSWYetqYUc+GrYe9S7e5eI2b9MTdbhmtiMqJ?=
 =?us-ascii?Q?2h8dl/MMYqPAYnm1ap+9ahzlnymEgakFRAmvm1RxVlDrT2rY3/vOC0MZ1JJ8?=
 =?us-ascii?Q?S8VzchqlqKTveNL0G66IxzujEXKxBiz/QAC/d5S8h5XpBaF0I7eoBJQLzBjn?=
 =?us-ascii?Q?mrd9TV9QWGVrbOGekTP00bwxKGIbwmbmfg8KFUkFHKSslXBG/CpSpZb8X5mn?=
 =?us-ascii?Q?l66QjYqxeIpS2aorU/70ERRuisswQBahhYQ4S1hmbayPVLV8krqNXuLITf/F?=
 =?us-ascii?Q?0fYZtlIKHj11s49yWrUcne2pOU+LyxiyLr3H8ljopZ93C3UjHWnAz/sfHIDl?=
 =?us-ascii?Q?pB1rynIp9kpRjRtiGMVcIW7gF8v4rlKi79K7eOxwizIzKWhN8CljfebP+Ycx?=
 =?us-ascii?Q?rPFbx+hJJs3zU6dwjTSWxAcsgIrK0OPt42rRpJN3M+JPHwR2zT4xaxc4/95W?=
 =?us-ascii?Q?owm0ULppbZe5lvCJC6t+jSG4U6LzdtdOAQD+2KhlVDttfV9PbeV7DracUZjT?=
 =?us-ascii?Q?e+qzgPhqaGvlnmEmWucEu9/WWEKgueT9IUP4Q2pPe133/CQAOZVJBEastWVe?=
 =?us-ascii?Q?rQ+/tGgr9P1H49Oe+B4L/9AX+xr3jDKeEkuPkQ6kIsG5oFz9FAyMuOMNLqN7?=
 =?us-ascii?Q?8JDWrrOBL9HEHKsUU3ZQgmMEbAaIYPG4+7XtQmgNyaMe0jpVe3qztaISrH2e?=
 =?us-ascii?Q?P08dReuOaOi/H9qLV8AinrqpG/ZykhVtlyXC8nxShTyQV+IRaD7ZPe7VGdAj?=
 =?us-ascii?Q?HxcvLQp8TqDcVqIJnuXn1phJE4wG3QX4B1Q+eJbuhPtmNe3lzFz67UzeSv6k?=
 =?us-ascii?Q?VfeLI2C0rZHsRZEvTg8ACNRdpsyx3VdK04i3sHigxnUXVpLj4EJtY7XISwVo?=
 =?us-ascii?Q?HSbm3po6ejMVQfIvroaldTqdfTWvOr91EBm4azTlbL54TeMOkp44q3JRyIXC?=
 =?us-ascii?Q?xrzDlOD8CXO/IVs8KcuhnZxOgDoJqJGz+LiO197X4VXDfegInnJbbDbRwveU?=
 =?us-ascii?Q?wFnPHTbbCxKqLlJUipNieY7OjdqlNik9YshXlC6S8xqhGMskrcz4dR88wp4s?=
 =?us-ascii?Q?M2JiFxvn4ioKE3wbbx18Z/a9MCMm6IFn8oAQiaw26NPSZZ2yD33jTZzDT+qv?=
 =?us-ascii?Q?pK8Bm3kOOQv/nPym3UDJHRjv55m7F4NubURNj6pwUW+vh31TlO8T9LJZvOiU?=
 =?us-ascii?Q?Mw=3D=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?us-ascii?Q?pl5ma0ayzNnXlvWIORqrIwcMf9+xNDZj07Y/TULSGE6q6eP744+qdcK54Sma?=
 =?us-ascii?Q?mVfTC4Gzo11XG8KOVrKjv6FRRX0sA32YWVGHmrLYGAHbjDA8bUxLuv4cC/8k?=
 =?us-ascii?Q?Iydkg7uIqdimQtCM18RrcxXjaOnYqVWhWGceSnXvSr1OkQjLamOAlq42ikU/?=
 =?us-ascii?Q?GkpQlflh6y8yKAc/7oLxnOBJeWlTQ209J9lGGTAzyPC2WtFmJ+7yhtCovCYm?=
 =?us-ascii?Q?wS41uIZjzYgLDowKanpBCoQSOnuQ4ynor7garL+21+ISGgLRjtbJlhftzjfH?=
 =?us-ascii?Q?1IWHT3/t9GguhKkjXuQnKEXOn64gOFamy4ERrvXTe5MRKeCrJMjWNqDhC5or?=
 =?us-ascii?Q?OTPK3iW2SrC1xYSQBFDpITAw4+9iQrW2xRweoWQxa7RexmtuodSreoNs/Zib?=
 =?us-ascii?Q?eE/rHDLTK+8a3l6CURJnrYLIds/EIiqtQMNn5Bvv5b38LoUPqZnXjGDO2KUY?=
 =?us-ascii?Q?0JXtKji5n2klaIl5U3D/UpzOt8+4Es11y4cd38xZUhtLsvv3oq8C9zFRRGUt?=
 =?us-ascii?Q?5+gZd2hnej2qTM+cNhHuNa9sTNfOLRiiGWJ0QWZL4ilb1tK2PSOrq4Vi1yMk?=
 =?us-ascii?Q?O3rSeaf6rcZxAcg9+G/zCOf3GmCIxgjX4cpKSqPEFwzpmPA3V7ISBG+6FlTh?=
 =?us-ascii?Q?uFsvYtbcKq/JpZ8bTz0IcPG0BzP/ILcGkOt0+6wZ6jCZPe6BU3Me7LLIOpXw?=
 =?us-ascii?Q?iikVzoZFfbaJDdGIMR6YW9w56exrkssJFdHehs0Ym9GzZhEVd83UhC5AQUMn?=
 =?us-ascii?Q?aW5kvMQuVZ/fBQrXBJcoTPB6ZqKg1u8i5hHv/Xg9ZmsoYL7STUJlBemH2tAz?=
 =?us-ascii?Q?i6JQ1ALicK/JooyY/7AV2FAnZ9xc+ITO2ZhGSNsZ4mLnyTlgIadpWzzzvBFu?=
 =?us-ascii?Q?Fr2cPBOCmZnPzQLhaJXo4XILtMAZZ4zB3mzNedLImE4MxU8JS9HBjlTfev4n?=
 =?us-ascii?Q?TDT6j4e4ma5DnJz9npIaH+ACBJAhrnxehAfAIa3gUlQ/KPGao0UVTBaUZrc1?=
 =?us-ascii?Q?T3Pm?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: a14f5fcc-d99b-4bcd-43a9-08dac7ec7ac0
X-MS-Exchange-CrossTenant-AuthSource: DS0PR10MB6798.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Nov 2022 16:06:11.2913
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: F8Wmpdeqi6UTlpI5GLfTp84s40MrvftVESnjf4JmKZeJIH6nLV3VQF6ebFvG7rfA513NfjSOCPVpp2km/VNvJQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR10MB6177
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.219,Aquarius:18.0.895,Hydra:6.0.545,FMLib:17.11.122.1
 definitions=2022-11-16_03,2022-11-16_01,2022-06-22_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 phishscore=0 bulkscore=0
 mlxlogscore=310 spamscore=0 adultscore=0 malwarescore=0 mlxscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2210170000
 definitions=main-2211160112
X-Proofpoint-GUID: cIK96hoJz8mV8AzYFaEjfvGYOULDhvD6
X-Proofpoint-ORIG-GUID: cIK96hoJz8mV8AzYFaEjfvGYOULDhvD6
X-Spam-Status: No, score=-2.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE 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: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 15 Nov 2022, Luis Chamberlain verbalised:

> On Mon, Nov 14, 2022 at 04:48:57PM +0000, Nick Alcock wrote:
>> [Added Steve Rostedt to Cc:]
>> 
>> On 13 Nov 2022, Luis Chamberlain stated:
>> > This does't say why we'd want this. So either you merge it with its
>> > first user or you explain here why anyone might find this useful.
>> 
>> Uh... the first user is later in this patch series? If you want each
>> commit to have a self-contained explanation, I could certainly note why
>> it's useful for said first user in this commit message (and adjust other
>> messages similarly), but I had previous complaints that commit log
>> messages and the cover letter were repeating points, so I was trying to
>> reduce that kind of thing.
>
> Commit logs should be self contained. The reason for *why* we want to
> add ranges should go here, not the cover letter. You can be terse in the
> cover letter over the general solution.

Ah, OK, my apologies: I was writing things precisely backwards then
(rationale in the cover letter, self-contained-but-no-rationale in the
individual patches then tying it together in the user at the end).
Whoops!

I'll rephrase as you suggest in the next round.

(And I'll definitely rephrase the cover letter again -- I tried to add a
decent rationale in there but from the sounds of things comprehensively
failed this time. Sorry!)

-- 
NULL && (void)