From owner-dev-commits-src-all@freebsd.org Sat Sep 25 15:55:40 2021 Return-Path: Delivered-To: dev-commits-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 17E726ABE5A; Sat, 25 Sep 2021 15:55:40 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from smtp.freebsd.org (smtp.freebsd.org [96.47.72.83]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "smtp.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4HGtlv6sf4z4jb8; Sat, 25 Sep 2021 15:55:39 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: from mail-qt1-f171.google.com (mail-qt1-f171.google.com [209.85.160.171]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "GTS CA 1O1" (verified OK)) (Authenticated sender: kevans) by smtp.freebsd.org (Postfix) with ESMTPSA id C4844C4B2; Sat, 25 Sep 2021 15:55:39 +0000 (UTC) (envelope-from kevans@freebsd.org) Received: by mail-qt1-f171.google.com with SMTP id x9so12559630qtv.0; Sat, 25 Sep 2021 08:55:39 -0700 (PDT) X-Gm-Message-State: AOAM530G18uBMwte2ftWlULJsBiLkJKKNJH1uQVO23n+QJQpt8IvkiXs t/dK16TZU50beHy88pCNN3xbdlsK2jMHs2Y4gN8= X-Google-Smtp-Source: ABdhPJx3SA10mseFFhwtF8kQh4FFsDl5AwDi8c0OXocWBh7ZksbWbuowRtyVzlF6L0+NXBdUD9lfTXMZf7rbvGbH2os= X-Received: by 2002:ac8:7384:: with SMTP id t4mr10148360qtp.83.1632585339166; Sat, 25 Sep 2021 08:55:39 -0700 (PDT) MIME-Version: 1.0 References: <202109251311.18PDBVqf077216@gitrepo.freebsd.org> <20210925131801.b4lyozg7b56646x4@mutt-hbsd> In-Reply-To: <20210925131801.b4lyozg7b56646x4@mutt-hbsd> From: Kyle Evans Date: Sat, 25 Sep 2021 10:55:28 -0500 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: git: 24a3897c2c32 - main - x86 bounce_bus_dmamem_alloc(): use malloc_aligned() only when possible To: Shawn Webb Cc: Konstantin Belousov , src-committers , "" , dev-commits-src-main@freebsd.org Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 25 Sep 2021 15:55:40 -0000 On Sat, Sep 25, 2021 at 8:18 AM Shawn Webb wrote: > > On Sat, Sep 25, 2021 at 01:11:31PM +0000, Konstantin Belousov wrote: > > The branch main has been updated by kib: > > > > URL: https://cgit.FreeBSD.org/src/commit/?id=24a3897c2c3205c2ec0cf323c555c403d3171e2c > > > > commit 24a3897c2c3205c2ec0cf323c555c403d3171e2c > > Author: Konstantin Belousov > > AuthorDate: 2021-09-24 17:46:47 +0000 > > Commit: Konstantin Belousov > > CommitDate: 2021-09-25 12:58:12 +0000 > > > > x86 bounce_bus_dmamem_alloc(): use malloc_aligned() only when possible > > > > malloc_domainset_aligned() requires that alignment is less than > > page size. Fall back to other allocation methods, most likely > > kmem_alloc_contig(), when malloc_aligned() cannot fullfill the driver > > request. > > > > Reported by: Loic F > > Reviewed by: markj > > Sponsored by: The FreeBSD Foundation > > MFC after: 1 week > > Differential revision: https://reviews.freebsd.org/D32127 > > Wrong Differential revision URL? The patch in the URL doesn't match > what was committed here. > No; there were two patches included, and Phabricator always updates the diff to the first one that closed it. You can check the history on the diff to find the last one pre-commit.