Date: Thu, 20 Jan 2011 17:42:42 +0000 (UTC) From: Pyun YongHyeon <yongari@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/dev/re if_re.c Message-ID: <201101201743.p0KHh1hi004862@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
yongari 2011-01-20 17:42:42 UTC FreeBSD src repository Modified files: (Branch: RELENG_7_4) sys/dev/re if_re.c Log: SVN rev 217648 on 2011-01-20 17:42:42Z by yongari MFC r217296: For re(4) controllers that uses new jumbo frame scheme(RTL8168C/D/E), limit maximum RX buffer size to RE_RX_DESC_BUFLEN instead of blindly configuring it to 16KB. Due to lack of documentation, re(4) didn't allow jumbo frame on these controllers. However it seems controller is confused with jumbo frame such that it can DMA the received frame to wrong address instead of splitting it into multiple RX buffers. Of course, this caused panic. Since re(4) does not support jumbo frames on these controllers, make controller drop frame that is longer than RE_RX_DESC_BUFLEN sized frame. Fortunately RTL810x controllers, which do not support jumbo frame, have no such issues but this change also limited maximum RX buffer size allowed to RTL810x controllers. Allowing 16KB RX buffer for controllers that have no such capability is meaningless. Approved by: re (bz) Revision Changes Path 1.95.2.65.2.2 +7 -2 src/sys/dev/re/if_re.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201101201743.p0KHh1hi004862>