Skip to content

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Jun 13, 2023

This is a partial fix for #7709; doc updates still to be done.

Quoting @cujomalainey in

I still don't see the benefit here. This branch and IPC3 are mostly
stable/archived. Hence anything we fuzz on main we should be able to
cherry-pick down. Nothing new should be landing on this branch so I
don't see the benefit of continued fuzzing past support on main.

Quoting @andyross in

I guess my gut says that fuzzing is a technique for validation of
new code. It has minimal (but sure, not zero) value on maintenance
branches that are expected to be protocol-compatible in perpetuity.

This is a partial fix for thesofproject#7709; doc updates still to be done.

Quoting @cujomalainey in
- thesofproject#7699
> I still don't see the benefit here. This branch and IPC3 are mostly
> stable/archived. Hence anything we fuzz on main we should be able to
> cherry-pick down. Nothing new should be landing on this branch so I
> don't see the benefit of continued fuzzing past support on main.

Quoting @andyross in
- thesofproject#7675
> I guess my gut says that fuzzing is a technique for validation of
> new code. It has minimal (but sure, not zero) value on maintenance
> branches that are expected to be protocol-compatible in perpetuity.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb marc-hb changed the title [SKIP SOF-TEST] .github: disable fuzzing [SKIP SOF-TEST] [STABLE-v2.2] .github: disable fuzzing Jun 13, 2023
@marc-hb
Copy link
Collaborator Author

marc-hb commented Jun 13, 2023

Unrelated Github glitches in https://github.com/thesofproject/sof/actions/runs/5260133903/jobs/9506630809?pr=7795

Everything else green.

@cujomalainey cujomalainey requested a review from mwasko June 13, 2023 22:51
@kv2019i kv2019i merged commit 3ddd15c into thesofproject:stable-v2.2 Jun 14, 2023
@marc-hb marc-hb deleted the no-22-fuzz branch July 12, 2023 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants