diff for duplicates of <20120208094009.GB24443@linux.vnet.ibm.com> diff --git a/a/1.txt b/N1/1.txt index e38a6f1..16b69a6 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -5,15 +5,15 @@ > > Changelog: (Since v10): Add code to clear REX.B prefix pointed out by Denys Vlasenko > > and fix suggested by Masami Hiramatsu. > ... -> > + /* -> > + * Point cursor at the modrm byte. The next 4 bytes are the -> > + * displacement. Beyond the displacement, for some instructions, -> > + * is the immediate operand. -> > + */ -> > + cursor = uprobe->insn + insn_offset_modrm(insn); -> > + insn_get_length(insn); -> > + if (insn->rex_prefix.nbytes) -> > + *cursor &= 0xfe; /* Clearing REX.B bit */ +> > + /* +> > + * Point cursor at the modrm byte. The next 4 bytes are the +> > + * displacement. Beyond the displacement, for some instructions, +> > + * is the immediate operand. +> > + */ +> > + cursor = uprobe->insn + insn_offset_modrm(insn); +> > + insn_get_length(insn); +> > + if (insn->rex_prefix.nbytes) +> > + *cursor &= 0xfe; /* Clearing REX.B bit */ > > It looks like cursor points to mod/reg/rm byte, not rex byte. > Comment above says it too. You seem to be clearing a bit @@ -33,10 +33,3 @@ Oh okay, Will correct this and send out a new patch. -- Thanks and Regards Srikar - --- -To unsubscribe, send a message with 'unsubscribe linux-mm' in -the body to majordomo@kvack.org. For more info on Linux MM, -see: http://www.linux-mm.org/ . -Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ -Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> diff --git a/a/content_digest b/N1/content_digest index 8b6d1fc..cca909b 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -33,15 +33,15 @@ "> > Changelog: (Since v10): Add code to clear REX.B prefix pointed out by Denys Vlasenko\n" "> > and fix suggested by Masami Hiramatsu.\n" "> ...\n" - "> > + /*\n" - "> > + * Point cursor at the modrm byte. The next 4 bytes are the\n" - "> > + * displacement. Beyond the displacement, for some instructions,\n" - "> > + * is the immediate operand.\n" - "> > + */\n" - "> > + cursor = uprobe->insn + insn_offset_modrm(insn);\n" - "> > + insn_get_length(insn);\n" - "> > + if (insn->rex_prefix.nbytes)\n" - "> > + *cursor &= 0xfe; /* Clearing REX.B bit */\n" + "> > + \302\240 \302\240 \302\240 /*\n" + "> > + \302\240 \302\240 \302\240 \302\240* Point cursor at the modrm byte. \302\240The next 4 bytes are the\n" + "> > + \302\240 \302\240 \302\240 \302\240* displacement. \302\240Beyond the displacement, for some instructions,\n" + "> > + \302\240 \302\240 \302\240 \302\240* is the immediate operand.\n" + "> > + \302\240 \302\240 \302\240 \302\240*/\n" + "> > + \302\240 \302\240 \302\240 cursor = uprobe->insn + insn_offset_modrm(insn);\n" + "> > + \302\240 \302\240 \302\240 insn_get_length(insn);\n" + "> > + \302\240 \302\240 \302\240 if (insn->rex_prefix.nbytes)\n" + "> > + \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 *cursor &= 0xfe; \302\240 \302\240 \302\240 \302\240/* Clearing REX.B bit */\n" "> \n" "> It looks like cursor points to mod/reg/rm byte, not rex byte.\n" "> Comment above says it too. You seem to be clearing a bit\n" @@ -60,13 +60,6 @@ "\n" "-- \n" "Thanks and Regards\n" - "Srikar\n" - "\n" - "--\n" - "To unsubscribe, send a message with 'unsubscribe linux-mm' in\n" - "the body to majordomo@kvack.org. For more info on Linux MM,\n" - "see: http://www.linux-mm.org/ .\n" - "Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/\n" - "Don't email: <a href=mailto:\"dont@kvack.org\"> email@kvack.org </a>" + Srikar -bd498b7387b0712f51d42ceb58e70cf5c1da0d4803814d023fcbe3760c78c371 +80cdb9332bb856d14c30d4e2fe3dcf48e96a342e9ec088f5b2cc619fb767a592
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.