[Devel] Re: [RFC v14-rc][PATCH 17/23] Record 'struct file' object instead of the file name for VMAs

[Dave Hansen]

On Fri, 2009-03-20 at 14:47 -0400, Oren Laadan wrote:
> +       /* if file-backed, add 'file' to the hash (will keep a reference) */
> +       if (vma->vm_file) {
> +               new = cr_obj_add_ptr(ctx, vma->vm_file,
> +                                    &objref, CR_OBJ_FILE, 0);
> +               cr_debug("vma %p objref %d file %p)\n",
> +                        vma, objref, vma->vm_file);
> +               if (new < 0) {
> +                       ret  = new;
> +                       goto out;
> +               }
> +       }

This is a very, very common pattern in these patches now.

Can't we do better than this?  The objhash already has intimate
knowledge of the files with which it deals.  So, why not also teach it
how to write those into the checkpoint stream?  
 
int cr_obj_add_ptr(struct cr_ctx *ctx, void *ptr, int *objref,
                   unsigned short type, unsigned short flags)
{
        struct cr_objref *obj;
        int ret = 0;

        obj = cr_obj_find_by_ptr(ctx, ptr);
        if (!obj) {
                obj = cr_obj_new(ctx, ptr, 0, type, flags);
                if (IS_ERR(obj))
                        return PTR_ERR(obj);
                else
                        ret = 1;

        } else if (obj->type != type)   /* sanity check */
                return -EINVAL;
        *objref = obj->objref;
-       return ret;
+	return cr_write_object(ctx, ptr, type);
}

int cr_write_object(ctx, ptr, type)
{
	switch(type) {
		...
		case CR_OBJ_FILE:
			return cr_write_file(ctx, (file *)ptr);
		...
	}
}

Why not?  It fits in with the rest of the objhash.


-- Dave

_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers