]> pilppa.com Git - linux-2.6-omap-h63xx.git/commit
[IA64] fix getpid and set_tid_address fast system calls for pid namespaces
authorPavel Emelyanov <xemul@openvz.org>
Fri, 28 Mar 2008 21:27:00 +0000 (14:27 -0700)
committerTony Luck <tony.luck@intel.com>
Wed, 9 Apr 2008 17:33:36 +0000 (10:33 -0700)
commit96ded9dadde397a9e372a650534a9ffbba97194a
tree9aaa92034e87218d3ab7ba7e33fad0661899a0ad
parentd167cb85150bd473a27df71e3116a9cc0008f5dd
[IA64] fix getpid and set_tid_address fast system calls for pid namespaces

The sys_getpid() and sys_set_tid_address() behavior changed from

return current->tgid

to

struct pid *pid;
pid = current->pids[PIDTYPE_PID].pid;
return pid->numbers[pid->level].nr;

But the fast system calls on ia64 still operate the old way.  Patch them
appropriately to let ia64 work with pid namespaces.  Besides, this is one more
step in deprecating of pid and tgid on task_struct.

The fsys_getppid() is to be patched as well, but its logic is much
more complex now, so I will make it later.

One thing I'm not 100% sure is the trick with the IA64_UPID_SHIFT.  On order
to access the pid->level's element of an array I have to perform the following
calculations

pid + sizeof(struct upid) * pid->level

The problem is that ia64 can only multiply float point registers, while all
the offsets I have in code are in rXX ones.  Fortunately, the sizeof(struct
upid) is 32 bytes on ia64 (and is very unlikely to ever change), so the
calculations get simpler:

pid + pid->level << 5

So, I introduce the IA64_UPID_SHIFT and use the shl instruction.  I also
looked at how gcc compiles the similar place and found that it makes it with
shift as well.  Is this OK to do so?

Tested with ski emulator with 2.6.24 kernel, but fits 2.6.25-rc4 and
2.6.25-rc4-mm1 as well.

Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Cc: David Mosberger-Tang <davidm@hpl.hp.com>
Cc: Hidetoshi Seto <seto.hidetoshi@jp.fujitsu.com>
Cc: Fenghua Yu <fenghua.yu@intel.com>
Cc: Amy Griffis <amy.griffis@hp.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Tony Luck <tony.luck@intel.com>
arch/ia64/kernel/asm-offsets.c
arch/ia64/kernel/fsys.S