On 6/5/2024 9:19 PM, Steve Sakoman wrote:
CAUTION: This email comes from a non Wind River email account!
Do not click links or open attachments unless you recognize the sender and know 
the content is safe.

On Wed, Jun 5, 2024 at 8:42 AM Randy MacLeod
<[email protected]>  wrote:
On 2024-06-05 10:56 a.m., Hemraj, Deepthi via lists.openembedded.org wrote:

Regression testing was done and below are the test results:
After update (v13.3)

=== gcc Summary ===

# of expected passes            248620
# of unexpected failures        9
# of expected failures          2484
# of unresolved testcases    25028
# of unsupported tests         4145

Before update (v13.2)

=== gcc Summary ===

# of expected passes            148304
# of unexpected failures        15
# of expected failures          1420
# of unresolved testcases     25136
# of unsupported tests          2688

Here are the computed differences for each category:

Expected Passes: +100316
Unexpected Failures: -6
Expected Failures: +1064
Unresolved Testcases: -108
Unsupported Tests: +1457

(Thanks to ChatGPT 4o !  Please do present the differences next time to save 
me, ChatGPT, and others from the task. )


Can you explain the huge increase expected passes?
i.e. did upstream really add ~100316 new tests?

Ah, you have a typo, and expected passes for 13.3 should be 148620, based on:
    https://lists.openembedded.org/g/openembedded-core/message/200008

The number of unexpected failures has decreased so that's good.


Also, did you check on the build problems that Steve reported?

That should be discussed in a cover letter for v2, v3, ...

Please learn how to write a v2 cover letter and send that with git:
$ git send-email -1 --cover-letter

Also, I saved your v2 patch and tried to apply it to scarthgap HEAD and that 
failed with:
I had no issues applying the patch when taken from patchworks, only at
build time when applying the CVE-2024-0151 patch:

CmdError('quilt --quiltrc
TOPDIR/tmp/work-shared/gcc-13.3.0-r0/recipe-sysroot-native/etc/quiltrc
push', 0, 'stdout: Applying patch CVE-2024-0151.patch
patching file gcc/config/arm/arm.cc
Hunk #1 succeeded at 19157 with fuzz 2 (offset -53 lines).
Hunk #2 succeeded at 23668 with fuzz 2 (offset -8 lines).
The next patch would create the file
gcc/testsuite/gcc.target/arm/cmse/extend-param.c,
which already exists!  Applying it anyway.
patching file gcc/testsuite/gcc.target/arm/cmse/extend-param.c
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- rejects in file
gcc/testsuite/gcc.target/arm/cmse/extend-param.c
The next patch would create the file
gcc/testsuite/gcc.target/arm/cmse/extend-return.c,
which already exists!  Applying it anyway.
patching file gcc/testsuite/gcc.target/arm/cmse/extend-return.c
Hunk #1 FAILED at 1.
1 out of 1 hunk FAILED -- rejects in file
gcc/testsuite/gcc.target/arm/cmse/extend-return.c
Patch CVE-2024-0151.patch can be reverse-applied

So I suspect that V3 of the patch needs to also remove
CVE-2024-0151.patch, since the version bump seems to include the fix.

Steve
Hi Steve,

I was able to reproduce the issue. since the patch CVE-2024-0151.patch was 
added yesterday after I had committed the changes, the above issue was caused I 
will send a V3 tomorrow. Thank you, Deepthi

$ git status
On branch scarthgap
Your branch is up to date with 'origin/scarthgap'.

$ git am /tmp/gcc-13.3-v2.eml
Applying: gcc : upgrade to v13.3
.git/rebase-apply/patch:121: trailing whitespace.
@@ -31,16 +31,16 @@
error: patch failed: meta/recipes-devtools/gcc/gcc-13.2.inc:65
error: meta/recipes-devtools/gcc/gcc-13.2.inc: patch does not apply
Patch failed at 0001 gcc : upgrade to v13.3
hint: Use 'git am --show-current-patch=diff' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Looking at the diff between v1 and v2, it seems that you have some line wrap 
issues
and perhaps more. Please test for this by using git send-email to send to 
yourself or
someone who is using a sane (non-MS Outlook) email client and save the patch 
and apply it
to an unpatched poky repo.


Thanks,

../Randy





--
# Randy MacLeod
# Wind River Linux
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#200413): 
https://lists.openembedded.org/g/openembedded-core/message/200413
Mute This Topic: https://lists.openembedded.org/mt/106504365/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to