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

2021-04-14 Thread Vijay Kumar Banerjee
On Wed, Apr 14, 2021 at 8:40 PM Chris Johns wrote: > > On 15/4/21 5:22 am, Vijay Kumar Banerjee wrote: > > On Wed, Apr 14, 2021 at 1:10 PM Gedare Bloom wrote: > >> > >> On Wed, Apr 14, 2021 at 12:36 PM Vijay Kumar Banerjee > >> wrote: > >>> > >>> --- > >>> netlegacy.py | 27

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

2021-04-14 Thread Chris Johns
On 15/4/21 5:22 am, Vijay Kumar Banerjee wrote: > On Wed, Apr 14, 2021 at 1:10 PM Gedare Bloom wrote: >> >> On Wed, Apr 14, 2021 at 12:36 PM Vijay Kumar Banerjee >> wrote: >>> >>> --- >>> netlegacy.py | 27 +-- >>> 1 file changed, 13 insertions(+), 14 deletions(-) >>>

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

2021-04-14 Thread Vijay Kumar Banerjee
On Wed, Apr 14, 2021 at 1:10 PM Gedare Bloom wrote: > > On Wed, Apr 14, 2021 at 12:36 PM Vijay Kumar Banerjee wrote: > > > > --- > > netlegacy.py | 27 +-- > > 1 file changed, 13 insertions(+), 14 deletions(-) > > > > diff --git a/netlegacy.py b/netlegacy.py > > index

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

2021-04-14 Thread Gedare Bloom
On Wed, Apr 14, 2021 at 12:36 PM Vijay Kumar Banerjee wrote: > > --- > netlegacy.py | 27 +-- > 1 file changed, 13 insertions(+), 14 deletions(-) > > diff --git a/netlegacy.py b/netlegacy.py > index 05cb78a..2936b8f 100644 > --- a/netlegacy.py > +++ b/netlegacy.py > @@

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

2021-04-14 Thread Vijay Kumar Banerjee
--- netlegacy.py | 27 +-- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/netlegacy.py b/netlegacy.py index 05cb78a..2936b8f 100644 --- a/netlegacy.py +++ b/netlegacy.py @@ -37,7 +37,7 @@ exclude_headers = ['rtems-bsd-user-space.h',

Re: Booting a rtems exe on Zynq UltraScale+ MPSoC ZCU106 board

2021-04-14 Thread Kinsey Moore
That exception dump doesn't say a lot other than the ELR pointing at the very first instruction in your image, so I suspect that u-boot is trying to start your image as AArch64 code instead of AArch32/ARMv7 code. The ESR just specifies that it was a 32-bit instruction that blew up (both

Re: Booting a rtems exe on Zynq UltraScale+ MPSoC ZCU106 board

2021-04-14 Thread Richi Dubey
Hi Jan, Kinsey, Thanks for your quick responses. I rebuilt the .img file with -O rtems option, and I think it is the standard uboot available from PetaLinux. It still fails, but I think it is related to RTEMS now: Welcome to minicom 2.7.1 OPTIONS:

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

2021-04-14 Thread Gedare Bloom
On Wed, Apr 14, 2021 at 12:14 PM Vijay Kumar Banerjee wrote: > > --- > netlegacy.py | 26 +- > 1 file changed, 13 insertions(+), 13 deletions(-) > > diff --git a/netlegacy.py b/netlegacy.py > index 05cb78a..427d6ad 100644 > --- a/netlegacy.py > +++ b/netlegacy.py > @@

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

2021-04-14 Thread Vijay Kumar Banerjee
--- netlegacy.py | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/netlegacy.py b/netlegacy.py index 05cb78a..427d6ad 100644 --- a/netlegacy.py +++ b/netlegacy.py @@ -37,7 +37,7 @@ exclude_headers = ['rtems-bsd-user-space.h',

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

2021-04-14 Thread Gedare Bloom
This is a nicer solution, thanks. On Tue, Apr 13, 2021 at 1:32 PM Kinsey Moore wrote: > > 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. >

Re: [PATCH rtems-libbsd 2/2] st-sdmmc: Handle short not cache aligned buffers

2021-04-14 Thread Gedare Bloom
These two look good if no one complains go ahead. (I'm not 100% on this libbsd stuff yet.) On Wed, Apr 14, 2021 at 1:21 AM Christian Mauderer wrote: > > Possible data sources for SD driver: > > - mmc_sd_switch(): > - length: 64 byte; > - buffer on stack > - mmc_test_bus_width(): > -

Re: [PATCH 2/2] main_rtrace.c: Fix Dereference after null check (CID #1399709)

2021-04-14 Thread Gedare Bloom
looks good. On Wed, Apr 14, 2021 at 9:33 AM Joel Sherrill wrote: > > I'll pick on you. :) > > Please don't use the "title" that says "Fix ABC". There has to be a better > description. > > Hopefully someone will ack the technical. I think it is right. > > --joel > > On Wed, Apr 14, 2021 at 9:20

Re: [PATCH 1/2] objectextendinformation.c: Fix Dereference after null check (CID #26033)

2021-04-14 Thread Gedare Bloom
On Wed, Apr 14, 2021 at 9:57 AM Gedare Bloom wrote: > > On Wed, Apr 14, 2021 at 8:49 AM Joel Sherrill wrote: > > > > > > > > On Mon, Mar 15, 2021 at 5:09 PM Gedare Bloom wrote: > >> > >> On Mon, Mar 15, 2021 at 2:27 PM Joel Sherrill wrote: > >> > > >> > > >> > > >> > On Mon, Mar 15, 2021, 3:21

Re: [PATCH 1/2] objectextendinformation.c: Fix Dereference after null check (CID #26033)

2021-04-14 Thread Gedare Bloom
On Wed, Apr 14, 2021 at 8:49 AM Joel Sherrill wrote: > > > > On Mon, Mar 15, 2021 at 5:09 PM Gedare Bloom wrote: >> >> On Mon, Mar 15, 2021 at 2:27 PM Joel Sherrill wrote: >> > >> > >> > >> > On Mon, Mar 15, 2021, 3:21 PM Gedare Bloom wrote: >> >> >> >> On Fri, Mar 12, 2021 at 7:55 AM Ryan

[PATCH v1 2/3] aarch64: Add support for mapping exceptions

2021-04-14 Thread Kinsey Moore
This adds support for mapping AArch64 machine exceptions to POSIX signals as necessary for running Ada applications. --- bsps/aarch64/shared/exceptions.c | 89 +++ .../cpu/aarch64/aarch64-exception-default.S | 81 ++---

[PATCH v1] c-user: Add documentation for exception mapping

2021-04-14 Thread Kinsey Moore
This adds the documention generated by rtems-central for CONFIGURE_APPLICATION_NEEDS_EXCEPTION_TO_SIGNAL_MAPPING. --- c-user/config/device-driver.rst | 32 +++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git a/c-user/config/device-driver.rst

[PATCH v1 3/3] testsuite: Add machine exception signal map test

2021-04-14 Thread Kinsey Moore
Add a test to verify that mapping of machine exceptions to POSIX signals operates properly when the application requests it. --- spec/build/testsuites/psxtests/grp.yml| 2 + .../build/testsuites/psxtests/psxsignal09.yml | 22 testsuites/psxtests/psxsignal09/init.c| 54

[PATCH v1 1/3] cpukit: Add signal mapping support

2021-04-14 Thread Kinsey Moore
This adds a confdef option allowing an application to request mapping machine exceptions to POSIX signals. This is required for some languages such as Ada. --- cpukit/doxygen/appl-config.h | 25 +++- cpukit/include/rtems/confdefs.h| 1 +

Re: [PATCH 2/2] main_rtrace.c: Fix Dereference after null check (CID #1399709)

2021-04-14 Thread Joel Sherrill
I'll pick on you. :) Please don't use the "title" that says "Fix ABC". There has to be a better description. Hopefully someone will ack the technical. I think it is right. --joel On Wed, Apr 14, 2021 at 9:20 AM Ryan Long wrote: > ping > > -Original Message- > From: Ryan Long > Sent:

Re: [PATCH 1/2] objectextendinformation.c: Fix Dereference after null check (CID #26033)

2021-04-14 Thread Joel Sherrill
On Mon, Mar 15, 2021 at 5:09 PM Gedare Bloom wrote: > On Mon, Mar 15, 2021 at 2:27 PM Joel Sherrill wrote: > > > > > > > > On Mon, Mar 15, 2021, 3:21 PM Gedare Bloom wrote: > >> > >> On Fri, Mar 12, 2021 at 7:55 AM Ryan Long > wrote: > >> > > >> > CID 26033: Dereference after null check in >

RE: [PATCH 2/2] main_rtrace.c: Fix Dereference after null check (CID #1399709)

2021-04-14 Thread Ryan Long
ping -Original Message- From: Ryan Long Sent: Friday, March 12, 2021 8:56 AM To: devel@rtems.org Cc: Ryan Long Subject: [PATCH 2/2] main_rtrace.c: Fix Dereference after null check (CID #1399709) CID 1399709: Dereference after null check in rtems_trace_buffering_shell_save(). Closes

RE: Booting a rtems exe on Zynq UltraScale+ MPSoC ZCU106 board

2021-04-14 Thread Jan.Sommer
Hi Richi, In your log it says: Image Type: ARM Linux Kernel Image (gzip compressed) At least for our zedboard devices we use the following main options for mkimage. mkimage -A arm -O rtems -T kernel Which yields for me: Image Type: ARM RTEMS Kernel Image (gzip compressed) IIRC the

Re: Booting a rtems exe on Zynq UltraScale+ MPSoC ZCU106 board

2021-04-14 Thread Kinsey Moore
That's an error I haven't seen before. It seems like the BOOT.BIN/u-boot you're using is relatively limited; where did it come from? Most recently, I've pulled the BOOT.BIN from the PetaLinux prebuilts for my particular dev boards. Kinsey On 4/14/2021 08:21, Richi Dubey wrote: Trying to

Re: Booting a rtems exe on Zynq UltraScale+ MPSoC ZCU106 board

2021-04-14 Thread Richi Dubey
Trying to boot directly from the .img file also fails: ZynqMP> tftpboot 0x300 rdubey/sp01.img Using ethernet@ff0e device TFTP from server 172.19.0.3; our IP address is 172.19.2.40 Filename 'rdubey/sp01.img'. Load address: 0x300 Loading: 6.1 MiB/s done Bytes

RE: Booting a rtems exe on Zynq UltraScale+ MPSoC ZCU106 board

2021-04-14 Thread Kinsey Moore
If you’re only running RTEMS, you should be able to drop the FDT commands since that what appears to be causing the problem and I don’t think that the arm/xilinx_zynqmp BSP uses it at all. Kinsey From: Richi Dubey Sent: Wednesday, April 14, 2021 01:01 To: Kinsey Moore ; rtems-de...@rtems.org

[PATCH rtems-libbsd 1/2] MMCSD: Use cache aligned alloc for ioctl

2021-04-14 Thread Christian Mauderer
Some drivers can't easily work with buffers that are not cache aligned and have an arbitrary length. Therefore use an aligned malloc instead. --- freebsd/sys/dev/mmc/mmcsd.c | 4 1 file changed, 4 insertions(+) diff --git a/freebsd/sys/dev/mmc/mmcsd.c b/freebsd/sys/dev/mmc/mmcsd.c index

[PATCH rtems-libbsd 2/2] st-sdmmc: Handle short not cache aligned buffers

2021-04-14 Thread Christian Mauderer
Possible data sources for SD driver: - mmc_sd_switch(): - length: 64 byte; - buffer on stack - mmc_test_bus_width(): - length: 4 or 8 byte - buffer in program memory or on stack - mmc_app_send_scr(): - length: 8 byte - buffer from device ivar structure -

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

2021-04-14 Thread Gedare Bloom
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 ++ > >

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

2021-04-14 Thread Gedare Bloom
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: > > > >

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

2021-04-14 Thread Gedare Bloom
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

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

2021-04-14 Thread Vijay Kumar Banerjee
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

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

2021-04-14 Thread Vijay Kumar Banerjee
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

Booting a rtems exe on Zynq UltraScale+ MPSoC ZCU106 board

2021-04-14 Thread Richi Dubey
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

2021-04-14 Thread Vijay Kumar Banerjee
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

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

2021-04-14 Thread Vijay Kumar Banerjee
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

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

2021-04-14 Thread Vijay Kumar Banerjee
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