cxgbei: Rework the pdu_append_data hook to support M_WAITOK.
ClosedPublic
Actions

Authored by jhb on Apr 21 2021, 11:59 PM.

Details

Reviewers

Commits

rG19224c9f8477: cxgbei: Rework the pdu_append_data hook to support M_WAITOK.
rG31df8ff73e3a: cxgbei: Rework the pdu_append_data hook to support M_WAITOK.

Summary

Only allocate 16K jumbo mbufs if the region of data to be appended is sufficiently large, and use a loop.

Use m_getm2() to allocate a chain for data less than 16K, or if m_getjcl() fails.

Use ENOMEM as the return value instead of '1' if the hook fails due to a memory allocation error.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 38777
Build 35666: arc lint + arc unit

Event Timeline

jhb created this revision.Apr 21 2021, 11:59 PM

Herald added subscribers: ae, imp. · View Herald TranscriptApr 21 2021, 11:59 PM

jhb requested review of this revision.Apr 21 2021, 11:59 PM

Harbormaster completed remote builds in B38777: Diff 87914.Apr 21 2021, 11:59 PM

jhb added a parent revision: D29908: cxgbei: Support DDP for target I/O S/G lists with more than one entry..Apr 21 2021, 11:59 PM

jhb added a child revision: D29910: cxgbei: Add support for zero-copy iSCSI target transmission/read..

This revision was not accepted when it landed; it landed in state Needs Review.May 14 2021, 7:22 PM

Closed by commit rG31df8ff73e3a: cxgbei: Rework the pdu_append_data hook to support M_WAITOK. (authored by jhb). · Explain Why

This revision was automatically updated to reflect the committed changes.

jhb added a commit: rG31df8ff73e3a: cxgbei: Rework the pdu_append_data hook to support M_WAITOK..

jhb added a commit: rG19224c9f8477: cxgbei: Rework the pdu_append_data hook to support M_WAITOK..Oct 30 2021, 12:00 AM

Revision Contents
Changeset List

Path

Size

sys/

dev/

cxgbe/

cxgbei/

icl_cxgbei.c

67 lines

Diff 87914

View Options

sys/dev/cxgbe/cxgbei/icl_cxgbei.c

cxgbei: Rework the pdu_append_data hook to support M_WAITOK.ClosedPublicActions