Date: Tue, 22 Sep 2015 02:04:50 +0300 From: Konstantin Belousov <kostikbel@gmail.com> To: John Baldwin <jhb@freebsd.org> Cc: "src-committers@freebsd.org" <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r287934 - head/sys/boot/efi/loader Message-ID: <20150921230450.GB67105@kib.kiev.ua> In-Reply-To: <7425989.fyHR6C7Hof@ralph.baldwin.cx> References: <13e1fdb2-6e22-4371-95e4-a556c357fa8d@me.com> <CAJ-Vmok6%2B2vcekUZLU0uRSGqQtgNqmBQBu82sOiSMR_BPy0U2w@mail.gmail.com> <7425989.fyHR6C7Hof@ralph.baldwin.cx>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Sep 21, 2015 at 03:31:11PM -0700, John Baldwin wrote: > (I'm of the opinion we should inline atomics and locks, etc. in modules > built as part of a kernel to match that kernel's config and only use shims > for modules built standalone. Most of this can probably be accomplished by > having modules built as part of a kernel not define KLD_MODULE during their > build but only use that for standalone modules. However, I haven't audited > all the uses of KLD_MODULE to see if we might want some of them to be on > for "bundled" modules in which case we'd need to add a new #define that > gets defined for "bundled" modules but not standalone modules (or gets > defined for both?) and fixup the code appropriately.) At least one use of KLD_MODULE outside the atomics and locks category is the definition of MAKEDEV_ETERNAL_KLD in sys/conf.h.
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20150921230450.GB67105>