On Wed, Jul 17, 2019 at 11:17:50AM +0200, Paolo Bonzini wrote: >On 30/04/19 05:44, Wei Yang wrote: >> RAMBlock->offset is calculated by find_ram_offset, which makes sure the >> offset is aligned to a word. >> >> This patch removes the alignment check on offset and unnecessary >> variable *word*. >> >> Signed-off-by: Wei Yang <richardw.y...@linux.intel.com> > >I would add an assertion instead, but overall leaving the condition >there is harmless. You still need the "else" part for the case where >the length is unaligned. >
Thanks, then I would leave current code untouched. >Paolo > -- Wei Yang Help you, Help me