From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f49.google.com (mail-ed1-f49.google.com [209.85.208.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8FA561A29A for ; Thu, 27 Jun 2024 07:13:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.49 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719472413; cv=none; b=TUAu65QoIMcEvcpBg1RgI6mHbpIybj04YhfSKh5uEjWIrizgAwtSUQElaqO10BGB4yJx4VSHCE6SjnB/nQY/iYuCyZXv7M/AcZeXYW9QErFc72Qx+eGxybAWJ4mBzWcuJW3LXBniXQjUBfNjG6KY2/eoMndPvFHWjOHRdsSdfmI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1719472413; c=relaxed/simple; bh=OunEzO/AYwbaIN0s6eaU896UIyKqDZ7uDcqDS/GgQpI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=BL6SqX5lAftSMArq1Nl7Hut3ff4x9b22p7SaxPDUR9byUv4hTFsJCjqgG/KDaDq+QcpaZjISsnOu2/fKzTfyTOyF49tD0Z0FseuMkENTWfYt8h3yL//8WzXAMQUYJ/rT2rgbufh1hC3zTx9Xez/gJYCK9nSilyKtQtH4ol5mikI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com; spf=pass smtp.mailfrom=suse.com; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b=cL6pkM+w; arc=none smtp.client-ip=209.85.208.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=suse.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=suse.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=suse.com header.i=@suse.com header.b="cL6pkM+w" Received: by mail-ed1-f49.google.com with SMTP id 4fb4d7f45d1cf-57d203d4682so1435606a12.0 for ; Thu, 27 Jun 2024 00:13:31 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=google; t=1719472410; x=1720077210; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=WjSqYVstGY+dnCOqDR7QEfoeCBWvYJSBI8B7JErfM6o=; b=cL6pkM+wLOXXQSqQnyCVoAoSXVvDiJXfHze7IBnrMy3C9R+UFNmKEBZy0wr+pz5Eza NbgYJyMDBaQUs5t349K72l86mtfn9um7AsQdxKebKOKrtMcoShYwj9kxVHyb4Z+9F2mQ PlM6XdEaT4xtYYsmBdQsrwoGjzD9EDUP8xBGPKsqy9wIk9PSRtduFDSGmWBShaS4hbPw XZ5B7hmC8ZIiDiuK538f3WXSuytK0mvuQeVUuHK7rTLJYsM6vMuW6THrc/rsoGfqM5S7 FJzwbBrfPTljX8D6PO2QiE6bTQH9YnF89fBjfxbTmo1zkikjA9GPynQetrXufZWtRKlI NNGg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1719472410; x=1720077210; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=WjSqYVstGY+dnCOqDR7QEfoeCBWvYJSBI8B7JErfM6o=; b=RPn7ah4uXT7WZuEcB8AC6mlnuQx5BqgmliUA+mK1xi5q3XO1t+wW1HeVNUYh6DNsd3 KVvBNwEHl4DoX7xY4/nhml7PpPyGhz5kek7ATw0fxt5BPRQch5ydmp7UepTsL/LBGzk9 zwDEhAeS+HYlyIlXrcnhKOcgZeDJnYME76wXQuUahEHzLTgKnBGKhSQzs5Pyqitmq+hN TG4/bgziUpCDOSxi08dQOYv//25snoGUEjYKddEB6K+B1scDQp1rYtphlieZhA+LM8gi fHSdmlL2r3LeaPZl5F/bfCXbhyAclCe/GjPz/JZ69pKRM+Q+3FQTz8XpYdr5+fFCf17C ZQzw== X-Forwarded-Encrypted: i=1; AJvYcCUGQz9fgjRA3esP5lNvOC4rbBgHKcXKjUKaaauDW8lqQbXY15QlqAIXmICtCkpOogmYKcrBie6bOluHKm8wfMco1qjO2VFaBQ== X-Gm-Message-State: AOJu0YwRihnB57dVBeBaNuEqlbo6JaPQZbF1BA19HMnC9XXwMJ3p7IoR UIazASzf75xovdyCeFF++Q0uPEAvwDPI37VfxDYXaswsMZy9jsjTylrnOruhIPI= X-Google-Smtp-Source: AGHT+IHQM+6N2TxfZxDlu1yZpXj1sEKyksdVN3G4/fMpiXtja6nFqYy2Yne2kCXQS07WmpUm6iav4w== X-Received: by 2002:a17:906:ba8b:b0:a6f:4f2c:1936 with SMTP id a640c23a62f3a-a727f706bf5mr398227766b.44.1719472409930; Thu, 27 Jun 2024 00:13:29 -0700 (PDT) Received: from localhost (nat2.prg.suse.com. [195.250.132.146]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-a729d71f5acsm31349466b.68.2024.06.27.00.13.29 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 27 Jun 2024 00:13:29 -0700 (PDT) Date: Thu, 27 Jun 2024 09:13:29 +0200 From: Michal Hocko To: Xiu Jianfeng Cc: hannes@cmpxchg.org, roman.gushchin@linux.dev, shakeel.butt@linux.dev, muchun.song@linux.dev, akpm@linux-foundation.org, cgroups@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH -next] mm: memcg: remove redundant seq_buf_has_overflowed() Message-ID: References: <20240626094232.2432891-1-xiujianfeng@huawei.com> Precedence: bulk X-Mailing-List: cgroups@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240626094232.2432891-1-xiujianfeng@huawei.com> On Wed 26-06-24 09:42:32, Xiu Jianfeng wrote: > Both the end of memory_stat_format() and memcg_stat_format() will call > WARN_ON_ONCE(seq_buf_has_overflowed()). However, memory_stat_format() > is the only caller of memcg_stat_format(), when memcg is on the default > hierarchy, seq_buf_has_overflowed() will be executed twice, so remove > the reduntant one. Shouldn't we rather remove both? Are they giving us anything useful actually? Would a simpl pr_warn be sufficient? Afterall all we care about is to learn that we need to grow the buffer size because our stats do not fit anymore. It is not really important whether that is an OOM or cgroupfs interface path. > Signed-off-by: Xiu Jianfeng > --- > mm/memcontrol.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index 974bd160838c..776d22bc66a2 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -1846,9 +1846,6 @@ static void memcg_stat_format(struct mem_cgroup *memcg, struct seq_buf *s) > vm_event_name(memcg_vm_event_stat[i]), > memcg_events(memcg, memcg_vm_event_stat[i])); > } > - > - /* The above should easily fit into one page */ > - WARN_ON_ONCE(seq_buf_has_overflowed(s)); > } > > static void memcg1_stat_format(struct mem_cgroup *memcg, struct seq_buf *s); > -- > 2.34.1 -- Michal Hocko SUSE Labs