From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752592AbdEEJ5o (ORCPT ); Fri, 5 May 2017 05:57:44 -0400 Received: from verein.lst.de ([213.95.11.211]:54177 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751023AbdEEJ5m (ORCPT ); Fri, 5 May 2017 05:57:42 -0400 Date: Fri, 5 May 2017 11:57:39 +0200 From: Christoph Hellwig To: Amir Goldstein Cc: Andy Shevchenko , Dan Williams , Linux ACPI , tpmdd-devel@lists.sourceforge.net, intel-gfx@lists.freedesktop.org, nouveau@lists.freedesktop.org, linux-input@vger.kernel.org, iommu@lists.linux-foundation.org, linux-mmc@vger.kernel.org, Netdev , linux-pci@vger.kernel.org, USB list , alsa-devel@alsa-project.org, "linux-kernel@vger.kernel.org" , "Rafael J . Wysocki" , Mika Westerberg , Borislav Petkov , Jarkko Sakkinen , Jani Nikula , Ben Skeggs , Benjamin Tissoires , Joerg Roedel , Adrian Hunter , Yisen Zhuang , Bjorn Helgaas , Zhang Rui , Felipe Balbi , Mathias Nyman , Heikki Krogerus , Liam Girdwood , Mark Brown , Christoph Hellwig Subject: Re: [PATCH v1] ACPI: Switch to use generic UUID API Message-ID: <20170505095739.GB6762@lst.de> References: <20170504092151.88646-1-andriy.shevchenko@linux.intel.com> <1493976265.30052.21.camel@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 05, 2017 at 12:50:31PM +0300, Amir Goldstein wrote: > To complete the picture for folks not cc'ed on my patches, > xfs use case suggests there is also justification for the additional helpers: > > uuid_is_null() / uuid_equal() > guid_is_null() / guid_equal() The is_null is useful and I bet we'll find other instances. I'm not sure _equals really adds much value over the existing _cmp helpers, but on the other hand they are so trivial that we might as well add them. The other thing XFS has is uuid_copy.