Thread (64 messages) flat view 64 messages, 10 authors, 2008-02-15

Re: [PATCH 3/8] pseries: phyp dump: use sysfs to release reserved mem

From: "Linas Vepstas" <linasvepstas@gmail.com>
Date: 2008-02-15 17:30:40

On 14/02/2008, Tony Breeds [off-list ref] wrote:
On Tue, Feb 12, 2008 at 01:11:58AM -0600, Manish Ahuja wrote:
 > +static ssize_t
 > +show_release_region(struct kset * kset, char *buf)
 > +{
 > +     return sprintf(buf, "ola\n");
 > +}
 > +
 > +static struct subsys_attribute rr = __ATTR(release_region, 0600,
 > +                                      show_release_region,
 > +                                      store_release_region);


Any reason this sysfs attribute can't be write only? The show method
 doesn't seem needed.
This was supposed to be a place-holder; a later patch would add detailed
info.  The goal was to  have user-land tools that would operate these files
to progressively dump and release memory regions; however, until these
userland tools get written, the proper interface remains murky  (e.g.
real addresses? virtual addresses? just delta's or a whole memory map?
some sort of numa flags or whatever?)

--linas
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help