Skip to content

eventfd: allow for all CPUs#16277

Closed
vszakats wants to merge 3 commits intocurl:masterfrom
vszakats:eventfd32
Closed

eventfd: allow for all CPUs#16277
vszakats wants to merge 3 commits intocurl:masterfrom
vszakats:eventfd32

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Feb 9, 2025

After fixing support for x32, unlock eventfd support for all CPUs.
Before this patch, it was explicitly limited to 64-bit ones.

You can disable eventfs manually on systems where it's auto-detected:

  • cmake: -DHAVE_EVENTFD=0
  • configure: export ac_cv_func_eventfd=0

Ref: c2aa504 #16239

@vszakats vszakats changed the title eventfd: enable for all CPUs eventfd: allow for all CPUs Feb 9, 2025
@dfandrich
Copy link
Copy Markdown
Contributor

dfandrich commented Feb 9, 2025 via email

@vszakats vszakats added the feature-window A merge of this requires an open feature window label Feb 9, 2025
@vszakats
Copy link
Copy Markdown
Member Author

vszakats commented Feb 9, 2025

Absolutely (#16239 (comment)).
Added the label.

@vszakats vszakats closed this in 4d01de3 Feb 21, 2025
@vszakats vszakats deleted the eventfd32 branch February 21, 2025 11:09
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
After fixing support for x32, unlock eventfd support for all CPUs.
Before this patch, it was explicitly limited to 64-bit ones.

You can disable eventfs manually on systems where it's auto-detected:
- cmake: `-DHAVE_EVENTFD=0`
- configure: `export ac_cv_func_eventfd=0`

Ref: c2aa504 curl#16239
Closes curl#16277
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature-window A merge of this requires an open feature window

Development

Successfully merging this pull request may close these issues.

3 participants