[Dovecot] Patch: More of kqueue() support.
Hi, the attached patch contains these changes:
2005-12-15 22:18 Vaclav Haisman <v.haisman@sh.cvut.cz>
* src/lib/ioloop-kqueue.c: Fix IO_ERROR behaviour.
* src/lib/ioloop-notify-kqueue.c: New file.
* configure.in: Improve kqueue detection and handling. Cleanup.
The configure.in changes that are not related to kqueue are there to make autoconf 2.59, automake 1.9.6 and libtool 1.5.20 are happy.
The directory change notification is not really well tested because Dovecot doesn't seem to use it that much.
As per Brad's request I have changed all occurrences of FreeBSD to just BSD.
Vaclav Haisman
I realise it is Christmas time and all but it has been a week without response, so I ping :)
VH
On Thu, 2005-12-15 at 22:42 +0100, Vaclav Haisman wrote:
The directory change notification is not really well tested because Dovecot doesn't seem to use it that much.
Give "IDLE" command in IMAP and it'll get used. dnotify works only with maildirs. I'm not sure if inotify works also with mboxes. If possible try to get kqueue() to work with mboxes too :)
Also is it necessary to use two kqueue()s?
Sorry for the delayed response. I was a bit busy.
Timo Sirainen wrote:
On Thu, 2005-12-15 at 22:42 +0100, Vaclav Haisman wrote:
The directory change notification is not really well tested because Dovecot doesn't seem to use it that much.
Give "IDLE" command in IMAP and it'll get used. dnotify works only with maildirs. I'm not sure if inotify works also with mboxes. If possible try to get kqueue() to work with mboxes too :)
I have tested the notification with IDLE command and it works fine for Maildir. It doesn't seem to work with mbox either.
Also is it necessary to use two kqueue()s?
Well, the notification about changes in directory and polling file handles are imho two different things. I think it is both logical and easier to keep it separate.
Vaclav Haisman
participants (4)
-
Timo Sirainen
-
Vaclav Haisman
-
Vaclav Haisman
-
Václav Haisman