[issue7211] select module - kevent ident field 64 bit issue

Ned Deily added the comment: This patch causes build errors on 32-bit/64-bit multi-architecture builds on OS X. See Issue7416 for more info and a patch. -- nosy: +ned.deily ___ Python tracker

[issue7211] select module - kevent ident field 64 bit issue

Antoine Pitrou added the comment: Patch was committed in r76108 (trunk) and r76111 (py3k) and didn't introduce any regression on the FreeBSD and OS X buildbots. Thanks! -- resolution: -> fixed stage: -> committed/rejected status: open -> closed ___

[issue7211] select module - kevent ident field 64 bit issue

Antoine Pitrou added the comment: For me, the patch is worth trying out on the buildbots - to see if there are any configuration problems we might have overlooked. -- ___ Python tracker

[issue7211] select module - kevent ident field 64 bit issue

Eric Smith added the comment: The patch (http://bugs.python.org/file15228/kevent.patch) works for me under OS X 10.5.8 Intel. All tests pass, except test_telnetlib which fails intermittently with and without the patch. Python 3.2a0 (py3k:75951, Oct 29 2009, 11:38:58) [GCC 4.0.1 (Apple Inc. bu

[issue7211] select module - kevent ident field 64 bit issue

Michael Broghton added the comment: Antoine, thanks for the tips and the example. I have updated the patch. I checked and this does apply cleanly to py3k. -- Added file: http://bugs.python.org/file15228/kevent.patch ___ Python tracker

[issue7211] select module - kevent ident field 64 bit issue

Antoine Pitrou added the comment: Apparently Roundup borked the URL. Let's try another one (or look into Modules/_io/_iomodule.h): http://code.python.org/hg/trunk/file/b9bc35171668/Modules/_io/_iomodule.h#l88 -- ___ Python tracker

[issue7211] select module - kevent ident field 64 bit issue

Antoine Pitrou added the comment: According to man pages on the Web, the kevent structure is: struct kevent { uintptr_t ident;/* identifier for this event */ short filter; /* filter for event */ u_short flags;/* action flag

[issue7211] select module - kevent ident field 64 bit issue

Michael Broghton added the comment: This is against release31-maint, if it matters. -- keywords: +patch Added file: http://bugs.python.org/file15212/kevent.patch ___ Python tracker _

[issue7211] select module - kevent ident field 64 bit issue

Michael Broghton added the comment: Martin, thanks for your responses. In regards to point three: Kqueue's are not just used for file descriptors. I believe this is the reason why the ident field is a uintptr_t and not an int. The example I gave was for kqueue timers. Since the operating syste

[issue7211] select module - kevent ident field 64 bit issue

Martin v. Löwis added the comment: > 1. What would be an appropriate member type for ident in > kqueue_event_members? It seems like T_PYSSIZET might work. Otherwise, I > am guessing that this will involve some #if's. IIUC, it needs to match *exactly* the field size inside struct kevent. So you'

[issue7211] select module - kevent ident field 64 bit issue

Michael Broghton added the comment: I'm not sure how to patch this so that it will work on both 32 and 64 bit systems. Issues: 1. What would be an appropriate member type for ident in kqueue_event_members? It seems like T_PYSSIZET might work. Otherwise, I am guessing that this will involve some

[issue7211] select module - kevent ident field 64 bit issue

Martin v. Löwis added the comment: Would you like to propose a patch? -- nosy: +loewis ___ Python tracker ___ ___ Python-bugs-list mai

[issue7211] select module - kevent ident field 64 bit issue

Changes by Antoine Pitrou : -- priority: -> normal type: behavior -> feature request versions: +Python 2.7, Python 3.2 -Python 3.1 ___ Python tracker ___

[issue7211] select module - kevent ident field 64 bit issue

Changes by Antoine Pitrou : -- nosy: +christian.heimes, therve ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: h

[issue7211] select module - kevent ident field 64 bit issue

New submission from Michael Broghton : On FreeBSD and MacOS 64-bit systems the ident field of a kevent is big enough to hold a 64-bit integer (uintptr_t). Looks like Python is casting it to an unsigned 32-bit integer. This is inconvenient for implementing kqueue timers, where id(timer_obj) is a