linux-kernel - Re: FInal kprobes rollup patches

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]

Date:	Mon, 17 Dec 2007 18:15:36 -0800
From:	Harvey Harrison <harvey.harrison@...il.com>
To:	Masami Hiramatsu <mhiramat@...hat.com>, Ingo Molnar <mingo@...e.hu>
Cc:	Jim Keniston <jkenisto@...ibm.com>,
	Srikar Dronamraju <srikar@...ux.vnet.ibm.com>,
	LKML <linux-kernel@...r.kernel.org>,
	Maneesh Soni <maneesh@...ux.vnet.ibm.com>,
	srinivasa@...ibm.com,
	Ananth N Mavinakayanahalli <ananth@...ibm.com>,
	Masami Hiramatsu <hiramatu@....hitachi.co.jp>,
	Rusty Lynch <rusty.lynch@...el.com>,
	Masami Hiramatsu <masami.hiramatsu.pt@...achi.com>,
	Keshavamurthy Anil S <anil.s.keshavamurthy@...el.com>
Subject: Re: FInal kprobes rollup patches

On Mon, 2007-12-17 at 19:27 -0500, Masami Hiramatsu wrote:
> Masami Hiramatsu wrote:
> >> These are:
> >>
> >> -add stack_addr() macro
> >> -I prefer the table defintion macros in mine as it avoids the need
> to
> >> cast the pointer passed to test_bit, but if you want them
> >> to be u32 as in your patch, I can change it.
> > 
> >  please do so. we'd like to reduce ifdefs as less as possible:-)
> > 
> >> -wrmsr/wrmsrl - use wrmsr() for both
> >> -call is_IF_modifier with p->ainsn.insn in both
> >> -check casting of jprobe_saved_sp, I get some compile warnings
> currently
> >> with pointer comparisons to signed/unsigned types.
> > 
> > Could you also add below?
> > - fix some comments (it clarifies the meanings of the code)
> > - add fix_riprel(). this useful to reduce ifdefs.
> > - expand reenter_kprobe(). I think it treat above two blocks.
> > - reassignment of regs->ip in kprobe_handler can be unified
> >   to "regs->ip = (unsigned long)addr;"
> 
> Oh, I forgot to point out important thing.
> - please make bugfix patches first. I think my bugfix patches
>   need to go upstream before unification. It would cause some
>   crashes.
> 

It's pretty hard to move those bugfixes to the head of the queue.

Attached is an mbox of my unification work.  Maybe you could get your
two bugfixes applied to mainline kprobes_32/64.c, then my series
could go in as a merge later on.  Most of my stuff is in kprobes.c
post-unify so the merge would be trivial later.


Ingo, what do you think?  This rollup replaces all of my kprobes
patches to date.  So you could apply patched 1,2/6 from Masami
into 2.6.24 and let mine come in during 2.6.25 as a merge, which
would avoid the conflicts in kprobes_32|64.c?


Harvey

Download attachment "final_kprobes_rollup.mbox.gz" of type "application/x-gzip" (47071 bytes)