public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Steve Lord <lord@xfs.org>
To: "K.R. Foley" <kr@cybsft.com>
Cc: Andrew Morton <akpm@osdl.org>,
	pozsy@uhulinux.hu, linux-kernel@vger.kernel.org,
	rusty@rustcorp.com.au
Subject: Re: Race condition in module load causing undefined symbols
Date: Tue, 14 Jun 2005 12:39:42 -0500	[thread overview]
Message-ID: <42AF165E.1020702@xfs.org> (raw)
In-Reply-To: <42AF0FA2.2050407@cybsft.com>

K.R. Foley wrote:
> Steve Lord wrote:
> <snip>
> 
>> So is this some P4 specific optimization which is not working as
>> intended?
>>
>> Steve
>>
>>
> 
> I'd say not since the first system I saw this on was a dual PIII Xeon. 
> While I am not 100% sure that the problems are related, the problem that 
> I saw on my 2.6 system also went away when I disabled hyper-threading in 
> the bios. It really just seems to me like it is some hard-to-trigger race.
> 

Not too hard for me :-(

Definitely a race, and it appears to be somewhere in the fork/exec/wait
complex at the very least. insmod is not built into nash, so is getting
run as a seperate process. Since module loading itself is synchronous,
the error would seem most likely to be happening in sys_wait4.

It could be the compiler doing a bad optimization, it could be
some other optimization code triggered by the Pentium 4 config
option, or, as you say, it could be a race which is being
opened up by the changed build flags.

Steve

  reply	other threads:[~2005-06-14 17:40 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-10 14:03 Race condition in module load causing undefined symbols Stephen Lord
2005-06-10 18:25 ` Andrew Morton
2005-06-10 19:06   ` Steve Lord
2005-06-11  3:30   ` Stephen Lord
2005-06-11  8:26   ` Pozsár Balázs
2005-06-11 13:23     ` Steve Lord
2005-06-11 15:05       ` Pozsár Balázs
2005-06-11 17:56         ` Stephen Lord
2005-06-11 19:00           ` Andrew Morton
2005-06-11 19:08             ` Pozsár Balázs
2005-06-11 20:09             ` Steve Lord
2005-06-11 20:18               ` Pozsár Balázs
2005-06-14 13:34             ` Steve Lord
2005-06-14 15:33               ` K.R. Foley
2005-06-14 15:36               ` K.R. Foley
2005-06-14 16:38                 ` Steve Lord
2005-06-14 16:56                   ` Andi Kleen
2005-06-14 17:16                     ` Steve Lord
2005-06-14 20:56                     ` Pozsár Balázs
2005-06-14 17:10                   ` K.R. Foley
2005-06-14 17:39                     ` Steve Lord [this message]
2005-06-14 18:23                       ` Prarit Bhargava
2005-06-14 19:27                         ` Steve Lord
2005-06-14 19:32                           ` Christoph Hellwig
2005-06-14 20:59                         ` Pozsár Balázs
2005-06-15 11:28                           ` Prarit Bhargava
2005-06-15 11:34                             ` Pozsár Balázs
2005-06-15 11:35                               ` Prarit Bhargava
2005-06-15 11:43                                 ` Pozsár Balázs
2005-06-15 12:33                             ` Stephen Lord
2005-07-28 19:42                               ` David Howells
2005-06-12  6:49 ` Rusty Russell

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=42AF165E.1020702@xfs.org \
    --to=lord@xfs.org \
    --cc=akpm@osdl.org \
    --cc=kr@cybsft.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pozsy@uhulinux.hu \
    --cc=rusty@rustcorp.com.au \
    /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