[TS-3315] Assert after try lock

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Fix Version/s: sometime
• Component/s: Cache
• Labels: newbie
• Assignee: Unassigned
• Reporter: Phil Sorber
• Created: 22/Jan/15 16:49
• Updated: 12/Jun/15 21:26

Description

In iocore/cache/Cache.cc there is the following:

  CACHE_TRY_LOCK(lock, cont->mutex, this_ethread());
  ink_assert(lock.is_locked());

Does it really make sense to try and assert when a try can fail?

Issue Links

• is related to: TS-3156 Mutex[Try]Lock bool() operator change and unused API removal