Wrong revert commit in stable channel

[Yury Zhuravlev]

Yes, this is right in 6.2-rc3

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit
/drivers/gpu/drm/amd/pm/powerplay/hwmgr?h=v6.2-rc3&id=f936f535fa70f35ce3369b1418ebae0e657cda6a


But somebody reverted it again for the stable stream:

https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?h=v6.1.4&id=9ccd11718d76b95c69aa773f2abedef560776037

On Wed, Jan 11, 2023 at 5:35 PM Chen, Guchun <Guchun.Chen at amd.com> wrote:

> Hi Yury,
>
>
>
> My understanding is though that’s a revert of your original patch, we did
> a revert again on top of the reverted patch later on. Can you please sync
> to below commit to check again? Or do I understand wrong?
>
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/gpu/drm/amd/pm/powerplay/hwmgr?h=v6.2-rc3&id=f936f535fa70f35ce3369b1418ebae0e657cda6a
>
>
>
> Regards,
>
> Guchun
>
>
>
> *From:* amd-gfx <amd-gfx-bounces at lists.freedesktop.org> *On Behalf Of *Yury
> Zhuravlev
> *Sent:* Wednesday, January 11, 2023 4:26 PM
> *To:* amd-gfx at lists.freedesktop.org
> *Subject:* Wrong revert commit in stable channel
>
>
>
> Hello,
>
>
>
> Something went wrong, and we commited what we diced not commit.
>
>
>
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.2-rc3&id=e5b781c56d46c44c52caa915f1b65064f2f7c1ba
> <https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2Fcommit%2F%3Fh%3Dv6.2-rc3%26id%3De5b781c56d46c44c52caa915f1b65064f2f7c1ba&data=05%7C01%7Cguchun.chen%40amd.com%7C9b399d97a2624b5bc46e08daf3ad88a1%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C638090223898249455%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=%2FkLymTjCMQ3uC669%2BWNXePiZ9ysaAGdvm7gFw5kIGeQ%3D&reserved=0>
>
>
>
> and
>
>
>
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.2-rc3&id=4545ae2ed3f2f7c3f615a53399c9c8460ee5bca7
> <https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Ftorvalds%2Flinux.git%2Fcommit%2F%3Fh%3Dv6.2-rc3%26id%3D4545ae2ed3f2f7c3f615a53399c9c8460ee5bca7&data=05%7C01%7Cguchun.chen%40amd.com%7C9b399d97a2624b5bc46e08daf3ad88a1%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C638090223898249455%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=I57dx%2FORRXDeu0TTENlejr0IcuSncVHfy7LCPOJNekw%3D&reserved=0>
>
>
>
> It's wrong reverts because, initially was an issue with a test case, not a
> patch itself.
>
> My GPU is not working correctly again after such "stable" patch.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20230113/a3c86aa9/attachment.htm>