All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gary Hade <garyhade@us.ibm.com>
To: Alex Chiang <achiang@hp.com>
Cc: akpm@linux-foundation.org, Gary Hade <garyhade@us.ibm.com>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	Badari Pulavarty <pbadari@us.ibm.com>,
	David Rientjes <rientjes@google.com>, Ingo Molnar <mingo@elte.hu>
Subject: Re: [PATCH v3 1/5] mm: add numa node symlink for memory section in sysfs
Date: Thu, 12 Nov 2009 09:26:40 -0800	[thread overview]
Message-ID: <20091112172640.GA9320@us.ibm.com> (raw)
In-Reply-To: <20091110223644.25636.77587.stgit@bob.kio>

On Tue, Nov 10, 2009 at 03:36:44PM -0700, Alex Chiang wrote:
> Commit c04fc586c (mm: show node to memory section relationship with
> symlinks in sysfs) created symlinks from nodes to memory sections, e.g.
> 
> /sys/devices/system/node/node1/memory135 -> ../../memory/memory135
> 
> If you're examining the memory section though and are wondering what
> node it might belong to, you can find it by grovelling around in
> sysfs, but it's a little cumbersome.
> 
> Add a reverse symlink for each memory section that points back to the
> node to which it belongs.

Hi Alex,
I'm kinda late to the party but I finally had a chance to review
and try it out on one of our systems today.  Looks good to me.
Tested-by: Gary Hade <garyhade@us.ibm.com>
Acked-by: Gary Hade <garyhade@us.ibm.com>

Gary

-- 
Gary Hade
System x Enablement
IBM Linux Technology Center
503-578-4503  IBM T/L: 775-4503
garyhade@us.ibm.com
http://www.ibm.com/linux/ltc


WARNING: multiple messages have this Message-ID (diff)
From: Gary Hade <garyhade@us.ibm.com>
To: Alex Chiang <achiang@hp.com>
Cc: akpm@linux-foundation.org, Gary Hade <garyhade@us.ibm.com>,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	Badari Pulavarty <pbadari@us.ibm.com>,
	David Rientjes <rientjes@google.com>, Ingo Molnar <mingo@elte.hu>
Subject: Re: [PATCH v3 1/5] mm: add numa node symlink for memory section in sysfs
Date: Thu, 12 Nov 2009 09:26:40 -0800	[thread overview]
Message-ID: <20091112172640.GA9320@us.ibm.com> (raw)
In-Reply-To: <20091110223644.25636.77587.stgit@bob.kio>

On Tue, Nov 10, 2009 at 03:36:44PM -0700, Alex Chiang wrote:
> Commit c04fc586c (mm: show node to memory section relationship with
> symlinks in sysfs) created symlinks from nodes to memory sections, e.g.
> 
> /sys/devices/system/node/node1/memory135 -> ../../memory/memory135
> 
> If you're examining the memory section though and are wondering what
> node it might belong to, you can find it by grovelling around in
> sysfs, but it's a little cumbersome.
> 
> Add a reverse symlink for each memory section that points back to the
> node to which it belongs.

Hi Alex,
I'm kinda late to the party but I finally had a chance to review
and try it out on one of our systems today.  Looks good to me.
Tested-by: Gary Hade <garyhade@us.ibm.com>
Acked-by: Gary Hade <garyhade@us.ibm.com>

Gary

-- 
Gary Hade
System x Enablement
IBM Linux Technology Center
503-578-4503  IBM T/L: 775-4503
garyhade@us.ibm.com
http://www.ibm.com/linux/ltc

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2009-11-12 17:26 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-10 22:36 [PATCH v3 0/5] Alex Chiang
2009-11-10 22:36 ` Alex Chiang
2009-11-10 22:36 ` [PATCH v3 1/5] mm: add numa node symlink for memory section in sysfs Alex Chiang
2009-11-10 22:36   ` Alex Chiang
2009-11-12 17:26   ` Gary Hade [this message]
2009-11-12 17:26     ` Gary Hade
2009-11-10 22:36 ` [PATCH v3 2/5] mm: refactor register_cpu_under_node() Alex Chiang
2009-11-10 22:36   ` Alex Chiang
2009-11-10 22:36 ` [PATCH v3 3/5] mm: refactor unregister_cpu_under_node() Alex Chiang
2009-11-10 22:36   ` Alex Chiang
2009-11-10 22:36 ` [PATCH v3 4/5] mm: add numa node symlink for cpu devices in sysfs Alex Chiang
2009-11-10 22:36   ` Alex Chiang
2009-11-10 22:37 ` [PATCH v3 5/5] Documentation: ABI: /sys/devices/system/cpu/cpu#/node Alex Chiang
2009-11-10 22:37   ` Alex Chiang

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=20091112172640.GA9320@us.ibm.com \
    --to=garyhade@us.ibm.com \
    --cc=achiang@hp.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mingo@elte.hu \
    --cc=pbadari@us.ibm.com \
    --cc=rientjes@google.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.