From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752104AbbCOKRw (ORCPT ); Sun, 15 Mar 2015 06:17:52 -0400 Received: from mail-bn1on0112.outbound.protection.outlook.com ([157.56.110.112]:32817 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751721AbbCOKRq (ORCPT ); Sun, 15 Mar 2015 06:17:46 -0400 X-WSS-ID: 0NL8ZXM-07-Y8N-02 X-M-MSG: Message-ID: <550558E5.2030309@amd.com> Date: Sun, 15 Mar 2015 12:03:17 +0200 From: Oded Gabbay Organization: AMD User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.5.0 MIME-Version: 1.0 To: Greg KH , Michel Machado CC: Bjorn Helgaas , netdev , linux-kernel , Dave Airlie Subject: Re: Linux XIA - merge proposal References: <54F5EF6E.9090303@digirati.com.br> <550301B6.3090302@digirati.com.br> <20150315093752.GA29005@kroah.com> In-Reply-To: <20150315093752.GA29005@kroah.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.20.0.84] X-EOPAttributedMessage: 0 Authentication-Results: spf=none (sender IP is 165.204.84.221) smtp.mailfrom=Oded.Gabbay@amd.com; digirati.com.br; dkim=none (message not signed) header.d=none; X-Forefront-Antispam-Report: CIP:165.204.84.221;CTRY:US;IPV:NLI;EFV:NLI;BMV:1;SFV:NSPM;SFS:(10019020)(6009001)(199003)(377454003)(189002)(479174004)(24454002)(15975445007)(77096005)(77156002)(54356999)(83506001)(86362001)(87936001)(23746002)(19580395003)(19580405001)(50986999)(76176999)(62966003)(93886004)(92566002)(33656002)(46102003)(101416001)(65806001)(65956001)(47776003)(50466002)(64126003)(105586002)(106466001)(2950100001)(36756003)(3556765001)(3940600001);DIR:OUT;SFP:1102;SCL:1;SRVR:BN3PR0201MB1043;H:atltwp01.amd.com;FPR:;SPF:None;MLV:sfv;A:1;MX:1;LANG:en; X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN3PR0201MB1043; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(5002010)(5005006);SRVR:BN3PR0201MB1043;BCL:0;PCL:0;RULEID:;SRVR:BN3PR0201MB1043; X-Forefront-PRVS: 05168A3970 X-OriginatorOrg: amd4.onmicrosoft.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Mar 2015 10:03:26.4857 (UTC) X-MS-Exchange-CrossTenant-Id: fde4dada-be84-483f-92cc-e026cbee8e96 X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=fde4dada-be84-483f-92cc-e026cbee8e96;Ip=[165.204.84.221];Helo=[atltwp01.amd.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR0201MB1043 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/15/2015 11:37 AM, Greg KH wrote: > On Fri, Mar 13, 2015 at 11:26:46AM -0400, Michel Machado wrote: >> On 03/10/2015 01:01 PM, Bjorn Helgaas wrote: >>> From my point of view (far outside the networking world and completely >>> incompetent to judge the merits of XIA), this email seems like a >>> reasonable starting point for a discussion, and I'm disappointed that >>> it got shut down so fast. It might not be in *exactly* the form >>> people are looking for, but hey, it's an introductory email, and >>> Michel seems open to constructive feedback. >> >> Thank you for the support, Bjorn. >> >> I've brainstormed with friends questions that could address the concerns >> that haven't been voiced in this thread, and I answered those questions in >> our FAQ page: >> >> https://github.com/AltraMayor/XIA-for-Linux/wiki/FAQ >> >> While I would like to have Linux XIA merged, I also don't want to become >> importunate. So, if nobody else shows interest in this thread, I'm going to >> leave it to rest. > > But you have yet to actually post the code, why give up now, you haven't > even tried. A number of people have given you hints here as to what you > should be doing / trying, to just run away without even attempting it > seems quite odd and defeatist. > > good luck, > > greg k-h > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ > Hi Michel, Although I don't know you and I'm not familiar with XIA, I would like to encourage you to listen to Greg and continue with your efforts to upstream XIA. I can tell you, from my recent experience of upstreaming AMD's HSA driver to drm (amdkfd), that I understand your response/frustration. However, you need to understand that the feedback you got is not because the maintainers/devs don't want your code. It is because they want your code to be in the best shape it can be before merging it to the kernel. Once it is in the kernel, it is distributed to millions of people and it is maintained practically forever. From my experience, the major changes that make your code better are usually done *before* upstreaming the code. Once the maintainers/devs realize that the code has reached some level of compliance with kernel standards, and the code can't get any better by leaving it outside the kernel, I'm sure they will accept the code. Especially if they will see that you are dedicated to improving it. Please see this enlightening post from drm maintainer, Dave Airlie, about trying to push code to upstream kernel: http://airlied.livejournal.com/80112.html It is called: "you have a long road to walk, but first you have to leave the house (or why publishing code is STEP ZERO)." BTW, I believe I was one of the reasons he wrote that post (it was published shortly after I published amdkfd's code) :) So bottom line, keep working on it. People gave you pointers what to do. Follow that and don't give up. In the end, I'm sure you will be very happy once the code is merged into the mainline kernel. Good luck, Oded