Date: Mon, 21 Nov 2005 13:16:54 -0500 From: John Baldwin <jhb@freebsd.org> To: "Damien Bergamini" <damien.bergamini@free.fr> Cc: cvs-src@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org, Nate Lawson <nate@root.org> Subject: Re: cvs commit: src/sys/modules/iwi Makefile src/sys/dev/iwi if_iwi.c if_iwireg.h if_iwivar.h Message-ID: <200511211316.56666.jhb@freebsd.org> In-Reply-To: <00ff01c5eec3$3b0e6640$0300a8c0@COMETE> References: <20051119165547.0A4BD16A43D@hub.freebsd.org> <4381FEDF.7080607@root.org> <00ff01c5eec3$3b0e6640$0300a8c0@COMETE>
next in thread | previous in thread | raw e-mail | index | archive | help
On Monday 21 November 2005 12:44 pm, Damien Bergamini wrote: > I don't like the idea of keeping the firmware in kernel memory. > It's a rather big file (~200KB). > And there is one for each operating mode (BSS, IBSS, monitor). > > The second reason why I don't like KLDs is because they require > user intervention and users must know which KLD to load for the > mode they want to operate in. And if you put all firmwares in > the same KLD, you end up with a big fat 1MB file. > > I won't go back to anything based on iwicontrol. People simply > don't know how to use it. Trust me. There is not a single day > where I don't get email from people complaining about it. Whatever logic you are doing in the kernel now to figure out which firmware to use you can just as easily do in the kerenl and trigger it by sending a devd event. You could have the userland side do an ioctl to get the type of firmware the driver wants for example. There is _ZERO_ reason that you have to do this in the kernel. You can move this logic out to userland and be much more robust in the process. As it is, trying to do VOP_READ() in the resume path is going to greatly diminish the robustness of suspend/resume. Moving this to userland is not a hard problem. -- John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve" = http://www.FreeBSD.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200511211316.56666.jhb>