Re: CVS commit: src/sys

[christos%zoulas.com@localhost (Christos Zoulas)]

On Nov 5, 11:17am, martin%netbsd.org@localhost (Martin Husemann) wrote:
-- Subject: Re: CVS commit: src/sys

| On Sat, Nov 04, 2017 at 06:17:55PM -0400, Christos Zoulas wrote:
| > Module Name:	src
| > Committed By:	christos
| > Date:		Sat Nov  4 22:17:55 UTC 2017
| > 
| > Modified Files:
| > 	src/sys/ddb: db_elf.c
| > 	src/sys/kern: kern_ksyms.c subr_kobj.c
| > 	src/sys/sys: ksyms.h
| > 
| > Log Message:
| > use Elf_Sym ** instead of casting.
| 
| The ksyms.h change to define ELFSIZE is not good, it breaks various
| builds.
| 
| Can we avoid it and include something defining it properly at the include
| sites (if it is not yet defined)?

Yes, it broke m68k and mips. I fixed m68k and examining mips. I am still
trying to understand what is defining where and why. I.e. DB_ELFSIZE vs.
ARCH_ELFSIZE. Unfortunately there were other build issues introduced in
the meantime (like the kdump ioctl mess that have been slowing me down.
I am not going to leave this the way it is now (for sure).

christos