RE: [PATCH] Resubmit: "sim buffering" support for AT2.
"Oded Arbel" <[email protected]>
| Newsgroups | gmane.comp.mobile.kannel.devel |
|---|---|
| Message-ID | <[email protected]> |
Hi Adnrea. I'm sorry, but I did not understand the second issue. as for the first issue - I did notice that sometimes messages land in SMS memory whether or no "SIM buffering" is enabled or not - _this is_ the issue the so called "SIM buffering" solution was meant to solve. I guess that setting CNMI to a more correct value would solve some part of this problem (for example - messages that land in memory because they are of some class or other), and this has nothing to do with "SIM buffering" - you will encounter this behavior, and be able to solve it correctly or not, no matter if the "SIM buffering" patch is applied or not. this patch, while not intended to solve this specific problem, does allow you to work around the problem by not caring whether messages land in memory or not. Also - initialization wise, the current incarnation of the "SIM buffering" patch will not affect the modem's setup in anyway other then explictly setting the location for memory storage. I think that part can be safely skipped for most modems. If you say its more correct, I will rewrite it not to issue AT+CPMS= if 'message-storage' isn't set in modems.conf. so, except for some internal code changes needed to support +CMGR: reading, the only difference in communication with the modem under normal conditions (that is - no messages are stored in memory at all) is the occasional AT+CPMS? command. now note : THIS WILL BREAK SOME MODEMS !!! specificly the Siemens M20 does not take well this simple command and immidietly after receiving this command will revert to sending all messages to memory regardless of +CNMI setting. do not enable sim-buffering with Siemens M20 unless you are sure you know what you are doing. That being said, while sim-buffering is set to 'false' or not at all, Kannel behaviour will remain unchanged from current CVS and therefor poses to threat to M20 users. that's why I think this patch should be included in CVS, as it adds needed functionality w/o harming current one. -- Oded Arbel m-Wise Inc. [email protected] Abstainer, n.: A weak person who yields to the temptation of denying himself a pleasure. -- Ambrose Bierce, "The Devil's Dictionary" > -----Original Message----- > From: Andrea Viscovich [mailto:[email protected]] > Sent: Monday, March 18, 2002 9:32 AM > To: Oded Arbel; Kannel-devel (E-mail) > Subject: Re: [PATCH] Resubmit: "sim buffering" support for AT2. > > > Hi Oded, > looking at it, it seems something has still to be fixed. > Even with your patch there is no AT+CNMI: check when receiving sms. > With wavecom happens to receive some sms which goes into sim > memory even if > we haven't sim-buffering. They may be class 2 messages > (reading manual about > AT+CNMI). > So, I think we should be able to receive those sms even if we have > sim-buffering=false, > but I don't mistake your patch alaways checks to have > sim-bufferiing=true > before managing those kind of sms that lands in sms memory. > Thanks > Andrea > >