RE: [Linux-ia64] Re: [PATCH] head.S fix for unusual load addrs

From: David Mosberger <davidm_at_napali.hpl.hp.com>
Date: 2003-05-15 13:05:35
The patch looks mostly fine to me.  I'm not too fond of the __tpa()
and __imva() macros, but that may be mainly a matter of preference.
What I definitely don't like is that the casting seems rather confused
and that the patch is adding __tpa() when we already have ia64_tpa().
As an example of the casting issues: __imva() returns a long, but at
times it's cast to "unsigned long" which doesn't make a lot of sense
(for assignments, anyhow).  Moreover, we should stick to the Linux
principle that kernel-space pointers have a type of "void *".

	--david
Received on Wed May 14 20:07:10 2003

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