From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752823AbbCIMlb (ORCPT ); Mon, 9 Mar 2015 08:41:31 -0400 Received: from mail-wi0-f176.google.com ([209.85.212.176]:40539 "EHLO mail-wi0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751360AbbCIMla (ORCPT ); Mon, 9 Mar 2015 08:41:30 -0400 Message-ID: <54FD94F6.4040600@plexistor.com> Date: Mon, 09 Mar 2015 14:41:26 +0200 From: Boaz Harrosh User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Ross Zwisler , Boaz Harrosh CC: Ingo Molnar , x86@kernel.org, linux-kernel , "Roger C. Pao" , Dan Williams , Thomas Gleixner , linux-nvdimm , "H. Peter Anvin" , Matthew Wilcox , Andy Lutomirski , Christoph Hellwig Subject: Re: [RFC 0/8] pmem: Submission of the Persistent memory block device References: <54F82CE0.4040502@plexistor.com> <54F830D4.7030205@plexistor.com> <1425667065.32009.7.camel@theros.lm.intel.com> In-Reply-To: <1425667065.32009.7.camel@theros.lm.intel.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/06/2015 08:37 PM, Ross Zwisler wrote: <> >> I maintain these patches on latest Kernels here: >> git://git.open-osd.org/pmem.git branch pmem >> >> Thanks for reviewing >> Boaz > > Hey Boaz, > > Regarding the PMEM series, my group has been working on an updated > version of this driver for the past 6 months or so since I initially > posted the beginnings of this series: > > https://lkml.org/lkml/2014/8/27/674 > > That new version should be ready for public viewing sometime in April. > > It's my preference that we wait to try and upstream any form of PMEM > until we've released our updated version of the driver, and you've had a > chance to review and add in any changes you need. I'm cool with > gathering additional feedback until then, of course. > Dear Ross We have a very grave problem with you guys. You do not develop an open source driver in stealth mode, and not release code until some undefined future. This is called forking. And forking means taking the extra hit when main stream is advancing. I've been changing this driver and supporting code, and even if not posting every change on ML, so not to make noise, I pushed any new version to the public tree every time. So anyone that monitors the tree can provision for my changes and see where I'm going with this. I've been monitoring your tree, and there was not a single change. If you are making progress you should make them available, as soon as they advance so we can see and adjust. (And/or maybe not make the same effort twice?) > Trying to upstream this older version and then merging it with the newer > stuff in-kernel seems like it'll just end up being more work in the end. > I don't think so. Did you see the latest version of this driver. It is so small. Any changes you will make, will probably be additions and enhancements. I do not see how it will make such an extra work. Show me the code, If I see that you are right I will make changes accordingly, but for now I cannot see why it can make any kind of problems. > Thanks, > - Ross Thanks Boaz