On 01/25/2012 01:48 PM, Avi Kivity wrote: > On 01/25/2012 01:26 PM, Orit Wasserman wrote: >> Implement Unsigned Little Endian Base 128. >> >> >> +/* ULEB128 */ >> +int uleb128_encode_small(uint8_t *out, uint32_t n); >> +int uleb128_decode_small(const uint8 *in, uint32_t *n); >> + >> #endif >> diff --git a/savevm.c b/savevm.c >> index 80be1ff..304db31 100644 >> --- a/savevm.c >> +++ b/savevm.c >> @@ -2297,3 +2297,29 @@ void vmstate_register_ram_global(MemoryRegion *mr) >> { >> vmstate_register_ram(mr, NULL); >> } >> + >> +/* ULEB128 */ >> +int uleb128_encode_small(uint8_t *out, uint32_t n) >> +{ > > assert(n <= 0x3fff); > >> + if (n < 0x80) { >> + *out++ = n; >> + return 1; >> + } else { >> + *out++ = (n & 0x7f) | 0x80; >> + *out++ = n >> 7; > > return 2? oops , where did it go ... I will fix it asp. > >> + } >> + return 0; >> +} >> + >> +int uleb128_decode_small(const uint8 *in, uint32_t *n) >> +{ >> + if (!(*in & 0x80)) { >> + *n = *in++; >> + return 1; >> + } else { >> + *n = *in++ & 0x7f; > > assert(!(*in & 0x80)); > >> + *n |= *in++ << 7; >> + return 0; > > return 2? > >> + } >> +} >> + > >