Chris Frey wrote:
On Sun, Jan 18, 2009 at 11:53:48AM -0600, Andy Herkey wrote:
Rick - Thanks for pointing that out - we do send out that packet (with the m_session_key) if the modem has a password but we never sent it without one. :-(

Chris - I'm working on some changes for Shannon and I'll add this to that patch.

Thanks Andy!  Does that packet make any difference in your testing?
Is it worth adding it as a small patch so Bill Paul can test, or is your
larger patch nearly ready?

I can be patient... just asking. :-)

- Chris

Chris,
Since we send that packet when using password I don't see those special packets. I only had rebooting problems when using the serial modem and that only happened when I went to this one web site. It would reboot every time I hit that site. When I got the ip_modem working with a password I never had any more problems. One time last November I pounded the hell out of it when my cable service went out during a .com site stress test. I was connected for over 4 hours and I had multiple monitors up feeding me data, multiple ssh windows open to web and app servers I even had a rdesktop session connected to my desktop at work. I was impressed to say the least with the BB and my service provider.

Anyway back to the patch stuff. Shannon is having a different problem since moving to FC10. It seems that pppd is not sending the chat script command to pppob. I also got another bug report for an 8820 that I wanted to work on before submitting another patch. And you'll be happy to know that I'm also going to remove the AT command code now that I'm seeing how other BB are responding.

For Bill's problem, if we move the closing curly bracket from line 318 to line 289 that will send the session_key packet for both password or non password devices. Below is a simple diff of the change:

[aher...@augie tmp]$ diff -c m_ipmodem.cc.cvs m_ipmodem.cc
*** m_ipmodem.cc.cvs    2009-01-20 20:33:56.592035108 -0600
--- m_ipmodem.cc        2009-01-20 20:35:55.573883681 -0600
***************
*** 287,292 ****
--- 287,293 ----
               m_dev.BulkRead(read_ep, data);
               ddout("IPModem: Start Response Packet:\n" << data);

+       }
               // send packet with the session_key
unsigned char response_header[] = { 0x00, 0, 0, 0, 0, 0, 0, 0, 3, 0, 0, 0, 0, 0xc2, 1, 0 }; memcpy(&response[0], response_header, sizeof(response_header));
***************
*** 315,321 ****
                       }
               }

-       }

       // see if the modem will respond to commands
       const char modem_command[] = { "AT\r" };
--- 316,321 ----

I hope that's clear enough to understand. :-)

- Andy


------------------------------------------------------------------------------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
Barry-devel mailing list
Barry-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/barry-devel

Reply via email to