linux-kernel - Re: [unused symbol] need to use init

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090501155130.GF27831@csn.ul.ie>
Date:	Fri, 1 May 2009 16:51:30 +0100
From:	Mel Gorman <mel@....ul.ie>
To:	Eric Lacombe <goretux@...il.com>
Cc:	Ingo Molnar <mingo@...e.hu>, linux-kernel@...r.kernel.org,
	davej@...hat.com, tglx@...utronix.de
Subject: Re: [unused symbol] need to use init_mm

On Thu, Apr 30, 2009 at 07:55:54PM +0200, Eric Lacombe wrote:
> On Thu, Apr 30, 2009, Ingo Molnar wrote :
> > * Mel Gorman <mel@....ul.ie> wrote:
> > > On Thu, Apr 30, 2009 at 12:32:04PM +0200, Eric Lacombe wrote:
> > > > Hi,
> > > >
> > > > Within a module I work on, I need to access to init_mm, but as it is
> > > > marked UNUSED, I wonder if there is another way (an API) to access to
> > > > it. Otherwise, what can I do to have this mark removed from init_mm in
> > > > the mainline? as my module is really not yet ready for inclusion.
> > > >
> > > > Thanks in advance for your response,
> > >
> > > It looks to be that it was removed already, somewhat behind schedule, in
> > > commit 9470565579f29486f4ed0ffa50774268b64994b0 after an initial attempt
> > > was made in commit 3abf024d2abb79614d8c4cb25a70d5596f77d0ad.  I don't
> > > know if there is a procedure for bringing it back but I would guess it's
> > > a call for the x86 maintainers. I imagine they will want to at least
> > > know;
> > >
> > > 1. Why do you need init_mm exported?
> 
> In brief, my module use hardware virtualization technologies (only Intel ones 
> for the moment) in order to prevent the kernel from behaving maliciously 
> (because of a kernel malware) and to prevent the installation of many kernel 
> malwares. One of my approaches is to preserve some kernel-constrained objects: 
> for instance, some simple objects like idtr, gdtr, ... or some more 
> complicated objects like the kernel address space layout, ...
> For the latter I use init_mm.pgd.
> 

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

> I wrote a paper and will talk about it at the EICAR conference (May 11th and 
> 12th) this year (http://www.eicar.org/conference/index.htm).
> 
> > > 2. Is there an alternative approach to using init_mm?
> 
> I don't really take the time to think about another approach, but it seems 
> natural to access to init_mm.pgd. Maybe, I could access to it by other means,  
> but that would certainly be ugly :/
> 
> > > 3. Assuming yes, what are the downsides?
> 
> ugliness ;) maybe kernel version dependent.
> 

Unfortunately, you are already hit with the kernel version problem as
2.6.29 has been released without init_mm exported for the x86 arch.

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

> >
> > yes, those questions have to be answered.
> 
> I hope to have adequately answered to your questions.
> 

We can't go back in time so check out if current->mm->pgd will do the job. If
not, make patch 1 of your set a re-export of init_mm until you're ready to
post. At that point, people can see what you are doing and suggest what an
alternative might be or if indeed the symbol should be exported after all.

-- 
Mel Gorman
Part-time Phd Student                          Linux Technology Center
University of Limerick                         IBM Dublin Software Lab
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/