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

Generate the Rotating Device Id based on Reset Counter #4224

Closed
hnnajh opened this issue Dec 15, 2020 · 5 comments · Fixed by #18514
Closed

Generate the Rotating Device Id based on Reset Counter #4224

hnnajh opened this issue Dec 15, 2020 · 5 comments · Fixed by #18514
Assignees
Labels
bug Something isn't working maybe done? Isssue may be complete/obsolete/not relevant anymore. Needs review to determine if to close or not. p1 priority 1 work spec Mismatch between spec and implementation V1.0
Milestone

Comments

@hnnajh
Copy link
Collaborator

hnnajh commented Dec 15, 2020

Currently the Rotating Device Id is hard-coded however it should be generated from the device serial number + reset counter

@issue-label-bot
Copy link

Issue-Label Bot is automatically applying the label feature_request to this issue, with a confidence of 0.90. Please mark this comment with 👍 or 👎 to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@bzbarsky-apple
Copy link
Contributor

Note that fixing this is a must-have for shipping rotating device ids.

@woody-apple woody-apple added p1 priority 1 work and removed feature request labels Feb 2, 2021
@woody-apple woody-apple added this to the V0.7 milestone Feb 2, 2021
@woody-apple woody-apple added V1.0 and removed V0.7 labels Feb 2, 2021
@woody-apple woody-apple modified the milestones: V0.7, V1.0 Feb 2, 2021
@tima-q
Copy link
Contributor

tima-q commented Mar 1, 2021

#4362 triggered a runtime issue inside our NVM implementation

The R/W to a LifetimePersistedCounter that is stored in NVM/Configuration is incorrect.
The key ID for that counter is not known at ConfigurationManager level.
Our NVM/Configuration manager implementation asserts on this unknown key as we only register the expected list.
Other vendor's might have the same issue.

Was there an idea to setup the relation between the new lifetimecounter and the configuration manager defines ?
There is currently no direct link between the define for the key (CHIP_CONFIG_LIFETIIME_PERSISTED_COUNTER_KEY)
and the list of key ID's listed in the different src\platform\xyz\xyzConfig.h files

We will disable the feature for now at build-time to proceed, but I think the structural solution would be to introduce a specific key in the ConfigurationManager for this counter.
@hnnajh What are/were your thoughts around this in terms of implementation ?

tima-q added a commit to tima-q/connectedhomeip that referenced this issue Mar 1, 2021
tima-q added a commit to tima-q/connectedhomeip that referenced this issue Mar 1, 2021
@bzbarsky-apple bzbarsky-apple added the spec Mismatch between spec and implementation label Mar 25, 2021
@andy31415 andy31415 added v1_triage_split_1 bug Something isn't working p1 priority 1 work V1.0 spec Mismatch between spec and implementation and removed bug Something isn't working p1 priority 1 work V1.0 spec Mismatch between spec and implementation v1_triage_split_1 labels Jan 30, 2022
@bzbarsky-apple
Copy link
Contributor

@hnnajh What remains to be done here?

@bzbarsky-apple bzbarsky-apple added the maybe done? Isssue may be complete/obsolete/not relevant anymore. Needs review to determine if to close or not. label May 4, 2022
@chrisdecenzo chrisdecenzo assigned chrisdecenzo and unassigned hnnajh May 16, 2022
@chrisdecenzo
Copy link
Contributor

It looks like rotating ID unique id input is coming from a #define (CHIP_DEVICE_CONFIG_ROTATING_DEVICE_ID_UNIQUE_ID). This needs to be something that device makers can populate at run time from storage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working maybe done? Isssue may be complete/obsolete/not relevant anymore. Needs review to determine if to close or not. p1 priority 1 work spec Mismatch between spec and implementation V1.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants