| Message ID | 20220324221818.81338-1-luiz.dentz@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 62e591578e3f948e187aacf44ede4286fad37ad7 |
| Headers | show |
| Series | [BlueZ] a2dp: Always invalidate the cache if its configuration fails | expand |
| Context | Check | Description |
|---|---|---|
| tedd_an/pre-ci_am | success | Success |
| tedd_an/checkpatch | success | Checkpatch PASS |
| tedd_an/gitlint | success | Gitlint PASS |
| tedd_an/setupell | success | Setup ELL PASS |
| tedd_an/buildprep | success | Build Prep PASS |
| tedd_an/build | success | Build Configuration PASS |
| tedd_an/makecheck | success | Make Check PASS |
| tedd_an/makecheckvalgrind | success | Make Check PASS |
| tedd_an/makedistcheck | success | Make Distcheck PASS |
| tedd_an/build_extell | success | Build External ELL PASS |
| tedd_an/build_extell_make | success | Build Make with External ELL PASS |
This is automated email and please do not reply to this email! Dear submitter, Thank you for submitting the patches to the linux bluetooth mailing list. This is a CI test results with your patch series: PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=626173 ---Test result--- Test Summary: CheckPatch PASS 0.71 seconds GitLint PASS 0.51 seconds Prep - Setup ELL PASS 54.69 seconds Build - Prep PASS 0.72 seconds Build - Configure PASS 10.80 seconds Build - Make PASS 1880.84 seconds Make Check PASS 12.88 seconds Make Check w/Valgrind PASS 571.74 seconds Make Distcheck PASS 301.33 seconds Build w/ext ELL - Configure PASS 10.99 seconds Build w/ext ELL - Make PASS 1833.37 seconds Incremental Build with patchesPASS 0.00 seconds --- Regards, Linux Bluetooth
Hello: This patch was applied to bluetooth/bluez.git (master) by Luiz Augusto von Dentz <luiz.von.dentz@intel.com>: On Thu, 24 Mar 2022 15:18:18 -0700 you wrote: > From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> > > When a configuration had been restored from last_used cache but it > fails invalidate it so another sep can be used. > > Fixes: https://github.com/bluez/bluez/issues/313 > > [...] Here is the summary with links: - [BlueZ] a2dp: Always invalidate the cache if its configuration fails https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=62e591578e3f You are awesome, thank you!
diff --git a/profiles/audio/a2dp.c b/profiles/audio/a2dp.c index d66c22b2b..c3ac432a7 100644 --- a/profiles/audio/a2dp.c +++ b/profiles/audio/a2dp.c @@ -872,12 +872,10 @@ static void store_remote_seps(struct a2dp_channel *chan) static void invalidate_remote_cache(struct a2dp_setup *setup, struct avdtp_error *err) { - if (err->category == AVDTP_ERRNO || - err->err.error_code != AVDTP_UNSUPPORTED_CONFIGURATION) + if (err->category == AVDTP_ERRNO) return; - /* Attempt to unregister Remote SEP if configuration - * fails with Unsupported Configuration and it was + /* Attempt to unregister Remote SEP if configuration fails and it was * loaded from cache. */ if (setup->rsep && setup->rsep->from_cache) {
From: Luiz Augusto von Dentz <luiz.von.dentz@intel.com> When a configuration had been restored from last_used cache but it fails invalidate it so another sep can be used. Fixes: https://github.com/bluez/bluez/issues/313 --- profiles/audio/a2dp.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-)