Messages in this thread | | | Date | Thu, 11 Sep 2014 21:44:06 -0700 | From | "H. Peter Anvin" <> | Subject | Re: [PATCH v8 07/10] x86, mpx: decode MPX instruction to get bound violation information |
| |
On 09/11/2014 04:37 PM, Thomas Gleixner wrote: >> >> Specifically because marshaling the data in and out of the generic >> decoder was more complex than a special-purpose decoder. > > I did not look at that detail and I trust your judgement here, but > that is in no way explained in the changelog. > > This whole patchset is a pain to review due to half baken changelogs > and complete lack of a proper design description. >
I'm not wedded to that concept, by the way, but using the generic parser had a whole bunch of its own problems, including the fact that you're getting bytes from user space.
It might be worthwhile to compare the older patchset which did use the generic parser to make sure that it actually made sense.
-hpa
| |