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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id E2F70C433F5 for ; Fri, 1 Apr 2022 15:00:56 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 556818D0002; Fri, 1 Apr 2022 11:00:46 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 504D36B0073; Fri, 1 Apr 2022 11:00:46 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3CD158D0002; Fri, 1 Apr 2022 11:00:46 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0028.hostedemail.com [216.40.44.28]) by kanga.kvack.org (Postfix) with ESMTP id 2E7376B0072 for ; Fri, 1 Apr 2022 11:00:46 -0400 (EDT) Received: from smtpin22.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id E12DD821EAD8 for ; Fri, 1 Apr 2022 15:00:35 +0000 (UTC) X-FDA: 79308621672.22.95E4549 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by imf19.hostedemail.com (Postfix) with ESMTP id 0D4D51A0033 for ; Fri, 1 Apr 2022 15:00:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1648825234; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=U4C2FtsuZxf0/Q/b0iTYDkQBo4MdhWNl+MwxPx5dtPM=; b=K7Ea6lP6QEdR5wTTtcDpKQCI4g047P5cnb5d3aZ2aIB9xiBa+6ul7QNHTAD1UUYibTdNRL Jjf2kgHCYkUaqegEpZzZnrD28wuNhOudXwBa6vl2ZvPAEMHowDOKe+DFiD0yujhuzE3mTH uo7i+8xQHAcZZIy8HzdZ7vhIeUIPhfQ= Received: from mail-ed1-f69.google.com (mail-ed1-f69.google.com [209.85.208.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-512-U9Y1FQ69MiiR-zpMtnEkwA-1; Fri, 01 Apr 2022 11:00:33 -0400 X-MC-Unique: U9Y1FQ69MiiR-zpMtnEkwA-1 Received: by mail-ed1-f69.google.com with SMTP id x1-20020a50f181000000b00418f6d4bccbso1688371edl.12 for ; Fri, 01 Apr 2022 08:00:33 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:references:from:in-reply-to :content-transfer-encoding; bh=U4C2FtsuZxf0/Q/b0iTYDkQBo4MdhWNl+MwxPx5dtPM=; b=S9/3nkT9f88T9BwEJdSe3DwhmU6GZ2eQ1AjthS/tDPnI/OGAzbbPT2Us1Doiuj+6sQ CxG8qn1JpO5OQHqbwLlhr0LLt5VR0dD6hfqitvv6a8kpVn+ExvF4sheYO1cVQEpbD4VZ IBBQ0v4z4lU74vv6XYRmmpiuJOweIjGRx3FfuPhQCIz/RoI4OUSqXFHGfj8z4Qgauvug Iwz8xA6solh36tS02IRTP6RRmU9Mdsg4R31zkONPW6DP4/Y2PJ/22xcDg+gYlMZRIUs2 4YswrtF4nnqOqTXV/uzNbOLji6nIVua/Hv/ipT3L2S2Htpy/jU4Z0ewLqElsmASpKSWk Os+g== X-Gm-Message-State: AOAM531SYzNWzUoEPYhywRvMl4mcfwjsOTuVwsb36YBK2zBD2L+GLs0I BpWNMl7JNiEgpEFp6hipTXXSc9/bYvrCkSAMuUBXsj90mC6Jxtj48k7sRnbjcLSFlVfqWeTjqoX 12LoQ83P/Ucw= X-Received: by 2002:a05:6402:27d2:b0:419:1a3d:442b with SMTP id c18-20020a05640227d200b004191a3d442bmr20841608ede.409.1648825232168; Fri, 01 Apr 2022 08:00:32 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzVAReUv2SvSmu/TQtO7iDA9+C4zYyJ+bufOOhAWSyh+evkJ1N0aCK5WKLJdbVvtRrCxIoc7A== X-Received: by 2002:a05:6402:27d2:b0:419:1a3d:442b with SMTP id c18-20020a05640227d200b004191a3d442bmr20841583ede.409.1648825231922; Fri, 01 Apr 2022 08:00:31 -0700 (PDT) Received: from ?IPV6:2001:1c00:c1e:bf00:1db8:22d3:1bc9:8ca1? (2001-1c00-0c1e-bf00-1db8-22d3-1bc9-8ca1.cable.dynamic.v6.ziggo.nl. [2001:1c00:c1e:bf00:1db8:22d3:1bc9:8ca1]) by smtp.gmail.com with ESMTPSA id gb29-20020a170907961d00b006e00c7b0f5asm1096476ejc.0.2022.04.01.08.00.30 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 01 Apr 2022 08:00:31 -0700 (PDT) Message-ID: Date: Fri, 1 Apr 2022 17:00:30 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 Subject: Re: mmotm 2022-03-30-13-01 uploaded (drivers/platform/x86/amd-pmc.o) To: "Limonciello, Mario" , Randy Dunlap , Andrew Morton , "broonie@kernel.org" , "mhocko@suse.cz" , "sfr@canb.auug.org.au" , "linux-next@vger.kernel.org" , "linux-fsdevel@vger.kernel.org" , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" , "mm-commits@vger.kernel.org" , "platform-driver-x86@vger.kernel.org" References: <20220330200158.2F031C340EC@smtp.kernel.org> <5a0b94c3-406e-463e-d93e-d1dc2a260b47@infradead.org> From: Hans de Goede In-Reply-To: X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Stat-Signature: 6591kq6t4s4zfnwbr5uu3shswspd6ynw Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=K7Ea6lP6; dmarc=pass (policy=none) header.from=redhat.com; spf=none (imf19.hostedemail.com: domain of hdegoede@redhat.com has no SPF policy when checking 170.10.133.124) smtp.mailfrom=hdegoede@redhat.com X-Rspam-User: X-Rspamd-Server: rspam11 X-Rspamd-Queue-Id: 0D4D51A0033 X-HE-Tag: 1648825234-640401 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Hi Mario, On 3/31/22 20:41, Limonciello, Mario wrote: > [AMD Official Use Only] > >> -----Original Message----- >> From: Randy Dunlap >> Sent: Thursday, March 31, 2022 10:51 >> To: Andrew Morton ; broonie@kernel.org; >> mhocko@suse.cz; sfr@canb.auug.org.au; linux-next@vger.kernel.org; linux- >> fsdevel@vger.kernel.org; linux-mm@kvack.org; linux- >> kernel@vger.kernel.org; mm-commits@vger.kernel.org; platform-driver- >> x86@vger.kernel.org; Limonciello, Mario >> Subject: Re: mmotm 2022-03-30-13-01 uploaded (drivers/platform/x86/amd- >> pmc.o) >> >> >> >> On 3/30/22 13:01, Andrew Morton wrote: >>> The mm-of-the-moment snapshot 2022-03-30-13-01 has been uploaded to >>> >>> >> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fww >> w.ozlabs.org%2F~akpm%2Fmmotm%2F&data=04%7C01%7Cmario.limo >> nciello%40amd.com%7Caa3aae02b7b6437c46ea08da132e4222%7C3dd8961fe >> 4884e608e11a82d994e183d%7C0%7C0%7C637843386674652995%7CUnknown >> %7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1ha >> WwiLCJXVCI6Mn0%3D%7C3000&sdata=WpfXOyTRBgvqCj3iZ%2BJjXVTb >> V%2FUWDP4ds5XtDfa5bPc%3D&reserved=0 >>> >>> mmotm-readme.txt says >>> >>> README for mm-of-the-moment: >>> >>> >> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fww >> w.ozlabs.org%2F~akpm%2Fmmotm%2F&data=04%7C01%7Cmario.limo >> nciello%40amd.com%7Caa3aae02b7b6437c46ea08da132e4222%7C3dd8961fe >> 4884e608e11a82d994e183d%7C0%7C0%7C637843386674652995%7CUnknown >> %7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1ha >> WwiLCJXVCI6Mn0%3D%7C3000&sdata=WpfXOyTRBgvqCj3iZ%2BJjXVTb >> V%2FUWDP4ds5XtDfa5bPc%3D&reserved=0 >>> >>> This is a snapshot of my -mm patch queue. Uploaded at random hopefully >>> more than once a week. >>> >>> You will need quilt to apply these patches to the latest Linus release (5.x >>> or 5.x-rcY). The series file is in broken-out.tar.gz and is duplicated in >>> >> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fozla >> bs.org%2F~akpm%2Fmmotm%2Fseries&data=04%7C01%7Cmario.limon >> ciello%40amd.com%7Caa3aae02b7b6437c46ea08da132e4222%7C3dd8961fe4 >> 884e608e11a82d994e183d%7C0%7C0%7C637843386674652995%7CUnknown >> %7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1ha >> WwiLCJXVCI6Mn0%3D%7C3000&sdata=adnYHchdMvieQ0lCqQ1er1jhHA >> UMomgFpaWBo%2BMnINk%3D&reserved=0 >>> >> >> on x86_64: >> when CONFIG_SUSPEND is not set: >> >> drivers/platform/x86/amd-pmc.o: in function `amd_pmc_remove': >> amd-pmc.c:(.text+0x11d): undefined reference to >> `acpi_unregister_lps0_dev' >> ld: drivers/platform/x86/amd-pmc.o: in function `amd_pmc_probe': >> amd-pmc.c:(.text+0x20be): undefined reference to `acpi_register_lps0_dev' >> >> >> >> -- >> ~Randy > > AFAICT you're missing 20e1d6402a71dba7ad2b81f332a3c14c7d3b939b. That is unlikely since the whole series got merged through the pdx86/for-next branch, so either some other tree has the whole series or none of it. Also note that Randy wrote: "when CONFIG_SUSPEND is not set" The problem is that all of drivers/acpi/x86/s2idle.c is wrapped by #ifdef CONFIG_SUSPEND ... #endif Since AFAIK the whole amd-pmc driver's whole purpose is to allow s2idle to work properly. I think this can be fixed by simply doing: --- a/drivers/platform/x86/Kconfig +++ b/drivers/platform/x86/Kconfig @@ -198,7 +198,7 @@ config ACER_WMI config AMD_PMC tristate "AMD SoC PMC driver" - depends on ACPI && PCI && RTC_CLASS + depends on ACPI && SUSPEND && PCI && RTC_CLASS help The driver provides support for AMD Power Management Controller primarily responsible for S2Idle transactions that are driven from If you agree, please submit a patch with this change and I'll queue it up in my fixes branch. Regards, Hans