Skip to content

tests: mimxrt1160_evk/mimxrt1166/cm7:samples/drivers/mbox_data/sample: drivers: mbox_data test Timeout #82841

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
hakehuang opened this issue Dec 11, 2024 · 3 comments · Fixed by #86871
Assignees
Labels
area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug platform: NXP NXP priority: low Low impact/importance bug

Comments

@hakehuang
Copy link
Collaborator

hakehuang commented Dec 11, 2024

Describe the bug
mbox_data test is Timeout on v4.0.0-1877-g90ff9c0883b2 on mimxrt1160_evk/mimxrt1166/cm7
testcase path is samples/drivers/mbox_data/

see logs for details

To Reproduce
1.

scripts/twister --device-testing --device-serial /dev/ttyACM0 -p mimxrt1160_evk/mimxrt1166/cm7  -T samples/drivers/mbox_data --sub-test mimxrt1160_evk/mimxrt1166/cm7:samples/drivers/mbox_data/sample.drivers.mbox_data.real_hw

or

# cd samples/drivers/mbox_data/
# west build -b mimxrt1160_evk/mimxrt1166/cm7
# west flash

git bisect found below commit introduce this failure 2e8868c

commit 71fd4d9c98c7fe0845b70693e8daea46c6646b80
Author: Torsten Rasmussen <Torsten.Rasmussen@nordicsemi.no>
Date:   Thu Nov 28 09:16:38 2024 +0100

    cmake: clear llvm bintool gapfill property

    The llvm-objcopy doesn't support a gap-fill argument, therefore clear
    the property to remove the error:
    > llvm-objcopy: error: unknown argument '--gap-fill'

    Signed-off-by: Torsten Rasmussen <Torsten.Rasmussen@nordicsemi.no>


2e8868c16e5ecfab8e823096892e02dac037c222 is the first bad commit
commit 2e8868c16e5ecfab8e823096892e02dac037c222
Author: Torsten Rasmussen <Torsten.Rasmussen@nordicsemi.no>
Date:   Thu Nov 28 09:07:00 2024 +0100

    cmake: deprecate BUILD_NO_GAP_FILL and introduce type specific gap fill

    Deprecate BUILD_NO_GAP_FILL as it gives a false impression that gap
    filling can be disabled in binary files.
    Binary files are always gap filled due to the fact they contain no
    address information. Only option for binary files is to control the gap
    fill pattern. When no gap fill is enabled in binary files, then a
    default pattern is used by the tool, which usually is 0x00.

    Generally the pattern 0x00 leads to unnecessary flash writes, as a
    flash generally contains 0xFF after an erase.
    Therefore provide a gap fill pattern Kconfig setting instead, with
    default value of 0xFF.

    For hex-files, intel hex and s19, then gap filling is generally not
    needed but in order to still support cases where gap filling is required
    then this commit introduces BUILD_OUTPUT_HEX_GAP_FILL and
    BUILD_OUTPUT_S19_GAP_FILL. Both settings are disabled per default.

    Signed-off-by: Torsten Rasmussen <Torsten.Rasmussen@nordicsemi.no>


  1. See error

Expected behavior
test pass

Impact

Logs and console output

*** Booting Zephyr OS build v4.0.0-1877-g90ff9c0883b2 ***
mbox_data Client demo started
Client send (on channel 3) value: 0
*** Booting Zephyr OS build v4.0.0-1877-g90ff9c0883b2 ***
mbox_data Client demo started
Client send (on channel 3) value: 0

    

Environment (please complete the following information):

  • OS: (e.g. Linux )
  • Toolchain (e.g Zephyr SDK)
  • Commit SHA or Version used: v4.0.0-1877-g90ff9c0883b2
@hakehuang hakehuang added area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug labels Dec 11, 2024
@hakehuang hakehuang transferred this issue from zephyrproject-rtos/test_results Dec 11, 2024
@hakehuang hakehuang changed the title tests: samples/drivers/mbox_data/sample: drivers: mbox_data test Timeout tests: mimxrt1160_evk/mimxrt1166/cm7:samples/drivers/mbox_data/sample: drivers: mbox_data test Timeout Dec 11, 2024
@hakehuang
Copy link
Collaborator Author

this case is ok on mimxrt1170_evk

@kartben kartben added the priority: low Low impact/importance bug label Dec 17, 2024
Copy link

This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time.

@github-actions github-actions bot added the Stale label Feb 16, 2025
@hakehuang hakehuang removed the Stale label Feb 16, 2025
hakehuang added a commit to nxp-upstream/zephyr that referenced this issue Mar 10, 2025
the device_state need realocate when boot as second core.
as the relocation does not know the alignment, this could
be a case that the next data is mis-place after relocation.

fix this by add a alignment in device_state section.

fixes: zephyrproject-rtos#82841

Signed-off-by: Hake Huang <hake.huang@oss.nxp.com>
hakehuang added a commit to nxp-upstream/zephyr that referenced this issue Mar 10, 2025
the device_state need realocate when boot as second core.
as the relocation does not know the alignment, this could
be a case that the next data is mis-place after relocation.

fix this by add a alignment in device_state section.

still need revert aec0355
reported in zephyrproject-rtos#86871

fixes: zephyrproject-rtos#82841

Signed-off-by: Hake Huang <hake.huang@oss.nxp.com>
Copy link

This issue has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this issue will automatically be closed in 14 days. Note, that you can always re-open a closed issue at any time.

@github-actions github-actions bot added the Stale label Apr 28, 2025
@hakehuang hakehuang removed the Stale label Apr 28, 2025
hakehuang added a commit to nxp-upstream/zephyr that referenced this issue May 15, 2025
the device_state need realocate when boot as second core.
as the relocation does not know the alignment, this could
be a case that the next data is mis-place after relocation.

fix this by add a alignment in device_state section.

still need revert aec0355
reported in zephyrproject-rtos#86871

fixes: zephyrproject-rtos#82841

Signed-off-by: Hake Huang <hake.huang@oss.nxp.com>
@nashif nashif closed this as completed in 8e2c053 May 30, 2025
Shreyas-Shankar155 pushed a commit to MihiraMadhava/zephyr that referenced this issue Jun 3, 2025
the device_state need realocate when boot as second core.
as the relocation does not know the alignment, this could
be a case that the next data is mis-place after relocation.

fix this by add a alignment in device_state section.

still need revert aec0355
reported in zephyrproject-rtos#86871

fixes: zephyrproject-rtos#82841

Signed-off-by: Hake Huang <hake.huang@oss.nxp.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Tests Issues related to a particular existing or missing test bug The issue is a bug, or the PR is fixing a bug platform: NXP NXP priority: low Low impact/importance bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants