Received: by 10.213.65.68 with SMTP id h4csp949605imn; Wed, 14 Mar 2018 05:18:18 -0700 (PDT) X-Google-Smtp-Source: AG47ELv8KqRmz7B/CQj0mNPvc/LVz+8Yk4FWj2z9KJrcVmmM7CwhH3TXrptFSRPsenp3Wbnb1QPD X-Received: by 10.101.83.194 with SMTP id z2mr3527831pgr.133.1521029898892; Wed, 14 Mar 2018 05:18:18 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521029898; cv=none; d=google.com; s=arc-20160816; b=CCx/k/LyH7nQUpgyzylSglpySFL9k49mBDGP1clTDcEpUIoDoF33fTI1+AvbJ4dXZJ 80iYNmOVPyk9xPTkQR2uMxNC/H9MVxy0pBDtzYxxORv9IDqHMnNp73Ny8WFKEHQgiFIu +byNdupIjv6Eh03ANVmPITmjrLDM3kWKk07D+e5ANCiaWya/tgAJThB9qPfyRcyJdFck jVNPUL+pUN6mbfbe2baY3QLNU93f250pCrW3DWKAXIYwseO4PJMu3bPneC6yb0qx7b9+ XaIS+jTGAw8Weoovx7XMMdoIItgDLLpx3SoKZ+oxWFz6SEcyJ2hXeTeSFCbm5bFnhnDI zmDQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:message-id:date:references :organization:in-reply-to:subject:cc:to:from :arc-authentication-results; bh=UEkHQSzMP8jcqcaF3S5Uj+4eyE2GCaK7QWuiBO4NJHc=; b=eeaTtmVnPnQr+H+0imabw1PilLCbdFm5GCp3dJmo1Y7C84p9bQ4T/QIcOvM1ruuyaC c3xxx6dd23jgumbhzKm4auKX8fvrhT1CfS0buh7KLczWJPb3zw+ibwbqwPnUdif2MnO/ z+Fbiquwq5Ms2iR06xoYJq+XESQrccXnSnDCsMgsXVmSsGqed1jb2BNX2HOk1xOtGqV5 jwwAhk+8ThFvQyfNP+9bra8Ur/X4Vc154oC2Q4s/YpDRZWjnglXVH7RZq8Fo5CFOtOql Rsp3uYZCvJhF8OhFlH9smPPI7ThpMlLxK45R/sR6IRHIgQH3BHCi94/njFrmqNCpUNwj Dl/g== 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 w3-v6si1859212plb.441.2018.03.14.05.17.59; Wed, 14 Mar 2018 05:18:18 -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 S1751959AbeCNMQP (ORCPT + 99 others); Wed, 14 Mar 2018 08:16:15 -0400 Received: from mga14.intel.com ([192.55.52.115]:18585 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751508AbeCNMQO (ORCPT ); Wed, 14 Mar 2018 08:16:14 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 14 Mar 2018 05:16:14 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.47,470,1515484800"; d="scan'208";a="37985401" Received: from jnikula-mobl2.fi.intel.com (HELO localhost) ([10.237.72.62]) by fmsmga001.fm.intel.com with ESMTP; 14 Mar 2018 05:16:11 -0700 From: Jani Nikula To: Salvatore Mesoraca , linux-kernel@vger.kernel.org Cc: dri-devel@lists.freedesktop.org, intel-gfx@lists.freedesktop.org, kernel-hardening@lists.openwall.com, David Airlie , Joonas Lahtinen , Kees Cook , Rodrigo Vivi , Salvatore Mesoraca Subject: Re: [PATCH] drm/i915: drop various VLAs in i915_debugfs.c In-Reply-To: <1520970688-19683-1-git-send-email-s.mesoraca16@gmail.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <1520970688-19683-1-git-send-email-s.mesoraca16@gmail.com> Date: Wed, 14 Mar 2018 14:17:07 +0200 Message-ID: <87fu52n96k.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 13 Mar 2018, Salvatore Mesoraca wrote: > 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. Thanks for your patch. However, Chris beat you to it with: 7aa0b14ede64 ("drm/i915: Remove variable length arrays from sseu debugfs printers") as well as adding -Wvla to our subdir-ccflags-y to prevent more from cropping up: c5c2b11894f4 ("drm/i915: Warn against variable length arrays") BR, Jani. -- Jani Nikula, Intel Open Source Technology Center