Page MenuHomeFreeBSD
Differential D42253

lltable: fix ddb show llentry l3_addr pretty printer
ClosedPublic
Actions

Authored by rcm on Oct 17 2023, 2:12 PM.
Tags
None
Referenced Files
F131795375: D42253.diff
Sat, Oct 11, 5:42 AM
Unknown Object (File)
Wed, Oct 1, 5:18 AM
Unknown Object (File)
Sun, Sep 28, 1:55 AM
Unknown Object (File)
Mon, Sep 22, 5:05 PM
Unknown Object (File)
Sun, Sep 14, 1:56 AM
Unknown Object (File)
Sep 5 2025, 9:37 AM
Unknown Object (File)
Sep 5 2025, 8:01 AM
Unknown Object (File)
Aug 25 2025, 2:09 PM
View All Files
Subscribers
ae
glebius
imp
pfsense

Details

Reviewers
kp
melifaro
Group Reviewers
network
Commits
rG6e281255ea35: lltable: fix ddb show llentry l3_addr pretty printer
Summary

The ddb commands for lltable do not produce useful l3_addr information.

This fixes the llentry pretty printer to correctly display the l3_addr

Sponsored by: Rubicon Communications, LLC ("Netgate")

Test Plan

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

rcm created this revision.Oct 17 2023, 2:12 PM
Herald added subscribers: glebius, ae, imp. · View Herald TranscriptOct 17 2023, 2:12 PM
rcm requested review of this revision.Oct 17 2023, 2:12 PM
kp added inline comments.Oct 17 2023, 2:44 PM
sys/net/if_llatbl.c
1095–1099

It might be worth NULL-checking lle->lle_tbl here first.

rcm updated this revision to Diff 128926.Oct 17 2023, 2:59 PM

Perform NULL-check on lle->lle_tbl

rcm updated this revision to Diff 128929.Oct 17 2023, 2:59 PM
rcm marked an inline comment as done.

typo

rcm updated this revision to Diff 128933.Oct 17 2023, 3:10 PM

Add missing \n on the l3_addr format string

kp accepted this revision as: kp.Oct 17 2023, 3:52 PM

LGTM.

I'll give melifaro@ a little more time to comment before I push this.

This revision is now accepted and ready to land.Oct 17 2023, 3:52 PM
Closed by commit rG6e281255ea35: lltable: fix ddb show llentry l3_addr pretty printer (authored by rcm, committed by kp). · Explain WhyOct 17 2023, 7:16 PM
This revision was automatically updated to reflect the committed changes.
kp added a commit: rG6e281255ea35: lltable: fix ddb show llentry l3_addr pretty printer.

Revision Contents
Changeset List

PathSize
sys/
net/
39 lines

Diff 128933

View Options

sys/net/if_llatbl.c

Loading...