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 44B1FC021B3 for ; Fri, 21 Feb 2025 21:01:33 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 41121280002; Fri, 21 Feb 2025 16:01:32 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 3BF2A280001; Fri, 21 Feb 2025 16:01:32 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2870D280002; Fri, 21 Feb 2025 16:01:32 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 0BD71280001 for ; Fri, 21 Feb 2025 16:01:32 -0500 (EST) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id 92F7012059C for ; Fri, 21 Feb 2025 21:01:31 +0000 (UTC) X-FDA: 83145172782.23.F928D9E Received: from mail-pj1-f53.google.com (mail-pj1-f53.google.com [209.85.216.53]) by imf08.hostedemail.com (Postfix) with ESMTP id 5DC17160020 for ; Fri, 21 Feb 2025 21:01:29 +0000 (UTC) Authentication-Results: imf08.hostedemail.com; dkim=pass header.d=chromium.org header.s=google header.b=itqPsKS2; dmarc=pass (policy=none) header.from=chromium.org; spf=pass (imf08.hostedemail.com: domain of senozhatsky@chromium.org designates 209.85.216.53 as permitted sender) smtp.mailfrom=senozhatsky@chromium.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1740171689; a=rsa-sha256; cv=none; b=WAPi/YhGkxm1Z8F6Vc/TOkTsndWdJyK7TdOuHJNSToakuuDd2XZuNKxxVGol7UTGBMlMhw lt0cIxFkx9+2TdMJoDUvyXLbf6jwR6On2rohMI4SSJSZfxG9lTgA07s/ze4ue3Txvo7XX9 j7ix3RszIBpnhgQQWQiYUG0Yja978sA= ARC-Authentication-Results: i=1; imf08.hostedemail.com; dkim=pass header.d=chromium.org header.s=google header.b=itqPsKS2; dmarc=pass (policy=none) header.from=chromium.org; spf=pass (imf08.hostedemail.com: domain of senozhatsky@chromium.org designates 209.85.216.53 as permitted sender) smtp.mailfrom=senozhatsky@chromium.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1740171689; 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=KB7zOw8AFw5HwaiJLyXfCqtg75nhb187Vb/57RmyT1g=; b=QGm/DtTbF1k29GjnkrqVTVyMAx4Y3jZUb9PtJqAQkJOjnbK+mrNkPXGo+a6EfvP5Z8LEkG MM7AbYVircy0UapCcx7z5OKtWVCBsgaErsXRBMPAA8s9OGN9P+ke6z2VEOVGRgZgCjkIKJ sxUU3bY+KPqB7nWRlq+/vXpotl2sS0Y= Received: by mail-pj1-f53.google.com with SMTP id 98e67ed59e1d1-2fc1f410186so6491528a91.0 for ; Fri, 21 Feb 2025 13:01:28 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; t=1740171688; x=1740776488; darn=kvack.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=KB7zOw8AFw5HwaiJLyXfCqtg75nhb187Vb/57RmyT1g=; b=itqPsKS2Gf9Z9NtaIVsOHKDXxEXu9L0QNS2ZncIxCM3EylnQZt5jpTvTr9ldbmOl5k Kyo964Fl/X1CnOlzT0OXSdmTbQbUIu+vI6xzMH7awYf1TUfbMDoZIzvvZYniiqxBpDBl a08kMqCfBphhVMHIuJazS8RkrZ29ot80BCzKg= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740171688; x=1740776488; 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=KB7zOw8AFw5HwaiJLyXfCqtg75nhb187Vb/57RmyT1g=; b=NmyNZY1nkEd+RHcbzfiPVRlbRTQewg0kzFpC+lh8FnempUQYBV3MYR8rlXHqt7T3Be opIdV/p+BJUP5i7UiaY82PA82Nn86NIwlar/QgL0F86+4F0kN/qdlBje1sssjG6NqmNj M5tSsQaDEHaCu297kW7Su3k/WirPcKdy+Dh6ry/uy0e9p0um40xIEAtiy6OJUViF4hP8 e1pM+QvKBUgGL545ef9R1NO6/V1WtOxTmg11T4wiSmcCQWfsJZUaok71k5zBwYKtwy46 ZtNOBBrx3qVe7q1U85qBx+IJSCzsw6+8Av86PAcPCxHdvrNaOb8HlKnFIdCzqyw2w/Jb bhkA== X-Forwarded-Encrypted: i=1; AJvYcCU9I1sZWNZY99fhDzsIC7AFTnp8sb+nxkWOkchZyYRDCozyykpFCfPcO3tQfFbd9PKuKd9jrlYHVA==@kvack.org X-Gm-Message-State: AOJu0YxjA8GMl75KfXBfm0XXQkgMIGD5fjHJlCPtJ8NqWHoiORdPGXkC GPpVdsWIqEwlFUenkoCFzVZntLgWquAF80wjB0CE8o4SYPW+P/dWSExA1dWdzw== X-Gm-Gg: ASbGncvemPYZyNOyUQq3HWtxTrMiWAZEqx27RLSPF8USBjKmEt25ZE+1PU9MXDwFVHr bFipW1027DoR5rKuwae46JKb/kKzdxxQSIpK+Niya/SzZsy7xEIOBN8+RQXbrqlMves4isv4z0n tu5ze5mH+5l+cWfkxBnXJC3dlkT8zysuyv4kR5V2L7qqTo2lsiUSYpzFC8ie8sQ4zbCRcBhP58N 6w7Egj4PHOWI4yXewmYLhpi/NAK8KGc/0DDDw3CMG4IE9G57TFr+hcKIv9kYgOn8kF/Wz1d8hkx mkzSvj4KSsAzKpHoiGBxc2TJ7kDt X-Google-Smtp-Source: AGHT+IEs01Hx2KIewi9vZQYw0yu8X2kEXs1yphv1va/bFuzNYCs+BRNeDhKdyaUblwTlOF6avYXpxA== X-Received: by 2002:a17:90b:3883:b0:2fa:42f3:e3e4 with SMTP id 98e67ed59e1d1-2fccc0f838amr13550504a91.3.1740171687113; Fri, 21 Feb 2025 13:01:27 -0800 (PST) Received: from google.com ([2401:fa00:8f:203:f987:e1e:3dbb:2191]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-2fceb05f2basm1833275a91.21.2025.02.21.13.01.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 21 Feb 2025 13:01:26 -0800 (PST) Date: Sat, 22 Feb 2025 06:01:20 +0900 From: Sergey Senozhatsky To: Yosry Ahmed Cc: Sergey Senozhatsky , Andrew Morton , Hillf Danton , Kairui Song , Sebastian Andrzej Siewior , Minchan Kim , linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v7 11/17] zsmalloc: make zspage lock preemptible Message-ID: <20250221-095831-265429292-neomutt-senozhatsky@chromium.org> References: <20250221093832.1949691-1-senozhatsky@chromium.org> <20250221093832.1949691-12-senozhatsky@chromium.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: 5DC17160020 X-Stat-Signature: 5bt7nhuyfd55rbnbb5rqqnw17dy3wron X-Rspam-User: X-HE-Tag: 1740171689-902159 X-HE-Meta: U2FsdGVkX18aAS5qspr1x7B+1xR3LRGIOVUaSTYzQp/SCcZK6noao3UCxijUUJqG0mKPgRH/OX1AyqvEvju+NKB5MHcXUm1g4Fe3dUi6tJPMJLbhV/QCiMpAEkM+jzALcHGUfzPkcRuIO46oy2SvU2llV9uxVu0xemzjOgL6qrjiAo0tC4szF0ndvYwyv+gAgPg42Y3nl4Oat2l6D+HPUCsP7u8vaZpLRjgI+LFsVilkpbo9VBbQYztWWBxQOjGPs5kzrCRK5VABZ03WDHymGin5O6/DR+HaZP9oTK+PT7wR2Vw0SvX4UMrFka/voy4wqpmGH1HfTyr6sOnxLWhUUv3uE8ZCJCd/oZOfa9W6VTEULvmvawapEUfrvvVh5f6Qznn87zS/nJnSJ5TIZrEjcoT+zyNmlA0+slNdIPyecOxV+l5DGtLgd0GV/J+kg0JUI7SnS6hK8jW1nEP5WM0PNirw8TQtToetTuvvqUkPdWUM9htiC44ghScCO9RMYrfbZZ6lTtv0rw+6c/xE61BB/vsKfnV7iKJvBHk2WFMfi2M+z0rrg7wDK3hM284mRRKk1CUuIiOOTLa2lJe/gnCq1C95PqCySyqjJeQRvqr7usoUDzhiqqarpq5phd/hWvW/ReTzxhJBBmTXVTZ217CWcXxVJC5jRjWzv4qw6h1+aN2hzUuM5rG3UT+x7PoHAQsOrShfz9lVuNmwkcMVVukQpKNkyd4DRpcpM7iVOsT9nDOGhiAAKP/wsi56IS7z5Z77QXdPV71regUvhInkrKIn61J8vd1NPDdo/O1K4WfPhf9Y9incm6X1aXf2ACg4nR7Pm+DRXEXXcQ8tSoHiZPxDlli+xFs7x/MgJ04lck+wdz8XTuU1WZhYpIE5Dv99HjAWNxfuTGOXRSZzbV3Z2rAfxvfica5R7HX7m2ouCDzIk51Z2H8Ri2K8Vza6UMUPRyeupYVjxSi9l+d/YSPFYYL Hj9jtCiD lxzkOaluRz0Uk9GUvuNYvZ/k3x1v9YCirtjG+rDrsTKMyO961LjOBLxi55c/8cF3LaEy9ZgmqqmSY9qB3MN6AKNtB7GuQrDltCgkGSqbG+g9yvc9qp/fkRVhx11Q93Gz03uhLXdBH8Rp3Mpe/dvGGvmMdIqeOFFXBLuq3sfQsqpLpCpr8zToI+aNDVhLj7pyzNptoodmHNvXECn0ZFdlWytwKFMets0Fhar2cdTFRz+6JyFhWT6yQFQNumuOPjuxoqm1QsjbLEu0TdkMTyoW8FIJBBg2wvMe7VZ405R19yjDluSSMI3S6jgrPz+crJyZMCz4mgA/dLAm+SN/QWat6tJz/CjH8Qa3vwluSq6BKBzb1uCA= X-Bogosity: Ham, tests=bogofilter, spamicity=0.023363, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Message-ID: <20250221210120.TVOGW7dsBqVjilblIFqr3r9pH8hc5sNynRiVvnihYSQ@z> On (25/02/21 19:48), Yosry Ahmed wrote: [..] > > +#ifdef CONFIG_DEBUG_LOCK_ALLOC > > +#define zsl_dep_map(zsl) (&(zsl)->dep_map) > > +#else > > +#define zsl_dep_map(zsl) NULL > > +#endif > > + > > +static void zspage_lock_init(struct zspage *zspage) > > +{ > > +#ifdef CONFIG_DEBUG_LOCK_ALLOC > > + lockdep_init_map(&zspage->zsl.dep_map, "zspage->lock", > > + &zspage->pool->lock_class, 0); > > Can't we remove this #ifdef as well by adding a similar macro? (e.g. > zsl_lock_class()) I thought about it but then was not sure if having a macro that we use in two place is worth it, but I guess we can. Let me send a v8.