Skip to content

stack overflow safe kdump (2.6.18-rc1-i386) try#2

Fernando Luis Vázquez Cao lkml 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 4 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4stack overflow safe kdump (2.6.18-rc1-i386) - safe_smp_processor_idarch/i386/kernel/smp.c include/asm-i386/smp.h+28 -02 replies
2/4stack overflow safe kdump (2.6.18-rc1-i386) - safe_smp_processor_id_voyagerarch/i386/mach-voyager/voyager_smp.c+7 -00 replies
3/4stack overflow safe kdump (2.6.18-rc1-i386) - crash_use_safe_smp_processor_idarch/i386/kernel/crash.c+2 -20 replies
4/4stack overflow safe kdump (2.6.18-rc1-i386) - safe_smp_send_nmi_allbutselfarch/i386/kernel/crash.c+4 -10 replies

Reviews trailers per patch

None of the 4 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchEric Biederman
SeriesA-b
1/4
2/4
3/4
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [PATCH 0/4] stack overflow safe kdump (2.6.18-rc1-i386) try#2

Hi,

I tried to incorporate all the ideas received after the previous post (thank you!). In particular I hope the new code is handling the Voyager case properly.

---

This is a the first of a series of patch-sets aiming at making kdump more robust against stack overflows.

This patch set does the following:

* Add safe_smp_processor_id function to i386 architecture (this function was inspired by the x86_64 function of the same name).

* Substitute "smp_processor_id" with the stack overflow-safe "safe_smp_processor_id" in the reboot path to the second kernel.

List of patches (the last two should be applied in the order of appearance):

[1/4] safe_smp_processor_id: stack overflow safe implementation of smp_processor_id.

[2/4] safe_smp_processor_id_voyager: stack overflow safe implementation of smp_processor_id for i386-voyager.

[3/4] crash_use_safe_smp_processor_id: replace smp_processor_id with safe_smp_processor_id in "arch/i386/kernel/crash.c".

[4/4] safe_smp_send_nmi_allbutself: re-implement smp_send_nmi_allbutself so that calls to smp_processor_id (through send_IPI_allbutself) can be replaced with safe_smp_processor_id without affecting other parts of the kernel (as suggested by Eric Biederman).

I am looking forward to your comments and suggestions.

Regards,

Fernando