Re: [Qemu-devel] [PATCH 9/9] Convert ram_list to RCU

[Paolo Bonzini <pbonzini@redhat.com>]

On 11/02/2015 09:47, Fam Zheng wrote:
>> > +    rcu_read_lock();
>> > +    block = atomic_rcu_read(&ram_list.mru_block);
>> >      if (block && block->host && host - block->host < block->max_length) {
>> >          goto found;
>> >      }
>> >  
>> > -    QLIST_FOREACH(block, &ram_list.blocks, next) {
>> > +    QLIST_FOREACH_RCU(block, &ram_list.blocks, next) {
>> >          /* This case append when the block is not mapped. */
>> >          if (block->host == NULL) {
>> >              continue;
>            }
>            if (host - block->host < block->max_length) {
>                goto found;
>            }
>        }
> ////
>        return NULL;
> 
> Should we call rcu_read_unlock at "////"?

Yes.

Paolo