On Nov 5, 11:17am, mar...@netbsd.org (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

Reply via email to