Re: [PATCH 2/14] firmware update for av7110 dvb driver

[Manuel Estrada Sainz]

On Wed, Oct 08, 2003 at 05:54:18PM +0200, Michael Hunold wrote:
> Hello Marcel,
> >>>... send to Linus only. (You don't want a 150kB bzip2 compressed 
> >>>firmware blob, don't you? In case you do, drop me a line.)
> >
> >
> >>the request_firmware() framework is part of Linux 2.4 and 2.6 and so for
> >>most drivers the firmware file can be loaded from userspace through proc
> >>or sysfs. Please take a look at it.
> 
> >Yes, we know. When I looked at it the last time, there were some 
> >problems that kept us from actually finishing the work.
> >
> >(If you did not use a hotplug agent, then the system was frozen, because 
> >the firmware foo did not use it's own workqueue)
> 
> As a follow-up to my post: I just checked what already got in and what's 
> still missing. I attached the stuff from request_firmwar() that didn't 
> make it into the kernel yet.
> 
> Please note that I did not check if the problem has been solved 
> otherwise. Perhaps Manuel Estrada Sainz <ranty@xxxxxxxxxx> can tell us 
> what the current state is?

 The patch should work but Andrew Morton didn't like the fact that in
 practice the kernel end's up having some threads around full time for
 the sole purpose of loading firmware once in a while.

 I should make a new patch to spawn a thread for each firmware load and
 have them die when the job is done. And stop using workqueues at all.

 	- Performance should not be an issue since firmware should
	  seldom be needed.
	- Code complexity shouldn't change much either.

 Comments are welcomed.

 If the issue seams urgent, the current "private workqueue" patch could
 be applied and replaced later with a cleaner solution.
 
 Just for the record I should get the job done within a couple of weeks.
 A proposal patch would of course help :)

 Have a nice day

 	Manuel

-- 
--- Manuel Estrada Sainz <ranty@xxxxxxxxxx>
                         <ranty@xxxxxxxxxxx>
			 <ranty@xxxxxxxxxxxxxxxxxxxxx>
------------------------ <manuel.estrada@xxxxxxxxxxxxx> -------------------
Let us have the serenity to accept the things we cannot change, courage to
change the things we can, and wisdom to know the difference.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/