On Thu, Jun 6, 2024 at 11:08 AM Deepthi H <deepthi.hem...@windriver.com> wrote:
>
>
> 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
> <randy.macl...@windriver.com> 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.

Sounds good, I'll watch for it tomorrow!

Steve

> $ 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 (#200414): 
https://lists.openembedded.org/g/openembedded-core/message/200414
Mute This Topic: https://lists.openembedded.org/mt/106504365/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to