Re: CVS commit: src/sys/arch/zaurus/conf

2020-01-05 Thread Nick Hudson
On 05/01/2020 13:47, Izumi Tsutsui wrote: [snip] My patch allows COPTS to be overridden as http://src.illumos.org/source/xref/netbsd-src/sys/conf/Makefile.kern.inc#69 has 69 DEFCOPTS?= -O2 70 COPTS?= ${DEFCOPTS} which I believe allows COPTS to be overridden by makeoptions

Re: CVS commit: src/sys/arch/amd64

2020-01-05 Thread Emmanuel Dreyfus
On Sun, Jan 05, 2020 at 02:43:43PM +0100, Maxime Villard wrote: > I have now requested to core@ that multiboot in amd64 be reverted entirely. So far I privilegied working on a fix to the boot problem that was reported, rather than spending time on a revert. This was not a futile effort, since at

Re: CVS commit: src/sys/arch/zaurus/conf

2020-01-05 Thread Izumi Tsutsui
> >> I don't see how as sys/arch/zaurus/conf/INSTALL doesn't define DDB. > > > > On zaurus GENERIC also has a size restriction (due to bootloader). > > Yes, I understand that. What I don't understand is how my > sys/arch/arm/conf/Makefile.arm change affects the zaurus INSTALL kernel > size

Re: CVS commit: src/sys/arch/amd64

2020-01-05 Thread Maxime Villard
Le 05/01/2020 à 13:56, Maxime Villard a écrit : > Le 05/01/2020 à 02:03, Emmanuel Dreyfus a écrit : >> On Sat, Jan 04, 2020 at 08:43:16AM +0100, Maxime Villard wrote: >>> +.section multiboot,"",@note >>> Why @note? It will be in the .text anyway. Also why no dot in the section >>> name? That's

Re: CVS commit: src/sys/arch/zaurus/conf

2020-01-05 Thread Nick Hudson
On 05/01/2020 13:31, Martin Husemann wrote: On Sun, Jan 05, 2020 at 01:21:46PM +, Nick Hudson wrote: Yes, I understand that. What I don't understand is how my sys/arch/arm/conf/Makefile.arm change affects the zaurus INSTALL kernel size because INSTALL specifically disables DDB and my patch

Re: CVS commit: src/sys/arch/zaurus/conf

2020-01-05 Thread Martin Husemann
On Sun, Jan 05, 2020 at 01:21:46PM +, Nick Hudson wrote: > Yes, I understand that. What I don't understand is how my > sys/arch/arm/conf/Makefile.arm change affects the zaurus INSTALL kernel > size because INSTALL specifically disables DDB and my patch is > conditional on DDB It did only

Re: CVS commit: src/sys/arch/zaurus/conf

2020-01-05 Thread Nick Hudson
On 04/01/2020 16:32, Izumi Tsutsui wrote: The problem is caused by sys/arch/arm/conf/Makefile.arm. It defines "COPTS+= -mapcs-frame" in recent rev 1.52 http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/arch/arm/conf/Makefile.arm#rev1.52 but MI sys/conf/Makefile.kern.inc defines COPTS+=-O2 only

Re: CVS commit: src/sys/arch/amd64

2020-01-05 Thread Maxime Villard
Le 05/01/2020 à 02:03, Emmanuel Dreyfus a écrit : > On Sat, Jan 04, 2020 at 08:43:16AM +0100, Maxime Villard wrote: >> +.section multiboot,"",@note >> Why @note? It will be in the .text anyway. Also why no dot in the section >> name? That's supposed to be the naming convention. > > The idea is