Hi folks
I've tried asking this in private mail for quite some time, but sadly
got no reply. So I end up needing to raise the question here.
Sangoma's s setup process includes a small patch to Zaptel. I have some
technical reservations with htat patch. It seems that under certain
circumstances it may cause exexpected behaviour when used with other
Zaptel channel drivers. I also don't understand why a safer method is
not used.
To be more specific:
The patch defines a number of ioctl numbers in the range of 60-65 in
zaptel.h . This is perfectly OK: that range is dedicated to
driver-specific ioctl-s. If userspace ever calls those ioctls on
channels belonging to spans handled by that specific driver, , zaptel
will deliver them to the driver.
But then comes the patch to zaptel.c . Under certain circumstances
(which don't seem to be specific to the Sangoma cards. Might be specific
to digital spans) zaptel is to call the ioctl handler.
Has it been tested with all other Zaptel drivers? Is it safe to apply
for a general-purpose zaptel-package? A better strategy would probably
be to add an extra function pointer to struct zt_span and call it in
case it is not NULL in such a case.
For that reason I have rejected the Sangoma patch from Debian packages
and tried contacting Sangoma's developers for clarifications. Sadly, I
have not recieved replies. Others have different opinions than me and
thus the Sangoma patch is included in some binary distributions of
Zaptel Maybe this is just because the potential problem was not clear.
So right now I'm waiting for clarifications. Until then I would
recommend for anybody using Zaptel with non-Sangoma digital cards not to
apply that patch.
I see no reason why a patch that slightly adapts Zaptel to allow better
ussage with Sangoma hardware could be applied. But such a patch should
first and formmost be technically sound.
--
Tzafrir Cohen
icq#16849755 jabber:tzafrir.cohen at xorcom.com
+972-50-7952406 mailto:tzafrir.cohen at xorcom.com
http://www.xorcom.com iax:guest at local.xorcom.com/tzafrir
