From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [RFC PATCH] page_alloc: use first half of higher order chunks when halving Date: Wed, 26 Mar 2014 12:34:42 -0400 Message-ID: <20140326163442.GC21368@phenom.dumpdata.com> References: <1395746524-9670-1-git-send-email-msw@linux.com> <20140325121922.GB12931@deinos.phlegethon.org> <20140325132740.GB11708@u109add4315675089e695.ant.amazon.com> <5331E269.9090708@gmail.com> <20140326095533.GA7885@deinos.phlegethon.org> <20140326101746.GA14195@u109add4315675089e695.ant.amazon.com> <20140326150801.GD18387@phenom.dumpdata.com> <20140326151507.GF14195@u109add4315675089e695.ant.amazon.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1WSqn1-00016m-QU for xen-devel@lists.xenproject.org; Wed, 26 Mar 2014 16:34:56 +0000 Content-Disposition: inline In-Reply-To: <20140326151507.GF14195@u109add4315675089e695.ant.amazon.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Matt Wilson Cc: Keir Fraser , Matt Wilson , Matthew Rushton , Andrew Cooper , Tim Deegan , Jan Beulich , xen-devel@lists.xenproject.org List-Id: xen-devel@lists.xenproject.org On Wed, Mar 26, 2014 at 05:15:08PM +0200, Matt Wilson wrote: > On Wed, Mar 26, 2014 at 11:08:01AM -0400, Konrad Rzeszutek Wilk wrote: > > > > Could you elaborate a bit more on the use-case please? > > My understanding is that most drivers use a scatter gather list - in which > > case it does not matter if the underlaying MFNs in the PFNs spare are > > not contingous. > > > > But I presume the issue you are hitting is with drivers doing dma_map_page > > and the page is not 4KB but rather large (compound page). Is that the > > problem you have observed? > > Drivers are using very large size arguments to dma_alloc_coherent() > for things like RX and TX descriptor rings. OK, but that call ends up using chunks from the SWIOTLB buffer which is contingously allocated. That shouldn't be a problem? > > --msw