linux-kernel - Re: [tip:x86/mm] x86, mm: fault.c, simplify kmmio

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] [day] [month] [year] [list]

Date:	Sun, 22 Feb 2009 10:26:08 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	Pekka Paalanen <pq@....fi>
Cc:	hpa@...or.com, mingo@...hat.com, torvalds@...ux-foundation.org,
	tglx@...utronix.de, linux-kernel@...r.kernel.org
Subject: Re: [tip:x86/mm] x86, mm: fault.c, simplify kmmio_fault()


* Pekka Paalanen <pq@....fi> wrote:

> On Fri, 20 Feb 2009 22:40:09 GMT
> Ingo Molnar <mingo@...e.hu> wrote:
> 
> > Author:     Ingo Molnar <mingo@...e.hu>
> > AuthorDate: Fri, 20 Feb 2009 22:32:10 +0100
> > Commit:     Ingo Molnar <mingo@...e.hu>
> > CommitDate: Sat, 21 Feb 2009 00:09:42 +0100
> > 
> > x86, mm: fault.c, simplify kmmio_fault()
> > 
> > Impact: cleanup
> > 
> > Remove an #ifdef from kmmio_fault() - we can do this by
> > providing default implementations for is_kmmio_active()
> > and kmmio_handler(). The compiler optimizes it all away
> > in the !CONFIG_MMIOTRACE case.
> > 
> > Also, while at it, clean up mmiotrace.h a bit:
> > 
> >  - standard header guards
> >  - standard vertical spaces for structure definitions
> > 
> > No code changed (both with mmiotrace on and off in the config):
> > 
> >    text	   data	    bss	    dec	    hex	filename
> >    2947	     12	     12	   2971	    b9b	fault.o.before
> >    2947	     12	     12	   2971	    b9b	fault.o.after
> > 
> > Cc: Pekka Paalanen <pq@....fi>
> > Cc: Linus Torvalds <torvalds@...ux-foundation.org>
> > Signed-off-by: Ingo Molnar <mingo@...e.hu>
> > 
> 
> Acked-by: Pekka Paalanen <pq@....fi>
> except for one minor detail below.
> 
> > 
> > ---
> >  arch/x86/mm/fault.c       |    5 ++-
> >  include/linux/mmiotrace.h |   78 +++++++++++++++++++++++++++------------------
> >  2 files changed, 50 insertions(+), 33 deletions(-)
> > 
> > diff --git a/arch/x86/mm/fault.c b/arch/x86/mm/fault.c
> > index 3e36614..fe99af4 100644
> > --- a/arch/x86/mm/fault.c
> > +++ b/arch/x86/mm/fault.c
> > @@ -55,13 +55,14 @@ enum x86_pf_error_code {
> >  	PF_INSTR	=		1 << 4,
> >  };
> >  
> > +/*
> > + * (returns 0 if mmiotrace is disabled)
> 
> Returns 0 if the fault was not handled in mmiotrace.

ok, i've updated the comment. I did not mean the comment to be 
an exclusive statement, but in hindsight it's indeed ambiguous.

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