author | Tero Marttila <terom@fixme.fi> |
Sat, 26 Sep 2009 23:51:31 +0300 | |
changeset 57 | 8c4032265c8c |
parent 52 | 722fc70a197a |
permissions | -rw-r--r-- |
39 | 1 |
from qmsk.net.lib.event2.event cimport * |
2 |
from qmsk.net.lib.event2.base cimport event_base |
|
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
3 |
cimport qmsk.net.py as py |
39 | 4 |
|
5 |
cdef lib.timeval* build_timeout (lib.timeval *tv, object timeout = None) except? <lib.timeval *>-1 : |
|
6 |
""" |
|
7 |
If a timeout is given, it is treated as a float and stored in *tv, which is returned. Otherwise, simply returns |
|
8 |
NULL. |
|
9 |
""" |
|
10 |
||
11 |
cdef double t |
|
12 |
||
41 | 13 |
if timeout is not None : |
39 | 14 |
t = timeout |
15 |
||
16 |
tv.tv_sec = <int>(t) |
|
17 |
tv.tv_usec = <int>((t - <int> t) * 100000) |
|
18 |
||
19 |
return tv |
|
20 |
||
21 |
else : |
|
22 |
return NULL |
|
23 |
||
24 |
||
25 |
# our shared callback handler |
|
26 |
# this must aquire the GIL before invoking the python object |
|
27 |
# this means that the libevent loop *must* be run with the GIL released! |
|
28 |
cdef void ev_callback (lib.evutil_socket_t fd, short mask, void *arg) with gil : |
|
29 |
""" |
|
30 |
Proxy event back to event object passed as arg |
|
31 |
""" |
|
32 |
||
33 |
# unpack opaque arg |
|
34 |
cdef event ev = <event>arg |
|
35 |
||
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
36 |
# drop self-ref if not pending anymore (keep the local ref) |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
37 |
# XXX: does the event show up as non-pending here? :/ |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
38 |
ev._unmark() |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
39 |
|
39 | 40 |
# invoke |
41 |
# XXX: logging? |
|
42 |
ev(fd, mask) |
|
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
43 |
|
39 | 44 |
|
45 |
cdef class event : |
|
46 |
||
42
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
47 |
def __init__ (self, event_base base not None, lib.evutil_socket_t fd, short events) : |
57
8c4032265c8c
event keeps a ref to its event_base - needed for later error propagation, and to ensure method calls remain valid
Tero Marttila <terom@fixme.fi>
parents:
52
diff
changeset
|
48 |
# hold on to our event_base |
8c4032265c8c
event keeps a ref to its event_base - needed for later error propagation, and to ensure method calls remain valid
Tero Marttila <terom@fixme.fi>
parents:
52
diff
changeset
|
49 |
self.base = base |
8c4032265c8c
event keeps a ref to its event_base - needed for later error propagation, and to ensure method calls remain valid
Tero Marttila <terom@fixme.fi>
parents:
52
diff
changeset
|
50 |
|
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
51 |
# we don't hold a self-reference yet |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
52 |
self.alive = False |
42
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
53 |
|
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
54 |
# libevent seems to segfault otherwise... |
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
55 |
if events & (lib.EV_READ | lib.EV_WRITE) and fd < 0 : |
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
56 |
raise ValueError(fd) |
39 | 57 |
|
58 |
# construct event using our internal callback, with self as the arg |
|
52
722fc70a197a
change CallbackEvent to replace fd with ev, event2 doc/style tweaks
Tero Marttila <terom@fixme.fi>
parents:
50
diff
changeset
|
59 |
self.ev = lib.event_new(base.ev_base, fd, events, ev_callback, <void *> self) |
39 | 60 |
|
61 |
if self.ev == NULL : |
|
62 |
raise Exception("event_new") |
|
63 |
||
64 |
||
42
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
65 |
def assign (self, event_base base not None, lib.evutil_socket_t fd, short events) : |
39 | 66 |
""" |
67 |
Re-assign our event parameters to the given ones. |
|
68 |
||
69 |
It is illegal to call this method after calling .add() but before del()/callback! |
|
70 |
""" |
|
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
71 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
72 |
if self.alive : |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
73 |
raise RuntimeError("Cannot change event while active") |
39 | 74 |
|
75 |
# interesting... no error return code |
|
52
722fc70a197a
change CallbackEvent to replace fd with ev, event2 doc/style tweaks
Tero Marttila <terom@fixme.fi>
parents:
50
diff
changeset
|
76 |
lib.event_assign(self.ev, base.ev_base, fd, events, ev_callback, <void *> self) |
39 | 77 |
|
78 |
||
79 |
def add (self, timeout = None) : |
|
80 |
""" |
|
81 |
Add this event to our event_base's set of monitored events. |
|
82 |
||
83 |
This event will be executed if any of the events specified in 'events' occurs, or the given timeout |
|
84 |
expires. If no timeout is given, no timeout will occur. |
|
85 |
||
86 |
If this event already has a shceduled timeout, the old timeout will be replaced by the new one. |
|
87 |
||
88 |
timeout - a float representing the timeout length |
|
89 |
||
90 |
XXX: "The event in the ev argument must be already initialized by event_set() and may not be used in calls to event_set() until it has timed out or been removed with event_del()" -- enforce |
|
91 |
""" |
|
92 |
||
93 |
cdef lib.timeval tv |
|
94 |
||
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
95 |
# aquire self-ref |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
96 |
self._mark() |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
97 |
|
39 | 98 |
if lib.event_add(self.ev, build_timeout(&tv, timeout)) < 0 : |
99 |
raise Exception("event_new") |
|
100 |
||
101 |
||
102 |
def delete (self) : |
|
103 |
""" |
|
104 |
Remove this event from our event_base's set of monitored events. |
|
105 |
||
106 |
If this event has already executed or has never been added this call with have no effect. |
|
107 |
""" |
|
108 |
||
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
109 |
|
39 | 110 |
if lib.event_del(self.ev) < 0 : |
111 |
raise Exception("event_del") |
|
112 |
||
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
113 |
# release self-ref |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
114 |
self._unmark() |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
115 |
|
39 | 116 |
|
117 |
def activate (self, int fd, short mask) : |
|
118 |
""" |
|
119 |
'Fake' this event as active, triggering the callback. |
|
120 |
""" |
|
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
121 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
122 |
# aquire self-ref |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
123 |
self._mark() |
39 | 124 |
|
125 |
lib.event_active(self.ev, fd, mask) |
|
126 |
||
127 |
||
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
128 |
cpdef pending (self, short mask = 0xffff) : |
39 | 129 |
""" |
130 |
Returns a bool indicating if this event is pending (that is, has been .add()'d). |
|
131 |
||
42
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
132 |
For convenience, this defaults to testing all possible flags, and so will return true if any events are |
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
133 |
pending. |
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
134 |
|
39 | 135 |
XXX: support returning timeout value? |
136 |
""" |
|
137 |
||
138 |
return bool(lib.event_pending(self.ev, mask, NULL)) |
|
139 |
||
140 |
||
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
141 |
cdef object _mark (self) : |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
142 |
""" |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
143 |
Refcount magic. Aquires an internal reference *if we don't have one yet*. |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
144 |
""" |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
145 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
146 |
if not self.alive : |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
147 |
self.alive = True |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
148 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
149 |
py.Py_INCREF(self) |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
150 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
151 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
152 |
cdef object _unmark (self) : |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
153 |
""" |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
154 |
Refcout magic. Releses the internal reference *if we have one* and libevent doesn't think this event is |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
155 |
pending anymore. |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
156 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
157 |
XXX: this might cause ourselves to be destroyed... we can't break if that happens. |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
158 |
""" |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
159 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
160 |
if self.alive and not self.pending() : |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
161 |
py.Py_DECREF(self) |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
162 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
163 |
self.alive = False |
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
164 |
|
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
165 |
|
39 | 166 |
property fd : |
167 |
def __get__ (self) : |
|
168 |
""" |
|
169 |
Get the OS file descriptor associated with this event. |
|
170 |
""" |
|
171 |
||
172 |
return lib.event_get_fd(self.ev) |
|
173 |
||
174 |
||
175 |
def __call__ (self, lib.evutil_socket_t fd, short mask) : |
|
176 |
""" |
|
41 | 177 |
The method invoked by the internal libevent callback when the event becomes active. |
178 |
||
179 |
fd - OS file descriptor the event occured on, or -1 |
|
180 |
mask - bitmask of EV_* flags that represents the triggered event |
|
181 |
||
182 |
||
183 |
The default implementation of __call__ does nothing. The method's return value will be ignored, and should |
|
184 |
be None. Any errors raised by the callback will be printed out as warning messages, and ignored. |
|
39 | 185 |
""" |
186 |
||
187 |
pass |
|
188 |
||
189 |
||
190 |
def __dealloc__ (self) : |
|
191 |
""" |
|
50
da394bb715af
try and keep lib.event2.event objects alive (via self-ref) while pending
Tero Marttila <terom@fixme.fi>
parents:
47
diff
changeset
|
192 |
Release the event object usign event_free. This should be completely safe as regards our event_base. |
39 | 193 |
|
194 |
XXX: what happens if event_base's __dealloc__ is triggered, but there are still event objects alive? |
|
195 |
""" |
|
42
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
196 |
|
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
197 |
if self.ev != NULL : |
0ff56f7216ee
fix some segfaults with event
Tero Marttila <terom@fixme.fi>
parents:
41
diff
changeset
|
198 |
lib.event_free(self.ev) |
39 | 199 |
|
46
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
200 |
class CallbackEvent (event) : |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
201 |
""" |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
202 |
Extends the event type to take a callback and additional arguments to invoke from the __call__ method. |
52
722fc70a197a
change CallbackEvent to replace fd with ev, event2 doc/style tweaks
Tero Marttila <terom@fixme.fi>
parents:
50
diff
changeset
|
203 |
|
722fc70a197a
change CallbackEvent to replace fd with ev, event2 doc/style tweaks
Tero Marttila <terom@fixme.fi>
parents:
50
diff
changeset
|
204 |
This will pass the event object itself to the callback in place of the fd. |
46
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
205 |
""" |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
206 |
|
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
207 |
def __init__ (self, event_base base, lib.evutil_socket_t fd, short events, object callback, *args, **kwargs) : |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
208 |
# parent |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
209 |
super(CallbackEvent, self).__init__(base, fd, events) |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
210 |
|
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
211 |
# store |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
212 |
self.callback = callback |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
213 |
self.args = args |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
214 |
self.kwargs = kwargs |
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
215 |
|
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
216 |
def __call__ (self, lib.evutil_socket_t fd, short mask) : |
52
722fc70a197a
change CallbackEvent to replace fd with ev, event2 doc/style tweaks
Tero Marttila <terom@fixme.fi>
parents:
50
diff
changeset
|
217 |
self.callback(self, mask, *self.args, **self.kwargs) |
46
64b4ffb44754
add CallbackEvent helper, and move getaddrinfo from endpoint into its own function
Tero Marttila <terom@fixme.fi>
parents:
39
diff
changeset
|
218 |