Skip to content
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

[Perf] Linux/x64: 2 Regressions on 8/29/2024 5:45:13 PM #107305

Open
performanceautofiler bot opened this issue Sep 3, 2024 · 9 comments
Open

[Perf] Linux/x64: 2 Regressions on 8/29/2024 5:45:13 PM #107305

performanceautofiler bot opened this issue Sep 3, 2024 · 9 comments
Assignees
Labels
arch-x64 area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI os-linux Linux OS (any supported distro) runtime-coreclr specific to the CoreCLR runtime
Milestone

Comments

@performanceautofiler
Copy link

Run Information

Name Value
Architecture x64
OS ubuntu 22.04
Queue TigerUbuntu
Baseline d48751b92d5989f8b052ba4a5c20a385f3d4a983
Compare 2a552ab9d4bf82a93aef97b4750fd1d7c88007fa
Diff Diff
Configs CompilationMode:tiered, RunKind:micro

Regressions in System.Tests.Perf_Uri

Benchmark Baseline Test Test/Base Test Quality Edge Detector Baseline IR Compare IR IR Ratio
111.22 ns 128.66 ns 1.16 0.16 False
114.87 ns 131.64 ns 1.15 0.09 False

graph
graph
Test Report

Repro

General Docs link: https://github.com/dotnet/performance/blob/main/docs/benchmarking-workflow-dotnet-runtime.md

git clone https://github.com/dotnet/performance.git
python3 .\performance\scripts\benchmarks_ci.py -f net8.0 --filter 'System.Tests.Perf_Uri*'

System.Tests.Perf_Uri.Ctor(input: "http://xn--hst-sna.with.xn--nicode-2ya")

ETL Files

Histogram

JIT Disasms

System.Tests.Perf_Uri.Ctor(input: "https://a.much.longer.domain.name")

ETL Files

Histogram

JIT Disasms

Docs

Profiling workflow for dotnet/runtime repository
Benchmarking workflow for dotnet/runtime repository

@performanceautofiler performanceautofiler bot added arch-x64 os-linux Linux OS (any supported distro) runtime-coreclr specific to the CoreCLR runtime untriaged New issue has not been triaged by the area owner labels Sep 3, 2024
@LoopedBard3 LoopedBard3 transferred this issue from dotnet/perf-autofiling-issues Sep 3, 2024
@dotnet-issue-labeler dotnet-issue-labeler bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Sep 3, 2024
@LoopedBard3
Copy link
Member

Seems to be #106988 @EgorBo

@jeffschwMSFT jeffschwMSFT added the area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI label Sep 4, 2024
Copy link
Contributor

Tagging subscribers to this area: @JulieLeeMSFT, @jakobbotsch
See info in area-owners.md if you want to be subscribed.

@JulieLeeMSFT JulieLeeMSFT removed the untriaged New issue has not been triaged by the area owner label Sep 4, 2024
@JulieLeeMSFT JulieLeeMSFT added this to the 9.0.0 milestone Sep 4, 2024
@EgorBo
Copy link
Member

EgorBo commented Sep 6, 2024

@EgorBot -intel -commit e1851a5 vs previous --envvars DOTNET_JitDisasm:CtorBench

using System;
using BenchmarkDotNet.Attributes;
using System.Collections.Generic;
public class Perf_Uri
{
    public static IEnumerable<object[]> Ctor_MemberData()
    {
        yield return ["https://a.much.longer.domain.name"];
    }

    [Benchmark]
    [ArgumentsSource(nameof(Ctor_MemberData))]
    public Uri CtorBench(string input) => new Uri(input);
}

@EgorBo

This comment was marked as resolved.

@EgorBo
Copy link
Member

EgorBo commented Sep 6, 2024

Not my change as it seems, let me inspect others.

@EgorBo
Copy link
Member

EgorBo commented Sep 6, 2024

@EgorBot -intel -commit 80187a3 vs previous --envvars DOTNET_JitDisasm:CtorBench

using System;
using BenchmarkDotNet.Attributes;
using System.Collections.Generic;
public class Perf_Uri
{
    public static IEnumerable<object[]> Ctor_MemberData()
    {
        yield return ["https://a.much.longer.domain.name"];
    }

    [Benchmark]
    [ArgumentsSource(nameof(Ctor_MemberData))]
    public Uri CtorBench(string input) => new Uri(input);
}

@EgorBo
Copy link
Member

EgorBo commented Sep 6, 2024

@EgorBot -intel -commit 8b34fb0 vs previous --envvars DOTNET_JitDisasm:CtorBench

using System;
using BenchmarkDotNet.Attributes;
using System.Collections.Generic;
public class Perf_Uri
{
    public static IEnumerable<object[]> Ctor_MemberData()
    {
        yield return ["https://a.much.longer.domain.name"];
    }

    [Benchmark]
    [ArgumentsSource(nameof(Ctor_MemberData))]
    public Uri CtorBench(string input) => new Uri(input);
}

@EgorBo
Copy link
Member

EgorBo commented Sep 6, 2024

So only Tiger regressed judging by ADX page, looks like the method itself is no longer 32b aligned: https://www.diffchecker.com/0qUcqaWc marking as 10.0.0

@EgorBo EgorBo modified the milestones: 9.0.0, 10.0.0 Sep 6, 2024
@EgorBo
Copy link
Member

EgorBo commented Sep 6, 2024

@EgorBot -intel -commit 9c9f6ad vs previous --envvars DOTNET_JitDisasm:CtorBench

using System;
using BenchmarkDotNet.Attributes;
using System.Collections.Generic;
public class Perf_Uri
{
    public static IEnumerable<object[]> Ctor_MemberData()
    {
        yield return ["https://a.much.longer.domain.name"];
    }

    [Benchmark]
    [ArgumentsSource(nameof(Ctor_MemberData))]
    public Uri CtorBench(string input) => new Uri(input);
}

@vcsjones vcsjones removed the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Sep 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arch-x64 area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI os-linux Linux OS (any supported distro) runtime-coreclr specific to the CoreCLR runtime
Projects
None yet
Development

No branches or pull requests

5 participants