Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Clear the busy flag if GetEvergreenInfo() fails #4461

Open
wants to merge 1 commit into
base: 25.lts.1+
Choose a base branch
from

Conversation

hlwarriner
Copy link
Contributor

With a crosstool evergreen build, GetEvergreenInfo() is called before SetEvergreenInfo() because of the early call to SbLogRawDumpStack(3) to work around b/151118284. GetEvergreenInfo() doesn't currently release the g_busy flag after it finds invalid g_evergreen_info, and this prevents SetEvergreenInfo() from acquiring it later.

b/369808307

Change-Id: Ibcfe894cf3c29be90d731a0fb934690df5b0e89a

With a crosstool evergreen build, GetEvergreenInfo() is called before
SetEvergreenInfo() because of the early call to SbLogRawDumpStack(3) to
work around b/151118284. GetEvergreenInfo() doesn't currently release
the g_busy flag after it finds invalid g_evergreen_info, and this
prevents SetEvergreenInfo() from acquiring it later.

b/369808307

Change-Id: Ibcfe894cf3c29be90d731a0fb934690df5b0e89a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants