Subject: Re: [Fastboot] [PATCH 01/03] kexec: Avoid overwriting the current
	pgd (V2, stubs)
From: Magnus Damm <magnus@valinux.co.jp>
To: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: fastboot@lists.osdl.org, linux-kernel@vger.kernel.org,
       Andrew Morton <akpm@osdl.org>
In-Reply-To: <m1wtcasu5b.fsf@ebiederm.dsl.xmission.com>
References: <20060524044232.14219.68240.sendpatchset@cherry.local>
	 <20060524044237.14219.15615.sendpatchset@cherry.local>
	 <m1wtcasu5b.fsf@ebiederm.dsl.xmission.com>
Content-Type: text/plain
Date: Thu, 25 May 2006 12:45:42 +0900
Message-Id: <1148528742.5793.135.camel@localhost>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1351
Lines: 35

On Wed, 2006-05-24 at 20:41 -0600, Eric W. Biederman wrote:
> Magnus Damm <magnus@valinux.co.jp> writes:
> 
> > --===============059810052910035161==
> >
> > kexec: Avoid overwriting the current pgd (V2, stubs)
> >
> > This patch adds an architecture specific structure "struct kimage_arch" to 
> > struct kimage. This structure is filled in with members by the architecture
> > specific patches followed by this one.
> 
> You should be able to completely remove the need for this by simply
> adding a single additional external variable to the control code
> page.  Given that you abuse this information and store way more
> than you need I am not persuaded that it is an interesting case.

I'm sorry, but I do not understand. Care to explain a bit more, maybe
with some examples?

Also, I get the impression that you dislike my patches. I'd like to work
with you and the community to merge my patches, but for that to happen
I'd appreciate if we both kept the language on a professional level.

Next time, please try to avoid strong words such as "abuse", "horrible"
and "ridiculous".

Thanks,

/ magnus

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/