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=-6.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, 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 C233EC433EF for ; Wed, 8 Sep 2021 12:32:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A3A1B6113E for ; Wed, 8 Sep 2021 12:32:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1349109AbhIHMdS (ORCPT ); Wed, 8 Sep 2021 08:33:18 -0400 Received: from mail.kernel.org ([198.145.29.99]:33620 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234005AbhIHMdQ (ORCPT ); Wed, 8 Sep 2021 08:33:16 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 62B646108D; Wed, 8 Sep 2021 12:32:08 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1631104328; bh=o6tcok3SiCX2NAJZ1TXHt2b3v45b5/2Tl2q5XnE75I8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=NlI/Ztq73jUW+HQpfB8jbmZVcaT48QxAjei8CKUiV+7f58DxZGF00hPiGlMJEsIna gnCuIBB4DM2fKqdW9qWv8ni8XkHXqcBWPp/rJAArbap1zGS8H4mHhtkXR3v71qPoNv qtXVD+JB0MJuYB78mgEJwBRZDeqQC5p35FFsQRC0= Date: Wed, 8 Sep 2021 14:32:06 +0200 From: Greg Kroah-Hartman To: Lang Yu Cc: Joe Perches , "Rafael J . Wysocki" , linux-kernel@vger.kernel.org Subject: Re: [PATCH] sysfs: Remove page boundary align limitation on sysfs_emit and sysfs_emit_at Message-ID: References: <20210908120723.3920701-1-lang.yu@amd.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210908120723.3920701-1-lang.yu@amd.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Sep 08, 2021 at 08:07:23PM +0800, Lang Yu wrote: > The key purpose of sysfs_emit and sysfs_emit_at is to > ensure that no overrun is done. Make them more equivalent > with scnprintf. That's not the only purpose. So why are you changing this? What in-kernel users are being tripped up by this, shouldn't we fix them instead? Remember, sysfs files are "one value per file", so why are the boundries not properly set here? thanks, greg k-h