On Wed, 2013-10-16 at 12:13 +0100, Ben Dooks wrote:
On 15/10/13 23:38, Taras Kondratiuk wrote:
Hi
I was debugging kprobes-test for BE8 and noticed that some data fields are stored in LE instead of BE. It happens because these data fields get interpreted as instructions.
Is it a known issue?
I reported the crashes to Tixy along with a different method of sovling the problem (changed to using pointers to the strings) a while ago.
I found that fix in the list archives: http://lists.infradead.org/pipermail/linux-arm-kernel/2013-July/186528.html
However it seems that nothing has happened to fix this.
Since kprobes seems to work with the fixed tests I forgot to follow up and prod Jon about looking into this problem.
Sorry, I sorta forgot/ignored the issue, strange compiler problem with a vague explanation on a big-endian kernel (who uses that ;-)
Jon, if you are not interested in fixing this, then please let me know and we can get a patch sorted to fix it.
Looking at your old patch again, it looks good to me. So if someone could post this to the lists again, with the commit message updated to have a clearer explanation as to the symptoms, e.g. some of Taras' analysis, that would be good. It should go to the list again to give people another chance to comment...
Thanks