Received: by 10.213.65.68 with SMTP id h4csp955220imn; Wed, 14 Mar 2018 05:29:10 -0700 (PDT) X-Google-Smtp-Source: AG47ELuPOTfrEOfINNO58DYrMozJfvS4YsdhGbamlWzLncwD5q//u9ZvyLWc74zm+1rewPUH0p9Y X-Received: by 10.98.79.90 with SMTP id d87mr4128989pfb.41.1521030550713; Wed, 14 Mar 2018 05:29:10 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521030550; cv=none; d=google.com; s=arc-20160816; b=cup9emMoiM4gudhEVnFGCidcrZwCjeYGubgJa4gAoK5zfzGGhQoesjjMq54Gx1pWke MYWmfn7UrkgKgkikDVhxELaumL7TAH5PQwnnlTa8vqOE+fkOgTo2wxa1dRyArEX3LIJY y9C9JnEEsQcqBxg8f7rQRDPsx8GRpFCM/ypxmfMAs77eDVadLsiTBppf+swj0TEHbXba SUK8G2aDG1u4w4zSYtXVUMKxf6nDi2vq/lnlxOC5HncvzQ1FxEx5JIqarJInHHG/DACH XGC/VRFczTp3SF59FDyOlnIY0lN9EL5zAbmtl5gYBzUbERuR1nukC7t7LIgu7iMCB/bq 6sWA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:date:subject:user-agent:message-id :references:cc:in-reply-to:organization:from:to :content-transfer-encoding:mime-version:arc-authentication-results; bh=TXH2wVFdZJxiMJUuaEmIquR5AUVQ1JkXDbjvF9PZApA=; b=L5zjcgsGCMmUsZiwq0XaEXk1YOqeElsMqvKRwRz22I0twW9IqCmGk4keFvESlbtJon +BdoeUAI08/orF+L+UEHW51S0hxUvVPdSNXfoEVShaCRcO5UAHUuP+TXM55it1aWPNFH DNQ7Iw01fTegfh4Cm1NJOAxB4SAVwMs068Z9OaRJbCZdsHoDh6VfQjyq66GysfTcu0GU xyyWz8yXnZXNVb2uFLhgfJBDL+QLZ2NU1PeN/OUROVva2kw7Le1wjSA980FDiORrGVqc PLqnJgU19OIq4CdihVvxdwaei2l+dx/k2uUFoAYKA/Pj/YcpnWQYYFUnP/ug2zMji1I1 lzCw== ARC-Authentication-Results: i=1; mx.google.com; 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 f9-v6si1907473pli.445.2018.03.14.05.28.54; Wed, 14 Mar 2018 05:29:10 -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; 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 S1751694AbeCNM1m convert rfc822-to-8bit (ORCPT + 99 others); Wed, 14 Mar 2018 08:27:42 -0400 Received: from mga07.intel.com ([134.134.136.100]:2108 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750855AbeCNM1l (ORCPT ); Wed, 14 Mar 2018 08:27:41 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 14 Mar 2018 05:27:40 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.47,470,1515484800"; d="scan'208";a="25168499" Received: from jlahtine-desk.ger.corp.intel.com (HELO localhost) ([10.249.35.141]) by orsmga008.jf.intel.com with ESMTP; 14 Mar 2018 05:27:38 -0700 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT To: Salvatore Mesoraca , linux-kernel@vger.kernel.org From: Joonas Lahtinen Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo In-Reply-To: <1520970688-19683-1-git-send-email-s.mesoraca16@gmail.com> Cc: dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, kernel-hardening@lists.openwall.com, David Airlie , Jani Nikula , Kees Cook , Rodrigo Vivi , Salvatore Mesoraca References: <1520970688-19683-1-git-send-email-s.mesoraca16@gmail.com> Message-ID: <152103045697.8738.16803220270496592360@jlahtine-desk.ger.corp.intel.com> User-Agent: alot/0.6 Subject: Re: [PATCH] drm/i915: drop various VLAs in i915_debugfs.c Date: Wed, 14 Mar 2018 14:27:37 +0200 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Salvatore Mesoraca (2018-03-13 21:51:28) > Avoid 3 VLAs[1] by using real constant expressions instead of variables. > The compiler should be able to optimize the original code and avoid using > any actual VLAs. Anyway this change is useful because it will avoid a false > positives with -Wvla, it might also help the compiler generating better > code. > > [1] https://lkml.org/lkml/2018/3/7/621 > > Signed-off-by: Salvatore Mesoraca > --- > drivers/gpu/drm/i915/i915_debugfs.c | 26 ++++++++++++++++---------- > 1 file changed, 16 insertions(+), 10 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c > index e968aea..bf0a8e3 100644 > --- a/drivers/gpu/drm/i915/i915_debugfs.c > +++ b/drivers/gpu/drm/i915/i915_debugfs.c > @@ -4259,19 +4259,20 @@ static ssize_t cur_wm_latency_write(struct file *file, const char __user *ubuf, > i915_cache_sharing_get, i915_cache_sharing_set, > "%llu\n"); > > +#define CHERRYVIEW_SS_MAX 2 CHV_SS_MAX should be good enough. Make these function scoped (so #define at the beginning and #undef at the end of function). Do use ARRAY_SIZE() instead of repeating. Regards, Joonas