Skip to content

file system detect for setup default debugfs for ipc flood test #191

Closed

Description

@fredoh9

When multiclient architecture changes are merged, thesofproject/linux#1841, defugfs root for
ipc flood test is changed to /sys/kernel/debug/ipc-flood-test.x/, where x is unique id. debugfs
entries under the root are remained same.

As of today(PR1841 is moving target), this is the changed location.

/sys/kernel/debug/sof/sof-ipc-test.0/
/sys/kernel/debug/sof/sof-ipc-test.1/

Debugfs file system detect might be required while this pr is getting review. One easy fix is just renaming /sys/kernel/debug/sof/ to /sys/kernel/debug/sof/sof-ipc-test.1/. When this PR is merged.

And as thesofproject/linux#1841 support multiple locations of debugfs for ipc flood test, we can improve more

  1. test with /sys/kernel/debug/sof/sof-ipc-test.0/
  2. test with /sys/kernel/debug/sof/sof-ipc-test.1/
  3. test both client1 and client2 simultaneously. This will be nice to see

The PR i made, #187, i dropped debugfs location change. I only pushed kmod change required from PR1841

cc:

Activity

  1. Bin-QA commented on Apr 16, 2020

    @Bin-QA
    Contributor

    How about use find /sys/kernel/debug/ -name ipc_flood_count to catch ipc_flood_count file ?

  2. fredoh9 commented on Apr 21, 2020

    @fredoh9
    ContributorAuthor

    How about use find /sys/kernel/debug/ -name ipc_flood_count to catch ipc_flood_count file ?

    Yes, that will work too. But it might hit multiple locations also. (see above description) So take first one should be same as current test.

  3. added
    P2Critical bugs or normal features
    on Jun 22, 2023
  4. marc-hb commented on Jun 22, 2023

    @marc-hb
    Collaborator

    @fredoh9 is this still an issue? BTW I noticed that check-ipc-flood is always skipped with IPC4

  5. marc-hb commented on Aug 13, 2024

    @marc-hb
    Collaborator

    There is a symbolic link that is currently in use. Problem solved, @fredoh9 can we close this?

    root@jf-cml-sku0983-sdw-1:~# file /sys/kernel/debug/sof/ipc_flood_count
    /sys/kernel/debug/sof/ipc_flood_count: symbolic link to snd_sof.ipc_flood.0/ipc_flood_count
    
  6. fredoh9 commented on Aug 14, 2024

    @fredoh9
    ContributorAuthor

    yes, we can close this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

P2Critical bugs or normal featuresstate:ping-for-closeWill be closed if submitter is not responsivetype:enhancementNew framework feature or request

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions