Robert,

I believe I've witnessed a similar problem (driver crash loading my PVR-500
(see email thread Re: [ivtv-devel] System hangs loading driver modules for
PVR-500), but only with the PVR-500 and only on my Dell 2850 (specifics in
the thread).  I had a partial fix inserting a sleep in a certain place, but
it only worked 50% of the time.  About half the time it would still crash.
It's possible your patch would work, but I no longer have access to the
machine for testing.  I had to replace the PVR-500s with PVR-250s, and now
it's stable.

The problem is, inserting the sleep as you've suggested is just masking the
real problem, and I don't think the IVTV developers (I am not one of them)
are willing to throw sleeps at a problem without really knowing what it is
doing and why.  I don't blame them.

I'd like someone to confirm this solution, and when I get a chance and
access to a similar system, I will test it myself.

Thanks,
Rich

--

Rich Kadel
Know'bout, Inc.
(858) 433-1747
www.knowbout.com

Do you know about Know'bout?


-----Original Message-----
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] On Behalf Of Robert Hardy
Sent: Tuesday, July 04, 2006 10:32 AM
To: Discussion list for development of the IVTV driver
Subject: Re: [ivtv-devel] Crash on loading of 2nd firmware file

On Tue, 27 Jun 2006, Robert Hardy wrote:
> # cat ivtv-0.6.0-delay_request_firmware.patch
> diff -urNbBp ivtv-0.6.0/driver/ivtv-firmware.c~
> ivtv-0.6.0/driver/ivtv-firmware.c
> --- ivtv-0.6.0/driver/ivtv-firmware.c~  2006-04-14 16:19:21.000000000
-0400
> +++ ivtv-0.6.0/driver/ivtv-firmware.c   2006-04-14 16:19:21.000000000
-0400
> @@ -122,6 +122,9 @@ static int load_fw_direct(const char *fn
>                 }
>                 release_firmware(fw);
>                 IVTV_INFO("loaded %s firmware (%d bytes)\n", fn, retval);
> +// This firmware loading is racy, try waiting to let things settle
> +               IVTV_DEBUG_INFO("Sleeping for 100ms\n");
> +               ivtv_sleep_timeout(HZ / 10, 0);
>         } else {
>                 IVTV_INFO("unable to open firmware %s\n", fn);
>                 IVTV_INFO("did you put the firmware in the hotplug
firmware directory?\n");
>
> This patch solved my firmware loading problems and I've had no problems
with
> the whole 0.6.x series.
>
> I just tried 2.6.17-rh1 with the latest ivtv 0.7 SVN (without my patch
> applied) and it locked my PVR solid trying to load the 2nd firmware file
> just as it used to. I applied the patch above, rebooted and everything
> has loaded fine and is working.

Is no one else seeing this problem?

I just upgraded to 0.7.0. The problem still occurred with unpatched 0.7.0;
applying my patch to 0.7.0 still fixed the problem. Can some one commit this
please (or suggest an alternative fix)?

Jul  4 12:33:24 vortex kernel: ivtv0: loaded v4l-cx2341x-dec.fw firmware
(262144 bytes)
Jul  4 12:33:24 vortex firmware_helper[7415]: Loading of
/lib/firmware/v4l-cx2341x-dec.fw for ivtv driver failed: No such file or
directory <- the file is there and md5sums match.
Jul  4 12:39:53 vortex syslogd 1.4.1: restart. <- Manual reset after hard
lockup

Regards,
Rob

-- 
---------------------"Happiness is understanding."----------------------
Robert Hardy, B.Eng Computer Systems                  C.E.O. Webcon Inc.
rhardy <at> webcon <dot> ca    GPG Key available          (613) 276-7327

_______________________________________________
ivtv-devel mailing list
[email protected]
http://ivtvdriver.org/mailman/listinfo/ivtv-devel



_______________________________________________
ivtv-devel mailing list
[email protected]
http://ivtvdriver.org/mailman/listinfo/ivtv-devel

Reply via email to