Project

General

Profile

Feature #9122

libstand: Consolidate tftp sendrecv into net.c sendrecv

Added by Toomas Soome over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Toomas Soome
Category:
bootloader
Start date:
2018-02-16
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:

Description

Update from FreeBSD: https://svnweb.freebsd.org/base?view=revision&revision=329339

bootp/arp/rarp/rpc all use the sendrecv implementation in net.c. tftp has
its own implementation because it passes an extra parameter into the recv
callback for the received payload type to be held.

These sendrecv implementations are otherwise equivalent, so consolidate
them. The other users of sendrecv won't be using the extra argument to recv,
but this gives us only one place to worry about respecting timeouts and one
consistent timeout behavior.

Tested by: sbruno
Reviewed by: sbruno, tsoome
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D14373

History

#1

Updated by Electric Monk about 1 year ago

  • % Done changed from 90 to 100
  • Status changed from In Progress to Closed

git commit 890c86716e81b6b22639ff07791007b82c26e6c5

commit  890c86716e81b6b22639ff07791007b82c26e6c5
Author: Toomas Soome <tsoome@me.com>
Date:   2018-05-14T17:00:24.000Z

    9122 libstand: Consolidate tftp sendrecv into net.c sendrecv
    Reviewed by: Yuri Pankov <yuripv@yuripv.net>
    Reviewed by: Andy Fiddaman <omnios@citrus-it.co.uk>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF