On 04/04/2014 03:57 AM, arei.gong...@huawei.com wrote: > From: ChenLiang <chenlian...@huawei.com> > > Rebuild the cache_is_cached function by cache_get_by_addr. And > drops the asserts because the caller is also asserting the same > thing.
Technically, the caller (parent - arch_init.c:save_xbzrle_page) is not asserting the same thing; rather, it is the callee (child - cache_get_by_addr() used first in the two methods where you dropped the asserts) that is making the same assertion. > > Signed-off-by: ChenLiang <chenlian...@huawei.com> > Signed-off-by: Gonglei <arei.gong...@huawei.com> > Reviewed-by: Dr. David Alan Gilbert <dgilb...@redhat.com> > --- > page_cache.c | 38 ++++++++++++++++---------------------- > 1 file changed, 16 insertions(+), 22 deletions(-) The commit message can be trivially amended; so Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
