On 27/07/17 16:20, Boris Ostrovsky wrote: > On 07/26/2017 07:56 AM, Andrew Cooper wrote: >> This allows svm_{get,set}_segment_register() to access the user segments by >> array index, capitalising on x86_seg_* matches the hardware encoding. > This does read correctly to me, I think some sort of preposition is missing. > > "capitalising on x86_seg_* matches *in* the hardware encoding"? *of*?
Oops yes. "capitalising on the fact that the x86_seg_* values match the hardware encodings." > > > >> While making this alteration, add some newlines for clarity, switch an int >> for >> a bool, and make the functions fail safe in a release build, rather than >> crashing Xen. >> >> Bloat-o-meter reports some modest improvements: >> >> add/remove: 0/0 grow/shrink: 0/2 up/down: 0/-130 (-130) >> function old new delta >> svm_set_segment_register 662 653 -9 >> svm_get_segment_register 409 288 -121 >> >> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> Thanks. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel