Skip to content

bpo-27585: Cancelled lock waiter wakes up the next one#1031

Merged
1st1 merged 1 commit into
python:masterfrom
msornay:fix-issue-27585
Jun 9, 2017
Merged

bpo-27585: Cancelled lock waiter wakes up the next one#1031
1st1 merged 1 commit into
python:masterfrom
msornay:fix-issue-27585

Conversation

@msornay
Copy link
Copy Markdown
Contributor

@msornay msornay commented Apr 7, 2017

Avoid a deadlock when the waiter who is about to take the lock is cancelled

Issue #27585: http://bugs.python.org/issue27585
Original PR on asyncio repository: python/asyncio#467

Cancelled lock waiter wakes up the next one
be53eaa 
Avoid a deadlock when the waiter who is about to take the lock is
cancelled

Issue python#27585
@mention-bot
Copy link
Copy Markdown

mention-bot commented Apr 7, 2017

@msornay, thanks for your PR! By analyzing the history of the files in this pull request, we identified @1st1, @asvetlov and @serhiy-storchaka to be potential reviewers.

@popravich popravich mentioned this pull request May 18, 2017
Closed
@popravich
Copy link
Copy Markdown

popravich commented May 18, 2017

Guys! @1st1, @asvetlov, @serhiy-storchaka,
Can anyone take a look at this patch, the issue is real PITA.

@msornay, probably, the same fix should be applied to asyncio.Condition.wait() method
as it uses the same logic as asyncio.Lock.acquire():
if await cond.wait() is resolved but owning task is cancelled -- other waiters could end up in a deadlock

@1st1 1st1 self-assigned this May 18, 2017
@asvetlov
Copy link
Copy Markdown
Contributor

asvetlov commented May 19, 2017

lgtm

@jettify jettify mentioned this pull request May 19, 2017
Closed
@popravich
Copy link
Copy Markdown

popravich commented May 26, 2017

Any news?

@popravich popravich mentioned this pull request May 26, 2017
Closed
@jettify
Copy link
Copy Markdown

jettify commented Jun 5, 2017

ping

@1st1
Copy link
Copy Markdown
Member

1st1 commented Jun 5, 2017

I'll take a look tomorrow/the day after.

@pfreixes pfreixes mentioned this pull request Jun 7, 2017
Merged
@1st1 1st1 merged commit 894a654 into python:master Jun 9, 2017
1st1 added a commit that referenced this pull request Jun 9, 2017
@1st1
bpo-27585: Add a NEWS entry for #1031
624f948
1st1 added a commit that referenced this pull request Jun 9, 2017
@1st1
bpo-27585: Add a NEWS entry for #1031 (#2036)
7be651c
1st1 pushed a commit that referenced this pull request Jun 9, 2017
@msornay @1st1
Fix waiter cancellation in asyncio.Lock (#1031)
b8023a3 
Avoid a deadlock when the waiter who is about to take the lock is
cancelled

Issue #27585
1st1 pushed a commit that referenced this pull request Jun 9, 2017
@msornay @1st1
Fix waiter cancellation in asyncio.Lock (#1031)
7e1cc8c 
Avoid a deadlock when the waiter who is about to take the lock is
cancelled

Issue #27585
1st1 added a commit that referenced this pull request Jun 9, 2017
@1st1
Fix waiter cancellation in asyncio.Lock (#1031) (#2037)
fa7f519 
Avoid a deadlock when the waiter who is about to take the lock is
cancelled

Issue #27585
1st1 added a commit that referenced this pull request Jun 9, 2017
@1st1
Fix waiter cancellation in asyncio.Lock (#1031) (#2038)
d913d1c 
Avoid a deadlock when the waiter who is about to take the lock is
cancelled

Issue #27585
@Mariatta
Copy link
Copy Markdown
Member

Mariatta commented Sep 7, 2017

Does this still need backport? If not please remove the label.

@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Sep 7, 2017

🐍🍒⛏🤖 Thanks @msornay for the PR, and @1st1 for merging it 🌮🎉.I'm working now to backport this PR to: 3.6.

@miss-islington
Copy link
Copy Markdown
Contributor

miss-islington commented Sep 7, 2017

Sorry, @msornay and @1st1, I could not cleanly backport this to 3.6 due to a conflict.
Please backport using cherry_picker on command line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@1st1 1st1 1st1 approved these changes

Assignees

@1st1 1st1

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

10 participants

@msornay @mention-bot @popravich @asvetlov @jettify @1st1 @Mariatta @miss-islington @serhiy-storchaka @the-knights-who-say-ni