From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH v1] ACPI: Switch to use generic UUID API Date: Fri, 05 May 2017 12:24:25 +0300 Message-ID: <1493976265.30052.21.camel@linux.intel.com> References: <20170504092151.88646-1-andriy.shevchenko@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Amir Goldstein , Dan Williams Cc: alsa-devel@alsa-project.org, Heikki Krogerus , Liam Girdwood , linux-pci@vger.kernel.org, Jarkko Sakkinen , Adrian Hunter , Benjamin Tissoires , Christoph Hellwig , Joerg Roedel , Linux ACPI , tpmdd-devel@lists.sourceforge.net, Ben Skeggs , nouveau@lists.freedesktop.org, linux-mmc@vger.kernel.org, Zhang Rui , Borislav Petkov , Yisen Zhuang , Mathias Nyman , intel-gfx@lists.freedesktop.org, linux-input@vger.kernel.org, Mark Brown , Bjorn Helgaas , Mika Westerberg , Felipe Balbi , Ne List-Id: alsa-devel@alsa-project.org T24gRnJpLCAyMDE3LTA1LTA1IGF0IDEwOjA2ICswMzAwLCBBbWlyIEdvbGRzdGVpbiB3cm90ZToK PiBPbiBGcmksIE1heSA1LCAyMDE3IGF0IDk6MjAgQU0sIERhbiBXaWxsaWFtcyA8ZGFuLmoud2ls bGlhbXNAaW50ZWwuY29tCj4gPiB3cm90ZToKPiA+IE9uIFRodSwgTWF5IDQsIDIwMTcgYXQgMjoy MSBBTSwgQW5keSBTaGV2Y2hlbmtvCj4gPiA8YW5kcml5LnNoZXZjaGVua29AbGludXguaW50ZWwu Y29tPiB3cm90ZToKCgo+ID4gPiDCoMKgwqDCoMKgwqDCoMKgZm9yIChpID0gMDsgaSA8IE5GSVRf VVVJRF9NQVg7IGkrKykKPiA+ID4gLcKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoGlmICht ZW1jbXAodG9fbmZpdF91dWlkKGkpLCBzcGEtPnJhbmdlX2d1aWQsIDE2KQo+ID4gPiA9PSAwKQo+ ID4gPiArwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgaWYgKCF1dWlkX2xlX2NtcF9wcCh0 b19uZml0X3V1aWQoaSksICh1dWlkX2xlCj4gPiA+ICopc3BhLT5yYW5nZV9ndWlkKSkKPiA+IAo+ ID4gV2hhdCBpcyBfY21wX3BwPyBXaHkgbm90IF9jb21wYXJlPwoKRGFuLCBpdCdzIGEgdHlwby4g SW4gdGhpcyBwYXRjaCBpdCBzaG91bGQgYmUganVzdCAuLi5fY21wKCksIHdoaWNoIGlzCmFscmVh ZHkgYSBwYXJ0IG9mIEFQSS4KCj4gPiAKPiAKPiBJIHNlY29uZCB0aGF0Lgo+IAo+IEFuZHksCgpB bWlyLCBqdXN0IHRvIGJlIGNsZWFyLiBUaGlzIHBhdGNoIGNhbiBiZSBhcHBsaWVkIHdpdGhvdXQg YW55IGFkZG9ucyB0bwphbiBleGlzdGluZyBBUEkuIEFib3ZlIGlzIGp1c3QgYSB0eXBvIGR1ZSB0 byByZWJhc2UgaW4gbXkgdHJlZS4gSSB3aWxsCnJlcGxhY2UgaXQgdG8ganVzdCB1dWlkX2xlX2Nt cCgpLgoKPiBJIG11Y2ggcmF0aGVyIHRoYXQgeW91IHNvcnQgb3V0IHV1aWQgaGVscGVycyBpbiBh IHdheSB0aGF0IHdpbGwKPiBzYXRpc2Z5IHRoZSBmaWxlc3lzdGVtCj4gbmVlZHMgKGp1c3QgcHJv dmlkZSB0aGUgaGVscGVycyBkb24ndCBuZWVkIHRvIGNvbnZlcnQgZmlsZXN5c3RlbXMKPiBjb2Rl KS4KCj4gVGhlIG9ubHkgcmVhc29uIEkgdG9vayBhIHN3aW5nIGF0IGhvaXN0aW5nIHRoZSB4ZnMg dXVpZCBoZWxwZXJzIGlzCj4gYmVjYXVzZSBpdCBkaWRuJ3QKPiBzZWVtIGxpa2UgeW91ciBwcm9w b3NhbCB3YXMgZ29pbmcgdG8gYmUgcG9zdGVkIHNvb24gb3Igd2Fzbid0IGdvaW5nIHRvCj4gc2F0 aXNmeQo+IHRoZSBmaWxlc3lzdGVtcyB1c2UgY2FzZS4KCj4gCj4gTXkgb3BpbmlvbiBub3csIGlz IHRoYXQgeW91ciBzdWdnZXN0aW9uIGlzIHByb2JhYmx5IG11Y2ggY2xvc2VyIHRvIHRoZQo+IHJl YWwgZGVhbAo+IHRoYW4gbWluZS4KPiAKPiBJTU8sIHlvdSBzaG91bGQgYWNrbm93bGVkZ2UgdGhh dCB0aGUgY29tbW9uIHVzZSBjYXNlIGZvciBmaWxlc3lzdGVtcwo+IGlzCj4gdG8gaGFuZGxlIGFu IG9wYXF1ZSBjaGFyWzE2XSB3aGljaCBtb3N0IGxpa2VseSBob2xkcyBhIHV1aWRfYmUgYW5kIHlv dQo+IHNob3VsZCBwcm92aWRlICduZXV0cmFsJyBoZWxwZXJzIHRvIHNhdGlzZnkgdGhpcyB1c2Ug Y2FzZS4KPiAKPiBUaGUgc2ltcGxlc3Qgd291bGQgYmUgdG8gdHlwZWRlZiB1dWlkX3QgdG8gc3Ry dWN0IHV1aWRfYmUgYW5kIHRvIG5hbWUKPiAnbmV1dHJhbCcKPiBoZWxwZXJzJyB1dWlkX2NtcC91 dWlkX2NvcHkodXVpZF90ICosIHV1aWRfdCAqKSwgc2ltaWxhciB0byBteQo+IHByb3Bvc2FsLgoK PiBJIHRoaW5rIHdpdGggdGhpcyBzZW1hbnRpYyBjaGFuZ2UsIG91ciBwcm9wb3NhbHMgY2FuIHJl YWNoIGNvbW1vbgo+IGdyb3VuZHMKPiBhbmQgc2F0aXNmeSBhIHdpZGVyIGdyb3VwIG9mIHVzZXJz IChpLmUuIGZpbGVzeXN0ZW0gZGV2ZWxvcGVycykuCj4gCj4gQ2hyaXN0b3BoIGFsc28gc3VnZ2Vz dGVkIGEgc2ltaWxhciB0cmVhdG1lbnQgdG8gdHlwZWRlZiBndWlkX3QgdG8KPiBzdHJ1Y3QgdXVp ZF9sZS4KPiBJIGRvbid0IGtub3cgdGhlIHVzZSBjYXNlcyBlbm91Z2ggdG8gY29tbWVudCBvbiB0 aGF0LgoKV2UgbWF5IGdvIHRoaXMgd2F5LiBCdXQgSSB3b3VsZG4ndCBwcmV2ZW50IGN1cnJlbnQg dXNlcnMgb2YgdXVpZF9sZSB0bwpjb250aW51ZSB1c2luZyBpdCB3aXRob3V0IGNvbnZlcnNpb24g KGl0IG1heSBiZSBkb25lIGNhc2UgYnkgY2FzZSBhZnRlcgp3ZSBzZXR0bGUgYW4gQVBJKQoKU28s IHN1bW1hcml6ZSB3aGF0IENocmlzdG9waCBzYWlkIGl0IHdpbGwgbG9vayBsaWtlCgp0eXBlZGVm IHV1aWRfYmUgdXVpZF90Owp0eXBlZGVmIHV1aWRfbGUgZ3VpZF90Cgp1dWlkX2NtcCgpIC8gdXVp ZF9jb3B5KCkgLyB1dWlkX3RvX2JpbigpIC8gZXRjCmd1aWRfY21wKCkgLyBndWlkX2NvcHkoKSAv IGd1aWRfdG9fYmluKCkgLyBldGMKCkNvcnJlY3Q/IENocmlzdG9waD8KCi0tIApBbmR5IFNoZXZj aGVua28gPGFuZHJpeS5zaGV2Y2hlbmtvQGxpbnV4LmludGVsLmNvbT4KSW50ZWwgRmlubGFuZCBP eQpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpJbnRlbC1n ZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9s aXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com ([134.134.136.65]:50382 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751116AbdEEJYf (ORCPT ); Fri, 5 May 2017 05:24:35 -0400 Message-ID: <1493976265.30052.21.camel@linux.intel.com> Subject: Re: [PATCH v1] ACPI: Switch to use generic UUID API From: Andy Shevchenko To: Amir Goldstein , Dan Williams Cc: 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 Date: Fri, 05 May 2017 12:24:25 +0300 In-Reply-To: References: <20170504092151.88646-1-andriy.shevchenko@linux.intel.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-pci-owner@vger.kernel.org List-ID: On Fri, 2017-05-05 at 10:06 +0300, Amir Goldstein wrote: > On Fri, May 5, 2017 at 9:20 AM, Dan Williams > wrote: > > On Thu, May 4, 2017 at 2:21 AM, Andy Shevchenko > > wrote: > > >         for (i = 0; i < NFIT_UUID_MAX; i++) > > > -               if (memcmp(to_nfit_uuid(i), spa->range_guid, 16) > > > == 0) > > > +               if (!uuid_le_cmp_pp(to_nfit_uuid(i), (uuid_le > > > *)spa->range_guid)) > > > > What is _cmp_pp? Why not _compare? Dan, it's a typo. In this patch it should be just ..._cmp(), which is already a part of API. > > > > I second that. > > Andy, Amir, just to be clear. This patch can be applied without any addons to an existing API. Above is just a typo due to rebase in my tree. I will replace it to just uuid_le_cmp(). > I much rather that you sort out uuid helpers in a way that will > satisfy the filesystem > needs (just provide the helpers don't need to convert filesystems > code). > The only reason I took a swing at hoisting the xfs uuid helpers is > because it didn't > seem like your proposal was going to be posted soon or wasn't going to > satisfy > the filesystems use case. > > My opinion now, is that your suggestion is probably much closer to the > real deal > than mine. > > IMO, you should acknowledge that the common use case for filesystems > is > to handle an opaque char[16] which most likely holds a uuid_be and you > should provide 'neutral' helpers to satisfy this use case. > > The simplest would be to typedef uuid_t to struct uuid_be and to name > 'neutral' > helpers' uuid_cmp/uuid_copy(uuid_t *, uuid_t *), similar to my > proposal. > I think with this semantic change, our proposals can reach common > grounds > and satisfy a wider group of users (i.e. filesystem developers). > > Christoph also suggested a similar treatment to typedef guid_t to > struct uuid_le. > I don't know the use cases enough to comment on that. We may go this way. But I wouldn't prevent current users of uuid_le to continue using it without conversion (it may be done case by case after we settle an API) So, summarize what Christoph said it will look like typedef uuid_be uuid_t; typedef uuid_le guid_t uuid_cmp() / uuid_copy() / uuid_to_bin() / etc guid_cmp() / guid_copy() / guid_to_bin() / etc Correct? Christoph? -- Andy Shevchenko Intel Finland Oy