From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-out.m-online.net (mail-out.m-online.net [212.18.0.9]) by mail.openembedded.org (Postfix) with ESMTP id 355367893C for ; Sun, 7 Jan 2018 18:57:45 +0000 (UTC) Received: from frontend01.mail.m-online.net (unknown [192.168.8.182]) by mail-out.m-online.net (Postfix) with ESMTP id 3zF72L70zNz1qsT4; Sun, 7 Jan 2018 19:57:46 +0100 (CET) Received: from localhost (dynscan1.mnet-online.de [192.168.6.70]) by mail.m-online.net (Postfix) with ESMTP id 3zF72L6flyz26T1q; Sun, 7 Jan 2018 19:57:46 +0100 (CET) X-Virus-Scanned: amavisd-new at mnet-online.de Received: from mail.mnet-online.de ([192.168.8.182]) by localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new, port 10024) with ESMTP id zAZvX_PpvrUd; Sun, 7 Jan 2018 19:57:46 +0100 (CET) X-Auth-Info: yI9XqqRhuG6GTWTSaIRJGwYfFg3FRM3Tw5W27LyMEEo= Received: from [IPv6:::1] (unknown [195.140.253.167]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.mnet-online.de (Postfix) with ESMTPSA; Sun, 7 Jan 2018 19:57:46 +0100 (CET) To: Bruce Ashfield , Richard Purdie References: <20180107161920.30937-1-marex@denx.de> <20180107161920.30937-2-marex@denx.de> <2c1c48a3-5355-b3eb-fab0-765a007b8475@denx.de> <1999cffb-3890-2271-3938-ba2be97b135d@denx.de> From: Marek Vasut Message-ID: <5ba2da95-dbe0-b65b-e2e9-8e8fff030d96@denx.de> Date: Sun, 7 Jan 2018 19:57:45 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Cc: Patches and discussions about the oe-core layer Subject: Re: [PATCH 2/2] kernel: Add optional patch_xenomai task X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 07 Jan 2018 18:57:46 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 01/07/2018 07:55 PM, Bruce Ashfield wrote: > On Sun, Jan 7, 2018 at 1:49 PM, Marek Vasut wrote: >> On 01/07/2018 06:55 PM, Bruce Ashfield wrote: >>> On Sun, Jan 7, 2018 at 12:53 PM, Marek Vasut wrote: >>>> On 01/07/2018 05:42 PM, Bruce Ashfield wrote: >>>>> On Sun, Jan 7, 2018 at 11:38 AM, Marek Vasut wrote: >>>>>> On 01/07/2018 05:37 PM, Bruce Ashfield wrote: >>>>>>> On Sun, Jan 7, 2018 at 11:19 AM, Marek Vasut wrote: >>>>>>>> Add additional task, do_patch_xenomai, inserted between do_patch and >>>>>>>> do_configure tasks. This task applies the cobalt patch to the kernel >>>>>>>> sources for a specific machine. This is disabled by default, so use >>>>>>>> PACKAGECONFIG[xenomai] of the kernel package to enable the patching. >>>>>>>> >>>>>>>> You will also need a kernel recipe for a kernel version with ipipe >>>>>>>> patch applied. >>>>>>> >>>>>>> This doesn't make any sense to me. >>>>>>> >>>>>>> Why on earth would this be in kernel.bbclass ? and part of a xenomai >>>>>>> recipe ? >>>>>> Do you have a better suggestion how to make this easily available for >>>>>> every kernel recipe ? >>>>> >>>>> There's no need to make it available for every kernel recipe. If someone >>>>> wants to build xenomai, they need to have a specific kernel recipe and >>>>> configuration to make it work. By providing that patch and allowing it to >>>>> be applied to any kernel doesn't mean it will actually work .. so you aren't >>>>> doing anyone any favours. >>>>> >>>>> The same thing could be said for the -rt patch, aufs, etc, any patch could >>>>> be made available for any kernel, but they aren't, since they will either fail >>>>> to patch, or won't work at runtime. >>>> We actually have a -rt patch available in the kernel recipes, so why not >>>> xenomai patch ? >>> >>> As a separate recipe, not as something in a common bbclass, which implies >>> that it applies and works everywhere. >>> >>> Anyone that is trying to apply -rt against a generic kernel .. is very mistaken. >> >> So I should pull this into linux-*-xenomai , just like the rt patch does? > > Yep. > > But how it is maintained is a RP question, oe-core has a set of reference kernel > versions, and those reference kernels have a similar set of functionality across > architectures, are actively maintained and all get the same fixes at > the same time. > > (and I just realized that RP isn't copied on this thread anymore, so I'm adding > him). > > If we introduce something like this, I'd argue that it needs to follow the same > kernel versions to be consistent with the other oe-core references. > > That is a question for the steering committee and the architecture mailing list. > > So while that happens, why not just put this in a meta-xenomai for the time > being and once all the details have been hammered out, it could merge. That's fine by me, although the main feedback I'm currently interested in is whether this has any chance of getting included in oe-core and if there are some other big issues with this . -- Best regards, Marek Vasut