Date: Mon, 21 Sep 2015 22:53:11 +0000 (GMT) From: Rui Paulo <rpaulo@me.com> To: Adrian Chadd <adrian.chadd@gmail.com> Cc: jhb@freebsd.org, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r287934 - head/sys/boot/efi/loader Message-ID: <35a0f1b6-0236-4b0e-b919-00cab07429be@me.com>
next in thread | raw e-mail | index | archive | help
No, that doesn't work very well. Not only the modules don't auto-load, th= e way the modules are compiled is different. See, for example, cam ctl wh= ich doesn't compile the sg code when it's built into the kernel, but compi= les it when it's built as a module. The sg code is currently buggy and ca= uses insta-panics with GNOME 3 (perhaps the auto-mounter in hald (?)=EF=BB= =BF).=0A=0A--=0ARui Paulo=0A=0A=0AOn Sep 21, 2015, at 11:24 AM, Adrian Cha= dd <adrian.chadd@gmail.com> wrote:=0A=0A> Hi,=0A>=0A> Warner has been work= ing on the modular kernel thing. But honestly, I=0A> think we should just = start biting that bullet and ship a modules-only=0A> GENERIC by default..=0A= >=0A>=0A> -a=0A>=0A>=0A> On 21 September 2015 at 11:02, Rui Paulo <rpaulo@= me.com> wrote:=0A>> So, we're going to keep ignoring the problem and keep = patching things up?=0A>> It's a bit sad that a single driver (pmspcv) is a= ble to cause so much=0A>> problems.=0A>>=0A>> --=0A>> Rui Paulo=0A>>=0A>>=0A= >> On Sep 17, 2015, at 01:36 PM, John Baldwin <jhb@FreeBSD.org> wrote:=0A>= >=0A>> Author: jhb=0A>> Date: Thu Sep 17 20:36:46 2015=0A>> New Revision: = 287934=0A>> URL: https://svnweb.freebsd.org/changeset/base/287934=0A>> =0A= >>=0A>> Log:=0A>> The EFI boot loader allocates a single chunk of contiguo= us memory to=0A>> hold the kernel, modules, and any other loaded data. Thi= s memory block=0A>> is relocated to the kernel's expected location during = the transfer of=0A>> control from the loader to the kernel.=0A>>=0A>> The = GENERIC kernel on amd64 has recently grown such that a kernel + zfs.ko=0A>= > no longer fits in the default staging size. Bump the default size from=0A= >> 32MB to 48MB to provide more breathing room.=0A>>=0A>> PR: 201679=0A>> = Reviewed by: imp=0A>> MFC after: 1 week=0A>> Differential Revision: https:= //reviews.freebsd.org/D3666=0A>> =0A>>=0A>> Modified:=0A>> head/sys/boot/= efi/loader/copy.c=0A>>=0A>> Modified: head/sys/boot/efi/loader/copy.c=0A>>= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=0A>> --- head/sys/boot/efi/loader/copy.c Thu Sep 17 20:36:34 201= 5=0A>> (r287933)=0A>> +++ head/sys/boot/efi/loader/copy.c Thu Sep 17 20:36= :46 2015=0A>> (r287934)=0A>> @@ -38,7 +38,7 @@ __FBSDID("$FreeBSD$");=0A>>= #include <efilib.h>=0A>>=0A>> #ifndef EFI_STAGING_SIZE=0A>> -#define EFI_= STAGING_SIZE 32=0A>> +#define EFI_STAGING_SIZE 48=0A>> #endif=0A>>=0A>> #d= efine STAGE_PAGES ((EFI_STAGING_SIZE) * 1024 * 1024 / 4096)=0A>>=0A=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?35a0f1b6-0236-4b0e-b919-00cab07429be>