linux-um archives
 help / color / mirror / Atom feed
From: BlaisorBlade <blaisorblade_spam@yahoo.it>
To: user-mode-linux-devel@lists.sourceforge.net
Cc: Gerd Knorr <kraxel@bytesex.org>
Subject: Re: [uml-devel] [patch] host-skas3 for 2.6.9-rc2
Date: Sat, 18 Sep 2004 18:56:44 +0200	[thread overview]
Message-ID: <200409181856.44696.blaisorblade_spam@yahoo.it> (raw)
In-Reply-To: <20040916133921.GA14509@bytesex>

On Thursday 16 September 2004 15:39, Gerd Knorr wrote:
> On Thu, Sep 16, 2004 at 01:03:23PM +0200, Gerd Knorr wrote:
> > Some change between -rc1 and -rc1-bk1 broke the skas patch, still
> > looking ...
>
> Found it: init_new_empty_context needs a arch_pick_mmap_layout call,
> otherwise we'll get a NULL ptr dereferences for the function pointers
> added recently to mm_struct.
Well, please cleanup it, because you are affecting the behaviour on fork() 
(the change should be harmless, however, just repeating 
arch_pick_mmap_layout()).

Without the SKAS patch, init_new_context does not call 
arch_pick_mmap_layout(), so it should not after ( and init_new_context calls, 
actually, init_new_empty_context()). I think that the proper place to add the 
call is open_proc_mm(). Do you agree on this?

Bye
-- 
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729


-------------------------------------------------------
This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170
Project Admins to receive an Apple iPod Mini FREE for your judgement on
who ports your project to Linux PPC the best. Sponsored by IBM.
Deadline: Sept. 24. Go here: http://sf.net/ppc_contest.php
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

  reply	other threads:[~2004-09-18 17:10 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200409122004.41777.blaisorblade_spam@yahoo.it>
     [not found] ` <20040913144250.GA17259@bytesex>
     [not found]   ` <200409132130.00661.blaisorblade_spam@yahoo.it>
     [not found]     ` <20040915144213.GA32509@bytesex>
     [not found]       ` <20040916110323.GB12785@bytesex>
2004-09-16 13:39         ` [uml-devel] [patch] host-skas3 for 2.6.9-rc2 Gerd Knorr
2004-09-18 16:56           ` BlaisorBlade [this message]
     [not found]             ` <20040920153737.GA9823@bytesex>
2004-09-20 18:34               ` BlaisorBlade

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=200409181856.44696.blaisorblade_spam@yahoo.it \
    --to=blaisorblade_spam@yahoo.it \
    --cc=kraxel@bytesex.org \
    --cc=user-mode-linux-devel@lists.sourceforge.net \
    /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