Page MenuHomeFreeBSD

iscsi: Remove icl_soft-only fields from struct icl_conn.
ClosedPublic

Authored by jhb on Aug 5 2021, 12:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 7:35 AM
Unknown Object (File)
Mar 8 2024, 11:03 AM
Unknown Object (File)
Jan 12 2024, 12:52 AM
Unknown Object (File)
Dec 14 2023, 8:29 PM
Unknown Object (File)
Nov 30 2023, 11:41 PM
Unknown Object (File)
Nov 6 2023, 6:28 PM
Unknown Object (File)
Nov 6 2023, 5:01 AM
Unknown Object (File)
Nov 2 2023, 6:05 AM
Subscribers

Details

Summary

Create a struct icl_soft_conn which extends struct icl_conn and
move fields only used by icl_soft from struct icl_conn to
struct icl_soft_conn.

Sponsored by: Chelsio Communications

Diff Detail

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

Event Timeline

jhb requested review of this revision.Aug 5 2021, 12:05 AM

With few comments it is fine for me. Few more fields could be made driver specific, but they are used at least by soft and cxgbei.

sys/dev/iscsi/icl_soft.c
79

I'd move receive_cv one line up, both alphabetically and by size, unless you had some other idea here.

735

Some disagree, but I personally prefer such assignment done up in definition lines.

This revision is now accepted and ready to land.Aug 5 2021, 12:57 AM
jhb marked 2 inline comments as done.Aug 5 2021, 6:48 PM
jhb added inline comments.
sys/dev/iscsi/icl_soft.c
79

I'd move receive_cv one line up, both alphabetically and by size, unless you had some other idea here.

735

I actually used it a fair bit in other places in this patch. I had only done it here to match existing style. The recent update to style(9) permits it when assigning values that don't change (which is true for both of these and things like softc pointers, etc.).

This revision was automatically updated to reflect the committed changes.
jhb marked 2 inline comments as done.