• tal
      link
      fedilink
      72 years ago

      If the Daily Mail’s editors do it, it must be a good idea!

    • @[email protected]
      link
      fedilink
      English
      382 years ago

      I can not tell you how many videos and channels I avoid because of that shit.

      also ones that have dumb ass thumbnails that do nothing to hint about the content.

    • poVoq
      link
      fedilink
      82 years ago

      AMD’s software has always been in a very sorry state. Only after non-inhouse developers created an alternative GPU driver from scratch for Linux it became feasible to use.

    • @[email protected]
      link
      fedilink
      English
      52 years ago

      Apparently there’s a bug in an AMD’s driver. It was supposed to assign processes based on each core’s self reported performance, but because of the bug it was random.

      This “self reported performance” is based on evaluation done to the cores in the fab process, by AMD. Meaning, due to imperfections some cores are a bit better than others.

      • @[email protected]
        link
        fedilink
        9
        edit-2
        2 years ago

        It’s not just random, it simply does not even work. Because they set this:

        +/*Preferred Core featue is supported*/
        +static bool prefcore = true;
        

        And later in the code they do the if condition wrong:

        +	if (prefcore)
        +		WRITE_ONCE(cpudata->highest_perf, AMD_PSTATE_PREFCORE_THRESHOLD);
        +	else
        +		WRITE_ONCE(cpudata->highest_perf, AMD_CPPC_HIGHEST_PERF(cap1));
        

        if should look like this:

        +	if (prefcore)
        +		WRITE_ONCE(cpudata->highest_perf, AMD_CPPC_HIGHEST_PERF(cap1));
        +	else
        +		WRITE_ONCE(cpudata->highest_perf, AMD_PSTATE_PREFCORE_THRESHOLD);
        

        There is probably even more wrong, looking at the code quality, but this at least makes the preferred core work.

        • @[email protected]
          link
          fedilink
          English
          2
          edit-2
          2 years ago

          I don’t know if we’re discussing semantics. A performance score is attributed, and before the fix their scores were all 166. It doesn’t work, as you said. So the consequence is the preferred core being “random”, isn’t it?

          • @[email protected]
            link
            fedilink
            12 years ago

            I don’t know if it’s random, the CPU scheduler still decides what thread to use. It will have its own semantics, but I don’t know on what those are based.

    • @[email protected]
      link
      fedilink
      162 years ago

      AMD patches for preferred core (prefer those cores which can clock higher) are a mess and ended up not working because of a wrong if condition. Showing that no one at AMD even tested it before submitting. The programmer in the video complains about AMDs developers being incompetent and shows how it’s fixed.

  • @[email protected]OP
    link
    fedilink
    22
    edit-2
    2 years ago

    I am not the creator of the video but amazed what it all makes it into the stable branch.