[Devel] Re: [PATCH 1/8] checkpoint: model ckpt_siginfo on signalfd_siginfo

Sukadev Bhattiprolu sukadev at linux.vnet.ibm.com
Thu Jul 15 17:55:46 PDT 2010


Nathan Lynch [ntl at pobox.com] wrote:
| ckpt_siginfo doesn't cover all of the possible siginfo fields; the
| current code overloads some fields (such as si_uid for si_overrun),
| which is confusing.  Signed-ness on some fields is not correct either.
| 
| signalfd_siginfo is a straightforward format for representing siginfo,
| so copy it and update the signal checkpoint code to use the new
| fields.  Although the ckpt_siginfo struct grows, there should be no
| behavioral changes.

| 
| Signed-off-by: Nathan Lynch <ntl at pobox.com>

Switching to the signalfd_siginfo style makes it more easier to
maintain.  So for all patches in this set:

Reviewed-by: Sukadev Bhattiprolu <sukadev at linux.vnet.ibm.com>
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list