Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp7756394imu; Fri, 28 Dec 2018 04:26:15 -0800 (PST) X-Google-Smtp-Source: AFSGD/V5i4dFCf71wc/DCwRY694kglngKkSWJgsB0en8gE3x6bNZiOCM1Mhbl3TxS5ngwWl/bMTS X-Received: by 2002:a62:f5da:: with SMTP id b87mr28354503pfm.253.1545999975864; Fri, 28 Dec 2018 04:26:15 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1545999975; cv=none; d=google.com; s=arc-20160816; b=DiFgAKmy6NQwj6wNwe8xHH/dH3HHSiaA8YIiQqQ0koir+iNiAFpUZ91RdMb4MsnKt5 YqW/AS9PA0C3J66/RytBRNTOPL7mkkoiI/tirHXXw+XML7O8mn6fZZJRRiCQRWgYk9DW MuGhIvm7L+YtbNdjVJfK0iZ21Qy3mUqVpcmK4kn8DIVGjlMdGn0x2vr/lz5BDvehZ06C dwQnOQfeLAoPBwXEdxXRsIoshMCPfC+jjQrCNFpfNfj90OHkgiX6KmfHy0hZteLla6Bt uVhLMub/6ZYLIwsdfdCIwhnRDr49obvtgZMfUPv7SBdGDm/zQNylEcuMTGT7zxq1+cFL eeIw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=V2AFYL2mLjRmFCoGRAlQVdhL6wSwHg2bszKZGFyNZuE=; b=Y0MjJJR8NKRQZgVsmnUrklKLhqQ1Y1aS2lsYzqSFADbsHMciD9ng/vwZb6oNTSc+iZ n5Ehr8MlLBP1UQnbh9rtkzPLHB+smX8LYiJ9Tnax4UEfFm78LgdB3FAbrBO4mikFCXm1 IPo72o3Fc9tjoIKzGq2jm6pBQzFkB//hjd1hOxeUCS/JG3x6cPRXThgZIumqpD89cMSH hX729GVPENNFzSgTyPNUXaU9Vg+0XiHcb525xYy/QJRSgMFT9q1U+18AodIgV4FmDYCG 6QIzTmM4T02VKI+3gT9DJxZYLOvASCmqNpf+XDngx1MoyE6OcG1lJ7MPJqpQFyxi3QDB sioQ== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 14si17895530pgg.425.2018.12.28.04.26.00; Fri, 28 Dec 2018 04:26:15 -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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728874AbeL1Isg (ORCPT + 99 others); Fri, 28 Dec 2018 03:48:36 -0500 Received: from verein.lst.de ([213.95.11.211]:60672 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726700AbeL1Isg (ORCPT ); Fri, 28 Dec 2018 03:48:36 -0500 Received: by newverein.lst.de (Postfix, from userid 2407) id C001468AFE; Fri, 28 Dec 2018 09:48:34 +0100 (CET) Date: Fri, 28 Dec 2018 09:48:34 +0100 From: Christoph Hellwig To: Huang Shijie Cc: akpm@linux-foundation.org, linux@armlinux.org.uk, arnd@arndb.de, gregkh@linuxfoundation.org, hch@lst.de, m.szyprowski@samsung.com, robin.murphy@arm.com, sfr@canb.auug.org.au, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org, shijie8@gmail.com Subject: Re: [PATCH] lib/genalloc.c: rename addr_in_gen_pool to gen_pool_has_addr Message-ID: <20181228084834.GA8658@lst.de> References: <20181228083950.20398-1-sjhuang@iluvatar.ai> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181228083950.20398-1-sjhuang@iluvatar.ai> User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Dec 28, 2018 at 04:39:50PM +0800, Huang Shijie wrote: > Follow the kernel conventions, rename addr_in_gen_pool to > gen_pool_has_addr. Which convention? The old name certainly looks more sensible to me. If we really want to change anything about this function I'd suggest to drop the size argument, as the address itself should describe the region good enough.