From: Greg KH <greg@kroah.com>
To: Dave Jones <davej@redhat.com>,
"Huang, Ying" <ying.huang@intel.com>,
Andrew Morton <akpm@linux-foundation.org>,
Dave Young <hidave.darkstar@gmail.com>,
"H. Peter Anvin" <hpa@zytor.com>,
linux-kernel@vger.kernel.org, clameter@sgi.com
Subject: Re: [PATCH -mm] x86 boot : export boot_params via sysfs (forward to Greg)
Date: Wed, 12 Dec 2007 14:18:22 -0800 [thread overview]
Message-ID: <20071212221822.GB16474@kroah.com> (raw)
In-Reply-To: <20071212191535.GB31044@redhat.com>
On Wed, Dec 12, 2007 at 02:15:35PM -0500, Dave Jones wrote:
> On Wed, Dec 12, 2007 at 09:45:07AM -0800, Greg Kroah-Hartman wrote:
>
> > Well, I respectively disagree. sysfs is NOT for exporting various
> > binary kernel structures to userspace directly. Again, the binary files
> > in sysfs are for chunks of memory that are PASS-THROUGH from hardware to
> > userspace, with no kernel intervention at all.
> > If you really need such a thing, use debugfs, as the only rule for
> > debugfs is that there is no rules :)
>
> Whilst on the subject, why wasn't /sys/slab done in debugfs ?
> The one-value-per-file thing has gone taken to ridiculous extremes there.
Heh, I think most of those files only show up if you have
CONFIG_SLUB_DEBUG enabled, and it's easier to add them to the original
sysfs entries instead of creating new debugfs ones.
> Having 3641 sysfs files that most people never use permanently taking up
> memory seems to be a massive waste of resources.
sysfs files have their dentries and inodes pushed out of memory if they
are never accessed, so no huge ammounts of memory are used here.
Otherwise we would never be able to support 20,000 block devices on a
31bit s390 with 128Mb of ram :)
thanks,
greg k-h
prev parent reply other threads:[~2007-12-12 22:30 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1197443265.14443.59.camel@caritas-dev.intel.com>
[not found] ` <20071212073502.GA31348@kroah.com>
[not found] ` <1197447079.14443.82.camel@caritas-dev.intel.com>
2007-12-12 17:45 ` [PATCH -mm] x86 boot : export boot_params via sysfs (forward to Greg) Greg KH
2007-12-12 19:15 ` Dave Jones
2007-12-12 22:18 ` Greg KH [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20071212221822.GB16474@kroah.com \
--to=greg@kroah.com \
--cc=akpm@linux-foundation.org \
--cc=clameter@sgi.com \
--cc=davej@redhat.com \
--cc=hidave.darkstar@gmail.com \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=ying.huang@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox