[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 02/11] x86emul: flatten twobyte_table[]

>>> On 10.02.17 at 18:13, <andrew.cooper3@xxxxxxxxxx> wrote:
> On 01/02/17 11:13, Jan Beulich wrote:
>> +static const struct {
>> +    opcode_desc_t desc;
>> +} twobyte_table[256] = {
>> +    [0x00] = { ModRM },
> This is definitely an improvement in readability, so Acked-by: Andrew
> Cooper <andrew.cooper3@xxxxxxxxxx> (I have briefly checked that
> everything appears to be the same, but not checked thoroughly)


> I had a plan to do this anyway, including the onebyte table, and adding
> instruction/group comments like the case statements for emulation.  Is
> that something you can introduce in your series, or shall I wait and
> retrofit a patch later?

Flattening the onebyte table was certainly an intention of mine too,
but with no specific plans time wise. Getting the additions taken care
of is proving time consuming enough for the moment. As to adding
comments, though: I had specifically considered this (for the twobyte
table) and considered it a bad idea, as it'll significantly clutter the
table, and be particularly unhelpful for table entries covering multiple
opcodes at once).


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.