From mboxrd@z Thu Jan  1 00:00:00 1970
From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Subject: Re: [RFC PATCH] page_alloc: use first half of higher
 order chunks when halving
Date: Wed, 26 Mar 2014 12:36:09 -0400
Message-ID: <20140326163609.GD21368@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>
	<5332F948.1020909@gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta3.messagelabs.com ([195.245.230.39])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <konrad.wilk@oracle.com>) id 1WSqoS-0001G2-Jf
	for xen-devel@lists.xenproject.org; Wed, 26 Mar 2014 16:36:24 +0000
Content-Disposition: inline
In-Reply-To: <5332F948.1020909@gmail.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Matthew Rushton <mvrushton@gmail.com>
Cc: Keir Fraser <keir@xen.org>, Matt Wilson <msw@amazon.com>, Matt Wilson <msw@linux.com>, Tim Deegan <tim@xen.org>, Jan Beulich <jbeulich@suse.com>, Andrew Cooper <andrew.cooper3@citrix.com>, xen-devel@lists.xenproject.org
List-Id: xen-devel@lists.xenproject.org

On Wed, Mar 26, 2014 at 08:59:04AM -0700, Matthew Rushton wrote:
> On 03/26/14 08:15, 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.

Large size like larger than 512kB? That would also cause problems
on baremetal then when swiotlb is activated I believe.

> >
> >--msw
> 
> It's the dma streaming api I've noticed the problem with, so
> dma_map_single(). Applicable swiotlb code would be
> xen_swiotlb_map_page() and range_straddles_page_boundary(). So yes
> for larger buffers it can cause bouncing.