Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp1791692ybl; Sat, 11 Jan 2020 02:57:23 -0800 (PST) X-Google-Smtp-Source: APXvYqzK787JiwMX1TxSyUdmE7lV0vGDXb2cmZvde8noLYHNjofUxNKBCLRF8scXlnToI0qhL9Zv X-Received: by 2002:aca:1012:: with SMTP id 18mr5695233oiq.151.1578740243445; Sat, 11 Jan 2020 02:57:23 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1578740243; cv=none; d=google.com; s=arc-20160816; b=uHmshS/KOFC4gPer7n6KlO0bmJQsJSdvzPfgdDVysuORg+XNfTUOG2Rxb7f4lQhIfT He4pDdBvUDgS6UkBFdY8LlZhRlT2ZosCsqPM9+KfD2KyDmy2Co21QPAaY5J6XiVx9bwj evZiYV1O8JghXz3TDsX662nlaI3DWGphRARpCmnXmLJvOobxEwn+zBoDRJjEn6NK4eMZ RqAdR4lhbuPgD/dC93bSDs3nfgNg6pe8d0wWtwnDK8/X/AHPY5AvYPPTLPpWNLG8I8F9 /gjqQmoUY/Kz8z7ZoA0iJLokko63IxcQFL02nhbUvh+4KvS65C04pPxh+opMeImZqRYK ujRg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=MZQVnsltuFIGEGv4VB2WGJgemVBmJS3Bgy+FfBsRHoc=; b=NIKND9O2qVofBnyYdJdDB25LWbjOGVzg7oyq4WliX9XQufUzfX1APUIxk7JwNdXj7T rUBjfsy5+Jmbffc4aH728KHZ0WeqjNqtUXWL+ohj0r54LXfXhGS8SvLxbOR+EqKAC4O6 fmS0EQ2BVn/V0b9/cxYZntDSX7kSLtdF1DKOHrR1rZp+SCHyq3l8ySYTWMG+Pf9VZ4cF 3GnkvPZ4BTowPlYSwFAMPAnzZLqD0XTI29l8rW3sZY1+6Bn4ZfvFwFuyJgDiO5rpshyf q0a0WmH9NY2LquSnKjQF1RGB6F14/6O9v+qDQ2F25oEwI24E4o3SeDcdBIhhKk0h010B xB5Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=H3aNzHKf; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l8si2785839oii.249.2020.01.11.02.57.12; Sat, 11 Jan 2020 02:57:23 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=H3aNzHKf; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729078AbgAKKzQ (ORCPT + 99 others); Sat, 11 Jan 2020 05:55:16 -0500 Received: from mail-pl1-f193.google.com ([209.85.214.193]:32778 "EHLO mail-pl1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728974AbgAKKzQ (ORCPT ); Sat, 11 Jan 2020 05:55:16 -0500 Received: by mail-pl1-f193.google.com with SMTP id ay11so1878329plb.0 for ; Sat, 11 Jan 2020 02:55:15 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=MZQVnsltuFIGEGv4VB2WGJgemVBmJS3Bgy+FfBsRHoc=; b=H3aNzHKfQEF2sU1RYhMU7P81CasygNH6mi0tGAiasCQkl7VqNPQ1sm8gx96zqYUsWM 3wCmeDoqd8ASepSb4y/RSiybh0RMeT4SxE2fq8RCYr4l/+vs5x7nk4/yWiQDXIlG+dM5 tvz1Zmo8t7sVV+dn44N1haiZvV2EgFhn6lM7hUpzXabkjxLzTf86Y3CN99a4MlRs3ntk vZpGujES41egbQt/Nk69Lah0PQxz4v9hAHB5N0FG4jsWIbxuJtIrriQZkCw7akuy0FXA l9xSwy2m4iaB3ZZ+ivzBFcVsHub54gwEXq4g4aAFm9NWCn+ZgD0xi6jEJaUJLpxyt511 uuXg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=MZQVnsltuFIGEGv4VB2WGJgemVBmJS3Bgy+FfBsRHoc=; b=W8FR8x7Y1V51hFXvffl6pK7VHxU52hWiVtkbOdsEZdKHhb8ADsoj286WA0gJG03+0U +XNXRN5LFXw0twg24zJlWkljpnfsLUGs0AGqL38XyMDST2ITv2GhdkZxXVFDfAEQJvvR vx4ENu4sO/6yxNW5czCgr1aQxm0qFGaRXEI1oUw+ulwgx4/PW901Uko412QL89kX7x// rnsVezY03sEFxKFRIkuVgH3bM9rDz6ZRNAifPecIRvMIuAAtQpuP0dVEmVsOn9ObLwrX PYh4MIsArYsYyNKzF2RrlGWmRGWTcH2nXPF+p4G0DO/mVQ9jeDLDAgksqR7mPil6mbCR TKTw== X-Gm-Message-State: APjAAAVQVlfSRaWX5Vr8ZprIivp7YGCpLHABv+6yB/AGcnU0Iy0o++Yy uzrZZ3XDzVACDtt6m+IrCAuxdifM/SZKulNoMYk= X-Received: by 2002:a17:90b:3109:: with SMTP id gc9mr11179878pjb.30.1578740115415; Sat, 11 Jan 2020 02:55:15 -0800 (PST) MIME-Version: 1.0 References: <20200110225602.91663-1-samitolvanen@google.com> <202001110830.00B8USV0024843@sdf.org> In-Reply-To: From: Andy Shevchenko Date: Sat, 11 Jan 2020 12:55:03 +0200 Message-ID: Subject: Re: [PATCH] lib/list_sort: fix function type mismatches To: George Spelvin Cc: samitolvanen@google.com, Andrew Morton , Andy Shevchenko , Kees Cook , Linux Kernel Mailing List , Rasmus Villemoes , Mauro Carvalho Chehab , Andrey Abramov Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Jan 11, 2020 at 12:35 PM Andy Shevchenko wrote: > On Sat, Jan 11, 2020 at 10:31 AM George Spelvin wrote: > > > > > typedef int __attribute__((nonnull(2,3))) (*cmp_func)(void *, > > > - struct list_head const *, struct list_head const *); > > > + struct list_head *, struct list_head *); > > > > I'd prefer to leave the const there for documentation. > > Not only. It's useful to show that we are not going to change those parameters. > > > Does anyone object to fixing it in the other direction by *adding* > > const to all the call sites? > > Agree. > Actually we have cmp_r_funct_t which might be used here (I didn't > check for the possibility, though). For the record, I have just checked users of list_sort() in regard to constify of priv parameter and only ACPI HMAT is using it as not const. UBIFS and XFS do not change the data (if I didn't miss anything). But the amount of changes there perhaps not worth of doing. So, maybe new type like cmp_w_func_t where priv is not const would be good enough for merge(). -- With Best Regards, Andy Shevchenko