All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans Verkuil <hverkuil-cisco@xs4all.nl>
To: linux-media@vger.kernel.org
Subject: [PATCH 0/7] CEC framework fixes
Date: Tue, 10 May 2022 17:00:15 +0200	[thread overview]
Message-ID: <20220510150022.1787112-1-hverkuil-cisco@xs4all.nl> (raw)

When testing what happens when randomly configuring and unconfiguring
a CEC adapter, and simultaneously setting an invalidating the physical
address, and starting and stopping monitoring and pin monitoring using
two cec-gpio instances, a whole bunch of race conditions were found.

This patch series addresses them.

Hans Verkuil (7):
  cec-pin: disabling the adapter cannot call kthread_stop
  cec-pin: don't zero work_pin_num_events in adap_enable
  cec-adap.c: don't unconfigure if already unconfigured
  cec-adap.c: stop trying LAs on CEC_TX_STATUS_TIMEOUT
  cec-adap.c: fix is_configuring state
  cec-adap.c: reconfigure if the PA changes during configuration
  cec-adap: drop activate_cnt, use state info instead

 drivers/media/cec/core/cec-adap.c | 188 ++++++++++++++----------------
 drivers/media/cec/core/cec-pin.c  |  57 +++++----
 include/media/cec.h               |   6 +-
 3 files changed, 124 insertions(+), 127 deletions(-)

-- 
2.34.1


             reply	other threads:[~2022-05-10 15:18 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-10 15:00 Hans Verkuil [this message]
2022-05-10 15:00 ` [PATCH 1/7] cec-pin: disabling the adapter cannot call kthread_stop Hans Verkuil
2022-05-10 15:00 ` [PATCH 2/7] cec-pin: don't zero work_pin_num_events in adap_enable Hans Verkuil
2022-05-10 15:00 ` [PATCH 3/7] cec-adap.c: don't unconfigure if already unconfigured Hans Verkuil
2022-05-10 15:00 ` [PATCH 4/7] cec-adap.c: stop trying LAs on CEC_TX_STATUS_TIMEOUT Hans Verkuil
2022-05-10 15:00 ` [PATCH 5/7] cec-adap.c: fix is_configuring state Hans Verkuil
2022-05-10 15:00 ` [PATCH 6/7] cec-adap.c: reconfigure if the PA changes during configuration Hans Verkuil
2022-05-10 15:00 ` [PATCH 7/7] cec-adap: drop activate_cnt, use state info instead Hans Verkuil

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220510150022.1787112-1-hverkuil-cisco@xs4all.nl \
    --to=hverkuil-cisco@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.