Praneeth,

srcrev in kernel recipe for kirkstone branch needs to be bumped. The last CICD 
build was 6-Sep - also a good idea to trigger another full build.

Regards,
Anand

-----Original Message-----
From: Bajjuri, Praneeth <[email protected]> 
Sent: Friday, September 15, 2023 9:47 PM
To: Balagopalakrishnan, Anand <[email protected]>; Denys Dmytriyenko 
<[email protected]>
Cc: [email protected]; Eatmon, Ryan <[email protected]>; Adivi, Sai 
Sree Kartheek <[email protected]>; Gupta, Gyan <[email protected]>; Khasim, Syed 
Mohammed <[email protected]>; Pothukuchi, Vijay <[email protected]>
Subject: Re: [EXTERNAL] Re: [meta-ti] [RFC|kirkstone] linux-ti-staging: add 
config preempt for am335x and am437x



On 9/14/2023 8:47 PM, Balagopalakrishnan, Anand wrote:
> Hi Denys,
> 
> This is only needed for v7 platforms like AM3 / AM4 and possibly AM57x also. 
> CONFIG_PREEMPT is enabled by default on arm64 platforms like AM65x.
> 
> Praneeth will send a kernel patch for TI kernel. This meta-ti patch is meant 
> to unblock anyone who wants to enable SGX drive in the meanwhile.


This patch can be abandoned .
https://git.ti.com/cgit/ti-linux-kernel/ti-linux-kernel/commit/?h=ti-linux-6.1.y-cicd&id=e025e31126fff01ded59d901af916abe87646ca6
 
enables CONFIG_PREEMPT


> 
> Regards,
> Anand
> 
> -----Original Message-----
> From: Denys Dmytriyenko <[email protected]>
> Sent: Friday, September 15, 2023 1:25 AM
> To: Balagopalakrishnan, Anand <[email protected]>
> Cc: [email protected]; Eatmon, Ryan <[email protected]>; Adivi, Sai 
> Sree Kartheek <[email protected]>; Gupta, Gyan <[email protected]>; Khasim, Syed 
> Mohammed <[email protected]>; Pothukuchi, Vijay <[email protected]>; Bajjuri, 
> Praneeth <[email protected]>
> Subject: [EXTERNAL] Re: [meta-ti] [RFC|kirkstone] linux-ti-staging: add 
> config preempt for am335x and am437x
> 
> On Thu, Sep 14, 2023 at 03:36:18PM -0400, Denys Dmytriyenko wrote:
>> On Thu, Sep 14, 2023 at 10:40:26PM +0530, Anand Balagopalakrishnan via 
>> lists.yoctoproject.org wrote:
>>> SGX needs a preemptible kernel. CONFIG_PREEMPT has always been
>>> enabled on TI Linux kernels upto K5.1.
>>>
>>> This is enabled on AM65x by default but missed out AM335x and AM437x.
>>> Add a kernel config fragment to enable config preempt.
>>
>> Hi, Anand,
>>
>> Is this only needed for am335x and am437x, as the subject suggests? If
>> so, why the code adds the config fragment to all platforms?
>>
>> Second question - historically this has been enabled in the TI kernel itself.
>> Why not do it like that again, why add the fragment to the OE metadata
>> instead of keeping it in-tree? How the distro-agnostic approach would be 
>> handled?
>>
>>
>>> Signed-off-by: Anand Balagopalakrishnan <[email protected]>
>>> ---
>>>   .../linux/files/configs/config-preempt.cfg           | 12 ++++++++++++
>>>   .../recipes-kernel/linux/linux-ti-staging_6.1.bb     |  1 +
>>>   2 files changed, 13 insertions(+)
>>>   create mode 100644
>>> meta-ti-bsp/recipes-kernel/linux/files/configs/config-preempt.cfg
>>>
>>> diff --git
>>> a/meta-ti-bsp/recipes-kernel/linux/files/configs/config-preempt.cfg
>>> b/meta-ti-bsp/recipes-kernel/linux/files/configs/config-preempt.cfg
>>> new file mode 100644
>>> index 00000000..c1e5ad7d
>>> --- /dev/null
>>> +++ b/meta-ti-bsp/recipes-kernel/linux/files/configs/config-preempt.
>>> +++ cfg
>>
>> This is not the right location for the file. You want to add to the
>> linux-ti-staging-rt-6.1/ directory, since you are only using it from
>> the corresponding linux-ti-staging_6.1.bb recipe.
> 
> Sorry, typo here - linux-ti-staging-6.1/ directory, if it's not RT-specific.
> 
>>
>>> @@ -0,0 +1,12 @@
>>> +CONFIG_PREEMPT_BUILD=y
>>> +# CONFIG_PREEMPT_NONE is not set
>>> +CONFIG_PREEMPT=y
>>> +CONFIG_PREEMPT_COUNT=y
>>> +CONFIG_PREEMPTION=y
>>> +CONFIG_PREEMPT_RCU=y
>>> +CONFIG_TASKS_RCU_GENERIC=y
>>> +CONFIG_TASKS_RCU=y
>>> +CONFIG_UNINLINE_SPIN_UNLOCK=y
>>> +# CONFIG_CEC_GPIO is not set
>>> +CONFIG_DEBUG_PREEMPT=y
>>> +# CONFIG_PREEMPT_TRACER is not set
>>> diff --git
>>> a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging_6.1.bb
>>> b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging_6.1.bb
>>> index 8a883878..0ee63358 100644
>>> --- a/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging_6.1.bb
>>> +++ b/meta-ti-bsp/recipes-kernel/linux/linux-ti-staging_6.1.bb
>>> @@ -33,6 +33,7 @@ PR = "${MACHINE_KERNEL_PR}"
>>>   
>>>   KERNEL_GIT_URI ?= 
>>> "git://git.ti.com/git/ti-linux-kernel/ti-linux-kernel.git"
>>>   KERNEL_GIT_PROTOCOL = "https"
>>> +KERNEL_CONFIG_FRAGMENTS += "${WORKDIR}/config-preempt.cfg"
>>
>> Since this file is out-of-tree and in the OE metadata, it would also
>> require listing it in SRC_URI.
>>
>>
>>>   SRC_URI += 
>>> "${KERNEL_GIT_URI};protocol=${KERNEL_GIT_PROTOCOL};branch=${BRANCH} \
>>>               file://defconfig"
>>>   
>>> --
>>> 2.34.1
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#16990): 
https://lists.yoctoproject.org/g/meta-ti/message/16990
Mute This Topic: https://lists.yoctoproject.org/mt/101362535/21656
Group Owner: [email protected]
Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to