Re: [PATCH] fix phys. address conversion in ia64_pal_tr_read

From: David Mosberger <davidm_at_napali.hpl.hp.com>
Date: 2004-11-17 04:57:55
>>>>> On Tue, 16 Nov 2004 09:01:49 -0800, Jesse Barnes <jbarnes@engr.sgi.com> said:

  Jesse> On Monday, November 15, 2004 6:03 pm, David Mosberger wrote:
  >> >>>>> On Mon, 15 Nov 2004 17:22:41 -0800, Jesse Barnes >>>>>
  >> <jbarnes@engr.sgi.com> said:

  Jesse> The palinfo module may call ia64_pal_tr_read using an IPI.
  Jesse> If the processor receiving the IPI is idle, the call will
  Jesse> fail since the idle process stack is in region 5 and
  Jesse> ia64_pal_tr_read uses __pa rather than ia64_tpa.  This small
  Jesse> patch fixes the problem by making it use ia64_tpa instead.
  >>  Perhaps its worthwhile to do any audit for this kind of
  >> problems?  It may not be very difficult to hack sparse to add a
  >> check for __pa() calls on (stack-)local variables.

  Jesse> I didn't see anything obvious in a quick check of all the
  Jesse> ia64 users of __pa (they were mostly referring to stuff in
  Jesse> the static data section), but I may have missed something.

OK, thanks for looking!

	--david
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Tue Nov 16 12:58:30 2004

This archive was generated by hypermail 2.1.8 : 2005-08-02 09:20:32 EST