Re: [PATCH v2 2/6] riscv: insn: Set sstatus.MXR when reading userspace instruction
"Yicong Yang" <[email protected]>
| Newsgroups | org.infradead.lists.linux-riscv |
|---|---|
| Message-ID | <[email protected]> |
On 8/25/26 8:06 AM, Paul Walmsley wrote: > Hi, > > On Tue, 28 Jul 2026, Yicong Yang wrote: > >> The __read_insn() uses get_user() to read userspace instructions. >> Bo Gan noticed that get_user() is insufficient in this case since >> it's possible for the instruction segment mapped as execute-only >> and get_user() doesn't have permissions to load it. The architecture >> provides sstatus.MXR for handling this so set MXR when reading >> userspace instruction. > > If this patch fixes a bug with the preceding patch, it would > better just to roll the fix into the original patch. sure, will squash into previous one. I used to split the refactor (preceding) and fix (current) to separate patches for better review. thanks. _______________________________________________ linux-riscv mailing list [email protected] http://lists.infradead.org/mailman/listinfo/linux-riscv