From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Winkler, Tomas" Subject: Re: [PATCH v1 1/8] lib/uuid: Introduce uuid_{be|le}_cmp_p{p}() helpers Date: Sun, 23 Apr 2017 20:20:33 +0000 Message-ID: <1492975128.3570.2.camel@intel.com> References: <20170421144645.45189-1-andriy.shevchenko@linux.intel.com> <5B8DA87D05A7694D9FA63FD143655C1B543E8399@hasmsx108.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: Content-Language: en-US Content-ID: <5D6A48AB91F1A947BA5214B76B765CAE-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "andy.shevchenko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org" Cc: "kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "alex.williamson-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org" , "Koul, Vinod" , "devel-tBiZLqfeLfOHmIFyCCdPziST3g8Odh+X@public.gmane.org" , "kwankhede-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org" , "kys-0li6OtcxBFHby3iVrkZq2A@public.gmane.org" , "ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" , "matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org" , "broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" , "akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org" , "rjw-LthD3rsA81gm4RdzfppkhA@public.gmane.org" , "sthemmin-0li6OtcxBFHby3iVrkZq2A@public.gmane.org" , "srinivas.pandruvada-VuQAYsv1563Yd54FQh9/CA@public.gmane.org" , "mika.westerberg-VuQAYsv1563Yd54FQh9/CA@public.gmane.org" List-Id: alsa-devel@alsa-project.org T24gU3VuLCAyMDE3LTA0LTIzIGF0IDE1OjQyICswMzAwLCBBbmR5IFNoZXZjaGVua28gd3JvdGU6 DQo+IE9uIFN1biwgQXByIDIzLCAyMDE3IGF0IDE6MjkgUE0sIFdpbmtsZXIsIFRvbWFzIDx0b21h cy53aW5rbGVyQGludGVsLg0KPiBjb20+IHdyb3RlOg0KPiA+ID4gTmV3IGhlbHBlcnMgdGFrZSBw b2ludGVycyB0byB1dWlkX3tiZXxsZX0gYXMgcGFyYW1ldGVycy4NCj4gPiA+IA0KPiA+ID4gV2hl biB1c2luZyB0aGVtIG9uIGEgcmF3IGRhdGEgd2UgZG9uJ3QgbmVlZCB0byBkbyBhbiB1Z2x5DQo+ ID4gPiBkZXJlZmVyZW5jZSBhbmQsDQo+ID4gPiBpbiBzb21lIGNhc2VzLCBhIHR5cGUgY2FzdGlu Zy4NCj4gPiBJIHRoaW5rIHRoaXMgZ29pbmcgb3ZlcmJvYXJkLCB0aGUgX3BwIHR5cGVzwqDCoGFy ZSBqdXN0IGVub3VnaC4NCj4gDQo+IEkgbG9va2VkIGF0IGV4aXN0aW5nIHVzZXJzIGFuZCB0aGVy ZSBhcmUgY2FzZXMgbGlrZQ0KPiAjZGVmaW5lIFhYWF9VVUlEIFVVSURfLi4uKGEsIGIsIGMsIC4u LikNCj4gDQo+IHV1aWRfLipfY21wKHZhbHVlLCBYWFhfVVVJRCkNCj4gDQo+IEZvciBfcHAgdmFy aWFudCBpZiB3b3VsZCBiZSBfY21wX3BwKHZhbHVlLCAmWFhYX1VVSUQpIHdoaWNoIGlzDQo+IHNs aWdodGx5IHdvcnNlIHRoYW4gZm9yIF9wIHZhcmlhbnQuDQoNCg0KTWF5YmUgaXQncyB3b3J0aCB0 byBhY3R1YWxseSByZXBsYWNlIHRoZSBkZWZpbmVzIHdpdGggdmFyaWFibGVzIHRoYW4gdG8NCmNy ZWF0ZSBhbiBpbnRlcmZhY2Ugd2l0aCBhbGwgdGhlIHBlcm11dGF0aW9ucy4NCg0KVG9tYXMNCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Winkler, Tomas" Subject: Re: [PATCH v1 1/8] lib/uuid: Introduce uuid_{be|le}_cmp_p{p}() helpers Date: Sun, 23 Apr 2017 20:20:33 +0000 Message-ID: <1492975128.3570.2.camel@intel.com> References: <20170421144645.45189-1-andriy.shevchenko@linux.intel.com> <5B8DA87D05A7694D9FA63FD143655C1B543E8399@hasmsx108.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: "kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "alex.williamson-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org" , "Koul, Vinod" , "devel-tBiZLqfeLfOHmIFyCCdPziST3g8Odh+X@public.gmane.org" , "kwankhede-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org" , "kys-0li6OtcxBFHby3iVrkZq2A@public.gmane.org" , "ard.biesheuvel-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" , "matt-mF/unelCI9GS6iBeEJttW/XRex20P6io@public.gmane.org" , "broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org" , "akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org" , "rjw-LthD3rsA81gm4RdzfppkhA@public.gmane.org" , "sthemmin-0li6OtcxBFHby3iVrkZq2A@public.gmane.org" , "srinivas.pandruvada-VuQAYsv1563Yd54FQh9/CA@public.gmane.org" , "mika.westerberg-VuQAYsv1563Yd54FQh9/CA@public.gmane.org" Return-path: In-Reply-To: Content-Language: en-US Content-ID: <5D6A48AB91F1A947BA5214B76B765CAE-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: kvm.vger.kernel.org T24gU3VuLCAyMDE3LTA0LTIzIGF0IDE1OjQyICswMzAwLCBBbmR5IFNoZXZjaGVua28gd3JvdGU6 DQo+IE9uIFN1biwgQXByIDIzLCAyMDE3IGF0IDE6MjkgUE0sIFdpbmtsZXIsIFRvbWFzIDx0b21h cy53aW5rbGVyQGludGVsLg0KPiBjb20+IHdyb3RlOg0KPiA+ID4gTmV3IGhlbHBlcnMgdGFrZSBw b2ludGVycyB0byB1dWlkX3tiZXxsZX0gYXMgcGFyYW1ldGVycy4NCj4gPiA+IA0KPiA+ID4gV2hl biB1c2luZyB0aGVtIG9uIGEgcmF3IGRhdGEgd2UgZG9uJ3QgbmVlZCB0byBkbyBhbiB1Z2x5DQo+ ID4gPiBkZXJlZmVyZW5jZSBhbmQsDQo+ID4gPiBpbiBzb21lIGNhc2VzLCBhIHR5cGUgY2FzdGlu Zy4NCj4gPiBJIHRoaW5rIHRoaXMgZ29pbmcgb3ZlcmJvYXJkLCB0aGUgX3BwIHR5cGVzwqDCoGFy ZSBqdXN0IGVub3VnaC4NCj4gDQo+IEkgbG9va2VkIGF0IGV4aXN0aW5nIHVzZXJzIGFuZCB0aGVy ZSBhcmUgY2FzZXMgbGlrZQ0KPiAjZGVmaW5lIFhYWF9VVUlEIFVVSURfLi4uKGEsIGIsIGMsIC4u LikNCj4gDQo+IHV1aWRfLipfY21wKHZhbHVlLCBYWFhfVVVJRCkNCj4gDQo+IEZvciBfcHAgdmFy aWFudCBpZiB3b3VsZCBiZSBfY21wX3BwKHZhbHVlLCAmWFhYX1VVSUQpIHdoaWNoIGlzDQo+IHNs aWdodGx5IHdvcnNlIHRoYW4gZm9yIF9wIHZhcmlhbnQuDQoNCg0KTWF5YmUgaXQncyB3b3J0aCB0 byBhY3R1YWxseSByZXBsYWNlIHRoZSBkZWZpbmVzIHdpdGggdmFyaWFibGVzIHRoYW4gdG8NCmNy ZWF0ZSBhbiBpbnRlcmZhY2Ugd2l0aCBhbGwgdGhlIHBlcm11dGF0aW9ucy4NCg0KVG9tYXMNCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1046019AbdDWUUt (ORCPT ); Sun, 23 Apr 2017 16:20:49 -0400 Received: from mga06.intel.com ([134.134.136.31]:25897 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1045951AbdDWUUi (ORCPT ); Sun, 23 Apr 2017 16:20:38 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.37,241,1488873600"; d="scan'208";a="77912947" From: "Winkler, Tomas" To: "andy.shevchenko@gmail.com" CC: "kvm@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "alex.williamson@redhat.com" , "Koul, Vinod" , "devel@linuxdriverproject.org" , "kwankhede@nvidia.com" , "kys@microsoft.com" , "ard.biesheuvel@linaro.org" , "matt@codeblueprint.co.uk" , "broonie@kernel.org" , "akpm@linux-foundation.org" , "rjw@rjwysocki.net" , "sthemmin@microsoft.com" , "srinivas.pandruvada@linux.intel.com" , "mika.westerberg@linux.intel.com" , "linux-efi@vger.kernel.org" , "lgirdwood@gmail.com" , "linux-input@vger.kernel.org" , "benjamin.tissoires@redhat.com" , "arnd@arndb.de" , "andriy.shevchenko@linux.intel.com" , "alsa-devel@alsa-project.org" , "haiyangz@microsoft.com" , "linux-acpi@vger.kernel.org" Subject: Re: [PATCH v1 1/8] lib/uuid: Introduce uuid_{be|le}_cmp_p{p}() helpers Thread-Topic: [PATCH v1 1/8] lib/uuid: Introduce uuid_{be|le}_cmp_p{p}() helpers Thread-Index: AQHSuq4ryU4x8feEVEm7l2P5RMMZXKHSws4A///ztYCAAG6zAA== Date: Sun, 23 Apr 2017 20:20:33 +0000 Message-ID: <1492975128.3570.2.camel@intel.com> References: <20170421144645.45189-1-andriy.shevchenko@linux.intel.com> <5B8DA87D05A7694D9FA63FD143655C1B543E8399@hasmsx108.ger.corp.intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.12.91.146] Content-Type: text/plain; charset="utf-8" Content-ID: <5D6A48AB91F1A947BA5214B76B765CAE@intel.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id v3NKKujq030914 On Sun, 2017-04-23 at 15:42 +0300, Andy Shevchenko wrote: > On Sun, Apr 23, 2017 at 1:29 PM, Winkler, Tomas com> wrote: > > > New helpers take pointers to uuid_{be|le} as parameters. > > > > > > When using them on a raw data we don't need to do an ugly > > > dereference and, > > > in some cases, a type casting. > > I think this going overboard, the _pp types  are just enough. > > I looked at existing users and there are cases like > #define XXX_UUID UUID_...(a, b, c, ...) > > uuid_.*_cmp(value, XXX_UUID) > > For _pp variant if would be _cmp_pp(value, &XXX_UUID) which is > slightly worse than for _p variant. Maybe it's worth to actually replace the defines with variables than to create an interface with all the permutations. Tomas