Received: by 2002:ad5:474a:0:0:0:0:0 with SMTP id i10csp2045284imu; Wed, 21 Nov 2018 06:02:18 -0800 (PST) X-Google-Smtp-Source: AFSGD/U4MIelcsTbidUXx1G0pkuiYWYKQ6SVO78cC3XMxTMFmIhPehMCe1bVqrxkXoaAAPwa9fBp X-Received: by 2002:a63:6b08:: with SMTP id g8mr6014279pgc.119.1542808937920; Wed, 21 Nov 2018 06:02:17 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1542808937; cv=none; d=google.com; s=arc-20160816; b=HxsFPUaK6bEozC2JgkHfMVy9r6XROHI21vcET0W1AbiETMkLfGWHcPnt7weJ5MsaCH ohs0SbMoRGUXa9peu8OfJyv1WqXBrQAkwc+iask/f9iiHUmiT+rbJziaGh6tYmgStonE dDgk2kEaBBhIwdgCjaRdxgBxxR4pOogl/IHSjiJtcVbZeiqHzncrDbNZYZCwsvFKWqcq K8qrZUvEDRTDjwf0sZfJZXfwvpsyiQTHU/qf9qmum+6jlp75R4Iov9Bl1jZjP/xxFe6U Oln14jb2ekW4nQkqHwGenVnW4jIgT0knEpF1h5ufqpbjqdLWJv7sdc5gmkLSiRmLSGhP 9WLw== 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; bh=Fg2AX1LsbzAAuoLp5DBUlVS6UoFph0SjoUg3TJ057mk=; b=OMIC5ou2s0Zq4u0l2nALW5Ox33pPJUKQKQ9jLBXuNJE6V32cni3rLka0QUlUHFqt8d +7f4afRtDpkxM9N8PMNI/byUxzzwQGehds7gpvWPVDEyjrNB1OMguNaNGtkN57118gfn WNVqTmD5PfbSE3ek/Xnt/vsKinpHx4+7SQ57IjE0vGuEMExvL0PQoEwlmfvi//RB46Wk OEGHG6/u1OuYXYf6AjLCkgrtW42dIv2tHQNZwd+ToJ0+V30rd/cI5Q/Ifjnwkmrncel8 Al2JctSI4GOysgf5zW5bAdWTfSb4hpMs8JNr5qS+m5FYBhX/tmiPHVf5iuYFl/8BLFGU v28A== 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id t12si10209197plr.311.2018.11.21.06.01.48; Wed, 21 Nov 2018 06:02:17 -0800 (PST) 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; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730456AbeKUXLz convert rfc822-to-8bit (ORCPT + 99 others); Wed, 21 Nov 2018 18:11:55 -0500 Received: from mga01.intel.com ([192.55.52.88]:16645 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729389AbeKUXLz (ORCPT ); Wed, 21 Nov 2018 18:11:55 -0500 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 21 Nov 2018 04:37:42 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.56,261,1539673200"; d="scan'208";a="109992510" Received: from jlahtine-desk.ger.corp.intel.com (HELO localhost) ([10.251.84.195]) by fmsmga001.fm.intel.com with ESMTP; 21 Nov 2018 04:37:40 -0800 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT To: Hans Holmberg From: Joonas Lahtinen Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo In-Reply-To: Cc: David Airlie , Jani Nikula , Rodrigo Vivi , intel-gfx@lists.freedesktop.org, Linux Kernel Mailing List , Hans Holmberg References: <20181121095423.20760-1-hans.ml.holmberg@owltronix.com> <154279503602.15074.8041259573781223214@jlahtine-desk.ger.corp.intel.com> Message-ID: <154280385988.24009.6122812249809585812@jlahtine-desk.ger.corp.intel.com> User-Agent: alot/0.6 Subject: Re: [PATCH] drm/i915: avoid rebuilding i915_gpu_error.o on version string updates Date: Wed, 21 Nov 2018 14:37:40 +0200 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Quoting Hans Holmberg (2018-11-21 13:35:19) > On Wed, Nov 21, 2018 at 11:10 AM Joonas Lahtinen > wrote: > > > > Quoting Hans Holmberg (2018-11-21 11:54:23) > > > From: Hans Holmberg > > > > > > There is no need to rebuild i915_gpu_error.o when the version string > > > changes as the version is available in init_utsname()->release. > > > > > > Signed-off-by: Hans Holmberg > > > > Seems reasonable to me. > > > > Reviewed-by: Joonas Lahtinen > > > > Out of curiosity, are you by any chance hashing the i915_gpu_error.o > > file (or the contents elsewhere) for some purpose? > > Oh no, I was just moderately annoyed by the file being rebuilt every > time the version was updated(I use my current branch name as > LOCALVERSION when building). That's a reasonable explanation, too :) I unblocked the message from moderation queue so that our CI picks this up for testing. I will then proceed to merge this once the results are back. Thanks for the patch! Regards, Joonas > > Thanks, > Hans > > > > Regards, Joonas > > > > > --- > > > drivers/gpu/drm/i915/i915_gpu_error.c | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c > > > index 8762d17b6659..958e1484a3dd 100644 > > > --- a/drivers/gpu/drm/i915/i915_gpu_error.c > > > +++ b/drivers/gpu/drm/i915/i915_gpu_error.c > > > @@ -27,7 +27,7 @@ > > > * > > > */ > > > > > > -#include > > > +#include > > > #include > > > #include > > > #include > > > @@ -650,7 +650,7 @@ int i915_error_state_to_str(struct drm_i915_error_state_buf *m, > > > > > > if (*error->error_msg) > > > err_printf(m, "%s\n", error->error_msg); > > > - err_printf(m, "Kernel: " UTS_RELEASE "\n"); > > > + err_printf(m, "Kernel: %s\n", init_utsname()->release); > > > ts = ktime_to_timespec64(error->time); > > > err_printf(m, "Time: %lld s %ld us\n", > > > (s64)ts.tv_sec, ts.tv_nsec / NSEC_PER_USEC); > > > -- > > > 2.17.1 > > >