Ewout van Bekkum | f7e38b3 | 2020-11-16 11:56:41 -0800 | [diff] [blame] | 1 | // Copyright 2020 The Pigweed Authors |
| 2 | // |
| 3 | // Licensed under the Apache License, Version 2.0 (the "License"); you may not |
| 4 | // use this file except in compliance with the License. You may obtain a copy of |
| 5 | // the License at |
| 6 | // |
| 7 | // https://www.apache.org/licenses/LICENSE-2.0 |
| 8 | // |
| 9 | // Unless required by applicable law or agreed to in writing, software |
| 10 | // distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 11 | // WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 12 | // License for the specific language governing permissions and limitations under |
| 13 | // the License. |
| 14 | |
| 15 | #include "pw_sync/counting_semaphore.h" |
| 16 | |
| 17 | #include <algorithm> |
| 18 | |
Wyatt Hepler | f298de4 | 2021-03-19 15:06:36 -0700 | [diff] [blame] | 19 | #include "pw_assert/check.h" |
Ewout van Bekkum | f7e38b3 | 2020-11-16 11:56:41 -0800 | [diff] [blame] | 20 | #include "pw_chrono/system_clock.h" |
| 21 | #include "pw_chrono_threadx/system_clock_constants.h" |
| 22 | #include "pw_interrupt/context.h" |
| 23 | #include "tx_api.h" |
| 24 | |
| 25 | using pw::chrono::SystemClock; |
Ewout van Bekkum | f7e38b3 | 2020-11-16 11:56:41 -0800 | [diff] [blame] | 26 | |
| 27 | namespace pw::sync { |
| 28 | |
Ewout van Bekkum | 5ff8cc5 | 2021-09-07 15:46:36 -0700 | [diff] [blame] | 29 | bool CountingSemaphore::try_acquire_for(SystemClock::duration timeout) { |
Ewout van Bekkum | f7e38b3 | 2020-11-16 11:56:41 -0800 | [diff] [blame] | 30 | // Enforce the pw::sync::CountingSemaphore IRQ contract. |
| 31 | PW_DCHECK(!interrupt::InInterruptContext()); |
| 32 | |
Ewout van Bekkum | 1dea2c8 | 2021-03-10 12:09:09 -0800 | [diff] [blame] | 33 | // Use non-blocking try_acquire for negative and zero length durations. |
Ewout van Bekkum | 5ff8cc5 | 2021-09-07 15:46:36 -0700 | [diff] [blame] | 34 | if (timeout <= SystemClock::duration::zero()) { |
Ewout van Bekkum | 1dea2c8 | 2021-03-10 12:09:09 -0800 | [diff] [blame] | 35 | return try_acquire(); |
| 36 | } |
Ewout van Bekkum | f7e38b3 | 2020-11-16 11:56:41 -0800 | [diff] [blame] | 37 | |
Ewout van Bekkum | a6642d7 | 2021-11-05 09:43:14 -0700 | [diff] [blame] | 38 | // In case the timeout is too long for us to express through the native |
| 39 | // ThreadX API, we repeatedly wait with shorter durations. Note that on a tick |
| 40 | // based kernel we cannot tell how far along we are on the current tick, ergo |
| 41 | // we add one whole tick to the final duration. However, this also means that |
| 42 | // the loop must ensure that timeout + 1 is less than the max timeout. |
Ewout van Bekkum | 1dea2c8 | 2021-03-10 12:09:09 -0800 | [diff] [blame] | 43 | constexpr SystemClock::duration kMaxTimeoutMinusOne = |
| 44 | pw::chrono::threadx::kMaxTimeout - SystemClock::duration(1); |
Ewout van Bekkum | 5ff8cc5 | 2021-09-07 15:46:36 -0700 | [diff] [blame] | 45 | while (timeout > kMaxTimeoutMinusOne) { |
Ewout van Bekkum | 1dea2c8 | 2021-03-10 12:09:09 -0800 | [diff] [blame] | 46 | const UINT result = tx_semaphore_get( |
| 47 | &native_type_, static_cast<ULONG>(kMaxTimeoutMinusOne.count())); |
Ewout van Bekkum | f7e38b3 | 2020-11-16 11:56:41 -0800 | [diff] [blame] | 48 | if (result != TX_NO_INSTANCE) { |
| 49 | // If we didn't time out (TX_NO_INSTANCE), then we should have succeeded. |
| 50 | PW_CHECK_UINT_EQ(TX_SUCCESS, result); |
| 51 | return true; |
| 52 | } |
Ewout van Bekkum | 5ff8cc5 | 2021-09-07 15:46:36 -0700 | [diff] [blame] | 53 | timeout -= kMaxTimeoutMinusOne; |
Ewout van Bekkum | f7e38b3 | 2020-11-16 11:56:41 -0800 | [diff] [blame] | 54 | } |
Ewout van Bekkum | a6642d7 | 2021-11-05 09:43:14 -0700 | [diff] [blame] | 55 | // On a tick based kernel we cannot tell how far along we are on the current |
| 56 | // tick, ergo we add one whole tick to the final duration. |
Ewout van Bekkum | 5ff8cc5 | 2021-09-07 15:46:36 -0700 | [diff] [blame] | 57 | const UINT result = |
| 58 | tx_semaphore_get(&native_type_, static_cast<ULONG>(timeout.count() + 1)); |
Ewout van Bekkum | f7e38b3 | 2020-11-16 11:56:41 -0800 | [diff] [blame] | 59 | if (result == TX_NO_INSTANCE) { |
| 60 | return false; // We timed out, there's still no token. |
| 61 | } |
| 62 | PW_CHECK_UINT_EQ(TX_SUCCESS, result); |
| 63 | return true; |
| 64 | } |
| 65 | |
| 66 | } // namespace pw::sync |