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 <de...@denix.org>
Sent: Friday, September 15, 2023 1:25 AM
To: Balagopalakrishnan, Anand <ana...@ti.com>
Cc: meta-ti@lists.yoctoproject.org; Eatmon, Ryan <reat...@ti.com>; Adivi, Sai Sree Kartheek
<s-ad...@ti.com>; Gupta, Gyan <g-gu...@ti.com>; Khasim, Syed Mohammed <kha...@ti.com>;
Pothukuchi, Vijay <vij...@ti.com>; Bajjuri, Praneeth <prane...@ti.com>
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 <ana...@ti.com>
---
.../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 (#16987):
https://lists.yoctoproject.org/g/meta-ti/message/16987
Mute This Topic: https://lists.yoctoproject.org/mt/101362535/21656
Group Owner: meta-ti+ow...@lists.yoctoproject.org
Unsubscribe:
https://lists.yoctoproject.org/g/meta-ti/leave/6695321/21656/1393940836/xyzzy
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
