DFU error: OPERATION NOT PERMITTED

RE: DFU error: OPERATION NOT PERMITTED

mdido — Thu, 25 Jun 2020 10:18:08 GMT

Thanks for the information

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Thu, 25 Jun 2020 09:20:23 GMT

1.10.4 released: https://github.com/NordicSemiconductor/Android-DFU-Library/releases/tag/v1.10.4

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Thu, 25 Jun 2020 07:33:34 GMT

Here's the issue:https://github.com/NordicSemiconductor/Android-DFU-Library/issues/252, a fix: https://github.com/NordicSemiconductor/Android-DFU-Library/pull/253

Also, I found out why it works for you on iOS: https://github.com/NordicSemiconductor/IOS-Pods-DFU-Library/issues/270 has already been fixed with: https://github.com/NordicSemiconductor/IOS-Pods-DFU-Library/pull/272

I'll fix one more issue in the DFU Library for Android and will release new version today.

RE: DFU error: OPERATION NOT PERMITTED

mdido — Wed, 24 Jun 2020 13:56:50 GMT

Thank you for the great support. While we wait for the new library, we will disable resuming in Android

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Wed, 24 Jun 2020 12:15:18 GMT

Yes, as with resume disabled the library does not try to execute the partially sent object. It ignores it and doesn't even check. Just starts by sending init packet again.

My recommendations are:

1. If your fw is not public yet, try the code snippet I posted above and remember to have it there in case of SDK update in the future. Use resuming, as this speeds up upload in case dfu breaks in the middle.

2. If devices are already in production and bootloader can't be modified, wait for my workaround in Android and use iOS library as is. Until 1.10.4 is released on Android you may disable resuming.

3. You may also ignore the workaround and just disable resuming on Android or both, and use your devices and bootloader as they are.

After releasing 1.10.4 I'll try to release nRF Toolbox for Android, but this most probably will not happen today. Also, nRF Connect for Android won't be using this version for some time, and there's not option to disable resuming yet.

RE: DFU error: OPERATION NOT PERMITTED

mdido — Wed, 24 Jun 2020 12:08:14 GMT

Here is the log from ios with resume disabled in nrfconnect. There is not the message:"Operation not permitted".

devzone.nordicsemi.com/.../log_5F00_ios_5F00_no_5F00_resume.txt

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Wed, 24 Jun 2020 10:56:41 GMT

Cool. So this is actually the workaround I wanted to do for DFU libraries. Seems like the iOS already had it (accidentally) implemented. I'll try to release DFU lib 1.10.4 today.

RE: DFU error: OPERATION NOT PERMITTED

mdido — Wed, 24 Jun 2020 10:53:25 GMT

Here is the log from ios. There is a line with string "Operation not permitted", but nrfconnect does not appear the error message, the uptade continues and ends succesfully.

devzone.nordicsemi.com/.../log_5F00_ios.txt

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Wed, 24 Jun 2020 10:25:44 GMT

As I understand it, the s_dfu_settings.progress.data_object_size is set to 0 when an object is executed. This change makes possible executing the same object when it's empty (when the last one has been executed, but no data were received/saved for the new object). This ensures that the current or previous object is executed correctly.

Regarding iOS, could you share logs from iOS as well? I wonder if the lib is sending execute command at all, or why you see it succeeding.

RE: DFU error: OPERATION NOT PERMITTED

mdido — Wed, 24 Jun 2020 10:10:10 GMT

The fix you suggest is not clear to me. It would seem that the two offset (image_offset and image_offset_last) are not initialized correctly on the resume update.

I confirm that I can't get error with iOS. Also if I have a device that is in error with android and I try a second update with iOS (wich restarts from percentage interrupted previously), the update is succesful.

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Wed, 24 Jun 2020 09:02:51 GMT

Hi,

In nrf_dfu_req_handler.c in sdk/components/libraries/bootloader/dfu, inmethod on_data_obj_execute_request you may add the following code:

static bool on_data_obj_execute_request(nrf_dfu_request_t * p_req, nrf_dfu_response_t * p_res)
{
    NRF_LOG_DEBUG("Handle NRF_DFU_OP_OBJECT_EXECUTE (data)");

    uint32_t const data_object_size = s_dfu_settings.progress.firmware_image_offset -
                                      s_dfu_settings.progress.firmware_image_offset_last;
    // NEW CODE BELOW:
    if (s_dfu_settings.progress.data_object_size == 0)
    {
        p_res->result = NRF_DFU_RES_CODE_SUCCESS;
        return true;
    }
    // END OF NEW CODE
    
    if (s_dfu_settings.progress.data_object_size != data_object_size)
    {
        /* The size of the written object was not as expected. */
        NRF_LOG_ERROR("Invalid data. expected: %d, got: %d",
                      s_dfu_settings.progress.data_object_size,
                      data_object_size);

        p_res->result = NRF_DFU_RES_CODE_OPERATION_NOT_PERMITTED;
        return true;
    }
    // ...
}

This hasn't been tested, but such solution was proposed by one of our developers.If you manager to implement this change, could you give us a feedback? Thanks!

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Wed, 24 Jun 2020 08:51:09 GMT

Yes, there's similar flag in the initiator: github.com/.../DFUServiceInitiator.swift

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Wed, 24 Jun 2020 08:49:45 GMT

Also, I just checked this issue and it was missed and isn't fixed in SDK 16 nor 17. That means I need to do some kind of workaround.

RE: DFU error: OPERATION NOT PERMITTED

mdido — Wed, 24 Jun 2020 08:31:22 GMT

thank you for your suggestion.

About iOS we have to check with our developers. Is the function called the same way?

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Wed, 24 Jun 2020 08:03:16 GMT

Regarding the last request: If you're using DFU library in your app, you may call

https://github.com/NordicSemiconductor/Android-DFU-Library/blob/2f61019d6da350ce3e577b5115e82b727b1c7112/dfu/src/main/java/no/nordicsemi/android/dfu/DfuServiceInitiator.java#L295

with value true. This will make the DFU always start from the beginning.

Also, I'm surprised the iOS DFU Library works. The issue should be independent from mobile OS. Do you also disable resume in iOS?

RE: DFU error: OPERATION NOT PERMITTED

mdido — Wed, 24 Jun 2020 07:59:01 GMT

Thank you for your support.

Here are some comments to what you wrote:

you wrote: "this looks like a bug in dfu from Sdk 15.3

mdido: I'm using Sdk 15.0, not 15.3

you wrote: "you may migrate to Sdk 16 or 17, where this bug should be fixed"

mdido: Is there a release note or change log where I can find this bug fixed?

you wrote: "I could try to implement a workaround in the Android and iOS library"

mdido: It will be a workaround officially approved by Nordic?. Instead of ignoring the error, could you restart the update from the beginning? It seems that iOS does not return the error

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Wed, 24 Jun 2020 04:06:18 GMT

Thank you. This looks like a bug in dfu from Sdk 15.3.

When a phone reconnects to a device and tries dfu it first checks if upload can be resumed. It compares offset and CRC. It the init packet matches, it checks the same for the data. In your case, 40960 bytes has been saved before, which is 10 full data objects, 4096 bytes each. The CRC is ok. At this point the library should ensure that the last data object has been executed. It should be fine to execute an already executed object, but in your case it falls.

There are 2 solutions. You may migrate to Sdk 16 or 17, where this bug should be fixed, or I could try to implement a workaround in the Android and iOS library, but I don't know if it will work. I would just ignore the returned error assuming all went ok, and continue.

RE: DFU error: OPERATION NOT PERMITTED

mdido — Tue, 23 Jun 2020 14:45:50 GMT

Thanks for the advice. Here is the log:

devzone.nordicsemi.com/.../Log-2020_2D00_06_2D00_23-16_5F00_30_5F00_40.txt

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Tue, 23 Jun 2020 14:27:37 GMT

You may do dfu from nRF Connect.

RE: DFU error: OPERATION NOT PERMITTED

mdido — Tue, 23 Jun 2020 14:27:32 GMT

I have already tried this solution, but without success

RE: DFU error: OPERATION NOT PERMITTED

mdido — Tue, 23 Jun 2020 14:26:40 GMT

I generated the previous log by connecting the device with nrf connect and then I started the dfu with nrf toolbox which returned the error. Please let me know by enabling the log directly from the nrf toolbox

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Tue, 23 Jun 2020 13:31:03 GMT

You may try enabling Packet Receipt Notifications. In nRF Toolbox you may do this in DFU -> Settings. Try PRN=12 and in case of problems, try with value 1.

Have a look at github.com/.../95

RE: DFU error: OPERATION NOT PERMITTED

Aleksander Nowakowski — Tue, 23 Jun 2020 13:23:17 GMT

Could you attach the logs showing this issue? In the attached logs there's no DFU operation.

Thank you