On 04/16/2014 01:44 AM, Masami Hiramatsu wrote: > Same thing can be done in awk part and insn.c, and we can encode it by > > #define INAT_MAKE_MEMSZ(size) (size << INAT_MEMSZ_OFFS) > > And decode it by > > insn->memsz_bytes = 1 << ((attr & INAT_MEMSZ_MASK) >> INAT_MEMSZ_OFFS) > > Thus, we only need 3 bits to represent 1, 2, 4, 8, 16 and 32. :)
We'll need 4 so that we could do 64 too :) btw, why aren't we using regular bitfields? this manual encoding thingie seems to be a bit confusing (try figuring out how many bits are left...). Thanks, Sasha -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

