From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7B9A0C433F5 for ; Thu, 14 Oct 2021 09:00:48 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 5BF0A611C1 for ; Thu, 14 Oct 2021 09:00:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230007AbhJNJCu (ORCPT ); Thu, 14 Oct 2021 05:02:50 -0400 Received: from mail.skyhub.de ([5.9.137.197]:47480 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229984AbhJNJCr (ORCPT ); Thu, 14 Oct 2021 05:02:47 -0400 Received: from zn.tnic (p200300ec2f0c72004cd8e1678c0b16c3.dip0.t-ipconnect.de [IPv6:2003:ec:2f0c:7200:4cd8:e167:8c0b:16c3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.skyhub.de (SuperMail on ZX Spectrum 128k) with ESMTPSA id 8BFA11EC051F; Thu, 14 Oct 2021 11:00:40 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=alien8.de; s=dkim; t=1634202040; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:in-reply-to:in-reply-to: references:references; bh=uSz2rZqEKZN/XdUySfMH+peSlaVrF7M5smPXGUk4r04=; b=py0fiiXqmN4jbKVkmkgXgCjskD/5/p9CDRsJlpMaz9oqtYXNXNUsNehQrYc48UOPDwzUEz jBdJHy8+T7bSJqN9Rab31QNGnNChHOiW2EPvJ11ToTdZxLRfc2eVRje4E/vjnc9FxuGyqs 43/6QqBPR8VM/LPLAs3OrDmgrv6u+5Q= Date: Thu, 14 Oct 2021 11:00:37 +0200 From: Borislav Petkov To: "Quan, Evan" Cc: Alex Deucher , amd-gfx list , LKML , "Deucher, Alexander" , "Pan, Xinhui" , "Chen, Guchun" Subject: Re: bf756fb833cb ("drm/amdgpu: add missing cleanups for Polaris12 UVD/VCE on suspend") Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Oct 14, 2021 at 02:02:48AM +0000, Quan, Evan wrote: > [Quan, Evan] Yes, but not(apply them) at the same time. One by one as you did before. > - try the patch1 first Ok, first patch worked fine. > - undo the changes of patch1 and try patch2 Did that, worked fine too except after the first resume cycle, the video didn't continue playing. Then I restarted the video and did a couple more suspend cycles to see if it would not continue again. In the subsequent tries it would resume fine and the video would continue playing too. So I'm going to chalk that single case of halted video with the second patch to a resume glitch or so. Btw, I don't have pm-suspend on that box but I did suspend to RAM assuming this is what you wanted, which is done as root with two commands: # echo "suspend" > /sys/power/disk # echo "mem" > /sys/power/state If you want me to do more extensive testing, just shoot. HTH. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette