From mboxrd@z Thu Jan 1 00:00:00 1970 From: Auke Kok Subject: Re: [PATCH 22/26] ixgb: Cache-align all TX components of the adapter struct. Date: Tue, 29 Aug 2006 15:20:44 -0700 Message-ID: <44F4BDBC.6090203@intel.com> References: <20060829164153.6872.1713.stgit@gitlost.site> <20060829164503.6872.93015.stgit@gitlost.site> <200608291933.56002.dada1@cosmosbay.com> <44F4AAC5.5030909@pobox.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Eric Dumazet , netdev@vger.kernel.org, akpm@osdl.org, "Brandeburg, Jesse" , "Kok, Auke" , "Ronciak, John" Return-path: Received: from mga01.intel.com ([192.55.52.88]:24177 "EHLO fmsmga101-1.fm.intel.com") by vger.kernel.org with ESMTP id S965122AbWH2Wk4 (ORCPT ); Tue, 29 Aug 2006 18:40:56 -0400 To: Jeff Garzik In-Reply-To: <44F4AAC5.5030909@pobox.com> Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Jeff Garzik wrote: > Eric Dumazet wrote: >> On Tuesday 29 August 2006 18:45, Kok, Auke wrote: >>> /* TX */ >>> - struct ixgb_desc_ring tx_ring; >>> + struct ixgb_desc_ring tx_ring ____cacheline_aligned; >>> unsigned long timeo_start; >> >> On UP, this would only enlarge the size of structure and might consume >> more cache lines... >> >> I guess ____cacheline_aligned_in_smp is preferred here. >> >> (see include/linux/netdevice.h) changed in our tree to ____cacheline_aligned_in_smp. Thanks, Auke