Re: [v6 PATCH 08/21] x86/insn-eval: Add utility function to get segment descriptor base address
From: Borislav Petkov
Date: Fri May 05 2017 - 13:19:53 EST
On Wed, Apr 26, 2017 at 03:37:44PM -0700, Ricardo Neri wrote:
> I need a human-readable way of identifying what segment selector (in
> pt_regs, vm86regs or directly reading the segment registers) to use.
> Since there is a segment override prefix for all of them, I thought I
> could use them.
Yes, you should...
> Perhaps I can rename enum segment to enum segment_selector and comment
> that the values in the enum are those of the override prefixes. Would
> that be reasonable?
... but you should call them what they are: "enum seg_override_pfxs" or
"enum seg_ovr_pfx" or...
Or somesuch. I suck at naming stuff.
SUSE Linux GmbH, GF: Felix ImendÃrffer, Jane Smithard, Graham Norton, HRB 21284 (AG NÃrnberg)