Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 16 Sep 2015 00:45:49 +0000 (UTC)
From:      Pyun YongHyeon <yongari@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-9@freebsd.org
Subject:   svn commit: r287840 - stable/9/sys/dev/msk
Message-ID:  <201509160045.t8G0jnMs086190@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: yongari
Date: Wed Sep 16 00:45:48 2015
New Revision: 287840
URL: https://svnweb.freebsd.org/changeset/base/287840

Log:
  MFC r287238:
    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.

Modified:
  stable/9/sys/dev/msk/if_mskreg.h
Directory Properties:
  stable/9/sys/   (props changed)
  stable/9/sys/dev/   (props changed)

Modified: stable/9/sys/dev/msk/if_mskreg.h
==============================================================================
--- stable/9/sys/dev/msk/if_mskreg.h	Wed Sep 16 00:25:40 2015	(r287839)
+++ stable/9/sys/dev/msk/if_mskreg.h	Wed Sep 16 00:45:48 2015	(r287840)
@@ -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?201509160045.t8G0jnMs086190>