Re: [PATCH v18 20/80] c/r: basic infrastructure for checkpoint/restart

Oren Laadan
2009-09-24T17:33:31Z

Daniel Walker wrote:
> On Wed, 2009-09-23 at 19:51 -0400, Oren Laadan wrote:
>> /
>> +static char *__ckpt_generate_fmt(struct ckpt_ctx *ctx, char *prefmt, char *fmt)
>> +{
>> + static int warn_notask = 0;
>> + static int warn_prefmt = 0;
>
> Shouldn't need the initializer since it's static..
>

Yup ...

>
>> +/* read the checkpoint header */
>> +static int restore_read_header(struct ckpt_ctx *ctx)
>> +{
>> + struct ckpt_hdr_header *h;
>> + struct new_utsname *uts = NULL;
>> + int ret;
>> +
>> + h = ckpt_read_obj_type(ctx, sizeof(*h), CKPT_HDR_HEADER);
>> + if (IS_ERR(h))
>> + return PTR_ERR(h);
>> +
>> + ret = -EINVAL;
>> + if (h->magic != CHECKPOINT_MAGIC_HEAD ||
>> + h->rev != CHECKPOINT_VERSION ||
>> + h->major != ((LINUX_VERSION_CODE >> 16) & 0xff) ||
>> + h->minor != ((LINUX_VERSION_CODE >> 8) & 0xff) ||
>> + h->patch != ((LINUX_VERSION_CODE) & 0xff))
>> + goto out;
>
> Do you still need this LINUX_VERSION_CODE stuff ? I would think once
> it's in mainline you wouldn't need to track that..

In short: yes.

This is our poor-man's way to telling the kernel version on which
a given checkpoint image was generated.

The image format is a blob that may change between kernel releases.
Conversion between version formats will be done by userspace tools.
Tagging the image with the version of the kernel serves two goals:
it indicates the image version to the kernel at restart time, and
it will be used by userspace conversion tools.

How the kernel and image versions are encoded is WIP and is likely
to become more comprehensive soon.

>
> These both got flagged by checkpatch .. Your series is marked in a
> couple other places with checkpatch errors .. If you haven't already
> reviewed those errors, it would be a good idea to review them.
>

Sure, will re-review to remove remaining errors that sneaked in.

Thanks,

Oren.