Serge E. Hallyn wrote: > Quoting Nathan Lynch (ntl@xxxxxxxxx): >> CC checkpoint/process.o >> checkpoint/process.c: In function ‘checkpoint_task_struct’: >> checkpoint/process.c:97: warning: assignment makes integer from pointer without a cast >> checkpoint/process.c:98: warning: assignment makes integer from pointer without a cast >> checkpoint/process.c: In function ‘restore_task_struct’: >> checkpoint/process.c:475: warning: assignment makes pointer from integer without a cast >> checkpoint/process.c:476: warning: assignment makes pointer from integer without a cast >> >> Signed-off-by: Nathan Lynch <ntl@xxxxxxxxx> >> --- >> checkpoint/process.c | 8 ++++---- >> 1 files changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/checkpoint/process.c b/checkpoint/process.c >> index 41566db..b7f4a43 100644 >> --- a/checkpoint/process.c >> +++ b/checkpoint/process.c >> @@ -94,8 +94,8 @@ static int checkpoint_task_struct(struct ckpt_ctx *ctx, struct task_struct *t) >> h->exit_signal = t->exit_signal; >> h->pdeath_signal = t->pdeath_signal; >> >> - h->set_child_tid = t->set_child_tid; >> - h->clear_child_tid = t->clear_child_tid; >> + h->set_child_tid = (unsigned long)t->set_child_tid; >> + h->clear_child_tid = (unsigned long)t->clear_child_tid; > > But it's defined as __u64 in checkpoint_hdr.h. So on 32-bit this cast > won't be right, right? Should be harmless. > > But a major ack to shutting this thing up. Definitely. In fact, it's addressed already in the PGID patch :) Oren. _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers
