Re: Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object creation with a specified id)

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Kirill Korotaev
Date: Tuesday, April 22, 2008 - 11:40 pm

> If the current interface is insufficient, we should first expand it in

iptables is a bad example. Luckily for checkpointing - it always had an interface
"load full state", "dump full state".

But even iptables are not working in current form for checkpointing - they can't
save/restore state of conntracks. Do you think netdev@/netfilter@ guys will be happy
to have APIs allowing to set conntracks and have all the pain related
to API stability - cause conntrack state changed a couple of times during last 2 years.

Consider more intimate kernel states like:
a. task statistics
b. task start time
c. load average
d. skb state and it's data.
e. mount tree.

If you think over, e.g. (b) is a bad thing. It was used to be accounted in jiffies, then in timespec.
(a) is another example of dataset which we can't predict. task statistics change over a time.
Why bother with such intimate data in user-space at all?
Why the hell user-space should know about it and be ABLE to modify it?

Why do we need to export ability to set IDs for some objects which none of the
operating systems do and then to have a burden to support it for application compatibility
the rest of our lifes? Do you really believe none of the applications except for checkpointing
will be using it?

My personal vision is that:
1. user space must initialize checkpointing/restore state via some system call,
   supply file descriptor from where data can be read/written to.
2. must call the syscall asking kernel to restore/save different subsytems one by one.
3. finalize cpt/restore state via the syscall
But user-space MUST NOT bother about data content. At least not about the data supplied by the kernel.
It can add additional sections if needed, e.g. about iptables state.

Having all this functionality in a signle syscall we specifically CLAIM a black box,
and that no one can use this interfaces for something different from checkpoint/restore.

So I think we have to know what other maintainers think before we can go.


--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/4] - v2 - Object creation with a specified id, Nadia.Derbey, (Thu Apr 17, 10:44 pm)
Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object cre ..., Alexey Dobriyan, (Tue Apr 22, 12:36 pm)
Re: Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object ..., Serge E. Hallyn, (Tue Apr 22, 12:51 pm)
Re: Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object ..., Kirill Korotaev, (Tue Apr 22, 11:40 pm)
Re: Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object ..., Kirill Korotaev, (Thu Apr 24, 12:00 am)
Re: Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object ..., Eric W. Biederman, (Wed Jul 9, 6:58 pm)
Re: Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object ..., Serge E. Hallyn, (Thu Jul 10, 10:32 am)
Re: Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object ..., Eric W. Biederman, (Thu Jul 10, 11:55 am)
Re: Checkpoint/restart (was Re: [PATCH 0/4] - v2 - Object ..., Eric W. Biederman, (Thu Jul 10, 12:21 pm)