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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id E33FEC04A94 for ; Thu, 10 Aug 2023 05:50:54 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 49E586B0075; Thu, 10 Aug 2023 01:50:54 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 44E336B0078; Thu, 10 Aug 2023 01:50:54 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2EE526B007B; Thu, 10 Aug 2023 01:50:54 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 1A74E6B0075 for ; Thu, 10 Aug 2023 01:50:54 -0400 (EDT) Received: from smtpin06.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id DB01CC012A for ; Thu, 10 Aug 2023 05:50:53 +0000 (UTC) X-FDA: 81107121186.06.98BD276 Received: from mail-yw1-f181.google.com (mail-yw1-f181.google.com [209.85.128.181]) by imf25.hostedemail.com (Postfix) with ESMTP id 1D4D7A000E for ; Thu, 10 Aug 2023 05:50:51 +0000 (UTC) Authentication-Results: imf25.hostedemail.com; dkim=pass header.d=google.com header.s=20221208 header.b=NAi9DyAm; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf25.hostedemail.com: domain of hughd@google.com designates 209.85.128.181 as permitted sender) smtp.mailfrom=hughd@google.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1691646652; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=x7yqFcfihO6UZ4Ofnmt86oPYT4hQDfDAsWIzATt1sTc=; b=W/RRBUFsRVK9H9rGRfPfo+bwVvmOwgtmvuOLBc8gxPGHqTQ0v+U3giQPxOg/ghTgXc4/n6 qJrV8Lo2ARZQkJIf8OQVhGkyX0s89RDNf5dRF0QRhkFeo/z6Z1M/R2JJZI9sfhq1XaicUp TT6p3y9/YK4hM2Ml7qFQS4i7vmEV+mY= ARC-Authentication-Results: i=1; imf25.hostedemail.com; dkim=pass header.d=google.com header.s=20221208 header.b=NAi9DyAm; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf25.hostedemail.com: domain of hughd@google.com designates 209.85.128.181 as permitted sender) smtp.mailfrom=hughd@google.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1691646652; a=rsa-sha256; cv=none; b=CNbGDDElPmHDZvbjtwXhNE3uYHe0OSV3EyZ0s09ufj2b04ydiGnNucRdW9Hh8VghLiwF1I e/IIustAoHE0UFoYjfULmWrqi8Dd/4nNn60jZC1QfKNM7qagGgWysk8jpQWAFOMaXtmg4t JLFzS2jmImleqSFRdckrg5ffDmjmYB0= Received: by mail-yw1-f181.google.com with SMTP id 00721157ae682-583f036d50bso7081527b3.3 for ; Wed, 09 Aug 2023 22:50:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20221208; t=1691646651; x=1692251451; h=mime-version:references:message-id:in-reply-to:subject:cc:to:from :date:from:to:cc:subject:date:message-id:reply-to; bh=x7yqFcfihO6UZ4Ofnmt86oPYT4hQDfDAsWIzATt1sTc=; b=NAi9DyAm+4viuBaK+2te/vEYT+Qzw6FmpDOVGbbjLStMiQomr4Av2iBnJIL6Uw4DBs pycOXzFZlP3T8OK1any+yj5dBc2FMkEEiNo30B+BKPwyfOs5+/L9b1fS2scHlVuFJypc yyO1X1LKQXDjxCQGYwSBAwJleAyDIevE3FpVAlXDZrKJXDwAWnEFZgfMSsBQVg8gBD08 YmNl+Si6DBE/oWAB0TFEay9z5QHBBPacQBGnidZZUFPoI5MUmOIXxsawP8l5ESkXe+L6 Oun/VSu/O91/o34Q99uYMUqp1tkHQJorTfYZmpWROFuHdkRgV4zPzgPskCBPjV0rYP1Y wglw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1691646651; x=1692251451; h=mime-version:references:message-id:in-reply-to:subject:cc:to:from :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=x7yqFcfihO6UZ4Ofnmt86oPYT4hQDfDAsWIzATt1sTc=; b=SOjoeGX6WDwcGYZA51XuDOb9CsktIAcy6QttBB/U5lp/IJAV13DrfzlFhFwZlxruBZ G0H2ZcmoE7RP3Zq3H2SHGFPq5vBr/tfDBlDqtwP3+cMYCWSqrsAGAtf7dro9IFfytoxu El+HiRwLIHEKLXOXsABhP6J6EmpYj3mSvt9sV0yrlsA1fCYqcenvVNvRIJ8U3ooLKNPN dcQAHcxOiHyZIp7aGmxCXVS5WiZkX8FTjzjKIBUZ8rekJChTVH0jtemmsvIe2Tv5YQAd F9KlN2rSsKl11vQGVyDkH3nkz3kPCtWFo3reHcOz54iOmNicsqB5wrrfyPJLEP1hLuXg wTZQ== X-Gm-Message-State: AOJu0Yy8HYSkOJK3y39H/+0GosS2qQENzCWa5yTKJbUDv13IJYF4qLX3 9PwoaDsGdrakot4RkYuA3/RT9w== X-Google-Smtp-Source: AGHT+IHL9XQObwck1BTnXd+TUHEB5iSg6FPFgszodJQAfdmJqAorzD2QzqndyFri9FLAViEk8CrsfA== X-Received: by 2002:a81:d54d:0:b0:577:189b:ad4 with SMTP id l13-20020a81d54d000000b00577189b0ad4mr1713495ywj.48.1691646651068; Wed, 09 Aug 2023 22:50:51 -0700 (PDT) Received: from ripple.attlocal.net (172-10-233-147.lightspeed.sntcca.sbcglobal.net. [172.10.233.147]) by smtp.gmail.com with ESMTPSA id e187-20020a8169c4000000b00559fb950d9fsm174825ywc.45.2023.08.09.22.50.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 09 Aug 2023 22:50:50 -0700 (PDT) Date: Wed, 9 Aug 2023 22:50:39 -0700 (PDT) From: Hugh Dickins X-X-Sender: hugh@ripple.attlocal.net To: Christian Brauner cc: Hugh Dickins , Andrew Morton , Oleksandr Tymoshenko , Carlos Maiolino , Jeff Layton , Chuck Lever , Jan Kara , Miklos Szeredi , Daniel Xu , Chris Down , Tejun Heo , Greg Kroah-Hartman , Matthew Wilcox , Christoph Hellwig , Pete Zaitcev , Helge Deller , Topi Miettinen , Yu Kuai , linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH vfs.tmpfs 0/5] tmpfs: user xattrs and direct IO In-Reply-To: <20230809-leitgedanke-weltumsegelung-55042d9f7177@brauner> Message-ID: References: <20230809-postkarten-zugute-3cde38456390@brauner> <20230809-leitgedanke-weltumsegelung-55042d9f7177@brauner> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-Rspamd-Queue-Id: 1D4D7A000E X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: 8im58pef6m5jxkm46dhtuia3umbnyzy3 X-HE-Tag: 1691646651-46909 X-HE-Meta: U2FsdGVkX19C6FOizvA5gETTJ2ay9zJAXwcX2hNRV0+TG23fasOHU2eqSHGChCoZIVYZGDk8Bf81MDLRORIfWZ1Wj2CSWUjIMjykjq4ux5jp7gzfa1DQy0ELOT4OjVR9YUQkkJBYN36ZnLISYb17o9FTRCO9TFssAvdncqIP5c2wOjr1IHibiH7mULIRXVItQG/J/sttToPemmaaaGH77Fileqgc67JyMkHWwU5cW5/N3haPyE5ShWj8sGa29no2cqHuqCY3Dk/q5gXuYubejRgwTTWaqSy5YBbEA/Fgob6TcGw/C0NcmIOCVxLIJg6l4qpxawR5eRv55L477nzrdoubJocofXZh4DfXcSOBU/ZhAtzzFlDcmAHrniGAV/6Gl4R+EVSrc5vMF4RT1qgQCosAkmEgKLV07c9FAF6RhcMbSNHIc/5e1ixYLKsSkE5eAKg4Eikfakv9smeSveFIyydd701c2pnZLkpl41MfX8MrNP35IuhR80jM82TaJ/eCWkMxIXX0g8NUSj1jxb28GE28GdTMlLQ20nk0UGRQ0pgOOOJtWrYvZpLLU0iLLu6zFuEb3AdCxefiMutWZctRYA0C+Ni+xQ6uKpdP8r8UjG4ah1zfzq4Ao1frewYcREf9teAME3+ZrAIc26EmGV3fe1ZlfBBsSfmPVyCLuaxJ90i8VCaX75PXSgmti42e7Xi9MFDBwXh3MKnHnkWrTB7btYQR1I6kGL8G04m/W4M3AC3ZoYn4LQNwTseNfjPPYa1Drna28kjt3p8eqwAb9lJEV4zLO2GxZsQK3NwhnQKO7rsVqFDp4zkudoyYn3gddptcU1HsHBSaTHcMcx0DcOkl4fbpIuSBCtG4BOQMdHaQoobZEo4aKu9lNoncEk0uH/8N7ccxV3o2aAA0df+1J4QEMeXgRixVclkSeXS05i2JtmrUkKUhAGsFsChTJzTu2R3wa/rGE0/m7O+2C+gkRxs 1Rx6kq0X SP+H5+9+vkzH+NrP4sLRCSUcrpwY4KX4uZm6Kmk+4vnXssC9Fv42c2H/8RmVHab5BeHhlJ6cY1wqpgwpCBtN1D8DHl61BqeSEz52BvWDgGJUrr3JJD1e/68ZID4TwSD+P8GaTnp0ywDTS1/ADh6XogNmHPmGNO+YH4H/7qrtYoixRBsZJ3lo2rRho7JemstBjc18jtM985hAVgL3v6FZGy9GXGzZZeUd7tMTZpvKB5mOj12JkTC7twtrOr8OmmB/c4frIC9hQgwl8zGzfp7L0PcWsAb5+JkTQkbgxiFCYivgfbN6w9BLPAxJqOuFPwG9wEtFucHHr6W5WcTT3wiFrVrjsB/l1QHhkErFzP/H3BQH6U3QbYiTD6uAN7nGl3REgXsPXPnvSq8eYSLU= 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: On Wed, 9 Aug 2023, Christian Brauner wrote: > On Wed, Aug 09, 2023 at 08:45:57AM +0200, Christian Brauner wrote: > > On Tue, Aug 08, 2023 at 09:28:08PM -0700, Hugh Dickins wrote: > > > This series enables and limits user extended attributes on tmpfs, > > > and independently provides a trivial direct IO stub for tmpfs. > > > > > > It is here based on the vfs.tmpfs branch in vfs.git in next-20230808 > > > but with a cherry-pick of v6.5-rc4's commit > > > 253e5df8b8f0 ("tmpfs: fix Documentation of noswap and huge mount options") > > > first: since the vfs.tmpfs branch is based on v6.5-rc1, but 3/5 in this > > > series updates tmpfs.rst in a way which depends on that commit. > > > > > > IIUC the right thing to do would be to cherry-pick 253e5df8b8f0 into > > > vfs.tmpfs before applying this series. I'm sorry that the series as > > > posted does not apply cleanly to any known tree! but I think posting > > > it against v6.5-rc5 or next-20230808 would be even less helpful. > > > > No worries, I'll sort that out. > > So, I hemmed and hawed but decided to rebase vfs.tmpfs onto v6.5-rc4 > which includes that fix as cherry picking is odd. Even better, thanks. And big thank you to you and Jan and Carlos for the very quick and welcoming reviews. If only Hugh were able to respond like that... Needing "freed = 0" in shmem_evict_inode(), as reported by robot: that was stupid of me (though it happens not to matter what the value is in the uninitialized case): I'll send you the fixup to 3/5 tomorrow (unless it turns out that you've typed in the " = 0" yourself already). And I'll send a replacement for 4/5, the direct IO one, following Christoph's guidance: but I'm wilting, and just didn't get to it today. Hugh