On 07/11/2012 10:20 AM, levin li wrote:
> +int objlist_cache_reclaim(uint32_t vid)
> +{
> +     struct objlist_reclaim_work *ow;
> +
> +     ow = zalloc(sizeof(*ow))

use xzalloc()

> +     if (vdi_exist(vid)) {
> +             eprintf("VDI (%" PRIx32 ") is still exist, can not reclaim\n",
> +                     vid);
> +             return;
> +     }

 VDI

should be "VDI (%" PRIx32 ") is still in use, can not be deleted\n"

Also add document in the source file to objlist_cache_{remove,delete},
and why we need two functions.

Thanks,
Yuan
-- 
sheepdog mailing list
[email protected]
http://lists.wpkg.org/mailman/listinfo/sheepdog

Reply via email to