Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys/kern




> On May 31, 2020, at 1:33 AM, Rin Okuyama <rin%netbsd.org@localhost> wrote:
> 
> db_show_callout(): struct callout_cpu and cpu_info are too much for stack.
> 
> XXX
> DDB can be running in the interrupt context, e.g., when activated from
> console. Therefore, use kmem_intr_alloc(9) instead of kmem_alloc(9).
> 
> Frame size, e.g. for m68k, becomes:
>    9212 (oops!) --> 0
> 

I'm not sure we want to be calling memory allocators from within ddb.  I think it would be better to simply allocate that space in the .bss segment -- ddb only runs on 1 CPU at a time.

-- thorpej



Home | Main Index | Thread Index | Old Index