Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <f72f7325-adc6-89d5-7cbc-647442308233@amd.com>
Date:   Thu, 22 Sep 2022 09:31:12 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCHv7 02/14] mm: Add support for unaccepted memory
Content-Language: en-US
To:     Dionna Amalie Glaze <dionnaglaze@google.com>,
        "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>
Cc:     Dave Hansen <dave.hansen@intel.com>,
        Mel Gorman <mgorman@techsingularity.net>,
        Vlastimil Babka <vbabka@suse.cz>,
        Borislav Petkov <bp@alien8.de>,
        Andy Lutomirski <luto@kernel.org>,
        Sean Christopherson <seanjc@google.com>,
        Andrew Morton <akpm@linux-foundation.org>,
        Joerg Roedel <jroedel@suse.de>,
        Ard Biesheuvel <ardb@kernel.org>,
        Andi Kleen <ak@linux.intel.com>,
        Kuppuswamy Sathyanarayanan 
        <sathyanarayanan.kuppuswamy@linux.intel.com>,
        David Rientjes <rientjes@google.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Peter Zijlstra <peterz@infradead.org>,
        Paolo Bonzini <pbonzini@redhat.com>,
        Ingo Molnar <mingo@redhat.com>,
        Dario Faggioli <dfaggioli@suse.com>,
        Mike Rapoport <rppt@kernel.org>,
        David Hildenbrand <david@redhat.com>,
        Marcelo Cerri <marcelo.cerri@canonical.com>,
        tim.gardner@canonical.com,
        Khalid ElMously <khalid.elmously@canonical.com>,
        philip.cox@canonical.com,
        the arch/x86 maintainers <x86@kernel.org>,
        Linux Memory Management List <linux-mm@kvack.org>,
        linux-coco@lists.linux.dev, linux-efi <linux-efi@vger.kernel.org>,
        LKML <linux-kernel@vger.kernel.org>,
        Mike Rapoport <rppt@linux.ibm.com>
References: <20220614120231.48165-3-kirill.shutemov@linux.intel.com>
 <8cf143e7-2b62-1a1e-de84-e3dcc6c027a4@suse.cz>
 <20220810141959.ictqchz7josyd7pt@techsingularity.net>
 <CAAH4kHa6s3sBRySNu-TZG_6vOaN4KheVy4kvxG5s=wOTDGy2=Q@mail.gmail.com>
 <2981e25e-9cda-518a-9750-b8694f2356b5@amd.com>
 <CAAH4kHbcfnVWNQHf6Mrg__bSFT6196Sx4kno6o0Zo7hsgOgnNw@mail.gmail.com>
 <984e07ed-914f-93ca-a141-3fc8677878e0@intel.com>
 <CAAH4kHawguTEuDVyz1ysSbH0X_mT=SvxLi=UhwEzXM0abbWefg@mail.gmail.com>
 <YxncAElGrPEGRYg1@linux.ibm.com>
 <CAAH4kHaP8JUh0Z4rF83=2RZTGMATT5MHot6rAnAwt79PL64mVQ@mail.gmail.com>
 <YxpCaQARczhZQmq2@linux.ibm.com>
From:   Tom Lendacky <thomas.lendacky@amd.com>
In-Reply-To: <YxpCaQARczhZQmq2@linux.ibm.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Szhzc2I1MmdiZi9Rd29GaHRqRDI0K2xTMzNQV1MxODFLTWVub1ZWVTFSdnFY?=
 =?utf-8?B?TmdtemEycjlqMmhhcXJDa3UwWEYwWEFONmM2c3lWb1F2OVVPWnlpYXZFTXhV?=
 =?utf-8?B?NjhXcnQySStHSGFrQmF6QmZQOGRrcENBY3pRL0VYV1h6VzAxSkhUVlFDek1s?=
 =?utf-8?B?UHZuNm9QNlhVSTMzcVl6QjM4bG0rTzlvZUdHSlFtamkydzgyV1kvSmpmWDVC?=
 =?utf-8?B?TVBjZks2SGtlWEFWUko5N3R4Vm5pNklRNGdpd1BwcGRucS9Qd2w1blY0MW95?=
 =?utf-8?B?VGdCYS9qYmMzTDdSRFhQWVp2bDJVSkpiNmRjTkUxeS9WZnFaSVUyMWFkb1Q5?=
 =?utf-8?B?cXk5K09PSDR1N0pldGVyUUZYRS9FbHdOS3RwL0JKaW1yMnJocDR5ZFBIbjlF?=
 =?utf-8?B?dVNzQmpDdkM1dVBPeENnRlNDTm9UR2pLNk9jeWYzYXNBMVZURlZMckpxR2RP?=
 =?utf-8?B?dHE2Vy80b1lwU0RGNDlva2ZKdThtZnVnNnNaaUlDNk43SVdya0ROTW8xNzZW?=
 =?utf-8?B?OFo5Uk1pMmVzaERZN05HZ3AvcWhyTlJKc1IwQlJoeERoUUhUY0s0QTBacXNO?=
 =?utf-8?B?bVNIa2tqQ0R6YVJnaExhS29qMjhsZkJXNDNpeFVRaVdCRnAxb2FNMmtub2JG?=
 =?utf-8?B?dlpiNis3cXhLQSsvaEN6eDVjU05MVVVXL0pLbGxodG1vYlY0SUtGUTBaNStF?=
 =?utf-8?B?cVQ3WDRPempLWVpPTXNobEM3RndJR2x3NThrT3VFVVNnbG83dC9DUTVZZHY1?=
 =?utf-8?B?VXFjT1Z3dmhZU2hGMFRhb1ptN0k3YUhJRE9Jc0crWGpISmR4dDRxMGV0ZjMy?=
 =?utf-8?B?S2dMQytOYzlNUys1MWtuTHk0M0M5OFEzZE4xWU5BRnFDQnZHdnA0SHAzRnRa?=
 =?utf-8?B?aUhxL3dsVHgwSzFCZDUxN0JnNUp2U3J0bFRsV2pOQU5SakIzaldldE9NbDNC?=
 =?utf-8?B?UG5KeTZYYlNQWGRTWWwya2luOU5rV2NrbDAwZnJzT2NoOGNhTDI4Ynl4L1VC?=
 =?utf-8?B?VUs2K29HK1VBek85alpkRXEwZ1RvVHVvYjJ3KysyNndkV0dEMGtsd2hlamht?=
 =?utf-8?B?SVhGWEtudWZuSXJWbkc4dVNSeTdBczFrK2l5MzdhVTc4ZHNQNjlqN2x2TUVQ?=
 =?utf-8?B?N0dpT2dXTDhHVHBiTytVUERObXhQQ3QxeTEwQk5jeFBoOWdZOUZhM0s2UGVt?=
 =?utf-8?B?bDk3OTB0V0RhbUpJRTAxWVN2T0VQYVR3V0tLSHBqeldNa0lUUVpaczc2ZUIv?=
 =?utf-8?B?Tk5TTUVSZVZqM2d0OERybnNnbTRKZ21wK1lSbVJZSTl1eEZwbE1nNUVzbU55?=
 =?utf-8?B?UnZqR2FKQW1PRWVjTHRYbG1reHMxNVYrOVdBU2lhL2pMQ0FPQldqdWh6ZE9R?=
 =?utf-8?B?Q3VMVDRxZUt3d2xBUnBhUDlnazA1a240aFBlZ3pNT21aYTlyanVzNS9Ganpk?=
 =?utf-8?B?SVJaL0VtRjQxV0ovUTNTb0hzQW8rODl0cjZZdnAyRUVLL203QUlVcUZUYllP?=
 =?utf-8?B?azVGSG1XR2FUWVg4Z3EveE1BZVBrVjM4dzVmaVpsVDA0bkF4aENSRWdoNWkr?=
 =?utf-8?B?M0xiQjFiZkpLc2NYOTNTUmVnZVJpV2J3dGthcHdZSzFUS0o3SzF6Zm1OQlVX?=
 =?utf-8?B?cG5FbzhLRU1PeUtSaGxzUDhnczErMDQ1LzByL0phdlNrNzRKZDFmSDZPd05v?=
 =?utf-8?B?YisySHRsaUpEQTllV2dYb2hqTFVMS0ZjRTRHam9uclY5NS94RFV0ZW5DdUFE?=
 =?utf-8?B?d1pQSWNWcGlmRElKT0RtLytBNzc1cmVrWVNORHpwa2xsWVp0ZzZxSUo3OEZv?=
 =?utf-8?B?RXdpZElTQmplMmVqd3dyRVorOFdCeUc4VVN1VEtYNVNoak12b09YdlE2WWpG?=
 =?utf-8?B?MnVMcXpwc1BhdmwxZWlZOWdPYmZ5RU9xblpocUt6MFNiMUw5Qk9hV09QN2V4?=
 =?utf-8?B?TzZrdzlGNkdiRU1rZ3htdVZqSjJQQWRrRW0xK3BQYTQ1VitJRUlOeHV2ZnBV?=
 =?utf-8?B?bS9EQkNGRFVpRWNvbXFidTVLQXFCbElnL3hjNlYveUpub29oSlJOUmRhS2tv?=
 =?utf-8?B?c2pwM280N2F5RzVvS3RjcHkyY1RmUzA4eW9kZUVpK0RyTlhuWE1zN1liMnlo?=
 =?utf-8?Q?DrI+c/pwQtgkSa8AD+5i1LcrA?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 5864a4ec-a94b-4eda-15e3-08da9ca71b93
X-MS-Exchange-CrossTenant-AuthSource: DM4PR12MB5229.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Sep 2022 14:31:16.4629
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: gH71Y7hMTqHifGHJMVppPsihNEBh3/lm+LYljJ2sTXpL/Jxfrnjk12mSCMrhg9sQ/QfFOZnWX88Hj5sjx6hEMA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5819
X-Spam-Status: No, score=-4.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 9/8/22 14:28, Mike Rapoport wrote:
> On Thu, Sep 08, 2022 at 09:23:07AM -0700, Dionna Amalie Glaze wrote:
>>>
>>> Looks like the first access to the memory map fails, although I think
>>> it's not in INIT_LIST_HEAD() but rather in init_page_count().
>>>
>>> I'd start with making sure that page_alloc::memmap_alloc() actually returns
>>> accepted memory. If you build kernel with CONFIG_DEBUG_VM=y the memory map
>>> will poisoned in this function, so my guess is it'd crash there.
>>>
>>
>> That's a wonderful hint, thank you! I did not run this test
>> CONFIG_DEBUG_VM set, but you think it's possible it could still be
>> here?
> 
> It depends on how you configured your kernel. Say, defconfig does not set
> it.
> 

I also hit the issue at 256GB. My config is using CONFIG_SPARSEMEM_VMEMMAP 
and fails in memmap_init_range() when attempting to add the first PFN. It 
looks like the underlying page that is backing the vmemmap has not been 
accepted (I receive a #VC 0x404 => page not validated).

Kirill, is this a path that you've looked at? It would appear that 
somewhere in the vmemmap_populate_hugepages() path, some memory acceptance 
needs to be done for the pages that are used to back vmemmap. I'm not very 
familiar with this code, so I'm not sure why everything works for a guest 
with 255GB of memory, but then fails for a guest with 256GB of memory.

Thanks,
Tom