Hello Sascha,
Am 18.06.20 um 13:59 schrieb Sascha Hauer:
> On Wed, Jun 17, 2020 at 09:32:45PM +0200, Daniel Glöckner wrote:
>>> +static void fastboot_start_download_net(struct fastboot *fb)
>>> +{
>>> + struct fastboot_net *fbn = container_of(fb, struct fastboot_net,
>>> + fastboot);
>>> +
>>> + fastboot_start_download_generic(fb);
>>> + fbn->active_download = true;
>>> +}
>>
>> You didn't base v4 on v3, did you?
>
> No, you sent v3 while I had different changes in my version already.
>
>> If FASTBOOT_INIT is received before active_download is set to true, nobody
>> will close fb->download_fd.
>
> Ok, I'll add a fastboot_abort() function like you did in v3.
What I was referring to are the three additional lines in v3 that call
fastboot_download_finished here instead of setting active_download to true
when reinit is already true.
>>> +static void fastboot_send_keep_alive(struct poller_struct *poller)
>>> +{
>>> + struct fastboot_net *fbn = container_of(poller, struct fastboot_net,
>>> + keep_alive_poller);
>>> +
>>> + if (!fbn->send_keep_alive)
>>> + return;
>>> +
>>> + if (!is_timeout_non_interruptible(fbn->host_waits_since,
>>> + 30ULL * NSEC_PER_SEC))
>>> + return;
>>> +
>>> + if (!fbn->may_send)
>>> + return;
>>> +
>>> + fastboot_tx_print(&fbn->fastboot, FASTBOOT_MSG_INFO, "still busy");
>>> +
>>> + fbn->host_waits_since = get_time_ns();
>>
>> Why do we need this line?
>> host_waits_since is assigned get_time_ns() when may_send is set to true.
>
> It is needed to reinitialize the timeout when we sent the message once.
> Without this we would send the "still busy" message every few ms after
> we've sent it for the first time.
No, because sending a message sets may_send to false and it stays false
until we update host_waits_since in fastboot_handle_type_fastboot.
Best regards,
Daniel
--
Dipl.-Math. Daniel Glöckner, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11,
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke
Ust-IdNr.: DE 205 198 055
emlix - your embedded linux partner
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
