Received: by 2002:a25:6193:0:0:0:0:0 with SMTP id v141csp1366726ybb; Thu, 9 Apr 2020 00:09:08 -0700 (PDT) X-Google-Smtp-Source: APiQypIIQ0dG3PKPE9hh3ypBU8zkFMqNQSBdt3uNT3JkjRPQM+pS3MSdzmBOnXemfDXleSLC2eRb X-Received: by 2002:a9d:17ec:: with SMTP id j99mr8491012otj.213.1586416148013; Thu, 09 Apr 2020 00:09:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1586416148; cv=none; d=google.com; s=arc-20160816; b=Xxc9/cPw1tLlu/uC1jLrkx3JZBGQ1iTQ0B8UsPifMHniVX02LYTUKgwiqwcq+NWuP4 XPkD1Ot2eqQtElGc7Jd+9X3YU5ANAEDiA4uV/4JuUqz6Y4Ff95FEXu15uaEfP0khtlp2 a2wtC0PC1zs945tSdExV1zVTsifAla/ovpjZbYHFrrCTxoJy2eK7x+V/kAMZ3joAHleB GZjpV8yQ1XVUTbssBxTfp71P/UIla5yunmu5ihrgB56q6JbK8SwsrDcNUENvhCCI6ExJ fqP6qKOWIZrqyzPnEmtcJPJHJM4H4aiPsN5OXsvLy0pguk1ROBnk5plQSoxb+iTfeTYr e3yw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:mail-followup-to:message-id:subject:cc:to :from:date:dkim-signature; bh=3fzlk+1a2b3q0EKRRkBiDFhrc/J/PvVywQ93uFwgEgk=; b=VUS6wGHIxsgRUEnvxD1RNIl2w70O/3CrE7uhAwUVOwIBHei1bLQmSfJkW1KjsPP20c 78c3AiYvESa3Vo9znFHDjXTSxGZ8I6Eqn+CLgf9gks6DDgiOEKwD4bnTSIkzEXdem9Kg 2Sh6j3BZa5kLCbX9/c1XR9rb3uKNyeKkwd3y2z02f2w57rqDq7AAKJSFiZKLci06GujV hc6cqG2BlCJByxLq7WDUifPUrC2NYVaVUqmlt0ApbeqvjR07riEIbEc2Ws+dYCTMFTC4 1BFxD2tJ2M2NFuIzKes6Z3uW9/wHbHycD6Ea3t3Xh7nYQN/Y+R9XgKZIPNLO+WI9zmEs Cj+w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ffwll.ch header.s=google header.b=SFCS+REr; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id w79si3413578oif.21.2020.04.09.00.08.52; Thu, 09 Apr 2020 00:09:07 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@ffwll.ch header.s=google header.b=SFCS+REr; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725985AbgDIHHi (ORCPT + 99 others); Thu, 9 Apr 2020 03:07:38 -0400 Received: from mail-wm1-f67.google.com ([209.85.128.67]:39973 "EHLO mail-wm1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725283AbgDIHHh (ORCPT ); Thu, 9 Apr 2020 03:07:37 -0400 Received: by mail-wm1-f67.google.com with SMTP id a81so3010936wmf.5 for ; Thu, 09 Apr 2020 00:07:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=date:from:to:cc:subject:message-id:mail-followup-to:references :mime-version:content-disposition:in-reply-to; bh=3fzlk+1a2b3q0EKRRkBiDFhrc/J/PvVywQ93uFwgEgk=; b=SFCS+RErUyseIyzdWmRg7SzDZMib4Xfg2GXUyA+nAt+SYSGs/E1Ed5M78k0qjfj4uG 1h0es5fAaghIusxx0DGZw84ApN8oVuD8UmwsSjZvtRNYy6+JCL5FSe9W9tsQi+gKmBIs 4g97B8u6ragxgWcFUotn0rOizq64hf2vkgLsA= 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:cc:subject:message-id :mail-followup-to:references:mime-version:content-disposition :in-reply-to; bh=3fzlk+1a2b3q0EKRRkBiDFhrc/J/PvVywQ93uFwgEgk=; b=S4+sUsgj+Ie6dIu/vVLfmH9xDeYNa4aCRInDjvsjU4d6hy3Y6OQhTokcusdBVOujFB dsutcc0lQ2CWckplxMBEetmsSqITVJAtSOuI8lD7/AZPuembG3L2teImLTu1vnLcRH9U d8UENJmt9hB4JVP21WdswqT8vqd6vEMBgvR6lX+CFBNmVwCz7Qrxl7ILDpvIR0Enu69x VsGYsMPWzhyaF8PqX93jXw7WDyFk2gpP2ERUGLEJQowFfPW9AAsvJ9gmLQbjt3uv1oqy gMyKOkvhRw6iiDOA5LURo4vCZKhrsbuSlPCH6PuJqZHfAvbOAn/l9xRRkW1DYBlBg23y d2Rg== X-Gm-Message-State: AGi0PuY84+/JIuMtaJwzSvKsYr8HcDpPwv/pBWRUFi/gARmyCEm1eVVa V0+f/m1Nx+NlnKff+/jnbeszRQ== X-Received: by 2002:a1c:dc8b:: with SMTP id t133mr8610965wmg.117.1586416056950; Thu, 09 Apr 2020 00:07:36 -0700 (PDT) Received: from phenom.ffwll.local ([2a02:168:57f4:0:efd0:b9e5:5ae6:c2fa]) by smtp.gmail.com with ESMTPSA id h6sm2627936wmf.31.2020.04.09.00.07.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Apr 2020 00:07:36 -0700 (PDT) Date: Thu, 9 Apr 2020 09:07:34 +0200 From: Daniel Vetter To: Linus Torvalds Cc: Nathan Chancellor , Chris Wilson , Linux Kernel Mailing List , dri-devel Subject: Re: [PATCH] drm/legacy: Fix type for drm_local_map.offset Message-ID: <20200409070734.GP3456981@phenom.ffwll.local> Mail-Followup-To: Linus Torvalds , Nathan Chancellor , Chris Wilson , Linux Kernel Mailing List , dri-devel References: <20200402203317.GA34560@ubuntu-m2-xlarge-x86> <20200402215926.30714-1-chris@chris-wilson.co.uk> <20200403013420.GA11516@ubuntu-m2-xlarge-x86> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 Fri, Apr 03, 2020 at 07:16:28PM +0200, Daniel Vetter wrote: > On Fri, Apr 3, 2020 at 7:14 PM Linus Torvalds > wrote: > > > > On Fri, Apr 3, 2020 at 1:29 AM Daniel Vetter wrote: > > > > > > > Tested-by: Nathan Chancellor # build > > > > > > This works too, missed it when replying to Linus > > > > > > Reviewed-by: Daniel Vetter > > > > > > Linus I guess this one is better, but like I explained it really > > > doesn't matter what we do with drm legacy code, it's a horror show > > > that should be disabled on all modern distros anyway. We just keep it > > > because of "never break old uapi". > > > > Ok, That patch from Chris looks fine to me too. > > > > dma_addr_t and resource_size_t aren't the same, but at least > > dma_addr_t should always be the bigger one. > > > > And it does look like nothing else ever takes the address of this > > field, so the ones that might want just the resource_size_t part will > > at least have enough bits. > > > > So I think Chris' patch is the way to go. I'm assuming I'll get it > > through the normal drm tree channels, this doesn't sound _so_ urgent > > that I'd need to expedite that patch into my tree and apply it > > directly. > > Ok, sounds good. > > Chris can you pls push this to drm-misc-next-fixes? That should be > enough for the pull request train next week. Ok I applied this now, seems to have fallen through a few cracks. Might only make it after easter :-/ -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch