Prev: [PATCH -tip 3/9] perf probe: Fix bugs in line range finder
Next: net: TCP thin-stream latency-improving modifications
From: Luca Barbieri on 18 Feb 2010 18:40
> The existing instruction parser is only present if KPROBES is configured
> in... this patch would make it obligatory. �Your patch doesn't reflect
> that. �Furthermore, it is ~16K of code and data which probably will make
> embedded people unhappy... although perhaps isn't out of line.
Didn't notice that.

> A good question, though, is if we actually need support for JMP or CALL
> as anything but the first instruction (usually the *only* instruction),
> which would make it a lot easier...

Probably not.
I think an even better option is to create a CALL if replacementlen is 0xff.
This would save some space for each callsite and make it clear we only
support this usage.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: [PATCH -tip 3/9] perf probe: Fix bugs in line range finder
Next: net: TCP thin-stream latency-improving modifications