From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D8637194083; Thu, 19 Sep 2024 11:18:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1726744724; cv=none; b=VVYySHwTfVChhRGMgY0fr0CPNe/mGmrRc6YWMWQn8ckVSGwmcSgIZG4/qrxlLWN6Es5qIjT6xIEBoRlefyI/jB1+xs/wEHzCzhn9H8nyPSk0piL3Igog6s9+6ba7AQ5g9rxAX065MOYht5k/a0Je+9AxfI3P8xcjpKUDclx1LTU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1726744724; c=relaxed/simple; bh=hIkLd8AvzpCoLbV7CT9rjtimplqyC+/Fv5vdmWL3vtI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=FqIhtzP3VrnfGkJXEqOG0GUgiA/3ElUNdYCL6WrKM5sP5B3U7jdjAQPLupCZAYB3gBX57wz3lkVQ8lOYw7I1vTrDRN5pVZxFUN7KDetiZasnPDnJ652AKsQTSP2Ci5JcoFyDamkiVzP+mwnCjBWsefzpqHmE0na+T6Q/XFEfmiQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=XmqBG8ns; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="XmqBG8ns" Received: by smtp.kernel.org (Postfix) with ESMTPSA id A0196C4CEC4; Thu, 19 Sep 2024 11:18:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1726744723; bh=hIkLd8AvzpCoLbV7CT9rjtimplqyC+/Fv5vdmWL3vtI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=XmqBG8ns2uJ+6E4E/IgTH8qLWzDPEqlixy+wG/Zne4Mt1niENwCleA7LucsTqxgTW qle3tzZtULuoXOjL0rJkxq/x1lJOeFLe8IDNSPFWznBJuogRumfG7J6Q8t0TJyrI1F o3x2t7G9a6J9FaICAQWcTCuWUIe71Oxk/mwNZ2NE= Date: Thu, 19 Sep 2024 13:18:39 +0200 From: Greg Kroah-Hartman To: Xi Ruoyao Cc: shankerwangmiao@gmail.com, stable@vger.kernel.org, Mateusz Guzik , Christian Brauner , Alexander Viro , Jan Kara , Miguel Ojeda , Alex Gaynor , Wedson Almeida Filho , Boqun Feng , Gary Guo , =?iso-8859-1?Q?Bj=F6rn?= Roy Baron , Benno Lossin , Andreas Hindborg , Alice Ryhl , linux-fsdevel@vger.kernel.org Subject: Re: [PATCH 0/3] Backport statx(..., NULL, AT_EMPTY_PATH, ...) Message-ID: <2024091900-sloppy-swept-0a2e@gregkh> References: <20240918-statx-stable-linux-6-10-y-v1-0-8364a071074f@gmail.com> <2024091801-segment-lurk-e67b@gregkh> Precedence: bulk X-Mailing-List: linux-fsdevel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Thu, Sep 19, 2024 at 01:37:17AM +0800, Xi Ruoyao wrote: > On Wed, 2024-09-18 at 19:33 +0200, Greg Kroah-Hartman wrote: > > On Wed, Sep 18, 2024 at 10:01:18PM +0800, Miao Wang via B4 Relay wrote: > > > Commit 0ef625bba6fb ("vfs: support statx(..., NULL, AT_EMPTY_PATH, > > > ...)") added support for passing in NULL when AT_EMPTY_PATH is given, > > > improving performance when statx is used for fetching stat informantion > > > from a given fd, which is especially important for 32-bit platforms. > > > This commit also improved the performance when an empty string is given > > > by short-circuiting the handling of such paths. > > > > > > This series is based on the commits in the Linus’ tree. Sligth > > > modifications are applied to the context of the patches for cleanly > > > applying. > > > > > > Tested-by: Xi Ruoyao > > > Signed-off-by: Miao Wang > > > > This really looks like a brand new feature wanting to be backported, so > > why does it qualify under the stable kernel rules as fixing something? > > > > I am willing to take some kinds of "fixes performance issues" new > > features when the subsystem maintainers agree and ask for it, but that > > doesn't seem to be the case here, and so without their approval and > > agreement that this is relevant, we can't accept them. > > Unfortunately the performance issue fix and the new feature are in the > same commit. Is it acceptable to separate out the performance fix part > for stable? (Basically remove "if (!path) return true;" from the 1st > patch.) What prevents you, if you wish to have the increased performance, from just moving to a newer kernel version? We add new features and improvements like this all the time, why is this one so special to warrant doing backports. Especially with no maintainer or subsystem developer asking for this to be done? thanks, greg k-h