From 706a44abb3aa7b4535ded7d5c934147b7de06ed1 Mon Sep 17 00:00:00 2001 From: Tomi Valkeinen Date: Fri, 24 Nov 2017 09:50:42 +0200 Subject: Update to latest pybind11 Update to latest pybind11 HEAD. We can't use the latest tag (v2.2.0) as it has a regression. There were two problems when updating: 1) Difficulty in managing DrmObject derived classes Most of the DrmObjects are owned by Card, and can't be allocated or freed, but a few of them are allocated and freed by the user. For the former, we need to use unique_ptr with py::nodelete, but that prevents the latter from working. The solution was to not tell the python that the latter classes derive from DrmObject. This seems to be missing feature in pybind11, but I think we can live with it. 2) DrmObjects in STL containers vector where T is a DrmObject derived class doesn't work. We need to have a manual wrapper to return vector> instead. This also seems to be a pybind11 missing feature. Signed-off-by: Tomi Valkeinen --- ext/pybind11 | 2 +- py/pykms/pykms.cpp | 5 +--- py/pykms/pykmsbase.cpp | 63 +++++++++++++++++++++++++++++++++++--------------- py/pykms/pykmsomap.cpp | 4 ++-- py/pykms/pyvid.cpp | 4 ++-- 5 files changed, 51 insertions(+), 27 deletions(-) diff --git a/ext/pybind11 b/ext/pybind11 index e70b2ab..086d53e 160000 --- a/ext/pybind11 +++ b/ext/pybind11 @@ -1 +1 @@ -Subproject commit e70b2abb6dee27a2889b01f245a2a28e6fcd4b01 +Subproject commit 086d53e8c66a84d0ec723d5435918c76edd878e8 diff --git a/py/pykms/pykms.cpp b/py/pykms/pykms.cpp index 1e54c9b..fec8417 100644 --- a/py/pykms/pykms.cpp +++ b/py/pykms/pykms.cpp @@ -15,9 +15,7 @@ void init_pyvid(py::module &m); void init_pykmsomap(py::module &m); #endif -PYBIND11_PLUGIN(pykms) { - py::module m("pykms", "kms bindings"); - +PYBIND11_MODULE(pykms, m) { init_pykmsbase(m); init_pykmstest(m); @@ -27,5 +25,4 @@ PYBIND11_PLUGIN(pykms) { #if HAS_LIBDRM_OMAP init_pykmsomap(m); #endif - return m.ptr(); } diff --git a/py/pykms/pykmsbase.cpp b/py/pykms/pykmsbase.cpp index fe4bc46..aae5ece 100644 --- a/py/pykms/pykmsbase.cpp +++ b/py/pykms/pykmsbase.cpp @@ -12,22 +12,36 @@ void init_pykmsbase(py::module &m) py::class_(m, "Card") .def(py::init<>()) .def_property_readonly("fd", &Card::fd) - .def("get_first_connected_connector", &Card::get_first_connected_connector) - .def_property_readonly("connectors", &Card::get_connectors) - .def_property_readonly("crtcs", &Card::get_crtcs) - .def_property_readonly("encoders", &Card::get_encoders) - .def_property_readonly("planes", &Card::get_planes) + .def_property_readonly("get_first_connected_connector", &Card::get_first_connected_connector) + + // XXX pybind11 can't handle vector where T is non-copyable, and complains: + // RuntimeError: return_value_policy = move, but the object is neither movable nor copyable! + // So we do this manually. + .def_property_readonly("connectors", [](Card* self) { + vector> v; + for (Connector* p : self->get_connectors()) + v.push_back(unique_ptr(p)); + return v; + }) + + .def_property_readonly("planes", [](Card* self) { + vector> v; + for (Plane* p : self->get_planes()) + v.push_back(unique_ptr(p)); + return v; + }) + .def_property_readonly("has_atomic", &Card::has_atomic) .def("get_prop", (Property* (Card::*)(uint32_t) const)&Card::get_prop) ; - py::class_(m, "DrmObject") + py::class_>(m, "DrmObject") .def_property_readonly("id", &DrmObject::id) .def_property_readonly("idx", &DrmObject::idx) .def_property_readonly("card", &DrmObject::card) ; - py::class_(m, "DrmPropObject", py::base()) + py::class_>(m, "DrmPropObject") .def("refresh_props", &DrmPropObject::refresh_props) .def_property_readonly("prop_map", &DrmPropObject::get_prop_map) .def("get_prop_value", (uint64_t (DrmPropObject::*)(const string&) const)&DrmPropObject::get_prop_value) @@ -36,7 +50,7 @@ void init_pykmsbase(py::module &m) .def("get_prop", &DrmPropObject::get_prop) ; - py::class_(m, "Connector", py::base()) + py::class_>(m, "Connector") .def_property_readonly("fullname", &Connector::fullname) .def("get_default_mode", &Connector::get_default_mode) .def("get_current_crtc", &Connector::get_current_crtc) @@ -49,7 +63,7 @@ void init_pykmsbase(py::module &m) .def("refresh", &Connector::refresh) ; - py::class_(m, "Crtc", py::base()) + py::class_>(m, "Crtc") .def("set_mode", (int (Crtc::*)(Connector*, const Videomode&))&Crtc::set_mode) .def("set_mode", (int (Crtc::*)(Connector*, Framebuffer&, const Videomode&))&Crtc::set_mode) .def("disable_mode", &Crtc::disable_mode) @@ -67,11 +81,11 @@ void init_pykmsbase(py::module &m) .def("refresh", &Crtc::refresh) ; - py::class_(m, "Encoder", py::base()) + py::class_>(m, "Encoder") .def("refresh", &Encoder::refresh) ; - py::class_(m, "Plane", py::base()) + py::class_>(m, "Plane") .def("supports_crtc", &Plane::supports_crtc) .def_property_readonly("formats", &Plane::get_formats) .def_property_readonly("plane_type", &Plane::plane_type) @@ -84,12 +98,12 @@ void init_pykmsbase(py::module &m) .value("Cursor", PlaneType::Cursor) ; - py::class_(m, "Property", py::base()) + py::class_>(m, "Property") .def_property_readonly("name", &Property::name) .def_property_readonly("enums", &Property::get_enums) ; - py::class_(m, "Blob", py::base()) + py::class_(m, "Blob") .def("__init__", [](Blob& instance, Card& card, py::buffer buf) { py::buffer_info info = buf.request(); if (info.ndim != 1) @@ -98,17 +112,30 @@ void init_pykmsbase(py::module &m) new (&instance) Blob(card, info.ptr, info.size * info.itemsize); }) .def_property_readonly("data", &Blob::data) - ; - py::class_(m, "Framebuffer", py::base()) + // XXX pybind11 doesn't support a base object (DrmObject) with custom holder-type, + // and a subclass with standard holder-type. + // So we just copy the DrmObject members here. + // Note that this means that python thinks we don't derive from DrmObject + .def_property_readonly("id", &DrmObject::id) + .def_property_readonly("idx", &DrmObject::idx) + .def_property_readonly("card", &DrmObject::card) ; - py::class_(m, "Framebuffer", py::base()) + py::class_(m, "Framebuffer") .def_property_readonly("width", &Framebuffer::width) .def_property_readonly("height", &Framebuffer::height) + + // XXX pybind11 doesn't support a base object (DrmObject) with custom holder-type, + // and a subclass with standard holder-type. + // So we just copy the DrmObject members here. + // Note that this means that python thinks we don't derive from DrmObject + .def_property_readonly("id", &DrmObject::id) + .def_property_readonly("idx", &DrmObject::idx) + .def_property_readonly("card", &DrmObject::card) ; - py::class_(m, "DumbFramebuffer", py::base()) + py::class_(m, "DumbFramebuffer") .def(py::init(), py::keep_alive<1, 2>()) // Keep Card alive until this is destructed .def(py::init(), @@ -120,7 +147,7 @@ void init_pykmsbase(py::module &m) .def("offset", &DumbFramebuffer::offset) ; - py::class_(m, "ExtFramebuffer", py::base()) + py::class_(m, "ExtFramebuffer") .def(py::init, vector, vector>(), py::keep_alive<1, 2>()) // Keep Card alive until this is destructed ; diff --git a/py/pykms/pykmsomap.cpp b/py/pykms/pykmsomap.cpp index 2662a18..4fc7084 100644 --- a/py/pykms/pykmsomap.cpp +++ b/py/pykms/pykmsomap.cpp @@ -10,11 +10,11 @@ using namespace std; void init_pykmsomap(py::module &m) { - py::class_(m, "OmapCard", py::base()) + py::class_(m, "OmapCard") .def(py::init<>()) ; - py::class_ omapfb(m, "OmapFramebuffer", py::base()); + py::class_ omapfb(m, "OmapFramebuffer"); // XXX we should use py::arithmetic() here to support or and and operators, but it's not supported in the pybind11 we use py::enum_(omapfb, "Flags") diff --git a/py/pykms/pyvid.cpp b/py/pykms/pyvid.cpp index 01177d5..6a6080e 100644 --- a/py/pykms/pyvid.cpp +++ b/py/pykms/pyvid.cpp @@ -11,7 +11,7 @@ using namespace std; void init_pyvid(py::module &m) { - py::class_(m, "VideoDevice") + py::class_(m, "VideoDevice") .def(py::init()) .def_property_readonly("fd", &VideoDevice::fd) .def_property_readonly("has_capture", &VideoDevice::has_capture) @@ -24,7 +24,7 @@ void init_pyvid(py::module &m) .def("get_capture_devices", &VideoDevice::get_capture_devices) ; - py::class_(m, "VideoStreamer") + py::class_(m, "VideoStreamer") .def_property_readonly("fd", &VideoStreamer::fd) .def_property_readonly("ports", &VideoStreamer::get_ports) .def("set_port", &VideoStreamer::set_port) -- cgit v1.2.3