[asterisk-ss7] Re: asterisk-ss7 Digest, Vol 24, Issue 18

[Alexandru Pirvulescu]

Yes, it works. Thanks.

Maybe the zapata.conf.sample should be patched with a real-life  
scenario, then.
Most of the operators are using timeslot 16 for signalling, so 1-15,  
17-31 is the most used configuration.
Right now the "cicbeginswith=" is not near "channel=" in the sample  
config, so no assumption can be made that
they are connected somehow. I thought that it is the startcic  
parameter in chan_ss7.

Regards,
Alex

On Feb 21, 2007, at 9:00 PM, asterisk-ss7-request at lists.digium.com  
wrote:

> Send asterisk-ss7 mailing list submissions to
> 	asterisk-ss7 at lists.digium.com
>
> To subscribe or unsubscribe via the World Wide Web, visit
> 	http://lists.digium.com/mailman/listinfo/asterisk-ss7
> or, via email, send a message with subject or body 'help' to
> 	asterisk-ss7-request at lists.digium.com
>
> You can reach the person managing the list at
> 	asterisk-ss7-owner at lists.digium.com
>
> When replying, please edit your Subject line so it is more specific
> than "Re: Contents of asterisk-ss7 digest..."
>
>
> Today's Topics:
>
>    1. Re: chan_zap.c patch (Matthew Fredrickson)
>
>
> ----------------------------------------------------------------------
>
> Message: 1
> Date: Wed, 21 Feb 2007 11:00:12 -0600
> From: Matthew Fredrickson <creslin at digium.com>
> Subject: Re: [asterisk-ss7] chan_zap.c patch
> To: asterisk-ss7 at lists.digium.com
> Message-ID: <7e1c0cce44ab27aedb6e263b01ff2ed8 at digium.com>
> Content-Type: text/plain; charset=ISO-8859-1; format=flowed
>
> It's probably because of a "bug" in your configs.  The way you  
> would to
> it properly is this:
> cicbeginswith=1
> channel=1-15
> cicbeginswith=17
> channel=17-31
>
> It seems to be a nonintuitive way (for most people) to do channel to
> CIC mapping.  Maybe this might be an area worth spending some time on
> improving.
>
> Matthew Fredrickson
>
> On Feb 21, 2007, at 10:33 AM, Alexandru Pirvulescu wrote:
>
>> Hello,
>>
>> The September/October version of SS7 implementation worked, but today
>> I've found out that the current trunk version is not working.
>> What is happening:
>>
>> If I have channel=>1-15 and channel=>17-31 in /etc/zapata.conf the
>> reset code will send GRS to CICs from 1 to 30 (which is wrong).
>> First, I thought that the break at channel 16 is ignored (which I
>> repeat, worked). The code is looking if the next zt_pvt->cic is the
>> following number after current and if it is not it sends a group  
>> reset
>> on the collected group (1-15 or 17-31). So, the problem was somewhere
>> at the ->cic member, which was ALWAYS the next number in sequence.
>>
>> Now, after digging more into the code, I've found that the CIC number
>> is taken incrementally (and 16 is a valid CIC number in this case) in
>> mkintf().
>>
>> I've patched chan_zap.c to set ->cic value to channel and it works
>> like a charm.
>>
>> Here is the patch, if any committer interested to put in the trunk:
>>
>> # svn diff channels/chan_zap.c
>> Index: channels/chan_zap.c
>> ===================================================================
>> --- channels/chan_zap.c (revision 55913)
>> +++ channels/chan_zap.c (working copy)
>> @@ -7538,7 +7538,8 @@
>>                                         return NULL;
>>                                 }
>> -                               tmp->cic = cur_cicbeginswith++;
>> +                               tmp->cic = channel;
>> +                               cur_cicbeginswith++;
>>                                 tmp->ss7 = ss7;
>>                                 tmp->ss7call = NULL;
>>
>>
>> Regards,
>> Alex
>>
>> _______________________________________________
>> --Bandwidth and Colocation provided by Easynews.com --
>>
>> asterisk-ss7 mailing list
>> To UNSUBSCRIBE or update options visit:
>>    http://lists.digium.com/mailman/listinfo/asterisk-ss7
>
>
>
> ------------------------------
>
> _______________________________________________
> --Bandwidth and Colocation provided by Easynews.com --
>
> asterisk-ss7 mailing list
> To UNSUBSCRIBE or update options visit:
>    http://lists.digium.com/mailman/listinfo/asterisk-ss7
>
>
> End of asterisk-ss7 Digest, Vol 24, Issue 18
> ********************************************