From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id DE57121A02F31 for ; Mon, 22 May 2017 11:59:29 -0700 (PDT) Message-ID: <1495479565.6967.72.camel@linux.intel.com> Subject: Re: [PATCH 23/23] uuid: remove uuid_be From: Andy Shevchenko Date: Mon, 22 May 2017 21:59:25 +0300 In-Reply-To: <20170518085629.GA6149@lst.de> References: <20170518062705.25902-1-hch@lst.de> <20170518062705.25902-24-hch@lst.de> <20170518085629.GA6149@lst.de> Mime-Version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" To: Christoph Hellwig , Amir Goldstein Cc: linux-xfs , linux-raid@vger.kernel.org, linux-nvdimm@lists.01.org, Shaohua Li , Hugh Dickins , linux-kernel , David Howells , linux-fsdevel , Mimi Zohar , Steven Whitehouse List-ID: T24gVGh1LCAyMDE3LTA1LTE4IGF0IDEwOjU2ICswMjAwLCBDaHJpc3RvcGggSGVsbHdpZyB3cm90 ZToKPiBPbiBUaHUsIE1heSAxOCwgMjAxNyBhdCAxMDo1NzoyNEFNICswMzAwLCBBbWlyIEdvbGRz dGVpbiB3cm90ZToKPiA+IEkgcmV2aWV3ZWQgdGhlIGVudGlyZSBzZXJpZXMuIFlvdSBtYXkgYWRk Cj4gPiBSZXZpZXdlZC1ieTogQW1pciBHb2xkc3RlaW4gPGFtaXI3M2lsQGdtYWlsLmNvbT4KPiA+ IAo+ID4gdG8gYW55IG9mIHRoZSBwYXRjaGVzIGFzIHlvdSBzZWUgZml0Lgo+IAo+IFRoYW5rcywg ZG9uZSEKClNpbWlsYXIgZnJvbSBteSBzaWRlLCBGV0lXOgpSZXZpZXdlZC1ieTogQW5keSBTaGV2 Y2hlbmtvIDxhbmRyaXkuc2hldmNoZW5rb0BsaW51eC5pbnRlbC5jb20+Cgo+IAo+ID4gTm93IGxl dCdzIHRhbGsgYWJvdXQgaG93IHRoaXMgaXMgZ29pbmcgdG8gYmUgbWVyZ2VkLgo+ID4gRG8geW91 IGludGVuZCB0byBzZW5kIExpbnVzIGEgcHVsbCByZXF1ZXN0Pwo+IAo+IFllcywgdGhhdCBtaWdo dCBiZSB0aGUgYmVzdCBpZGVhLsKgwqBJJ20gYWxzbyBjb250ZW1wbGF0aW5nIGxpc3RpbmcKPiBt ZSBwbHVzIGFueW9uZSB2b2x1bnRlZXJpbmcgKHlvdT8sIEFuZHkpIGFzIG1haW50YWluZXMgZm9y IHRoZSB1dWlkCj4gY29kZS4KCkkgd291bGQgYWdyZWUgdG8gYmUgYSBkZXNpZ25hdGVkIHJldmll d2VyIGZvciBub3cgKHRvbyBtYW55IHN0dWZmIHRvCmZvbGxvdyBhcyBhIChjby0pbWFpbnRhaW5l cikuCgo+ID4gVGhlIHJlYXNvbiBJIGFtIGFza2luZyBpcyBiZWNhdXNlIHRoaXMgbGFzdCByZW1v dmFsIHBhdGNoIHNob3VsZAo+ID4gcHJvYmFibHkKPiA+IGJlIGFwcGxpZWQgbmVhciB0aGUgZW5k IG9mIHRoZSBtZXJnZSB3aW5kb3cgKD8pLgo+ID4gQmVjYXVzZSBtYWludGFpbmVycyBjYW5ub3Qg YXBwbHkgcGF0Y2hlcyB3aXRoIGNvZGUgdGhhdCB1c2VzIHRoZSBuZXcKPiA+IHV1aWRfdCB0byBs aW51eC1uZXh0IGJyYW5jaGVzIGFuZCB3ZSBkb24ndCB3YW50IGxpbnV4LW5leHQgYnVpbGQgdG8K PiA+IGZhaWwKPiA+IHdpdGggbmV3IGNvZGUgdGhhdCB1c2VzIHV1aWRfYmUuLi4KPiAKPiBZZWFo LCB3ZSBjYW4gcHJvYmFibHkgZGVmZXIgdGhhdCBvbmUgZm9yIG5vdy4KCkkgd291bGQgYWxzbyBs aWtlIHRvIGFwcGVuZCBzb21lIG9mIHBhdGNoZXMgZnJvbSBteSBzaWRlLCB0aG91Z2ggdGhleQph cmUgV0lQLgoKLS0gCkFuZHkgU2hldmNoZW5rbyA8YW5kcml5LnNoZXZjaGVua29AbGludXguaW50 ZWwuY29tPgpJbnRlbCBGaW5sYW5kIE95Cl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCkxpbnV4LW52ZGltbSBtYWlsaW5nIGxpc3QKTGludXgtbnZkaW1tQGxp c3RzLjAxLm9yZwpodHRwczovL2xpc3RzLjAxLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LW52 ZGltbQo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [PATCH 23/23] uuid: remove uuid_be Date: Mon, 22 May 2017 21:59:25 +0300 Message-ID: <1495479565.6967.72.camel@linux.intel.com> References: <20170518062705.25902-1-hch@lst.de> <20170518062705.25902-24-hch@lst.de> <20170518085629.GA6149@lst.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20170518085629.GA6149-jcswGhMUV9g@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linux-nvdimm-bounces-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org Sender: "Linux-nvdimm" To: Christoph Hellwig , Amir Goldstein Cc: linux-xfs , linux-raid-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-nvdimm-hn68Rpc1hR1g9hUCZPvPmw@public.gmane.org, Shaohua Li , Hugh Dickins , linux-kernel , David Howells , linux-fsdevel , Mimi Zohar , Steven Whitehouse List-Id: linux-raid.ids T24gVGh1LCAyMDE3LTA1LTE4IGF0IDEwOjU2ICswMjAwLCBDaHJpc3RvcGggSGVsbHdpZyB3cm90 ZToKPiBPbiBUaHUsIE1heSAxOCwgMjAxNyBhdCAxMDo1NzoyNEFNICswMzAwLCBBbWlyIEdvbGRz dGVpbiB3cm90ZToKPiA+IEkgcmV2aWV3ZWQgdGhlIGVudGlyZSBzZXJpZXMuIFlvdSBtYXkgYWRk Cj4gPiBSZXZpZXdlZC1ieTogQW1pciBHb2xkc3RlaW4gPGFtaXI3M2lsQGdtYWlsLmNvbT4KPiA+ IAo+ID4gdG8gYW55IG9mIHRoZSBwYXRjaGVzIGFzIHlvdSBzZWUgZml0Lgo+IAo+IFRoYW5rcywg ZG9uZSEKClNpbWlsYXIgZnJvbSBteSBzaWRlLCBGV0lXOgpSZXZpZXdlZC1ieTogQW5keSBTaGV2 Y2hlbmtvIDxhbmRyaXkuc2hldmNoZW5rb0BsaW51eC5pbnRlbC5jb20+Cgo+IAo+ID4gTm93IGxl dCdzIHRhbGsgYWJvdXQgaG93IHRoaXMgaXMgZ29pbmcgdG8gYmUgbWVyZ2VkLgo+ID4gRG8geW91 IGludGVuZCB0byBzZW5kIExpbnVzIGEgcHVsbCByZXF1ZXN0Pwo+IAo+IFllcywgdGhhdCBtaWdo dCBiZSB0aGUgYmVzdCBpZGVhLsKgwqBJJ20gYWxzbyBjb250ZW1wbGF0aW5nIGxpc3RpbmcKPiBt ZSBwbHVzIGFueW9uZSB2b2x1bnRlZXJpbmcgKHlvdT8sIEFuZHkpIGFzIG1haW50YWluZXMgZm9y IHRoZSB1dWlkCj4gY29kZS4KCkkgd291bGQgYWdyZWUgdG8gYmUgYSBkZXNpZ25hdGVkIHJldmll d2VyIGZvciBub3cgKHRvbyBtYW55IHN0dWZmIHRvCmZvbGxvdyBhcyBhIChjby0pbWFpbnRhaW5l cikuCgo+ID4gVGhlIHJlYXNvbiBJIGFtIGFza2luZyBpcyBiZWNhdXNlIHRoaXMgbGFzdCByZW1v dmFsIHBhdGNoIHNob3VsZAo+ID4gcHJvYmFibHkKPiA+IGJlIGFwcGxpZWQgbmVhciB0aGUgZW5k IG9mIHRoZSBtZXJnZSB3aW5kb3cgKD8pLgo+ID4gQmVjYXVzZSBtYWludGFpbmVycyBjYW5ub3Qg YXBwbHkgcGF0Y2hlcyB3aXRoIGNvZGUgdGhhdCB1c2VzIHRoZSBuZXcKPiA+IHV1aWRfdCB0byBs aW51eC1uZXh0IGJyYW5jaGVzIGFuZCB3ZSBkb24ndCB3YW50IGxpbnV4LW5leHQgYnVpbGQgdG8K PiA+IGZhaWwKPiA+IHdpdGggbmV3IGNvZGUgdGhhdCB1c2VzIHV1aWRfYmUuLi4KPiAKPiBZZWFo LCB3ZSBjYW4gcHJvYmFibHkgZGVmZXIgdGhhdCBvbmUgZm9yIG5vdy4KCkkgd291bGQgYWxzbyBs aWtlIHRvIGFwcGVuZCBzb21lIG9mIHBhdGNoZXMgZnJvbSBteSBzaWRlLCB0aG91Z2ggdGhleQph cmUgV0lQLgoKLS0gCkFuZHkgU2hldmNoZW5rbyA8YW5kcml5LnNoZXZjaGVua29AbGludXguaW50 ZWwuY29tPgpJbnRlbCBGaW5sYW5kIE95Cl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fCkxpbnV4LW52ZGltbSBtYWlsaW5nIGxpc3QKTGludXgtbnZkaW1tQGxp c3RzLjAxLm9yZwpodHRwczovL2xpc3RzLjAxLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LW52 ZGltbQo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com ([134.134.136.24]:51761 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760421AbdEVS7b (ORCPT ); Mon, 22 May 2017 14:59:31 -0400 Message-ID: <1495479565.6967.72.camel@linux.intel.com> Subject: Re: [PATCH 23/23] uuid: remove uuid_be From: Andy Shevchenko Date: Mon, 22 May 2017 21:59:25 +0300 In-Reply-To: <20170518085629.GA6149@lst.de> References: <20170518062705.25902-1-hch@lst.de> <20170518062705.25902-24-hch@lst.de> <20170518085629.GA6149@lst.de> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: Christoph Hellwig , Amir Goldstein Cc: linux-fsdevel , Shaohua Li , Dan Williams , David Howells , Steven Whitehouse , Mimi Zohar , linux-xfs , linux-raid@vger.kernel.org, linux-nvdimm@lists.01.org, linux-kernel , Hugh Dickins On Thu, 2017-05-18 at 10:56 +0200, Christoph Hellwig wrote: > On Thu, May 18, 2017 at 10:57:24AM +0300, Amir Goldstein wrote: > > I reviewed the entire series. You may add > > Reviewed-by: Amir Goldstein > > > > to any of the patches as you see fit. > > Thanks, done! Similar from my side, FWIW: Reviewed-by: Andy Shevchenko > > > Now let's talk about how this is going to be merged. > > Do you intend to send Linus a pull request? > > Yes, that might be the best idea.  I'm also contemplating listing > me plus anyone volunteering (you?, Andy) as maintaines for the uuid > code. I would agree to be a designated reviewer for now (too many stuff to follow as a (co-)maintainer). > > The reason I am asking is because this last removal patch should > > probably > > be applied near the end of the merge window (?). > > Because maintainers cannot apply patches with code that uses the new > > uuid_t to linux-next branches and we don't want linux-next build to > > fail > > with new code that uses uuid_be... > > Yeah, we can probably defer that one for now. I would also like to append some of patches from my side, though they are WIP. -- Andy Shevchenko Intel Finland Oy From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761053AbdEVS7h (ORCPT ); Mon, 22 May 2017 14:59:37 -0400 Received: from mga09.intel.com ([134.134.136.24]:51761 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760421AbdEVS7b (ORCPT ); Mon, 22 May 2017 14:59:31 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.38,378,1491289200"; d="scan'208";a="860074981" Message-ID: <1495479565.6967.72.camel@linux.intel.com> Subject: Re: [PATCH 23/23] uuid: remove uuid_be From: Andy Shevchenko To: Christoph Hellwig , Amir Goldstein Cc: linux-fsdevel , Shaohua Li , Dan Williams , David Howells , Steven Whitehouse , Mimi Zohar , linux-xfs , linux-raid@vger.kernel.org, linux-nvdimm@ml01.01.org, linux-kernel , Hugh Dickins Date: Mon, 22 May 2017 21:59:25 +0300 In-Reply-To: <20170518085629.GA6149@lst.de> References: <20170518062705.25902-1-hch@lst.de> <20170518062705.25902-24-hch@lst.de> <20170518085629.GA6149@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 Thu, 2017-05-18 at 10:56 +0200, Christoph Hellwig wrote: > On Thu, May 18, 2017 at 10:57:24AM +0300, Amir Goldstein wrote: > > I reviewed the entire series. You may add > > Reviewed-by: Amir Goldstein > > > > to any of the patches as you see fit. > > Thanks, done! Similar from my side, FWIW: Reviewed-by: Andy Shevchenko > > > Now let's talk about how this is going to be merged. > > Do you intend to send Linus a pull request? > > Yes, that might be the best idea.  I'm also contemplating listing > me plus anyone volunteering (you?, Andy) as maintaines for the uuid > code. I would agree to be a designated reviewer for now (too many stuff to follow as a (co-)maintainer). > > The reason I am asking is because this last removal patch should > > probably > > be applied near the end of the merge window (?). > > Because maintainers cannot apply patches with code that uses the new > > uuid_t to linux-next branches and we don't want linux-next build to > > fail > > with new code that uses uuid_be... > > Yeah, we can probably defer that one for now. I would also like to append some of patches from my side, though they are WIP. -- Andy Shevchenko Intel Finland Oy