public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Hansen <haveblue-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org>
To: Keiichiro Tokunaga
	<tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
Cc: naveen.b.s-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org,
	Matthew E Tolentino
	<matthew.e.tolentino-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
	acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org,
	lhms
	<lhms-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org>
Subject: Re: [Lhms-devel] Re: [PATCH] ACPI based Memory Hotplug Driver Patch
Date: Mon, 13 Sep 2004 01:41:05 -0700	[thread overview]
Message-ID: <1095064865.4671.5.camel@localhost> (raw)
In-Reply-To: <20040913172318.791db349.tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>

On Mon, 2004-09-13 at 01:23, Keiichiro Tokunaga wrote:
> On Fri, 10 Sep 2004 10:32:35 -0700 Dave Hansen wrote:
> > At this point, adding stuff to /proc in mainline has a pretty slim
> > chance of happening.  sysfs is really the way to go, plus we already
> > have code in the mhp tree to export the hotplug memory controls via
> > sysfs.  This stuff seems to me to belong in /sys/firmware.
> >
> > But, this brings up something that's been nagging me for a while: what
> > should /sys/devices/system/memory contain?  A logical view of memory for
> > use with memory hotplug, or a hardware description of memory?  I think
> > both are probably necessary, and should link to each other, but how
> > should we present them?
> 
> How about this?
> 
> # ls /sys/firmware/acpi/memory/MEM1/
> resource_type
> producer_consumer
> decode
> min_address
> max_address
> granularity
> min_address_range
> max_address_range
> address_translation_offset
> address_length

The placement looks right to me.

> # ls /sys/devices/system/memory/memsection4/
> start_physaddr
> <something else>
> 
> We can determine which memsection belongs to which ACPI
> memory object by using min_address_range, max_address_range,
> and start_physaddr.  In the above case, start_physaddr is between
> min_address_range and max_address_range if MEM1 contains
> memsection4.

sysfs really has a legacy of being linked together with regular old
symlinks.  There's plenty of stuff in there that we *could* derive by
looking all over the place, but don't because it's so easy to just
create links.  I say we create links from one to the other, and make
sure not to duplicate any information.  

For instance, there shouldn't really need to be a start_addr in the acpi
directory if it can easily be determined from the memsections that
belong to that acpi memory object.  (or the othe way around, maybe)

What we want to discourage is anybody writing ACPI-specific (or
OpenFirmware on ppc for that matter) memory hotplug applications when a
generic Linux one would do.  That might be impossible in the long run,
but it's worth a try.  

-- Dave



-------------------------------------------------------
This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170
Project Admins to receive an Apple iPod Mini FREE for your judgement on
who ports your project to Linux PPC the best. Sponsored by IBM. 
Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php

  parent reply	other threads:[~2004-09-13  8:41 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-09-09 10:54 [PATCH] ACPI based Memory Hotplug Driver Patch S, Naveen B
     [not found] ` <FEB6C4E97F6CAF41978FB2059D5454180F37E8-OkeUvhg1trkFyVwBAnZdSLfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2004-09-09 13:11   ` Dmitry Torokhov
2004-09-09 15:50   ` [Lhms-devel] " Dave Hansen
2004-09-09 15:52   ` Bjorn Helgaas
2004-09-10 13:45   ` Keiichiro Tokunaga
     [not found]     ` <20040910224510.7f78b1b5.tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
2004-09-11  6:08       ` [Lhms-devel] " Hirokazu Takahashi
     [not found]         ` <20040911.150854.71102255.taka-jCdQPDEk3idL9jVzuh4AOg@public.gmane.org>
2004-09-11  7:14           ` Dave Hansen
2004-09-10 14:02   ` Keiichiro Tokunaga
     [not found]     ` <20040910230200.73eb0374.tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
2004-09-10 17:32       ` [Lhms-devel] " Dave Hansen
2004-09-13  8:23         ` Keiichiro Tokunaga
     [not found]           ` <20040913172318.791db349.tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
2004-09-13  8:41             ` Dave Hansen [this message]
2004-09-14  2:58               ` Keiichiro Tokunaga
     [not found]                 ` <20040914115844.1e12203d.tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
2004-09-14  4:11                   ` Dave Hansen
2004-09-14  9:29                     ` Keiichiro Tokunaga
     [not found]                       ` <20040914182956.49c9381d.tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
2004-09-14 17:12                         ` Dave Hansen
2004-09-15  0:08                           ` Keiichiro Tokunaga
     [not found]                             ` <20040915090827.7f86a54f.tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
2004-09-15  0:17                               ` Dave Hansen
2004-09-16  2:51                                 ` Keiichiro Tokunaga
2004-09-14 16:21                     ` Joel Schopp
2004-09-16 10:47       ` [PATCH] /sys/firmware/acpi/memory/ for showing info of ACPI memory object Keiichiro Tokunaga
     [not found]         ` <20040916194725.562b50f0.tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org>
2004-09-24  8:46           ` Keiichiro Tokunaga

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=1095064865.4671.5.camel@localhost \
    --to=haveblue-r/jw6+rmf7hqt0dzr+alfa@public.gmane.org \
    --cc=acpi-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=lhms-devel-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org \
    --cc=matthew.e.tolentino-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=naveen.b.s-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
    --cc=tokunaga.keiich-+CUm20s59erQFUHtdCDX3A@public.gmane.org \
    /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