From owner-svn-src-head@freebsd.org Mon Nov 2 16:41:04 2015 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 12CEBA2445B for ; Mon, 2 Nov 2015 16:41:04 +0000 (UTC) (envelope-from oliver.pinter@hardenedbsd.org) Received: from mail-wm0-x230.google.com (mail-wm0-x230.google.com [IPv6:2a00:1450:400c:c09::230]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 906F51244 for ; Mon, 2 Nov 2015 16:41:03 +0000 (UTC) (envelope-from oliver.pinter@hardenedbsd.org) Received: by wmeg8 with SMTP id g8so65641370wme.1 for ; Mon, 02 Nov 2015 08:41:02 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hardenedbsd_org.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; bh=3AkJK1BPaR0+RpG4zpwoR3GVJFzVwX6r8pcMwlRZJB8=; b=Vp2urvaoX4i9u0j29ss0X7rv8X3AebJ9l0wNm8QeEOBZ4QUc1QMkkes3xaZw1IIPjY gyAXpU+CaU89Qkll36xODaSTPfz0R86k5fS6w7s4HkV7T+h8PFBJZnzf6Z5zkAH7vzrJ AWPju4E2yxVr5HY7pNwiIFlmqtPn45Rc1iiquzUMjZ9tys35voqTVrskw9NZHG5WWas3 Iyic9oAwk6DwU0n2vpLOTacjU7cCgDfK2spxUM1Y+PsRnQsnTqPd3HZfDzg7/ThHRi+J 6LVL6mQl2hJaLvhm+1almZvBzFDetNJsW+TgdtsgGi00wTc6UspPrGTcK6YBioOpNdtv eXtA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=3AkJK1BPaR0+RpG4zpwoR3GVJFzVwX6r8pcMwlRZJB8=; b=RnfP8I2p6KoZRC6vCjSYyYkD19iPex1Fc5kZ38BaIZyh2c70sS6KulUbuL6NcnOous ZZ8IY7gRk3sRM1NoZy9Yz9bJvpXmWUKve4oqfcpooL60x+V/xVBhJZZlqxr0QP8Aj5aT dpjZU7tX7sg/bZkAFMcKz4Ct4hXXWQSfXpOdIqFVZS3wEVGdoJcK/HlqNFIuaLGwMFrm 8k65CUvOHd50XH0jBUxee119l2mJkiWNsCVJzCHn82LOd9eHba0PPiIAHvqhikWfMANK XZFFIXH6q+QtnN/ncLBEdmn0Z8RYqNSM/Y5J7uSAOflnfb5EHX7ipPZwkcC88Ua+zLJe CXtg== X-Gm-Message-State: ALoCoQnxgrOZL1X+lY47KtEbS5Jzgw49zqgS3DIdYF99iRE/TwVggzAe+/RnHZnFlwCBkreFysAX MIME-Version: 1.0 X-Received: by 10.28.93.145 with SMTP id r139mr13879689wmb.20.1446482461988; Mon, 02 Nov 2015 08:41:01 -0800 (PST) Received: by 10.194.240.226 with HTTP; Mon, 2 Nov 2015 08:41:01 -0800 (PST) In-Reply-To: References: <201510231539.t9NFdxPe018130@repo.freebsd.org> <562A564D.9080703@citrix.com> <562E1F8F.3010304@FreeBSD.org> <5632447B.1020402@FreeBSD.org> <56376567.9090100@FreeBSD.org> Date: Mon, 2 Nov 2015 17:41:01 +0100 Message-ID: Subject: Re: svn commit: r289834 - head/sys/x86/x86 From: Oliver Pinter To: =?UTF-8?Q?Roger_Pau_Monn=C3=A9?= Cc: Adrian Chadd , "svn-src-head@freebsd.org" , "svn-src-all@freebsd.org" , "src-committers@freebsd.org" Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 02 Nov 2015 16:41:04 -0000 On Mon, Nov 2, 2015 at 4:06 PM, Oliver Pinter wrote: > On Mon, Nov 2, 2015 at 2:44 PM, Oliver Pinter > wrote: >> On 11/2/15, Roger Pau Monn=C3=A9 wrote: >>> El 29/10/15 a les 22.48, Oliver Pinter ha escrit: >>>> Yes, at next week I have time for the test. Please ping me again at >>>> Monday. >>> >>> Ping :). >> >> Pong. :) I now stared a new build with the patch, so I will report >> back 1-2 hours from now. > > It's better. :) > > root@nyi-01 src# git show | head -30 > commit 8f6110bd2e258b3f4224eb580e99fa142bebc727 > Author: Roger Pau Monne > Date: Thu Oct 29 12:58:23 2015 +0100 > > x86/dma_bounce: rework _bus_dmamap_load_ma implementation > > The implementation of bus_dmamap_load_ma_triv currently calls > _bus_dmamap_load_phys on each page that is part of the passed in buff= er. > Since each page is treated as an individual buffer, the resulting beh= aviour > is different from the behaviour of _bus_dmamap_load_buffer. This brea= ks > certain drivers, like Xen blkfront. > > If an unmapped buffer of size 4096 that starts at offset 13 into the = first > page is passed to the current _bus_dmamap_load_ma implementation (so = the ma > array contains two pages), the result is that two segments are create= d, one > with a size of 4083 and the other with size 13 (because two independa= nt > calls to _bus_dmamap_load_phys are performed, one for each physical p= age). > If the same is done with a mapped buffer and calling _bus_dmamap_load= _buffer > the result is that only one segment is created, with a size of 4096. > > This patch relegates the usage of bus_dmamap_load_ma_triv in x86 boun= ce > buffer code to drivers requesting BUS_DMA_KEEP_PG_OFFSET and implemen= ts > _bus_dmamap_load_ma so that it's behaviour is the same as the mapped = version > (_bus_dmamap_load_buffer). This patch only modifies the x86 bounce bu= ffer > code, other arches are left untouched. > > Reviewed by: kib, jah > Differential Revision: https://reviews.freebsd.org/D888 > Sponsored by: Citrix Systems R&D > > root@nyi-01 src# uname -a > FreeBSD nyi-01.build.hardenedbsd.org 11.0-CURRENT-HBSD FreeBSD > 11.0-CURRENT-HBSD #14 8f6110b(jenkins-world-kernel-20151102): Mon Nov > 2 09:42:26 EST 2015 > root@nyi-01.build.hardenedbsd.org:/usr/obj/usr/src/sys/JENKINS amd64 > root@nyi-01 src# uptime > 10:05AM up 3 mins, 2 users, load averages: 0.46, 1.09, 0.57 > root@nyi-01 src# > > Now I started a new jenkins build to see the patch under stress. Okay, the server survived a full buildworld + installer generation with the new patch. > >>> >>>