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
|