public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: "Bob Picco" <bob.picco@hp.com>
To: Paul Jackson <pj@sgi.com>
Cc: Yasunori Goto <y-goto@jp.fujitsu.com>,
	akpm@osdl.org, apw@shadowen.org, bob.picco@hp.com,
	linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org,
	linux-ia64@vger.kernel.org, tony.luck@intel.com, ak@suse.de,
	len.brown@intel.com, discuss@x86-64.org
Subject: Re: [PATCH 001/003]Fix unify mapping from pxm to node id.
Date: Sat, 28 Jan 2006 22:44:34 -0500	[thread overview]
Message-ID: <20060129034434.GO7306@localhost> (raw)
In-Reply-To: <20060127231517.4c0ce573.pj@sgi.com>

Paul Jackson wrote:	[Sat Jan 28 2006, 02:15:17AM EST]
> Yasunori-san,
> 
> Thank-you for updating your patch.
> 
> However I am still puzzled by one detail.
> 
> Your latest patchset removes the defines:
> -#define MAX_PXM_DOMAINS	256	/* 1 byte and no promises about values */
> 
> and:
> -#define MAX_PXM_DOMAINS (256)
> 
> but continues to have code using MAX_PXM_DOMAINS.  I am unable
> to compile ia64 with it now, for lack of this define.
> 
> -- 
>                   I won't rest till it's the best ...
>                   Programmer, Linux Scalability
>                   Paul Jackson <pj@sgi.com> 1.925.600.0401
Hi Paul. 

MAX_PXM_DOMAINS is defined in include/acpi/acpi_numa.h.  So I'm confused.

I just built sn2 with Yasunori's patches applied to -mm3. I had previously
built sn2 with a slightly different patch series of mine which Yasunori 
later incorporated most of into this patch series. Would you please send your 
.config and the error output from the build? 

thanks,

bob

  reply	other threads:[~2006-01-29  3:48 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-20 11:24 [RFT/PATCH]Unify mapping from pxm to node id (take 2) Yasunori Goto
2006-01-23  7:57 ` Yasunori Goto
2006-01-26 15:48   ` Paul Jackson
2006-01-28  3:34     ` [PATCH 001/003]Fix unify mapping from pxm to node id Yasunori Goto
2006-01-28  7:15       ` Paul Jackson
2006-01-29  3:44         ` Bob Picco [this message]
2006-01-29  5:04           ` Paul Jackson
2006-01-29 10:38           ` Paul Jackson
2006-01-28  3:34     ` [PATCH 002/003]Fix " Yasunori Goto
2006-01-28  3:34     ` [PATCH 003/003]Fix " Yasunori Goto

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=20060129034434.GO7306@localhost \
    --to=bob.picco@hp.com \
    --cc=ak@suse.de \
    --cc=akpm@osdl.org \
    --cc=apw@shadowen.org \
    --cc=discuss@x86-64.org \
    --cc=len.brown@intel.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-ia64@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pj@sgi.com \
    --cc=tony.luck@intel.com \
    --cc=y-goto@jp.fujitsu.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