fds garbage. Invalid offset on new page.

RE: fds garbage. Invalid offset on new page.

Håkon Alseth — Thu, 10 Mar 2016 12:27:15 GMT

Hi Petr,

I can confirm the issue that you are seeing, and the validity of your workaround.

I had a chat with the developers of FDS, and looked into the internal repository. They have actually implemented the same fix that you have found, and this will be included in the next release of the SDK.

Best regards, Håkon

RE: fds garbage. Invalid offset on new page.

Petr — Wed, 09 Mar 2016 06:24:00 GMT

Hi! I save bond data and periodic save my custom parameters. When save my custom parameters and the flash page ends, fds_write() return FDS_ERR_NO_SPACE_IN_FLASH. It runs fds_gc(). In the second iteration, result of fds_gc() is erased page and new page with data at error offset, see pics. This error offset equal length of bond data. FDS_VIRTUAL_PAGES =2 for simplify.

bool saveFile (dataFile_t &data)

{ ret_code_t retval; fds_record_key_t key; fds_record_chunk_t chunk; fds_record_desc_t descriptor;

clearFile(data);

// Fill in the record keys.
key.type     = SETUP_PARAMETERS_TYPE;
key.instance = data.fileId;

// Initialize the record chunk, pointing it to the data to be stored.
chunk.p_data       = data.pointer;
chunk.length_words = data.length;

// Write the record to flash.
retval = fds_write /*fds_update*/ (&descriptor,
                   key,
                   1,
                   &chunk);

if (retval != NRF_SUCCESS)
{
    if (retval == FDS_ERR_NO_SPACE_IN_FLASH)
    {
        //Garbage collection reclaims the flash space occupied by records which have been cleared using fds_clear
        fds_gc();
        vTaskDelay(GC_WAIT_100MLS);
        // Write the record to flash.
        retval = fds_write (&descriptor,
                            key,
                            1,
                            &chunk);
        if (retval != NRF_SUCCESS) return false;
    }
    else return false;
}
return true;

}

RE: fds garbage. Invalid offset on new page.

Håkon Alseth — Wed, 02 Mar 2016 15:22:40 GMT

Hi,

Could you post a test-snippet on how you are re-creating this scenario? Is it storing+erasing data, storing+erasing, then running a GC?

Cheers, Håkon