Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp4581601pxu; Wed, 9 Dec 2020 23:28:23 -0800 (PST) X-Google-Smtp-Source: ABdhPJxk7EuX2oKRkHqRURJNs5ndZ8PDhc6bPqyFbvR1C38vxeeUGPn/qTjuSSqWyjKsSz10fTGh X-Received: by 2002:a17:906:d8dc:: with SMTP id re28mr5271857ejb.168.1607585303541; Wed, 09 Dec 2020 23:28:23 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607585303; cv=none; d=google.com; s=arc-20160816; b=xzJQWYqiJJG9WxZBd4G4stY8h5TGoL1NePXgGzMZupKZ9TWAn7jWxXG3jFWA6RO8vg BgkOWqO10ji/5ApoPTX1bCz2Hwuftwps4Kk8eEmm63upNmMaS3L6/sJVqiuE+R2il8LT X9NQp8Rt0eWnd0SMHa7qA2XQ4WZxT18ZyZz6kOiZpgbnidE/RFlzRLV0ocDBE2FGtyBa 5n7wtBYUujAXLvnYqgaXsI2jZAAXuTprQuGHzdTfkKlysg//cVCZv1wRvkjr78piE20Y q7qKcgzREc0ZK2iEGRBah1UGDAkmxoKODxr0rdKEUtKoVlN+jIid64JrmM+gz96uhRi+ sh1g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:from:references :cc:to:subject:dkim-signature; bh=yUO807Um276FtA7btiIrTKbtdq/+/urED8qmMHlTed4=; b=TX5s7Uo81vkdhjW7kLN7q0Lbq6LWnjz/K58quFIyUJTFdzUksMWGxk8u7xG0isdqZw LKs69isW0p5XIn/qO0ibZSDdcF0Ms4/rlNOan3oeRgmGngixTZzGjnktCSlhTbJ/w+ni xUN0TCRcMAHYFwTpGthb2U7ZWDbGzbTV5wU2yJbTDlDZqm5EFEC9dHXen3pdWc07S0Xm 85JWK/yKG7ABCvUNfISDUzrsMxzYdCUgv2OMwDzpCh9Qgvta8ps11ntbN0kcJ3eZVELt 77/S0pTbjTmV0wQMSicWE1nT5j0GmSR+86aN4Sbgn5J69W0GPZ06bI8nE+ddmSxpjY2l 6rwQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@infradead.org header.s=merlin.20170209 header.b="uHD/s79Z"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id mb18si2065447ejb.284.2020.12.09.23.28.00; Wed, 09 Dec 2020 23:28:23 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@infradead.org header.s=merlin.20170209 header.b="uHD/s79Z"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1733231AbgLJGRR (ORCPT + 99 others); Thu, 10 Dec 2020 01:17:17 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41352 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728264AbgLJGRQ (ORCPT ); Thu, 10 Dec 2020 01:17:16 -0500 Received: from merlin.infradead.org (merlin.infradead.org [IPv6:2001:8b0:10b:1231::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 616DCC0613CF; Wed, 9 Dec 2020 22:16:36 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=merlin.20170209; h=Content-Transfer-Encoding:Content-Type: In-Reply-To:MIME-Version:Date:Message-ID:From:References:Cc:To:Subject:Sender :Reply-To:Content-ID:Content-Description; bh=yUO807Um276FtA7btiIrTKbtdq/+/urED8qmMHlTed4=; b=uHD/s79Zc5ENvsKx0qHo2KxKmA pJBiDhpJWeNP/K7QAEdEgmuAtjnhBBlL+eTqEB2F+l5Z9bSt4ZJgg8EDuPvI0OS/FrsD6V1bxjxuZ vdn3/eMlL+MoNqLnRzJgbj/ubvDBcir5gISijnyCscRqoKzimp6Riin/CUyC0LWLqxt7WDu8HPUN9 XPEfymjb71OOUQEdY8t0NOA/IGnW9v7/LqMiFYG8JwnPYFx2rhbnzvSrk3U4WdI9C4ho3Z2tEkj8w H0r24Lod3LzAKiPxMkOUFqtSwPQyGjgKDQBFgseqbI5C/FRHsrdW4Ihcr6GSI1J1tLDAqq5GqeTe3 w6NxkaMQ==; Received: from [2601:1c0:6280:3f0::1494] by merlin.infradead.org with esmtpsa (Exim 4.92.3 #3 (Red Hat Linux)) id 1knFFa-0000lG-Tj; Thu, 10 Dec 2020 06:16:27 +0000 Subject: Re: [PATCH] block: blk-iocost: fix build for ARCH with missing local64.h files To: Christoph Hellwig Cc: linux-kernel@vger.kernel.org, Tejun Heo , Jens Axboe , linux-block@vger.kernel.org, Ley Foon Tan , Mark Salter , Aurelien Jacquiot , linux-c6x-dev@linux-c6x.org, Peter Zijlstra References: <20201209204657.6676-1-rdunlap@infradead.org> <20201210060742.GA19263@infradead.org> From: Randy Dunlap Message-ID: <04828b4e-8791-6f3e-8984-9de06f40c85a@infradead.org> Date: Wed, 9 Dec 2020 22:16:20 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0 MIME-Version: 1.0 In-Reply-To: <20201210060742.GA19263@infradead.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/9/20 10:07 PM, Christoph Hellwig wrote: > On Wed, Dec 09, 2020 at 12:46:57PM -0800, Randy Dunlap wrote: >> When building block/blk-iocost.c on arch/x6x/ or arch/nios2/, the >> build fails due to missing the file. > > Please mark it mandatory-y if the asm-generic version is suitable > for everyone and random pieces of kernel code are supposed to include > it. include/asm-generic/local64.h has comments about some $arch could do its things better/faster instead of using asm-generic, but no $arch has done that since 2010 when it was added. Is that conclusive? If it is, why even use mandatory-y? Why not just change all occurrences of to ? thanks. -- ~Randy