Page MenuHomeFreeBSD
Differential D34829

i40e_validate_filter_settings: Remove unused variables.
ClosedPublic
Actions

Authored by jhb on Apr 7 2022, 6:02 PM.
Tags
None
Referenced Files
F163460394: D34829.diff
Thu, Jul 23, 9:23 AM
Unknown Object (File)
Wed, Jul 22, 2:23 AM
Unknown Object (File)
Tue, Jul 21, 12:03 PM
Unknown Object (File)
Tue, Jul 21, 8:36 AM
Unknown Object (File)
Sun, Jul 19, 6:16 PM
Unknown Object (File)
Sun, Jul 12, 4:45 PM
Unknown Object (File)
Fri, Jul 10, 11:24 PM
Unknown Object (File)
Jun 17 2026, 12:09 PM
View All Files
Subscribers
ae
imp

Details

Reviewers
erj
kgalazka
Commits
rGc434b26cf3eb: i40e_validate_filter_settings: Remove unused variables.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb created this revision.Apr 7 2022, 6:02 PM
Herald added subscribers: ae, imp. · View Herald TranscriptApr 7 2022, 6:02 PM
jhb requested review of this revision.Apr 7 2022, 6:02 PM
Harbormaster completed remote builds in B45062: Diff 104738.Apr 7 2022, 6:02 PM
erj added a reviewer: kgalazka.Apr 11 2022, 6:39 PM
erj requested changes to this revision.Apr 11 2022, 6:46 PM

Checking internally, it looks like those variables can just be removed. I also don't see anything similar to the GLHMC_FCOEFMAX register in the hardware for the PE filters or contexts.

This revision now requires changes to proceed.Apr 11 2022, 6:46 PM
jhb retitled this revision from ixl: Mark two variables as unused. to i40e_validate_filter_settings: Remove unused variables..Apr 12 2022, 5:21 PM
jhb edited the summary of this revision. (Show Details)
jhb updated this revision to Diff 104923.Apr 12 2022, 5:28 PM
  • Remove the variables instead of marking them as unused.
Harbormaster completed remote builds in B45128: Diff 104923.Apr 12 2022, 5:28 PM
erj accepted this revision.Apr 12 2022, 8:15 PM
This revision is now accepted and ready to land.Apr 12 2022, 8:15 PM
Closed by commit rGc434b26cf3eb: i40e_validate_filter_settings: Remove unused variables. (authored by jhb). · Explain WhyApr 12 2022, 9:53 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rGc434b26cf3eb: i40e_validate_filter_settings: Remove unused variables..

Revision Contents
Changeset List

PathSize
sys/
dev/
ixl/
5 lines

Diff 104934

View Options

sys/dev/ixl/i40e_common.c

Loading...