diff options
author | Sukadev Bhattiprolu <sukadev@us.ibm.com> | 2006-09-29 05:00:07 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-29 12:18:12 -0400 |
commit | f400e198b2ed26ce55b22a1412ded0896e7516ac (patch) | |
tree | a3d78bfc1c20635e199fe0fe85aaa1d8792acc58 /arch | |
parent | 959ed340f4867fda7684340625f60e211c2296d6 (diff) |
[PATCH] pidspace: is_init()
This is an updated version of Eric Biederman's is_init() patch.
(http://lkml.org/lkml/2006/2/6/280). It applies cleanly to 2.6.18-rc3 and
replaces a few more instances of ->pid == 1 with is_init().
Further, is_init() checks pid and thus removes dependency on Eric's other
patches for now.
Eric's original description:
There are a lot of places in the kernel where we test for init
because we give it special properties. Most significantly init
must not die. This results in code all over the kernel test
->pid == 1.
Introduce is_init to capture this case.
With multiple pid spaces for all of the cases affected we are
looking for only the first process on the system, not some other
process that has pid == 1.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: Sukadev Bhattiprolu <sukadev@us.ibm.com>
Cc: Dave Hansen <haveblue@us.ibm.com>
Cc: Serge Hallyn <serue@us.ibm.com>
Cc: Cedric Le Goater <clg@fr.ibm.com>
Cc: <lxc-devel@lists.sourceforge.net>
Acked-by: Paul Mackerras <paulus@samba.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/alpha/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/arm/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/arm26/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/i386/lib/usercopy.c | 2 | ||||
-rw-r--r-- | arch/i386/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/ia64/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/m32r/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/m68k/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/mips/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/powerpc/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/powerpc/platforms/pseries/ras.c | 2 | ||||
-rw-r--r-- | arch/ppc/kernel/traps.c | 2 | ||||
-rw-r--r-- | arch/ppc/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/s390/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/sh/mm/fault.c | 2 | ||||
-rw-r--r-- | arch/sh64/mm/fault.c | 6 | ||||
-rw-r--r-- | arch/um/kernel/trap.c | 2 | ||||
-rw-r--r-- | arch/x86_64/mm/fault.c | 4 | ||||
-rw-r--r-- | arch/xtensa/mm/fault.c | 2 |
19 files changed, 22 insertions, 22 deletions
diff --git a/arch/alpha/mm/fault.c b/arch/alpha/mm/fault.c index 622dabd8468..8871529a34e 100644 --- a/arch/alpha/mm/fault.c +++ b/arch/alpha/mm/fault.c | |||
@@ -193,7 +193,7 @@ do_page_fault(unsigned long address, unsigned long mmcsr, | |||
193 | /* We ran out of memory, or some other thing happened to us that | 193 | /* We ran out of memory, or some other thing happened to us that |
194 | made us unable to handle the page fault gracefully. */ | 194 | made us unable to handle the page fault gracefully. */ |
195 | out_of_memory: | 195 | out_of_memory: |
196 | if (current->pid == 1) { | 196 | if (is_init(current)) { |
197 | yield(); | 197 | yield(); |
198 | down_read(&mm->mmap_sem); | 198 | down_read(&mm->mmap_sem); |
199 | goto survive; | 199 | goto survive; |
diff --git a/arch/arm/mm/fault.c b/arch/arm/mm/fault.c index a5b33ff3924..5e658a87449 100644 --- a/arch/arm/mm/fault.c +++ b/arch/arm/mm/fault.c | |||
@@ -198,7 +198,7 @@ survive: | |||
198 | return fault; | 198 | return fault; |
199 | } | 199 | } |
200 | 200 | ||
201 | if (tsk->pid != 1) | 201 | if (!is_init(tsk)) |
202 | goto out; | 202 | goto out; |
203 | 203 | ||
204 | /* | 204 | /* |
diff --git a/arch/arm26/mm/fault.c b/arch/arm26/mm/fault.c index a7c4cc92209..a1f6d8a9cc3 100644 --- a/arch/arm26/mm/fault.c +++ b/arch/arm26/mm/fault.c | |||
@@ -185,7 +185,7 @@ survive: | |||
185 | } | 185 | } |
186 | 186 | ||
187 | fault = -3; /* out of memory */ | 187 | fault = -3; /* out of memory */ |
188 | if (tsk->pid != 1) | 188 | if (!is_init(tsk)) |
189 | goto out; | 189 | goto out; |
190 | 190 | ||
191 | /* | 191 | /* |
diff --git a/arch/i386/lib/usercopy.c b/arch/i386/lib/usercopy.c index efc7e7d5f4d..08502fc6d0c 100644 --- a/arch/i386/lib/usercopy.c +++ b/arch/i386/lib/usercopy.c | |||
@@ -739,7 +739,7 @@ survive: | |||
739 | retval = get_user_pages(current, current->mm, | 739 | retval = get_user_pages(current, current->mm, |
740 | (unsigned long )to, 1, 1, 0, &pg, NULL); | 740 | (unsigned long )to, 1, 1, 0, &pg, NULL); |
741 | 741 | ||
742 | if (retval == -ENOMEM && current->pid == 1) { | 742 | if (retval == -ENOMEM && is_init(current)) { |
743 | up_read(¤t->mm->mmap_sem); | 743 | up_read(¤t->mm->mmap_sem); |
744 | blk_congestion_wait(WRITE, HZ/50); | 744 | blk_congestion_wait(WRITE, HZ/50); |
745 | goto survive; | 745 | goto survive; |
diff --git a/arch/i386/mm/fault.c b/arch/i386/mm/fault.c index 50d8617391d..2581575786c 100644 --- a/arch/i386/mm/fault.c +++ b/arch/i386/mm/fault.c | |||
@@ -589,7 +589,7 @@ no_context: | |||
589 | */ | 589 | */ |
590 | out_of_memory: | 590 | out_of_memory: |
591 | up_read(&mm->mmap_sem); | 591 | up_read(&mm->mmap_sem); |
592 | if (tsk->pid == 1) { | 592 | if (is_init(tsk)) { |
593 | yield(); | 593 | yield(); |
594 | down_read(&mm->mmap_sem); | 594 | down_read(&mm->mmap_sem); |
595 | goto survive; | 595 | goto survive; |
diff --git a/arch/ia64/mm/fault.c b/arch/ia64/mm/fault.c index d8b1b4ac7f2..59f3ab93761 100644 --- a/arch/ia64/mm/fault.c +++ b/arch/ia64/mm/fault.c | |||
@@ -280,7 +280,7 @@ ia64_do_page_fault (unsigned long address, unsigned long isr, struct pt_regs *re | |||
280 | 280 | ||
281 | out_of_memory: | 281 | out_of_memory: |
282 | up_read(&mm->mmap_sem); | 282 | up_read(&mm->mmap_sem); |
283 | if (current->pid == 1) { | 283 | if (is_init(current)) { |
284 | yield(); | 284 | yield(); |
285 | down_read(&mm->mmap_sem); | 285 | down_read(&mm->mmap_sem); |
286 | goto survive; | 286 | goto survive; |
diff --git a/arch/m32r/mm/fault.c b/arch/m32r/mm/fault.c index dc18a33eefe..8d5f551b575 100644 --- a/arch/m32r/mm/fault.c +++ b/arch/m32r/mm/fault.c | |||
@@ -299,7 +299,7 @@ no_context: | |||
299 | */ | 299 | */ |
300 | out_of_memory: | 300 | out_of_memory: |
301 | up_read(&mm->mmap_sem); | 301 | up_read(&mm->mmap_sem); |
302 | if (tsk->pid == 1) { | 302 | if (is_init(tsk)) { |
303 | yield(); | 303 | yield(); |
304 | down_read(&mm->mmap_sem); | 304 | down_read(&mm->mmap_sem); |
305 | goto survive; | 305 | goto survive; |
diff --git a/arch/m68k/mm/fault.c b/arch/m68k/mm/fault.c index 5e2d87c10c8..911f2ce3f53 100644 --- a/arch/m68k/mm/fault.c +++ b/arch/m68k/mm/fault.c | |||
@@ -181,7 +181,7 @@ good_area: | |||
181 | */ | 181 | */ |
182 | out_of_memory: | 182 | out_of_memory: |
183 | up_read(&mm->mmap_sem); | 183 | up_read(&mm->mmap_sem); |
184 | if (current->pid == 1) { | 184 | if (is_init(current)) { |
185 | yield(); | 185 | yield(); |
186 | down_read(&mm->mmap_sem); | 186 | down_read(&mm->mmap_sem); |
187 | goto survive; | 187 | goto survive; |
diff --git a/arch/mips/mm/fault.c b/arch/mips/mm/fault.c index a4f8c45c4e8..8423d859077 100644 --- a/arch/mips/mm/fault.c +++ b/arch/mips/mm/fault.c | |||
@@ -171,7 +171,7 @@ no_context: | |||
171 | */ | 171 | */ |
172 | out_of_memory: | 172 | out_of_memory: |
173 | up_read(&mm->mmap_sem); | 173 | up_read(&mm->mmap_sem); |
174 | if (tsk->pid == 1) { | 174 | if (is_init(tsk)) { |
175 | yield(); | 175 | yield(); |
176 | down_read(&mm->mmap_sem); | 176 | down_read(&mm->mmap_sem); |
177 | goto survive; | 177 | goto survive; |
diff --git a/arch/powerpc/mm/fault.c b/arch/powerpc/mm/fault.c index 77953f41d75..e8fa50624b7 100644 --- a/arch/powerpc/mm/fault.c +++ b/arch/powerpc/mm/fault.c | |||
@@ -386,7 +386,7 @@ bad_area_nosemaphore: | |||
386 | */ | 386 | */ |
387 | out_of_memory: | 387 | out_of_memory: |
388 | up_read(&mm->mmap_sem); | 388 | up_read(&mm->mmap_sem); |
389 | if (current->pid == 1) { | 389 | if (is_init(current)) { |
390 | yield(); | 390 | yield(); |
391 | down_read(&mm->mmap_sem); | 391 | down_read(&mm->mmap_sem); |
392 | goto survive; | 392 | goto survive; |
diff --git a/arch/powerpc/platforms/pseries/ras.c b/arch/powerpc/platforms/pseries/ras.c index 903115d67fd..311ed1993fc 100644 --- a/arch/powerpc/platforms/pseries/ras.c +++ b/arch/powerpc/platforms/pseries/ras.c | |||
@@ -337,7 +337,7 @@ static int recover_mce(struct pt_regs *regs, struct rtas_error_log * err) | |||
337 | err->disposition == RTAS_DISP_NOT_RECOVERED && | 337 | err->disposition == RTAS_DISP_NOT_RECOVERED && |
338 | err->target == RTAS_TARGET_MEMORY && | 338 | err->target == RTAS_TARGET_MEMORY && |
339 | err->type == RTAS_TYPE_ECC_UNCORR && | 339 | err->type == RTAS_TYPE_ECC_UNCORR && |
340 | !(current->pid == 0 || current->pid == 1)) { | 340 | !(current->pid == 0 || is_init(current))) { |
341 | /* Kill off a user process with an ECC error */ | 341 | /* Kill off a user process with an ECC error */ |
342 | printk(KERN_ERR "MCE: uncorrectable ecc error for pid %d\n", | 342 | printk(KERN_ERR "MCE: uncorrectable ecc error for pid %d\n", |
343 | current->pid); | 343 | current->pid); |
diff --git a/arch/ppc/kernel/traps.c b/arch/ppc/kernel/traps.c index d7a433049b4..aafc8e8893d 100644 --- a/arch/ppc/kernel/traps.c +++ b/arch/ppc/kernel/traps.c | |||
@@ -119,7 +119,7 @@ void _exception(int signr, struct pt_regs *regs, int code, unsigned long addr) | |||
119 | * generate the same exception over and over again and we get | 119 | * generate the same exception over and over again and we get |
120 | * nowhere. Better to kill it and let the kernel panic. | 120 | * nowhere. Better to kill it and let the kernel panic. |
121 | */ | 121 | */ |
122 | if (current->pid == 1) { | 122 | if (is_init(current)) { |
123 | __sighandler_t handler; | 123 | __sighandler_t handler; |
124 | 124 | ||
125 | spin_lock_irq(¤t->sighand->siglock); | 125 | spin_lock_irq(¤t->sighand->siglock); |
diff --git a/arch/ppc/mm/fault.c b/arch/ppc/mm/fault.c index bc776beb313..465f451f3bc 100644 --- a/arch/ppc/mm/fault.c +++ b/arch/ppc/mm/fault.c | |||
@@ -291,7 +291,7 @@ bad_area: | |||
291 | */ | 291 | */ |
292 | out_of_memory: | 292 | out_of_memory: |
293 | up_read(&mm->mmap_sem); | 293 | up_read(&mm->mmap_sem); |
294 | if (current->pid == 1) { | 294 | if (is_init(current)) { |
295 | yield(); | 295 | yield(); |
296 | down_read(&mm->mmap_sem); | 296 | down_read(&mm->mmap_sem); |
297 | goto survive; | 297 | goto survive; |
diff --git a/arch/s390/mm/fault.c b/arch/s390/mm/fault.c index f2b9a84dc2b..9c3c19fe62f 100644 --- a/arch/s390/mm/fault.c +++ b/arch/s390/mm/fault.c | |||
@@ -353,7 +353,7 @@ no_context: | |||
353 | */ | 353 | */ |
354 | out_of_memory: | 354 | out_of_memory: |
355 | up_read(&mm->mmap_sem); | 355 | up_read(&mm->mmap_sem); |
356 | if (tsk->pid == 1) { | 356 | if (is_init(tsk)) { |
357 | yield(); | 357 | yield(); |
358 | down_read(&mm->mmap_sem); | 358 | down_read(&mm->mmap_sem); |
359 | goto survive; | 359 | goto survive; |
diff --git a/arch/sh/mm/fault.c b/arch/sh/mm/fault.c index 507f2891470..68663b8f99a 100644 --- a/arch/sh/mm/fault.c +++ b/arch/sh/mm/fault.c | |||
@@ -149,7 +149,7 @@ no_context: | |||
149 | */ | 149 | */ |
150 | out_of_memory: | 150 | out_of_memory: |
151 | up_read(&mm->mmap_sem); | 151 | up_read(&mm->mmap_sem); |
152 | if (current->pid == 1) { | 152 | if (is_init(current)) { |
153 | yield(); | 153 | yield(); |
154 | down_read(&mm->mmap_sem); | 154 | down_read(&mm->mmap_sem); |
155 | goto survive; | 155 | goto survive; |
diff --git a/arch/sh64/mm/fault.c b/arch/sh64/mm/fault.c index f08d0eaf649..8e2f6c28b73 100644 --- a/arch/sh64/mm/fault.c +++ b/arch/sh64/mm/fault.c | |||
@@ -277,7 +277,7 @@ bad_area: | |||
277 | show_regs(regs); | 277 | show_regs(regs); |
278 | #endif | 278 | #endif |
279 | } | 279 | } |
280 | if (tsk->pid == 1) { | 280 | if (is_init(tsk)) { |
281 | panic("INIT had user mode bad_area\n"); | 281 | panic("INIT had user mode bad_area\n"); |
282 | } | 282 | } |
283 | tsk->thread.address = address; | 283 | tsk->thread.address = address; |
@@ -319,14 +319,14 @@ no_context: | |||
319 | * us unable to handle the page fault gracefully. | 319 | * us unable to handle the page fault gracefully. |
320 | */ | 320 | */ |
321 | out_of_memory: | 321 | out_of_memory: |
322 | if (current->pid == 1) { | 322 | if (is_init(current)) { |
323 | panic("INIT out of memory\n"); | 323 | panic("INIT out of memory\n"); |
324 | yield(); | 324 | yield(); |
325 | goto survive; | 325 | goto survive; |
326 | } | 326 | } |
327 | printk("fault:Out of memory\n"); | 327 | printk("fault:Out of memory\n"); |
328 | up_read(&mm->mmap_sem); | 328 | up_read(&mm->mmap_sem); |
329 | if (current->pid == 1) { | 329 | if (is_init(current)) { |
330 | yield(); | 330 | yield(); |
331 | down_read(&mm->mmap_sem); | 331 | down_read(&mm->mmap_sem); |
332 | goto survive; | 332 | goto survive; |
diff --git a/arch/um/kernel/trap.c b/arch/um/kernel/trap.c index 61a23fff439..c7b195c7e51 100644 --- a/arch/um/kernel/trap.c +++ b/arch/um/kernel/trap.c | |||
@@ -120,7 +120,7 @@ out_nosemaphore: | |||
120 | * us unable to handle the page fault gracefully. | 120 | * us unable to handle the page fault gracefully. |
121 | */ | 121 | */ |
122 | out_of_memory: | 122 | out_of_memory: |
123 | if (current->pid == 1) { | 123 | if (is_init(current)) { |
124 | up_read(&mm->mmap_sem); | 124 | up_read(&mm->mmap_sem); |
125 | yield(); | 125 | yield(); |
126 | down_read(&mm->mmap_sem); | 126 | down_read(&mm->mmap_sem); |
diff --git a/arch/x86_64/mm/fault.c b/arch/x86_64/mm/fault.c index 9ba54cc2b5f..3751b4788e2 100644 --- a/arch/x86_64/mm/fault.c +++ b/arch/x86_64/mm/fault.c | |||
@@ -244,7 +244,7 @@ static int is_errata93(struct pt_regs *regs, unsigned long address) | |||
244 | 244 | ||
245 | int unhandled_signal(struct task_struct *tsk, int sig) | 245 | int unhandled_signal(struct task_struct *tsk, int sig) |
246 | { | 246 | { |
247 | if (tsk->pid == 1) | 247 | if (is_init(tsk)) |
248 | return 1; | 248 | return 1; |
249 | if (tsk->ptrace & PT_PTRACED) | 249 | if (tsk->ptrace & PT_PTRACED) |
250 | return 0; | 250 | return 0; |
@@ -580,7 +580,7 @@ no_context: | |||
580 | */ | 580 | */ |
581 | out_of_memory: | 581 | out_of_memory: |
582 | up_read(&mm->mmap_sem); | 582 | up_read(&mm->mmap_sem); |
583 | if (current->pid == 1) { | 583 | if (is_init(current)) { |
584 | yield(); | 584 | yield(); |
585 | goto again; | 585 | goto again; |
586 | } | 586 | } |
diff --git a/arch/xtensa/mm/fault.c b/arch/xtensa/mm/fault.c index a945a33e85a..dd0dbec2e57 100644 --- a/arch/xtensa/mm/fault.c +++ b/arch/xtensa/mm/fault.c | |||
@@ -144,7 +144,7 @@ bad_area: | |||
144 | */ | 144 | */ |
145 | out_of_memory: | 145 | out_of_memory: |
146 | up_read(&mm->mmap_sem); | 146 | up_read(&mm->mmap_sem); |
147 | if (current->pid == 1) { | 147 | if (is_init(current)) { |
148 | yield(); | 148 | yield(); |
149 | down_read(&mm->mmap_sem); | 149 | down_read(&mm->mmap_sem); |
150 | goto survive; | 150 | goto survive; |