Re: [PATCH rtems-net-legacy v3] nfsclient: Build nfsclient from the top level netlegacy script

looks good, other than the comment I made on the other patch about '${PREFIX}' On Wed, Apr 14, 2021 at 12:01 AM Vijay Kumar Banerjee wrote: > > Ping :) > > On Wed, Apr 7, 2021 at 10:51 AM Vijay Kumar Banerjee wrote: > > > > --- > > netlegacy.py | 31 ++ > > nfsclien

Re: [PATCH rtems-net-legacy] netlegacy: Use os.path instead of hardcoded Unix like path

On Wed, Apr 14, 2021 at 12:18 AM Gedare Bloom wrote: > > this one looks good, but unrelated to the direct changes, is > '${PREFIX}' well-defined or is there a way to grab that as a variable > from something like bld.env? > > On Wed, Apr 14, 2021 at 12:04 AM Vijay Kumar Banerjee wrote: > > > > Pin

Re: [PATCH rtems-net-legacy] netlegacy: Use os.path instead of hardcoded Unix like path

this one looks good, but unrelated to the direct changes, is '${PREFIX}' well-defined or is there a way to grab that as a variable from something like bld.env? On Wed, Apr 14, 2021 at 12:04 AM Vijay Kumar Banerjee wrote: > > Ping :) > > On Wed, Apr 7, 2021 at 10:52 AM Vijay Kumar Banerjee wrote:

Re: [PATCH rtems-net-legacy] netlegacy: Use os.path instead of hardcoded Unix like path

Ping :) On Wed, Apr 7, 2021 at 10:52 AM Vijay Kumar Banerjee wrote: > > --- > netlegacy.py | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/netlegacy.py b/netlegacy.py > index 05cb78a..ddd1db3 100644 > --- a/netlegacy.py > +++ b/netlegacy.py > @@ -37,7 +3

Re: [PATCH rtems-net-legacy] netlegacy: Use os.path to get the file extension

On Sat, Apr 3, 2021 at 8:36 AM Gedare Bloom wrote: > > looks good > Pushed. Thanks. > On Fri, Apr 2, 2021 at 7:32 PM Vijay Kumar Banerjee wrote: > > > > --- > > bsp_drivers.py | 5 +++-- > > netlegacy.py | 5 +++-- > > 2 files changed, 6 insertions(+), 4 deletions(-) > > > > diff --git a/bsp_

Booting a rtems exe on Zynq UltraScale+ MPSoC ZCU106 board

Hi, I followed the 8.2.23 docs to build rtems for the xilinx_zynqmp_ultra96 bsp since it was the only bsp corresponding to xilinx-zynqmp in the rtems-bsp. Then I followed the boot via Uboot section 8.2.1.1 on docs

Re: [PATCH rtems-net-legacy v3] nfsclient: Build nfsclient from the top level netlegacy script

Ping :) On Wed, Apr 7, 2021 at 10:51 AM Vijay Kumar Banerjee wrote: > > --- > netlegacy.py | 31 ++ > nfsclient/wscript | 56 --- > wscript | 2 +- > 3 files changed, 28 insertions(+), 61 deletions(-) > delete m

Re: [PATCH rtems-net-legacy] leon, greth: added support for variable sized descriptor table sizes

On Tue, Mar 30, 2021 at 10:22 AM Gedare Bloom wrote: > > thanks, go ahead > Pushed. Thanks > On Tue, Mar 30, 2021 at 9:52 AM Vijay Kumar Banerjee wrote: > > > > From: Daniel Hellstrom > > > > The descriptor table size is equal to its alignment and set when > > configuring the HW IP thro

Re: [PATCH rtems-net-legacy v2] Remove RTEMS_NETWORKING checks

On Mon, Apr 5, 2021 at 10:33 AM Gedare Bloom wrote: > > On Sat, Apr 3, 2021 at 2:13 PM Vijay Kumar Banerjee wrote: > > > > > > > > On Sat, Apr 3, 2021 at 12:27 PM Gedare Bloom wrote: > > > > > > On Fri, Apr 2, 2021 at 7:32 PM Vijay Kumar Banerjee > > > wrote: > > > > > > > > * Remove nfsclient

[PATCH rtems-net-legacy] bsp_drivers: Use os.path for compatibility with non Unix host

--- bsp_drivers.py | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/bsp_drivers.py b/bsp_drivers.py index 3ca10c6..4a07847 100644 --- a/bsp_drivers.py +++ b/bsp_drivers.py @@ -36,18 +36,18 @@ def bsp_files(bld): include_dirs = {} include_fi

[PATCH rtems-net-legacy] netlegacy: Remove telnetd build

--- netlegacy.py | 13 ++--- testsuites/telnetd01/wscript | 5 + 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/netlegacy.py b/netlegacy.py index ddd1db3..327d7cc 100644 --- a/netlegacy.py +++ b/netlegacy.py @@ -31,8 +31,7 @@ import os source_files

[PATCH rtems 2/2] testsuites: Remove telnetd01

--- spec/build/testsuites/libtests/grp.yml | 2 - spec/build/testsuites/libtests/telnetd01.yml | 22 testsuites/libtests/telnetd01/init.c | 120 --- testsuites/libtests/telnetd01/telnetd01.doc | 24 testsuites/libtests/telnetd01/telnetd01.scn | 11 --

[PATCH rtems 1/2] telnetd.c: Remove RTEMS_NETWORKING check

Set the priority manually to make telnetd compatible with the --- cpukit/telnetd/telnetd.c | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/cpukit/telnetd/telnetd.c b/cpukit/telnetd/telnetd.c index b8adec297a..7dad2f2f92 100644 --- a/cpukit/telnetd/telnetd.c +++ b/cp

Re: Tool update procedure (was Re: Fwd: GCC 10.3 Released)

On 12/4/21 6:15 pm, Sebastian Huber wrote: > Hello Chris, > > On 12/04/2021 01:34, Chris Johns wrote: >> On 10/4/21 1:19 am, Joel Sherrill wrote: >>> On Fri, Apr 9, 2021, 9:07 AM Sebastian Huber >>> >> > wrote: >>> >>> On 08/04/2021 14:50, Joel Sher

Re: [PATCH 0/3] rtemstoolkit: Add SMTP login to mailer

On 13/4/21 12:09 pm, Joel Sherrill wrote: > On Mon, Apr 12, 2021, 6:47 PM Chris Johns > wrote: > > Hi Alex, > > This looks great. Thank you for adding this feature. OK to push. > > Do you think a small section called "Posting Reports" that explains how > to

Re: [PATCH] rtemstoolkit/mailer.py: Add --use-gitconfig option

OK to push with a minor fix noted below ... On 14/4/21 7:33 am, Alex White wrote: > This adds the option to pull mail-related configuration values from the > user's git configuration. > > Closes #4384 > --- > rtemstoolkit/mailer.py | 46 -- > 1 file change

[PATCH] rtemstoolkit/mailer.py: Add --use-gitconfig option

This adds the option to pull mail-related configuration values from the user's git configuration. Closes #4384 --- rtemstoolkit/mailer.py | 46 -- 1 file changed, 44 insertions(+), 2 deletions(-) diff --git a/rtemstoolkit/mailer.py b/rtemstoolkit/mailer.py

Re: [PATCH v3] c-user: Add scheduler glossary terms

I like it, thanks. On Tue, Apr 13, 2021 at 12:20 PM Sebastian Huber wrote: > > Add glossary terms which characterize the relationship of a scheduler to > a task. > --- > v3: > > Address review comments. Use "ineligible" instead of "alien" or > "foreign". > > c-user/glossary.rst | 24 +++

Re: [PATCH v2] c-user: Add scheduler glossary terms

> On Apr 13, 2021, at 11:23 , Gedare Bloom wrote: > >> What about "ineligible" as Gedare originally suggested. >> > that would be my preference. It is a cleaner terminology fit. > > From reading the mailings this is in contrast to the "eligible scheduler". So "ineligible" is good. That's b

[PATCH v2] bsps/xilinx-zynqmp: Avoid constant UART reinit

Constantly reinitializing the Cadence UART on every character output causes data corruption/loss on some ZynqMP hardware. Only initialize the UART once for early output and give it a kick on startup. --- bsps/aarch64/xilinx-zynqmp/console/console.c | 6 ++ bsps/shared/dev/serial/zynq-uart-poll

Re: [PATCH v3] smpfatal08: halt secondary processors if necessary

ok to push, thanks On Tue, Apr 13, 2021 at 11:48 AM Jiri Gaisler wrote: > > > Just one nit, I think we still want to keep > > - (void) cpu_self; > > because the variable is unused. This suppresses unused variable warnings. > I never understood why it was there, but now I know ... :-) > > smpfata

[PATCH v3] c-user: Add scheduler glossary terms

Add glossary terms which characterize the relationship of a scheduler to a task. --- v3: Address review comments. Use "ineligible" instead of "alien" or "foreign". c-user/glossary.rst | 24 1 file changed, 24 insertions(+) diff --git a/c-user/glossary.rst b/c-user/glos

[PATCH v3] smpfatal08: halt secondary processors if necessary

Just one nit, I think we still want to keep - (void) cpu_self; because the variable is unused. This suppresses unused variable warnings. I never understood why it was there, but now I know ... :-) smpfatal08 fails on systems where all cpus are started by the boot-loader and thereby clobber the

Re: [PATCH] clock:_TOD_To_seconds(): Fix year 2514 overflow

Hello, On 08/04/2021 15:41, Frank Kuehndel wrote: +/** + * @brief Latest year to which a time of day can be initialized. + * + * The following constant defines the latest year to which an + * RTEMS time of day can be set using rtems_clock_set(). + * + * 32 bits can accept as latest point in

Re: [PATCH v2] c-user: Add scheduler glossary terms

On Tue, Apr 13, 2021 at 9:22 AM Sebastian Huber wrote: > > On 13/04/2021 16:23, Sebastian Huber wrote: > > > On 13/04/2021 16:02, Joel Sherrill wrote: > > > >> +foreign scheduler > >> +An foreign scheduler of a :term:`task` is a > >> :term:`scheduler` which is not > >>

Re: [PATCH v2] c-user: Add scheduler glossary terms

On Tue, Apr 13, 2021 at 8:23 AM Sebastian Huber wrote: > > On 13/04/2021 16:02, Joel Sherrill wrote: > > > +foreign scheduler > > +An foreign scheduler of a :term:`task` is a > > :term:`scheduler` which is not > > +an :term:`eligible scheduler`. > > + > > >

Re: [PATCH v2] smpfatal08: halt secondary processors if necessary

Hi Jiri, Just one nit, I think we still want to keep - (void) cpu_self; because the variable is unused. This suppresses unused variable warnings. On Tue, Apr 13, 2021 at 7:18 AM Jiri Gaisler wrote: > > smpfatal08 fails on systems where all cpus are started by the boot-loader and > thereby clob

Re: [PATCH v2] c-user: Add scheduler glossary terms

On 13/04/2021 16:23, Sebastian Huber wrote: On 13/04/2021 16:02, Joel Sherrill wrote:     +    foreign scheduler     +        An foreign scheduler of a :term:`task` is a     :term:`scheduler` which is not     +        an :term:`eligible scheduler`.     + Can we come up with an adjective othe

[PATCH v3] score/aarch64: Align context validation frame

Ensure the stack remains aligned by keeping the context frame at a multiple of 16 bytes. This avoids stack alignment exceptions which occur when the stack pointer is not 16 byte aligned. --- .../cpu/aarch64/aarch64-context-validate.S| 29 --- 1 file changed, 18 insertions(+), 1

Re: [PATCH v2] score/aarch64: Align context validation frame

On 13/04/2021 16:02, Kinsey Moore wrote: Ensure the stack remains aligned by keeping the context frame at a multiple of 16 bytes. This avoids stack alignment exceptions which occur when the stack pointer is not 16 byte aligned. It would be good to check this requirement with static assertions.

Re: [PATCH v2] c-user: Add scheduler glossary terms

On 13/04/2021 16:02, Joel Sherrill wrote: +    foreign scheduler +        An foreign scheduler of a :term:`task` is a :term:`scheduler` which is not +        an :term:`eligible scheduler`. + Can we come up with an adjective other than foreign. The scheduler did not originat

Re: [PATCH v2] c-user: Add scheduler glossary terms

On Tue, Apr 13, 2021 at 2:31 AM Sebastian Huber < sebastian.hu...@embedded-brains.de> wrote: > Add glossary terms which characterize the relationship of a scheduler to > a task. > --- > > v2: > > Address review comments. Use "foreign" instead of "alien". > > c-user/glossary.rst | 24

[PATCH v2] score/aarch64: Align context validation frame

Ensure the stack remains aligned by keeping the context frame at a multiple of 16 bytes. This avoids stack alignment exceptions which occur when the stack pointer is not 16 byte aligned. --- .../cpu/aarch64/aarch64-context-validate.S| 25 +++ 1 file changed, 15 insertions(+), 1

[PATCH v2] smpfatal08: halt secondary processors if necessary

smpfatal08 fails on systems where all cpus are started by the boot-loader and thereby clobber the test output. This patch stops the secondary cpus with a call to _CPU_Thread_Idle_body(). Tested on RISC-V and SPARC. From 4429fd6b391e3feebd72a0e7d403cc4745e35a3a Mon Sep 17 00:00:00 2001 From: Jir

Re: [PATCH] smpfatal08: halt secondary RISC-V processors

On 4/13/21 7:44 AM, Sebastian Huber wrote: On 12/04/2021 21:40, Jiri Gaisler wrote: I just realized that I can change the patch from an assembly WFI to a simple busy loop instead. This would work on any architecture without assembly. Should I provide a new patch with this cleaner solution?

[PATCH] My first RTEMS application.

looking forward to working with RTEMS!! Cheers! --- .gitmodules | 3 +++ hello.c | 14 ++ init.c | 15 +++ rtems_waf | 1 + wscript | 35 +++ 5 files changed, 68 insertions(+) create mode 100644 .gitmodules create mode 100

AW: [PATCH 0/1] grlib/genirq: Issue when enabling/disabling interrupt

>Yes, rtems 5 also has it. I can create a ticket. Here is the link to the ticket https://devel.rtems.org/ticket/4385#ticket. BTW the proposed solution for the master branch can be also applied to 5 as well. >Regarding the proposed solution, I wanted to start a thread for discussing it >(maybe t

[PATCH v2] c-user: Add scheduler glossary terms

Add glossary terms which characterize the relationship of a scheduler to a task. --- v2: Address review comments. Use "foreign" instead of "alien". c-user/glossary.rst | 24 1 file changed, 24 insertions(+) diff --git a/c-user/glossary.rst b/c-user/glossary.rst index