Paths

Table of Contentst

Differential D22834

Convert routing stats to VNET_PCPUSTAT
ClosedPublic
Actions

Authored by olivier on Dec 16 2019, 5:51 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Feb 13, 7:20 AM

	Unknown Object (File)
	Thu, Feb 13, 3:39 AM

	Unknown Object (File)
	Wed, Feb 12, 2:28 AM

	Unknown Object (File)
	Thu, Feb 6, 5:11 AM

	Unknown Object (File)
	Fri, Jan 31, 1:44 AM

	Unknown Object (File)
	Nov 22 2024, 3:16 PM

	Unknown Object (File)
	Nov 16 2024, 7:18 AM

	Unknown Object (File)
	Nov 14 2024, 8:33 PM

View All 41 Files

Subscribers

Contributor Reviews (src)

Details

Reviewers

glebius
melifaro
ae

Commits

rS355840: Convert routing statistics to VNET_PCPUSTAT.

Summary

Looks like those old stats should be converted to VNET_PCPUSTAT

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

olivier created this revision.Dec 16 2019, 5:51 PM

Herald added subscribers: Contributor Reviews (src), ae, imp. · View Herald TranscriptDec 16 2019, 5:51 PM

Harbormaster completed remote builds in B28177: Diff 65709.Dec 16 2019, 5:51 PM

olivier added reviewers: glebius, melifaro, ae.Dec 16 2019, 5:52 PM

melifaro added inline comments.Dec 16 2019, 6:58 PM

sys/net/route.c
668 ↗	(On Diff #65709)	Nit: previous logic increased that under `if (error == 0)` condition
sys/net/route.h
257 ↗	(On Diff #65709)	Do we need counters that can go backwards ? :-) On a serious note, zero statistics can potentially be useful.

melifaro added inline comments.Dec 16 2019, 7:00 PM

sys/net/route.h
257 ↗	(On Diff #65709)	Sorry, I meant macro, zeroing current statistics counters.

This revision was not accepted when it landed; it landed in state Needs Review.Dec 17 2019, 2:02 AM

Closed by commit rS355840: Convert routing statistics to VNET_PCPUSTAT. (authored by glebius). · Explain Why

This revision was automatically updated to reflect the committed changes.

glebius added a commit: rS355840: Convert routing statistics to VNET_PCPUSTAT..

Revision Contents
Changeset List

Path

Size

head/

sys/

net/

13 lines

24 lines

usr.bin/

netstat/

12 lines

Diff 65737

head/sys/net/route.h

Loading...

head/sys/net/route.c

Loading...

head/usr.bin/netstat/route.c

Loading...