Received: by 2002:a25:6193:0:0:0:0:0 with SMTP id v141csp233773ybb; Fri, 3 Apr 2020 01:31:15 -0700 (PDT) X-Google-Smtp-Source: APiQypKTXUQAVTGQ/pO3tEhZ02Ej2YWzQ1r3ZwwvpabOsjkZluIeAeNAzcWpdn7S52nSV7Woaw2Q X-Received: by 2002:a54:4181:: with SMTP id 1mr2214103oiy.158.1585902674839; Fri, 03 Apr 2020 01:31:14 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1585902674; cv=none; d=google.com; s=arc-20160816; b=rjVGwGb7XDdgLOAMCzAfAnWGP3EMbDzkzW92y2/sEYagl+4VwT10r8e9RMm8UnfTjn Kzkm1adpLX1pa707H0RDTmAFS5j2SvH3is0M9UY2Fj+lGpp0/K7+9tQT67HVivLqvSZg 3Y6MjuBdwwMzJ9igDpG0DswATSlnm+rd9NEeFaXaMGOo2AeBdHzJ+krncPmdB3d9TT3s I42qrCEIzc+GTPwVMbEybRsVU31hg5DM4GD7knHatESpJA4zpZj0BauxvgZp2dwZ1obr JL5gF9mra+YWDklE2FHirI4vJbdAaDiYeUwEp6K9pg/VMax0fTVIacDIToPQgO6fFU1t yOag== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=02Izn5NIEOgQ17Wwu9B2nDjWYnskhkbWYrLbCws9cys=; b=TyCHnlIt72Onf1KJuJCoX/WOGOYA/HfMjfMdS7As1jBUxeUhdQJ/+r7P0m798aCY6S zQaLLkpW1o53n2gBvY+UlitpdipDD/ugM21WEVTW+inCnp/ebQpUSuJE/Bc6dwbVqYPg e1BAt2j/YSlZ7n+sgWiYOyPqR+UEn3vDIWeZQ0saqdaHI9sCCvaTBJCtBnHi/eaAJhH+ Bd8iwk2mGO1wnY0dXAg1pjNfo970/ox7mv70qBIPBdDVivxXBbewf439RkkmTOQtGOnW 5EDfZ4lXcZMaZ8p5EcgKNgb/V+ZfWGPHCAo75J5+1wKUkJAhXMkIa9zBj1RrwbzmEdFn 7rIA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ffwll.ch header.s=google header.b=AePlV6cv; 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 p204si3507608oib.241.2020.04.03.01.31.02; Fri, 03 Apr 2020 01:31:14 -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=AePlV6cv; 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 S2389388AbgDCI3F (ORCPT + 99 others); Fri, 3 Apr 2020 04:29:05 -0400 Received: from mail-ot1-f67.google.com ([209.85.210.67]:44830 "EHLO mail-ot1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727835AbgDCI3F (ORCPT ); Fri, 3 Apr 2020 04:29:05 -0400 Received: by mail-ot1-f67.google.com with SMTP id a49so6412122otc.11 for ; Fri, 03 Apr 2020 01:29:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ffwll.ch; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=02Izn5NIEOgQ17Wwu9B2nDjWYnskhkbWYrLbCws9cys=; b=AePlV6cvRFaia2Flr22OaglBCSan4Fn1fu1TjdH+PdqjokxAikNigzQZUfp4HAtll6 YGMdl+BcQ1Graq9IqqFvdG6d375dVL46YFsYvHdO39lzISMurfDlqYGtDvDCsUlXkMqv sssHVyogZ9QM32uJngkP6RMbY0sg1Hsp8mtmI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=02Izn5NIEOgQ17Wwu9B2nDjWYnskhkbWYrLbCws9cys=; b=LiLYsy3Zf3xBtg1qZeuZxXRKNF7QdS+P0JnEIj+/XjvLGmlSLAUfVzKG2KI6ZCZkMo uCAkmgS0ilWbRIyvu+2KYVxnZ13YFpqKqQzQTgaWbmZgUA834Jax6mJx5c6HEazXVL9v B9iS2VjiAtmeUR5S4UGqHXiQcSPWT1t2p5MWN+eSXXKHi6fmwqSz2m+rGMoCVqIgqv+u PxJnBcvPMoyTjaDHF6LQz6oGHGdrgdWdwmC8z+STUuHVKuZs6oCcEp5qpz23NRk75by/ heaYFUKrxNElV+uX5NvB8dCbH4vhIRB+6Mb5te013lTMKeF1V8Vzu4ceCsRbgbPU02Vs OjUg== X-Gm-Message-State: AGi0PuYxkznrXsnxZCyYtiQRs/28dSPEFRtE4wLE0vTBpMqrcRsFcSBF 48f3+XH6C46sZ13AQheLsj5M+cBh16JdtxRQpLXCzw== X-Received: by 2002:a4a:c819:: with SMTP id s25mr5852558ooq.6.1585902543776; Fri, 03 Apr 2020 01:29:03 -0700 (PDT) MIME-Version: 1.0 References: <20200402203317.GA34560@ubuntu-m2-xlarge-x86> <20200402215926.30714-1-chris@chris-wilson.co.uk> <20200403013420.GA11516@ubuntu-m2-xlarge-x86> In-Reply-To: <20200403013420.GA11516@ubuntu-m2-xlarge-x86> From: Daniel Vetter Date: Fri, 3 Apr 2020 10:28:52 +0200 Message-ID: Subject: Re: [PATCH] drm/legacy: Fix type for drm_local_map.offset To: Nathan Chancellor Cc: Chris Wilson , Linus Torvalds , Linux Kernel Mailing List , dri-devel Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 3, 2020 at 8:54 AM Nathan Chancellor wrote: > > On Thu, Apr 02, 2020 at 10:59:26PM +0100, Chris Wilson wrote: > > drm_local_map.offset is not only used for resource_size_t but also > > dma_addr_t which may be of different sizes. > > > > Reported-by: Nathan Chancellor > > Fixes: 8e4ff9b56957 ("drm: Remove the dma_alloc_coherent wrapper for internal usage") > > Signed-off-by: Chris Wilson > > Cc: Dave Airlie > > Cc: Nathan Chancellor > > Cc: Linus Torvalds > > --- > > include/drm/drm_legacy.h | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/include/drm/drm_legacy.h b/include/drm/drm_legacy.h > > index dcef3598f49e..aed382c17b26 100644 > > --- a/include/drm/drm_legacy.h > > +++ b/include/drm/drm_legacy.h > > @@ -136,7 +136,7 @@ struct drm_sg_mem { > > * Kernel side of a mapping > > */ > > struct drm_local_map { > > - resource_size_t offset; /**< Requested physical address (0 for SAREA)*/ > > + dma_addr_t offset; /**< Requested physical address (0 for SAREA)*/ > > unsigned long size; /**< Requested physical size (bytes) */ > > enum drm_map_type type; /**< Type of memory to map */ > > enum drm_map_flags flags; /**< Flags */ > > -- > > 2.20.1 > > > > Thanks for the quick fix! > > I ran it through my set of build tests and nothing else seems to have > broken (at least not any more than it already is...). > > Tested-by: Nathan Chancellor # build This works too, missed it when replying to Linus Reviewed-by: Daniel Vetter Linux 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". Maybe in a few years more ... -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch