diff options
author | Devin Heitmueller <dheitmueller@kernellabs.com> | 2009-05-20 22:58:16 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-06-16 18:14:15 -0400 |
commit | a5beb7b32364a236e833631c3ee5c910cbbfa5bb (patch) | |
tree | 1c5bdfdfe8ddbd05443e0625cbcfe974da17fe1f | |
parent | 9c9c68a8c06623b1f1b68652b1a3bf6d8ef415df (diff) |
V4L/DVB (11875): dvb_frontend: fix case where fepriv->exit not reset
The fact that we now explicitly set fepriv->exit = 1 when the thread is
shutting down exposed an edge case where it was not being reset back to zero
once the thread went away in some cases. This resulted in failures in cases
where the frontend was closed, and then opened O_RDONLY, since in that case
the thread is not being restarted but it was checking the fepriv->exit flag.
Thanks to Thierry Lelegard, who and encountered and debugged a large portion
of the issue in the same twelve hours that I did (as well as testing my patch).
Cc: Thierry Lelegard <thierry.lelegard@tv-numeric.com>
Signed-off-by: Devin Heitmueller <dheitmueller@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
-rw-r--r-- | drivers/media/dvb/dvb-core/dvb_frontend.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/media/dvb/dvb-core/dvb_frontend.c b/drivers/media/dvb/dvb-core/dvb_frontend.c index 1f38dd86f542..f50ca7292a7d 100644 --- a/drivers/media/dvb/dvb-core/dvb_frontend.c +++ b/drivers/media/dvb/dvb-core/dvb_frontend.c | |||
@@ -657,6 +657,7 @@ restart: | |||
657 | } | 657 | } |
658 | 658 | ||
659 | fepriv->thread = NULL; | 659 | fepriv->thread = NULL; |
660 | fepriv->exit = 0; | ||
660 | mb(); | 661 | mb(); |
661 | 662 | ||
662 | dvb_frontend_wakeup(fe); | 663 | dvb_frontend_wakeup(fe); |