From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752645AbdFEQXh (ORCPT ); Mon, 5 Jun 2017 12:23:37 -0400 Received: from mga09.intel.com ([134.134.136.24]:39640 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751701AbdFEQXe (ORCPT ); Mon, 5 Jun 2017 12:23:34 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.39,302,1493708400"; d="scan'208";a="864757218" Message-ID: <1496679785.22624.19.camel@linux.intel.com> Subject: Re: [PATCH v2 4/5] ACPI / extlog: Switch to use new generic UUID API From: Andy Shevchenko To: Christoph Hellwig Cc: linux-kernel@vger.kernel.org, "Rafael J . Wysocki" , linux-acpi@vger.kernel.org, Dan Williams , Mika Westerberg , Borislav Petkov , intel-gfx@lists.freedesktop.org Date: Mon, 05 Jun 2017 19:23:05 +0300 In-Reply-To: <20170605160626.GD29981@lst.de> References: <20170531194152.65927-1-andriy.shevchenko@linux.intel.com> <20170531194152.65927-5-andriy.shevchenko@linux.intel.com> <20170605160626.GD29981@lst.de> Organization: Intel Finland Oy Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.22.6-1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2017-06-05 at 18:06 +0200, Christoph Hellwig wrote: > > @@ -165,11 +165,11 @@ static int extlog_print(struct notifier_block > > *nb, unsigned long val, > >   err_seq++; > >   gdata = (struct acpi_hest_generic_data *)(tmp + 1); > >   if (gdata->validation_bits & CPER_SEC_VALID_FRU_ID) > > - fru_id = (uuid_le *)gdata->fru_id; > > + fru_id = (guid_t *)gdata->fru_id; > >   if (gdata->validation_bits & CPER_SEC_VALID_FRU_TEXT) > >   fru_text = gdata->fru_text; > > - sec_type = (uuid_le *)gdata->section_type; > > - if (!uuid_le_cmp(*sec_type, CPER_SEC_PLATFORM_MEM)) { > > + sec_type = (guid_t *)gdata->section_type; > > From a quick look over the tree it seems like both fru_id and > section_type should be declared as guid_t in > struct acpi_hest_generic_data. They are arrays of 16 u8:s. And since it's defined in ACPI table definition I'm not sure we can change it. > > > + if (!guid_equal(sec_type, &CPER_SEC_PLATFORM_MEM)) { > > The "!" seems incorrect here. Right. -- Andy Shevchenko Intel Finland Oy