Differential D34929

cxgbei: Return limits for the specific adapter the socket is on.
ClosedPublic
Actions

Authored by jhb on Apr 15 2022, 5:52 PM.

Tags

None

Referenced Files

	F102836933: D34929.diff
	Sun, Nov 17, 7:45 PM

	Unknown Object (File)
	Mon, Nov 11, 3:01 PM

	Unknown Object (File)
	Sep 23 2024, 5:39 PM

	Unknown Object (File)
	Sep 21 2024, 9:03 PM

	Unknown Object (File)
	Sep 21 2024, 12:30 AM

	Unknown Object (File)
	Sep 20 2024, 10:43 AM

	Unknown Object (File)
	Sep 19 2024, 3:38 PM

	Unknown Object (File)
	Sep 16 2024, 1:16 PM

View All 43 Files

Subscribers

Details

Reviewers

mav
trasz
np

Commits

rG2c273cf4ceb7: cxgbei: Return limits for the specific adapter the socket is on.
rG43d5661a9d0f: cxgbei: Return limits for the specific adapter the socket is on.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 45188
Build 42076: arc lint + arc unit

Event Timeline

jhb created this revision.Apr 15 2022, 5:52 PM

Herald added subscribers: ae, imp. · View Herald TranscriptApr 15 2022, 5:52 PM

jhb requested review of this revision.Apr 15 2022, 5:52 PM

Harbormaster completed remote builds in B45188: Diff 105052.Apr 15 2022, 5:53 PM

jhb added a parent revision: D34928: iscsi: Fetch limits based on a socket rather than assuming global limits..Apr 15 2022, 5:53 PM

mav accepted this revision.Apr 18 2022, 1:43 AM

mav added inline comments.

sys/dev/cxgbe/cxgbei/icl_cxgbei.c
1825	This part looks mostly duplicated from cxgbei_limits().

This revision is now accepted and ready to land.Apr 18 2022, 1:43 AM

jhb added inline comments.Apr 18 2022, 7:50 PM

sys/dev/cxgbe/cxgbei/icl_cxgbei.c
1825	It is, but without the min/max computations. I could perhaps put cxgbei_limits() under #ifdef COMPAT_FREEBSD13 in fact since it should only be used in the compat shim for older binaries.

Closed by commit rG43d5661a9d0f: cxgbei: Return limits for the specific adapter the socket is on. (authored by jhb). · Explain WhyApr 18 2022, 7:54 PM

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG43d5661a9d0f: cxgbei: Return limits for the specific adapter the socket is on..

mav added inline comments.Apr 18 2022, 8:22 PM

sys/dev/cxgbe/cxgbei/icl_cxgbei.c
1825	I don't see a problem from min() in cxgbei_limits(). I was more concerned by it not returning result code, whatever it means. But as you wish...

jhb added a commit: rG2c273cf4ceb7: cxgbei: Return limits for the specific adapter the socket is on..Nov 10 2022, 9:49 PM

Revision Contents
Changeset List

Path

Size

sys/

dev/

cxgbe/

cxgbei/

62 lines

Diff 105052

sys/dev/cxgbe/cxgbei/icl_cxgbei.c

Loading...