Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

checkOnline in controller ZMemError #22

Open
splitice opened this issue May 9, 2017 · 2 comments
Open

checkOnline in controller ZMemError #22

splitice opened this issue May 9, 2017 · 2 comments

Comments

@splitice
Copy link

splitice commented May 9, 2017

There is a function in controller.js _checkOnline which attempts by way of iteration of the database to determine what devices are online and joined to the network. The node descriptor request in this function always returns 0x10 - 0x11 corresponding to ZMemError & ZBufferFull which ensures the devices are never detected and hence marked as online.

This is using the ZNP build from Zigbeer.

@splitice
Copy link
Author

splitice commented May 9, 2017

I slowed the querying down and now most of the time it returns with a status code of 0, still ZNP online devices time out. & I can see with a packet sniffer that no node description request is actually being sent by the coordinator dongle.

Route Request packets do go out for those devices which are not on the network however. Those on the network are sent no packets.

@splitice
Copy link
Author

Ok,

so there is two bugs in this function:

  1. Performing all of these (and anything else going on in the background, which is quite alot during initialization) at once results in memory exhaustion on the dongle.

  2. The timeout of 5s is insufficient for devices that sleep. In fact the devices I tested never return when queried here. If the device is not allowed to sleep during init, i.e for an IAS sensor if it is being toggled throught init then it does respond.
    '
    I propose replacement of this method with an LQI based query to the routers in the network. This way the sleeping end devices are not involved.

allofmex pushed a commit to allofmex/zigbee-shepherd that referenced this issue Apr 23, 2019
* First Test to maybe fix undefined

* Updates

* Updates
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant