Skip to content

Conversation

@pguyot
Copy link
Collaborator

@pguyot pguyot commented Jun 4, 2023

This is based on a specific select thread loop using esp-idf's poll(2).

Qemu-based test uses a custom simple vfs pipe implementation to demonstrate usage.

These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).

SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

@pguyot pguyot force-pushed the w22/add-select-support-to-esp32 branch 5 times, most recently from 3e7d5c8 to 549b5c5 Compare June 6, 2023 19:40
@pguyot pguyot force-pushed the w22/add-select-support-to-esp32 branch from 549b5c5 to d9335ef Compare June 11, 2023 16:57
@pguyot pguyot force-pushed the w22/add-select-support-to-esp32 branch from d9335ef to 0e98675 Compare July 16, 2023 22:42
@bettio bettio added this to the v0.6 milestone Jul 28, 2023
@pguyot pguyot force-pushed the w22/add-select-support-to-esp32 branch from 0e98675 to 10f5b9e Compare July 30, 2023 19:45
@pguyot pguyot marked this pull request as ready for review July 30, 2023 19:47
This is based on a specific select thread loop using esp-idf's `poll(2)`.

Qemu-based test uses a custom simple vfs pipe implementation to
demonstrate usage.

Signed-off-by: Paul Guyot <pguyot@kallisys.net>
@pguyot pguyot force-pushed the w22/add-select-support-to-esp32 branch from 10f5b9e to c205c23 Compare July 31, 2023 06:48
Copy link
Collaborator

@fadushin fadushin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with atomvm_lib:httpd

Copy link
Collaborator

@fadushin fadushin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on esp32 using httpd from atomvm_lib

@fadushin fadushin merged commit 6d588de into atomvm:master Aug 6, 2023
@pguyot pguyot deleted the w22/add-select-support-to-esp32 branch August 7, 2023 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants