Date: Fri, 28 Aug 2015 01:32:42 +0000 (UTC) From: Pyun YongHyeon <yongari@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r287238 - head/sys/dev/msk Message-ID: <201508280132.t7S1Wg7n059051@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: yongari Date: Fri Aug 28 01:32:42 2015 New Revision: 287238 URL: https://svnweb.freebsd.org/changeset/base/287238 Log: Set DMA alignment constraint of status, TX and RX LEs(List Elements in Marvell terms) to 32768. 32768 looks overkill but it will ensure correct DMAed update. This change addresses occasional watchdog timeouts reported on 10.2-RELEASE. Tested by: Johann Hugo <jhugo@meraka.csir.co.za> MFC after: 2 weeks Modified: head/sys/dev/msk/if_mskreg.h Modified: head/sys/dev/msk/if_mskreg.h ============================================================================== --- head/sys/dev/msk/if_mskreg.h Fri Aug 28 00:49:30 2015 (r287237) +++ head/sys/dev/msk/if_mskreg.h Fri Aug 28 01:32:42 2015 (r287238) @@ -2175,13 +2175,8 @@ #define MSK_ADDR_LO(x) ((uint64_t) (x) & 0xffffffffUL) #define MSK_ADDR_HI(x) ((uint64_t) (x) >> 32) -/* - * At first I guessed 8 bytes, the size of a single descriptor, would be - * required alignment constraints. But, it seems that Yukon II have 4096 - * bytes boundary alignment constraints. - */ -#define MSK_RING_ALIGN 4096 -#define MSK_STAT_ALIGN 4096 +#define MSK_RING_ALIGN 32768 +#define MSK_STAT_ALIGN 32768 /* Rx descriptor data structure */ struct msk_rx_desc {
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201508280132.t7S1Wg7n059051>