Messages in this thread | | | Subject | Re: [PATCH 1/2 v3] x86: introduce int3-based instruction patching | From | Joe Perches <> | Date | Thu, 11 Jul 2013 16:01:29 -0700 |
| |
On Thu, 2013-07-11 at 22:26 +0200, Jiri Kosina wrote: > Introduce a method for run-time instrucntion patching on a live SMP kernel
Some of the typos anyway.
s/instrunction/instruction
[]
> v2 -> v3: > + fixed few typos (Joe Perches) [] > + /* we are not interested in non-int3 faults and ring > 0 faults */ > + if (val != DIE_INT3 || !args->regs || user_mode_vm(args->regs) > + || (unsigned long) bp_int3_addr != args->regs->ip) > + return NOTIFY_DONE; > + > + /* set up the specified breakpoint handler */ > + args->regs->ip = (unsigned long) bp_int3_handler;
Still think the test and set should be in the same order.
> +/* > + * text_poke_bp() -- update instructions on live kernel on SMP > + * @addr: address to patch > + * @opcode: opcode of new instruction > + * @len: length to copy > + * @handler: address to jump to when the temporary breakpoint is hit > + *
This sure looks like kernel-doc but misses the leading /**
| |