Skip to content

gh-76785: Add SendChannel.send_buffer() #110246

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.

Sign up for GitHub

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

Merged
ericsnowcurrently merged 20 commits into from
Oct 9, 2023
Merged

gh-76785: Add SendChannel.send_buffer() #110246

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
ab2aa3b
Export _PyInterpreterState_LookUpID().
ericsnowcurrently Sep 9, 2023
4d103dd
Optionally free the pending call's data.
ericsnowcurrently Sep 11, 2023
faf7778
Add _Py_CallInInterpreter() and _Py_CallInInterpreterAndRawFree().
ericsnowcurrently Sep 11, 2023
5c5662d
Add PyUnstable_Buffer_ReleaseInInterpreter() and PyUnstable_Buffer_Re…
ericsnowcurrently Sep 11, 2023
573abef
Add CrossInterpreterBufferView.
ericsnowcurrently Sep 11, 2023
be7e927
Add _get_current_xibufferview_type().
ericsnowcurrently Sep 11, 2023
955989d
Add xibufferview_from_xid().
ericsnowcurrently Sep 11, 2023
05d13f5
Register memoryview for cross-interpreter use.
ericsnowcurrently Sep 11, 2023
3e4e977
Add _channels.send_buffer().
ericsnowcurrently Sep 11, 2023
1699495
Use _get_current_module_state().
ericsnowcurrently Sep 18, 2023
0c216ff
Add SendChannel.send_buffer().
ericsnowcurrently Sep 19, 2023
0ad6f41
Keep some C-API functions private for now.
ericsnowcurrently Oct 2, 2023
56e3774
Add a test.
ericsnowcurrently Oct 2, 2023
272488a
Fix SendChannel.send_buffer_nowait().
ericsnowcurrently Oct 2, 2023
ceb3dae
Fix a typo.
ericsnowcurrently Oct 2, 2023
873eec2
Add tests.
ericsnowcurrently Oct 2, 2023
da9779f
Fix SendChannel.send_buffer().
ericsnowcurrently Oct 2, 2023
dea173a
Clear the XID classes during module fini.
ericsnowcurrently Oct 3, 2023
0271b4a
Merge branch 'main' into channel-send-buffer
ericsnowcurrently Oct 4, 2023
fc15f77
Fix ref leaks.
ericsnowcurrently Oct 9, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Fix ref leaks.
  • Loading branch information
@ericsnowcurrently
ericsnowcurrently committed Oct 9, 2023
commit fc15f77681f819b5c195673f9ced76fa8fae7fc9
22 changes: 14 additions & 8 deletions Modules/_xxinterpchannelsmodule.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ register_xid_class(PyTypeObject *cls, crossinterpdatafunc shared,
int res = _PyCrossInterpreterData_RegisterClass(cls, shared);
if (res == 0) {
assert(classes->count < MAX_XID_CLASSES);
Py_INCREF(cls);
// The class has refs elsewhere, so we need to incref here.
classes->added[classes->count].cls = cls;
classes->count += 1;
}
Expand All @@ -111,7 +111,6 @@ clear_xid_class_registry(struct xid_class_registry *classes)
classes->count -= 1;
PyTypeObject *cls = classes->added[classes->count].cls;
_PyCrossInterpreterData_UnregisterClass(cls);
Py_DECREF(cls);
}
}

Expand Down Expand Up @@ -2770,17 +2769,18 @@ module_exec(PyObject *mod)
}
struct xid_class_registry *xid_classes = NULL;

module_state *state = get_module_state(mod);
if (state == NULL) {
goto error;
}
xid_classes = &state->xid_classes;

/* Add exception types */
if (exceptions_init(mod) != 0) {
goto error;
}

/* Add other types */
module_state *state = get_module_state(mod);
if (state == NULL) {
goto error;
}
xid_classes = &state->xid_classes;

// ChannelID
state->ChannelIDType = add_new_type(
Expand All @@ -2799,7 +2799,7 @@ module_exec(PyObject *mod)
goto error;
}

// Make sure chnnels drop objects owned by this interpreter
/* Make sure chnnels drop objects owned by this interpreter. */
PyInterpreterState *interp = _get_current_interp();
PyUnstable_AtExit(interp, clear_interpreter, (void *)interp);

Expand Down Expand Up @@ -2847,7 +2847,13 @@ module_free(void *mod)
{
module_state *state = get_module_state(mod);
assert(state != NULL);

// Before clearing anything, we unregister the various XID types. */
clear_xid_class_registry(&state->xid_classes);

// Now we clear the module state.
clear_module_state(state);

_globals_fini();
}

Expand Down