Message-Id: <200107230349.WAA04057@ccure.karaya.com>
To: Andrea Arcangeli <andrea@suse.de>
Cc: linux-kernel@vger.kernel.org
Subject: Re: 2.4.7pre8aa1 
In-Reply-To: Your message of "Mon, 23 Jul 2001 03:11:46 +0200."
             <20010723031146.D23517@athlon.random> 
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Date: Sun, 22 Jul 2001 22:49:42 -0500
From: Jeff Dike <jdike@karaya.com>
Sender: linux-kernel-owner@vger.kernel.org
Original-Recipient: rfc822;linux-kernel-outgoing

andrea@suse.de said:
> I should have said "compiles" fine (not "works" fine :). 

2.4.6 "works" fine :-)

I just finished fixing 2.4.7.  My fixes are similar to yours (I moved 
stext rather than moving .data.init).

> __initdata is broken in uml and the kernel deadlocks because the wait
> list is empty in complete() despite wait_for_completion actually
> registered correctly into it. This because wait_for_completion runs in
> a different address space than complete() and the virtual memory is
> not shared across the two address spaces (it is not rempped in a
> MAP_SHARED so it generates a cow). The registration is basically only
> private to the entity that is registrating and it will never get
> visible to the waker task that will do nothing. This is a severe bug
> not just for the completion code in 2.4.7.

Nice debugging, btw.  I spent an hour this morning chasing that problem.

My 2.4.7 patch is available from the usual places:

       http://ftp.nl.linux.org/pub/uml/uml-patch-2.4.7-1.bz2
       http://uml-pub.ists.dartmouth.edu/uml/uml-patch-2.4.7-1.bz2
       http://prdownloads.sourceforge.net/user-mode-linux/uml-patch-2.4.7-1.bz2

				Jeff

-
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/