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 C31A2C54EAA for ; Mon, 30 Jan 2023 13:02:48 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4A3736B0072; Mon, 30 Jan 2023 08:02:48 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 4539B6B0073; Mon, 30 Jan 2023 08:02:48 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 31C216B0074; Mon, 30 Jan 2023 08:02:48 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 24BB76B0072 for ; Mon, 30 Jan 2023 08:02:48 -0500 (EST) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay07.hostedemail.com (Postfix) with ESMTP id E5E3F1604BB for ; Mon, 30 Jan 2023 13:02:47 +0000 (UTC) X-FDA: 80411479974.28.64D6693 Received: from smtp-out1.suse.de (smtp-out1.suse.de [195.135.220.28]) by imf15.hostedemail.com (Postfix) with ESMTP id D5A81A0022 for ; Mon, 30 Jan 2023 13:02:44 +0000 (UTC) Authentication-Results: imf15.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=pyJcnYwB; spf=pass (imf15.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.28 as permitted sender) smtp.mailfrom=mhocko@suse.com; dmarc=pass (policy=quarantine) header.from=suse.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1675083765; h=from:from:sender: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:dkim-signature; bh=ARwEGyGpwYF9H5slVIYtCFzph0d26JmSdEf25hcilbY=; b=7r1f4JUNqxawJ0rax8FNgOqCX16KGSa38cieUQdZPqK/+f6ir0GdX+jhfN06ZqBS0OHYfU Xyn2PS4iACbVoZKnv6Y63XvMIuqF4QNtdG109c0TrBxuFol6+R3CdzQ+U2HsKRiUvIZBEU zPUS4nzwv76bLS4R6ouk81Jqw0zH3yw= ARC-Authentication-Results: i=1; imf15.hostedemail.com; dkim=pass header.d=suse.com header.s=susede1 header.b=pyJcnYwB; spf=pass (imf15.hostedemail.com: domain of mhocko@suse.com designates 195.135.220.28 as permitted sender) smtp.mailfrom=mhocko@suse.com; dmarc=pass (policy=quarantine) header.from=suse.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1675083765; a=rsa-sha256; cv=none; b=7reGdhAfVt5G9/v6J1BnV2OXpXZHsMoaFiCbIoc5vVeho7BQKVHIaud9Hp4x7XiWSAQX/P QFozyvrwg4Hsd0G2SQ63cxYvpzdutehOfqrp2KwCB3E4YofDhzBFjFZ82B/lrG4MXCQKor eib6FoPTWTpF157PlpDZkCg9+M0iiSM= Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 3F4C121BB2; Mon, 30 Jan 2023 13:02:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=susede1; t=1675083763; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=ARwEGyGpwYF9H5slVIYtCFzph0d26JmSdEf25hcilbY=; b=pyJcnYwBl3DH/jl+/129tNUOIFyqyViNLmoDtFmngEAY4W5QwguCfIassNpLWPbUwRip5Q Z+RahvE57ghKk8jep3/EAoHBgMp7qdYPURDYQRlHKF4OEFFpfS8h7xHSkIfBx29ubvbPnC EUuasEmCb8a1ZTWBgJ4rSbPj1ooYlNQ= Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id 3007913A06; Mon, 30 Jan 2023 13:02:43 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id GpzmCvO/12NKUQAAMHmgww (envelope-from ); Mon, 30 Jan 2023 13:02:43 +0000 Date: Mon, 30 Jan 2023 14:02:42 +0100 From: Michal Hocko To: Kefeng Wang Cc: Andrew Morton , Tejun Heo , Jens Axboe , Jan Kara , Shakeel Butt , Naoya Horiguchi , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Ma Wupeng , shy828301@gmail.com Subject: Re: [PATCH] mm: memcg: fix NULL pointer in mem_cgroup_track_foreign_dirty() Message-ID: References: <20230129024451.121590-1-wangkefeng.wang@huawei.com> <20230129134815.21083b65ef3ae4c3e7fae8eb@linux-foundation.org> <568c10e8-c225-b3c4-483a-5bb3329de4c5@huawei.com> <13e4e6ee-414e-7e36-5ac1-fa0fa555ba41@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <13e4e6ee-414e-7e36-5ac1-fa0fa555ba41@huawei.com> X-Rspamd-Queue-Id: D5A81A0022 X-Stat-Signature: erzr3jri5rx9tsybh4en4ytakccq5n1q X-Rspam-User: X-Rspamd-Server: rspam08 X-HE-Tag: 1675083764-430076 X-HE-Meta: U2FsdGVkX19CFlYPNtNpGGCiGcTXCgVM3dgelCU0PZ0LIj3+m8suUfyqtowOQ3mf1/tNUBO5OMCOwvUxLJ5TwvDAmcHYqXbxcOfCqh7mz8py2l2K7FRXRL0Zigwrofw4mv5WM016q4VqXoeSRVZjqXqinm6O6BWG3vuwwpevkQUb4rzmuPCxSpwPiCrYbjtGW8R7dbmyOqMOC3/RM+1xDdRr/01N6Rq7GVKPdDQPG5wxBokN8htSuXv5aaQ4Zp8R/CVevc6k7hhxb0c/jEbnDX9jwrq6Iqm+24ptIFH3SobB+m0yLgFjvjRKWxPF5hCgE7lCWH1u8T4SBarYershVfO7qU8NLuEQLnKMG5st2Qnbn4rvejnk6PHoxS8UHG/KQCLcCaRltHv4CRYsElhuXxseO8AIHVKvmyu21leEnb42Cc40TWJIfFulgC3ydn0WBpfoZG5j1f8gMkbRqFAfWDMehEVyBjlLkb/VZwPrSXyJFZMZn1TdqozpZLZW464mpMYsgcR03s6qXLG5/HoMWSCUkLrHS3kMcJeAiJbcl7sb160W4kTg3ZiW5LbH88MKeiJ2CWQYZOOruUuqwp6NBckhmI9LuVgEuMo9by2A+t+sHQkWnuclE9BlxXIWrbyNecKyRbglmHyPk0vOZKrqv4+vE3lETlXlvzToyexYApKxEJPn6CWNRYFjiKF6gyvPFLo/tWLIQ75E6SclsS3t0aWV1e98f+GluwIMnM7Ya25oAbm4a4FhgB/INdZqHjh/WH4s5RrorQFtE/7vmDn0JzsyCBvoF5pBEdIHyl4mDC3XdBI0eahMTO5RsfjgDWWg7m902SmVb2cwfxjGyaBxc5b1cnZSWq96NWF2UJF4KhrMQFVNn0pM9/hw01W90qForaIGqx+DGYS2TWuCJI6w3PqGLb/tMgA0akwjdxjZBsWr5u8vZ7hDSoEetbywhHZMuEpz9auGXPQI8yDjGFo nQ20fLq9 ymsy1gf0ZLfFGeyVPVVMsXJrCesq3nWNdOz/Xy258pxGzMIvyrlks+d6I6kyhc3ocrQhqXSSZwQLMvcfDsMKAFhBRnFJTyvr1MuYDRBFZvNtOinlJdOCO4RvkHf2Bj9T8w82bzt7J1G5dccPfFZJf9hTFnfxWeEtXffEsnjqs74Wwz/Kkta813U2GK1VSyDGtL0cyVt2TR0RD9hQp+pHxe+sWSHEAgb2XzA+VjwpNP3aYrEO0HmVDILIgf9cGJc4zpcH0seOn6mM8wInE8Zfn3Buq/6BBcDigBw1M48LXNsOQ/u1zM3e3qoq5QCXVE3y0Dh0bVox9iBleqEg0kUmewSe5rT1S2gbFih7orc7D3/xtJhk/j3w48ya3qk80y2QMes6t5p+IOhiQiV2QEfvjNJq3ebAXkDdghr5BR0YU0VkbSVM= 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: On Mon 30-01-23 20:20:16, Kefeng Wang wrote: > > > On 2023/1/30 16:48, Michal Hocko wrote: > > On Mon 30-01-23 09:16:13, Kefeng Wang wrote: > > > > > > > > > On 2023/1/30 5:48, Andrew Morton wrote: > > > > On Sun, 29 Jan 2023 10:44:51 +0800 Kefeng Wang wrote: > > > > > > > > > As commit 18365225f044 ("hwpoison, memcg: forcibly uncharge LRU pages"), > > > > > > > > Merged in 2017. > > > > > > > > > hwpoison will forcibly uncharg a LRU hwpoisoned page, the folio_memcg > > > > > could be NULl, then, mem_cgroup_track_foreign_dirty_slowpath() could > > > > > occurs a NULL pointer dereference, let's do not record the foreign > > > > > writebacks for folio memcg is null in mem_cgroup_track_foreign() to > > > > > fix it. > > > > > > > > > > Reported-by: Ma Wupeng > > > > > Fixes: 97b27821b485 ("writeback, memcg: Implement foreign dirty flushing") > > > > > > > > Merged in 2019. > > > > > ... > > > > Just to make sure I understand. The page has been hwpoisoned, uncharged > > but stayed in the page cache so a next page fault on the address has blowned > > up? > > > > Say we address the NULL memcg case. What is the resulting behavior? > > Doesn't userspace access a poisoned page and get a silend memory > > corruption? > > + Yang Shi > > Check previous link[1], seems that it is a known issue, and there is a TODO > list for storage backed filesystems from Yang. OK, so IIUC this patch will just help the test to not blow up but it will not allow the test to behave consistently. From my past experience the hwpoisoning is not really something that any production environment should be relying on working properly. But this patch is straightforward so no objection from me. Acked-by: Michal Hocko Thanks! > [1] https://lore.kernel.org/all/20211020210755.23964-6-shy828301@gmail.com/T/#m1d40559ca2dcf94396df5369214288f69dec379b -- Michal Hocko SUSE Labs