From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 08F90C282DD for ; Fri, 10 Jan 2020 16:45:09 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id BF58D206ED for ; Fri, 10 Jan 2020 16:45:08 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=chrisdown.name header.i=@chrisdown.name header.b="grsTIxw6" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org BF58D206ED Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=chrisdown.name Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 5A7C68E0005; Fri, 10 Jan 2020 11:45:08 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 5582B8E0001; Fri, 10 Jan 2020 11:45:08 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 46DCC8E0005; Fri, 10 Jan 2020 11:45:08 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0190.hostedemail.com [216.40.44.190]) by kanga.kvack.org (Postfix) with ESMTP id 30A508E0001 for ; Fri, 10 Jan 2020 11:45:08 -0500 (EST) Received: from smtpin28.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with SMTP id D27A41269 for ; Fri, 10 Jan 2020 16:45:07 +0000 (UTC) X-FDA: 76362299454.28.neck99_66ebccb9da55f X-HE-Tag: neck99_66ebccb9da55f X-Filterd-Recvd-Size: 5085 Received: from mail-wr1-f65.google.com (mail-wr1-f65.google.com [209.85.221.65]) by imf24.hostedemail.com (Postfix) with ESMTP for ; Fri, 10 Jan 2020 16:45:07 +0000 (UTC) Received: by mail-wr1-f65.google.com with SMTP id q6so2423847wro.9 for ; Fri, 10 Jan 2020 08:45:07 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chrisdown.name; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=ympT5AY/DLl6UvuOCQnir9hVpWD64MMWty4h6MSY0Uc=; b=grsTIxw69aF1wk5b95935hZWjq6OuMW2LUVE+UZgYwNKRK2xArvuAAXFlmsla9UBic QoxoszrvgwSlKPFJlE6/tOUAJ+8skkaNMQWEI6v7uWzO3mvd9qiUS/otfZMFQUAlHm/w 8up97pyFZiEUl3kFEo+kNrPE/vlE2s0yDRVBA= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=ympT5AY/DLl6UvuOCQnir9hVpWD64MMWty4h6MSY0Uc=; b=CI7nVNFdm2MuqRObNtNuVu2Iu2oPIMM3+oLSPunY62kULHWounQQJiKUTU2DjCyRpi nOmo+B2figq/oVO7nwQWDPBUNFujsm8JLf2983hhuYWEsWlqVSuOaBCvP1xKMvZbTELY /U1/EDpIFQJ9+7fTD++x7BlUgoNIatm1CrOIdCJi6G/7CL9+ecMoaK1YHQMawaRVJ9PT bUKOou8jVH98swiAbOhdtonR3rmwh/+ky/nWd2FcLBRXi74IBTOrwf4duLnQxlmZX5eH /u31Xog1mj6b8MCM49uuubfYsmBh+RALThrSD0SN0m5MAKtWbQyNFcvNJXUsxbl0rAzo adqw== X-Gm-Message-State: APjAAAVAAkEJb9pYJDNIkuNcMfFAUffNscc7fWmdt0lhdBf72i+xtuRN ji5MZA2sTIM5fr4KAqL1fDJRKw== X-Google-Smtp-Source: APXvYqx8rXwpLQ7ifI81LfMYzyilHJtApNJMIaEHlbBH7ZTahixW6jrbn4b5p2Rj+1+wR/12UcK4Gw== X-Received: by 2002:a05:6000:1288:: with SMTP id f8mr4518896wrx.66.1578674705827; Fri, 10 Jan 2020 08:45:05 -0800 (PST) Received: from localhost ([2a01:4b00:8432:8a00:63de:dd93:20be:f460]) by smtp.gmail.com with ESMTPSA id x10sm2823222wrp.58.2020.01.10.08.45.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 10 Jan 2020 08:45:05 -0800 (PST) Date: Fri, 10 Jan 2020 16:45:03 +0000 From: Chris Down To: Hugh Dickins , Dave Chinner Cc: Chris Mason , Amir Goldstein , Linux MM , Andrew Morton , Al Viro , Matthew Wilcox , Jeff Layton , Johannes Weiner , Tejun Heo , linux-fsdevel , linux-kernel , Kernel Team Subject: Re: [PATCH v5 2/2] tmpfs: Support 64-bit inums per-sb Message-ID: <20200110164503.GA1697@chrisdown.name> References: <20200107001039.GM23195@dread.disaster.area> <20200107001643.GA485121@chrisdown.name> <20200107003944.GN23195@dread.disaster.area> <20200107210715.GQ23195@dread.disaster.area> <4E9DF932-C46C-4331-B88D-6928D63B8267@fb.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Hi Hugh, Dave, Hugh Dickins writes: >Dave, Amir, Chris, many thanks for the info you've filled in - >and absolutely no need to run any scan on your fleet for this, >I think we can be confident that even if fb had some 15-year-old tool >in use on its fleet of 2GB-file filesystems, it would not be the one >to insist on a kernel revert of 64-bit tmpfs inos. > >The picture looks clear now: while ChrisD does need to hold on to his >config option and inode32/inode64 mount option patch, it is much better >left out of the kernel until (very unlikely) proved necessary. Based on Mikael's comment above about Steam binaries, and the lack of likelihood that they can be rebuilt, I'm inclined to still keep inode{64,32}, but make legacy behaviour require explicit opt-in. That is: - Default it to inode64 - Remove the Kconfig option - Only print it as an option if tmpfs was explicitly mounted with inode32 The reason I suggest keeping this is that I'm mildly concerned that the kind of users who might be impacted by this change due to 32-bit _FILE_OFFSET_BITS -- like the not-too-uncommon case that Mikael brings up -- seem unlikely to be the kind of people that would find it in an rc. Other than that, the first patch could be similar to how it is now, incorporating Hugh's improvements to the first patch to put everything under the same stat_lock in shmem_reserve_inode. What do you folks think? Thanks, Chris