All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Lacombe <goretux@gmail.com>
To: Mel Gorman <mel@csn.ul.ie>
Cc: Ingo Molnar <mingo@elte.hu>,
	linux-kernel@vger.kernel.org, davej@redhat.com,
	tglx@linutronix.de
Subject: Re: [unused symbol] need to use init_mm
Date: Sat, 2 May 2009 20:05:36 +0200	[thread overview]
Message-ID: <200905022005.37101.goretux@gmail.com> (raw)
In-Reply-To: <20090501155130.GF27831@csn.ul.ie>

Le vendredi 1 mai 2009 17:51:30 Mel Gorman, vous avez écrit :
> On Thu, Apr 30, 2009 at 07:55:54PM +0200, Eric Lacombe wrote:
> > On Thu, Apr 30, 2009, Ingo Molnar wrote :
> > > * Mel Gorman <mel@csn.ul.ie> wrote:
> > > > On Thu, Apr 30, 2009 at 12:32:04PM +0200, Eric Lacombe wrote:

[...]

> If it's the kernel portion of the address space is what you care about,
> is current->mm->pgd usable?

Not currently, because this one is precisely compared to init_mm.pgd.

[...]

> > > > 4. When do you think the module will be ready for posting?  i.e. is
> > > > this really something destined for mainline or some
> > > > perma-out-of-tree?
> >
> > It is destined for mainline as far as I'm concerned. But it needs to be
> > accepted by the kernel community when it will be ready.
> > (I work for a french public laboratory, so it will be GPL 2 or 3 licensed
> > anyway)
>
> If it really is absolutly necessary that you have init_mm and the
> alternatives really are too ugly, patch 1 of your set becomes a
> re-export of init_mm. The leader of that patch should explain why the
> alternatives to init_mm suck.
>
> You can try posting a patch to export init_mm without an in-tree user but
> I wouldn't bet money on the x86 maintainers picking it up without an
> extremely compelling reason.

Thanks for your comments. I recently thought about an alternative that seems 
to be acceptable.
Nonetheless, I need to modify my security paradigm a little, but this one will 
still be correct. Besides, no ugliness ;)

Best regards,

	Eric


      reply	other threads:[~2009-05-02 18:06 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-30 10:32 [unused symbol] need to use init_mm Eric Lacombe
2009-04-30 14:09 ` Mel Gorman
2009-04-30 14:40   ` Ingo Molnar
2009-04-30 17:55     ` Eric Lacombe
2009-05-01 15:51       ` Mel Gorman
2009-05-02 18:05         ` Eric Lacombe [this message]

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=200905022005.37101.goretux@gmail.com \
    --to=goretux@gmail.com \
    --cc=davej@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mel@csn.ul.ie \
    --cc=mingo@elte.hu \
    --cc=tglx@linutronix.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.