On Fri, Apr 20, 2007 at 01:15:34AM +0800, Zou, Nanhai wrote:
>
> > -----Original Message-----
> > From: Simon Horman [mailto:horms@xxxxxxxxxxxx]
> > Sent: 2007年4月16日 16:21
> > To: fastboot@xxxxxxxxxxxxxx; linux-ia64@xxxxxxxxxxxxxxx
> > Cc: Vivek Goyal; Luck, Tony; Zou, Nanhai
> > Subject: Re: [PATCH] kexec/kdump: Use generic elf code on ia64
[snip]
> > -crash_save_this_cpu(void)
> > +ia64_kexec_elf_core_copy_regs(elf_gregset_t *elfregs, struct pt_regs *regs)
> > {
> > - void *buf;
> > unsigned long cfm, sof, sol;
> > -
> > - int cpu = smp_processor_id();
> > - struct elf_prstatus *prstatus = &per_cpu(elf_prstatus, cpu);
> > -
> > - elf_greg_t *dst = (elf_greg_t *)&(prstatus->pr_reg);
> > - memset(prstatus, 0, sizeof(*prstatus));
> > - prstatus->pr_pid = current->pid;
> > + elf_greg_t *dst = (elf_greg_t *)elfregs;
>
> There was a concern about struct elf_prstatus is so big on IA64 that it better not to put it on stack, consider crash happen with a deep stack..., So I put the structure into percpu data.
>
> With this patch, it fallbacks to stack.
Thanks, I was wondering why was per_cpu rather than on the stack.
I'll rework things a bit so per_cpu data is used once again.
[snip]
--
Horms
H: http://www.vergenet.net/~horms/
W: http://www.valinux.co.jp/en/
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
|