Devicetree
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
To: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>
Cc: "parch-QRwYI7m9GJLYtjvyW6yDsg@public.gmane.org"
	<parch-QRwYI7m9GJLYtjvyW6yDsg@public.gmane.org>,
	devicetree-discuss
	<devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org>,
	David Gibson <dwg-8fk3Idey6ehBDgjK7y7TUQ@public.gmane.org>,
	Stuart Yoder <b08248-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
	Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org>,
	Matthew McClintock <msm-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
Subject: Re: [Power.org:parch] devicetree: Musings on reserved regions
Date: Tue, 08 Feb 2011 17:45:21 +1100	[thread overview]
Message-ID: <1297147521.14982.192.camel@pasglop> (raw)
In-Reply-To: <AANLkTikgBqJM=kWf+FK1nxmi+BSV-nEJo=ToL7gyinkN-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On Mon, 2011-02-07 at 22:05 -0700, Grant Likely wrote:
> 
> Hmmm, of all the ideas, I think I like this one the best.  It is a
> little more verbose than I was thinking, but making each reserved
> region (or set of reserved regions) have separate node has some very
> real advantages.  For one, the address decoding *just works* with our
> existing helpers.  Also it means that device nodes can directly
> indicate the reserved regions associated with it by way of a phandle.

But that means that anything that needs to setup a simple memory
allocator (and thus needs to know what to avoid) will need to walk the
entire tree for nodes with a "reserved" property, be able to parse
addresses down to that node (not always obvious accross some bus types),
etc...

Or do we propose to keep a "consolidated" reserve map in addition to
having nodes self-describe which part of this reserve map they are
responsible for, which thus gives the OS the option to:

 - Reserve it all
 - Then, based on specific knowledge of some nodes, maybe release
or juts "tag" (for diag purposes) portions of reserved memory

Ben.

  parent reply	other threads:[~2011-02-08  6:45 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-07 20:59 devicetree: Musings on reserved regions Grant Likely
     [not found] ` <AANLkTinjPk1rJp1pCvM3ZTsLnEym57qU6u72vL8JQ1V3-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-07 21:39   ` [Power.org:parch] " Benjamin Herrenschmidt
2011-02-07 21:45     ` Grant Likely
     [not found]       ` <AANLkTiniLf6f7FKMdZ1BWZ9jiA=KHEb0wsp84hCnm73Z-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-07 21:53         ` Yoder Stuart-B08248
     [not found]           ` <9F6FE96B71CF29479FF1CDC8046E15030C1955-TcFNo7jSaXPiTqIcKZ1S2K4g8xLGJsHaLnY5E4hWTkheoWH0uzbU5w@public.gmane.org>
2011-02-07 21:56             ` Grant Likely
     [not found]               ` <AANLkTinO40aKu9VjFrB2GQmpqwGGk+vR=LjD5E49wyHf-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-07 22:01                 ` Jimi Xenidis
     [not found]                   ` <EA0D723E-90B1-4D35-8A3D-A5193E00F740-aZOuKsOsJu3MbYB6QlFGEg@public.gmane.org>
2011-02-07 22:50                     ` Dan Hettena
2011-02-08  1:13                     ` David VomLehn
2011-02-07 21:58             ` Scott Wood
2011-02-07 22:00         ` Jimi Xenidis
     [not found]           ` <DEB310FA-724C-4D8B-A8EC-E222A51CD5D4-e+AXbWqSrlAAvxtiuMwx3w@public.gmane.org>
2011-02-08  5:05             ` Grant Likely
     [not found]               ` <AANLkTikgBqJM=kWf+FK1nxmi+BSV-nEJo=ToL7gyinkN-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-08  6:45                 ` Benjamin Herrenschmidt [this message]
2011-02-08 10:56                   ` Jimi Xenidis
2011-02-08  2:20         ` Benjamin Herrenschmidt
2011-02-07 21:46     ` Scott Wood
2011-02-08 12:24   ` David Gibson

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=1297147521.14982.192.camel@pasglop \
    --to=benh-xvmvhmargas8u2djnn8i7kb+6bgklq7r@public.gmane.org \
    --cc=b08248-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
    --cc=devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \
    --cc=dwg-8fk3Idey6ehBDgjK7y7TUQ@public.gmane.org \
    --cc=grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org \
    --cc=msm-KZfg59tc24xl57MIdRCFDg@public.gmane.org \
    --cc=parch-QRwYI7m9GJLYtjvyW6yDsg@public.gmane.org \
    --cc=scottwood-KZfg59tc24xl57MIdRCFDg@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