From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 36FFCC3F2CE for ; Mon, 16 Mar 2020 09:56:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 0694120658 for ; Mon, 16 Mar 2020 09:56:56 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=ffwll.ch header.i=@ffwll.ch header.b="MKnh4eqi" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730513AbgCPJ4z (ORCPT ); Mon, 16 Mar 2020 05:56:55 -0400 Received: from mail-wr1-f67.google.com ([209.85.221.67]:43670 "EHLO mail-wr1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1730438AbgCPJ4y (ORCPT ); Mon, 16 Mar 2020 05:56:54 -0400 Received: by mail-wr1-f67.google.com with SMTP id b2so14088827wrj.10 for ; Mon, 16 Mar 2020 02:56:52 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=date:from:to:subject:message-id:mail-followup-to:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=4uFjuIar5NrPLcBZJPObOsb7IgzEi+xWM2W1XFlAhG0=; b=MKnh4eqiajGVxiWBSEZMhJK78EufIWCaAajO2U8EZBruc02ewFnzmSll3FPGSyz4Hk xPq8uLRmHc0q0D4U9xamBCevqwUDOHipFL6D5kF9zcnF8d7pv6w3+C3wGNepqSZVK7sA 1GiLkSHxkP0/BmZATcbxmT7B1ucb7SosFQg/0= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:subject:message-id:mail-followup-to :references:mime-version:content-disposition :content-transfer-encoding:in-reply-to; bh=4uFjuIar5NrPLcBZJPObOsb7IgzEi+xWM2W1XFlAhG0=; b=YE5SWRD0UVhF8Kqi8sVxDmkhFZFuKQbdOCVA7OvmB7AxufO8sEAPeQus4lJ03ovrDs L27CufzGW+dOKlaB6onJh4XA5gX265EV3HRj20yey9ozapfo+2EEvGtMufuZcu1xDPdB vMjma7Ph/Bh9Mt3m163xFI8VHSvUEbAjv/e28BMFByTL/33hl/wWHu7XqCos12wWELji aMjfBn7RhPuXMOW9ByzsDq2ZG5ITzIcE3a+BJR+QOwBRSGp2nsnwETak0xfvw7HRGzaH sIuh+JqqdD1sm/kELJvwhycuYYZLo+mBINTxSYBNjy8Wmcytxi1c72ZFTqpPkqJMFIU0 d1Tw== X-Gm-Message-State: ANhLgQ1RnYksdNr+QH/XeVFIx2U+q0cd8n6eksfdaM7XQYYlCVluLVRR iDPcsQt+6V2+5LKu0tZKS/1nIOEaXaw2DYDi X-Google-Smtp-Source: ADFU+vtMt31tP7+bdrL8Xnvjkbvor50IyCbDn+WnCnsbJXMAyj9Melvbj6Mpc4exxa3r1An2EKb0dg== X-Received: by 2002:a5d:6150:: with SMTP id y16mr35501106wrt.352.1584352611440; Mon, 16 Mar 2020 02:56:51 -0700 (PDT) Received: from phenom.ffwll.local ([2a02:168:57f4:0:efd0:b9e5:5ae6:c2fa]) by smtp.gmail.com with ESMTPSA id i6sm29154761wru.40.2020.03.16.02.56.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Mar 2020 02:56:50 -0700 (PDT) Date: Mon, 16 Mar 2020 10:56:49 +0100 From: Daniel Vetter To: Joe Perches , Christian =?iso-8859-1?Q?K=F6nig?= , Lukas Bulwahn , intel-gfx@lists.freedesktop.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Sebastian Duda , Sumit Semwal , linux-media@vger.kernel.org Subject: Re: [Intel-gfx] [PATCH] MAINTAINERS: adjust to reservation.h renaming Message-ID: <20200316095649.GK2363188@phenom.ffwll.local> Mail-Followup-To: Joe Perches , Christian =?iso-8859-1?Q?K=F6nig?= , Lukas Bulwahn , intel-gfx@lists.freedesktop.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, Sebastian Duda , Sumit Semwal , linux-media@vger.kernel.org References: <20200304120711.12117-1-lukas.bulwahn@gmail.com> <20200306103946.GT2363188@phenom.ffwll.local> <155f99baffe11836fc9d794ff297bdcee7831050.camel@perches.com> <20200316095007.GI2363188@phenom.ffwll.local> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20200316095007.GI2363188@phenom.ffwll.local> X-Operating-System: Linux phenom 5.3.0-3-amd64 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Mar 16, 2020 at 10:50:07AM +0100, Daniel Vetter wrote: > On Fri, Mar 06, 2020 at 02:56:06AM -0800, Joe Perches wrote: > > On Fri, 2020-03-06 at 11:39 +0100, Daniel Vetter wrote: > > > On Wed, Mar 04, 2020 at 01:08:32PM +0100, Christian König wrote: > > > > Am 04.03.20 um 13:07 schrieb Lukas Bulwahn: > > > > > Commit 52791eeec1d9 ("dma-buf: rename reservation_object to dma_resv") > > > > > renamed include/linux/reservation.h to include/linux/dma-resv.h, but > > > > > missed the reference in the MAINTAINERS entry. > > > > > > > > > > Since then, ./scripts/get_maintainer.pl --self-test complains: > > > > > > > > > > warning: no file matches F: include/linux/reservation.h > > > > > > > > > > Adjust the DMA BUFFER SHARING FRAMEWORK entry in MAINTAINERS. > > > > > > > > > > Co-developed-by: Sebastian Duda > > > > > Signed-off-by: Sebastian Duda > > > > > Signed-off-by: Lukas Bulwahn > > > > > > > > Reviewed-by: Christian König > > > > > > You'll push this too? > > > -Daniel > > > > > > > > --- > > > > > Christian, please pick this patch. > > > > > applies cleanly on current master and next-20200303 > > > > > > > > > > MAINTAINERS | 2 +- > > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > > > index 6158a143a13e..3d6cb2789c9e 100644 > > > > > --- a/MAINTAINERS > > > > > +++ b/MAINTAINERS > > > > > @@ -5022,7 +5022,7 @@ L: dri-devel@lists.freedesktop.org > > > > > L: linaro-mm-sig@lists.linaro.org (moderated for non-subscribers) > > > > > F: drivers/dma-buf/ > > > > > F: include/linux/dma-buf* > > > > > -F: include/linux/reservation.h > > > > > +F: include/linux/dma-resv.h > > > > > F: include/linux/*fence.h > > > > > F: Documentation/driver-api/dma-buf.rst > > > > > K: dma_(buf|fence|resv) > > > > Slightly unrelated: > > > > The K: entry matches a lot of other things > > and may have a lot of false positive matches > > like any variable named dma_buffer > > > > This should also use (?:...) to avoid a perl > > capture group. > > > > Perhaps: > > > > K: '\bdma_(?:buf|fence|resv)\b' > > Hm either people aren't using get_maintainers.pl consistently, or it > doesn't seem to be a real world problem. I'm not seeing any unrelated > patches on dri-devel at least. > > But happy to merge such a patch if it shows up ofc, it's definitely the > more correct thing :-) Ofc as usual if you lean out the window you immediately get to eat your hat, right after sending this I got a mail from syzbot about some random stuff because of this :-) I'm gonna do the patch now ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch