forked from luck/tmp_suning_uos_patched
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>
This commit is contained in:
parent
9c9c68a8c0
commit
a5beb7b323
@ -657,6 +657,7 @@ static int dvb_frontend_thread(void *data)
|
|||||||
}
|
}
|
||||||
|
|
||||||
fepriv->thread = NULL;
|
fepriv->thread = NULL;
|
||||||
|
fepriv->exit = 0;
|
||||||
mb();
|
mb();
|
||||||
|
|
||||||
dvb_frontend_wakeup(fe);
|
dvb_frontend_wakeup(fe);
|
||||||
|
Loading…
Reference in New Issue
Block a user