Quoting Oren Laadan (orenl@xxxxxxxxxxx): > > > 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 :) Right, I thought it looked familiar :) _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers
