Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754174Ab3EMRsY (ORCPT ); Mon, 13 May 2013 13:48:24 -0400 Received: from webmail.solarflare.com ([12.187.104.25]:29930 "EHLO webmail.solarflare.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752197Ab3EMRsW (ORCPT ); Mon, 13 May 2013 13:48:22 -0400 Message-ID: <1368467297.3305.28.camel@bwh-desktop.uk.solarflarecom.com> Subject: [PATCH net] sfc: Reduce RX scatter buffer size to multiple of 256 From: Ben Hutchings To: David Miller , Heiko Carstens , Geert Uytterhoeven CC: , Linux Kernel Development , , Date: Mon, 13 May 2013 18:48:17 +0100 Organization: Solarflare Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3 (3.2.3-3.fc16) Content-Transfer-Encoding: 7bit MIME-Version: 1.0 X-Originating-IP: [10.17.20.137] X-TM-AS-Product-Ver: SMEX-10.0.0.1412-7.000.1014-19862.005 X-TM-AS-Result: No--5.042700-0.000000-31 X-TM-AS-User-Approved-Sender: Yes X-TM-AS-User-Blocked-Sender: No Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2324 Lines: 57 efx_start_datapath() asserts that we can fit 2 RX scatter buffers plus a software structure, each cache-aligned, into a single page. Where L1_CACHE_BYTES == 256 and PAGE_SIZE == 4096, which is the case on s390, this assertion fails. Reduce EFX_RX_USR_BUF_SIZE to make this work. This should also be good for performance, as it ensures that each RX scatter buffer covers whole cache lines and slightly reduces the use of DMA writes that can require a read-modify-write on inter-processor links. (We could use 2048 - L1_CACHE_BYTES, but EFX_RX_USR_BUF_SIZE also affects user-level networking where a larger amount of housekeeping data may be needed. Although this version of the driver does not support user-level networking, I prefer to keep scattering behaviour consistent with the out-of-tree version.) Reported-by: Geert Uytterhoeven Reported-by: Heiko Carstens Signed-off-by: Ben Hutchings --- Heiko or Geert, please confirm that this really does fix the build failure - I don't have an s390 toolchain. Ben. drivers/net/ethernet/sfc/net_driver.h | 6 ++++-- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/sfc/net_driver.h b/drivers/net/ethernet/sfc/net_driver.h index 9bd433a..6c8e1a8 100644 --- a/drivers/net/ethernet/sfc/net_driver.h +++ b/drivers/net/ethernet/sfc/net_driver.h @@ -72,8 +72,10 @@ /* Maximum possible MTU the driver supports */ #define EFX_MAX_MTU (9 * 1024) -/* Size of an RX scatter buffer. Small enough to pack 2 into a 4K page. */ -#define EFX_RX_USR_BUF_SIZE 1824 +/* Size of an RX scatter buffer. Small enough to pack 2 into a 4K page, + * and should be a multiple of the cache line size. + */ +#define EFX_RX_USR_BUF_SIZE (2048 - 256) /* Forward declare Precision Time Protocol (PTP) support structure. */ struct efx_ptp_data; -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/