This simple change removes a level of nesting in the ARP code. It was made mostly to test a Git configuration change, but is a reasonable change in its own right.
Change-Id: I55aed697c3b7e4ba589260ed14002dd3d44028bf Signed-off-by: Dan Cross <[email protected]> --- kern/src/net/arp.c | 23 +++++++++++------------ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/kern/src/net/arp.c b/kern/src/net/arp.c index c821abd..f0add94 100644 --- a/kern/src/net/arp.c +++ b/kern/src/net/arp.c @@ -121,18 +121,17 @@ static struct arpent *newarp6(struct arp *arp, uint8_t * ip, struct Ipifc *ifc, freeblist(xp); xp = next; } - } else { // queue icmp unreachable for rxmitproc later on, w/o arp lock - if (xp) { - if (arp->dropl == NULL) - arp->dropf = xp; - else - arp->dropl->list = xp; - - for (next = xp->list; next; next = next->list) - xp = next; - arp->dropl = xp; - rendez_wakeup(&arp->rxmtq); - } + } else if (xp) { + /* queue icmp unreachable for rxmitproc later, w/o arp lock */ + if (arp->dropl == NULL) + arp->dropf = xp; + else + arp->dropl->list = xp; + + for (next = xp->list; next; next = next->list) + xp = next; + arp->dropl = xp; + rendez_wakeup(&arp->rxmtq); } /* take out of current chain */ -- 2.8.0.rc3.226.g39d4020 -- You received this message because you are subscribed to the Google Groups "Akaros" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. For more options, visit https://groups.google.com/d/optout.
