device_manager_peripheral.c variable word alignment bug report.

[Georgi Olentšenko]

It's probably a rare bug, but it is still worth noting. I have been debugging this issue for a day. I am using SDK 6.1 at the moment.

The problem was that Device manager was failing to initialize itself, if optimizations were set to -Os and p_init_param->clear_persistent_data == false in dm_init(). Digging deeper i found out that

*err_code = pstorage_load((uint8_t )&m_peer_table[index], &block_handle, sizeof(peer_id_t), 0);

was returning NRF_ERROR_INVALID_ADDR because

static peer_id_t m_peer_table[DEVICE_MANAGER_MAX_BONDS];

was not aligned by word (4 bytes) is RAM, which the function checked. The simplest solution was to force the compiler to align the array as was expected in pstorage_load():

static peer_id_t m_peer_table[DEVICE_MANAGER_MAX_BONDS] attribute ((aligned (4)));

This problem did not occur when using other optimization levels. One thing i do not understand is why do we need to align the memory in RAM? As far as i know it is not written by pages. Am I wrong?

[Petter Myhre]

Sounds very similar to this. I have reported it. Thank you for letting us know about the bug and the solution.

Edit 16.02: This should be fixed in SDK 8.0

[RK]

It's ok - I'm not that worried about the release date, just was a little surprised after how recent 7.x was that 8.x already sounds like it's around the corner. Hopefully the directory structures and general usage from 7.x will remain, 6->7 was a pretty painful conversion, although I will agree that the new directory structure mostly makes more sense than the old one.