From patchwork Thu Nov 22 10:06:20 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Hildenbrand X-Patchwork-Id: 10693799 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 2AEB15A4 for ; Thu, 22 Nov 2018 10:06:45 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 1E04E2BA1E for ; Thu, 22 Nov 2018 10:06:45 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 124AD2BB0D; Thu, 22 Nov 2018 10:06:45 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.9 required=2.0 tests=BAYES_00,MAILING_LIST_MULTI, RCVD_IN_DNSWL_NONE autolearn=unavailable version=3.3.1 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 94C692BA1E for ; Thu, 22 Nov 2018 10:06:44 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 6AC396B2AF1; Thu, 22 Nov 2018 05:06:43 -0500 (EST) Delivered-To: linux-mm-outgoing@kvack.org Received: by kanga.kvack.org (Postfix, from userid 40) id 609F86B2AF2; Thu, 22 Nov 2018 05:06:43 -0500 (EST) X-Original-To: int-list-linux-mm@kvack.org X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 40E436B2AF3; Thu, 22 Nov 2018 05:06:43 -0500 (EST) X-Original-To: linux-mm@kvack.org X-Delivered-To: linux-mm@kvack.org Received: from mail-qk1-f200.google.com (mail-qk1-f200.google.com [209.85.222.200]) by kanga.kvack.org (Postfix) with ESMTP id 0EF9A6B2AF1 for ; Thu, 22 Nov 2018 05:06:43 -0500 (EST) Received: by mail-qk1-f200.google.com with SMTP id 80so8993683qkd.0 for ; Thu, 22 Nov 2018 02:06:43 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-original-authentication-results:x-gm-message-state:from:to:cc :subject:date:message-id:in-reply-to:references; bh=Kd7IBInTJuE5UyuhiTUmlbkfVgxPmpI0fHwb4kJFqF0=; b=jxo3dTfPTjcyr0qs4doqtbD2QMq3D0dCEXwwqTjph5K7GUPsTwe2N4AkXq8pEBOEUx D/HtfvzWgm6kbiPqWm/y8Y4n83WMqVeD6w9FHUtXMuThGVAdpvxKMZ+cdAveI4+uVPyZ E1LZVS5dBiYjXjl/RC34cCbosiKvHaq9E1dKsNb7d3Hoz+uWAW1iLVZuczpYedMa0Nkh L6/uXHFRpVmtlHI9XKrugDPlT3P9JCwhN2nAUp7ekRL0mqW9U79HqKtu1CoPk7w0JH3p rAQyxkYTMUvsR73aq7L70VMJ5n9fk+o5v36NCySqnL7p3JQhhWkQPcU6WKSgVyY/deua SY4w== X-Original-Authentication-Results: mx.google.com; spf=pass (google.com: domain of david@redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=david@redhat.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com X-Gm-Message-State: AA+aEWZBO9vnalRi94lBLNE87YSruErw8Qr7nKvouGY0RHzwDpb8ZGxS 4d1GMsWxFnNZw4jFIzH0HBbkL2sJvU7zznEk1xQcsbtcn/d5H374iDWs0ezt+I8vsdk4JrzDX9e DSq91KchddMCS0PyQg7DerrBVEYCZpkAm8qUfrbsCGynbl8bU8PDNcNPaZvQmuq0vgg== X-Received: by 2002:a37:a0d5:: with SMTP id j204mr8925379qke.261.1542881202824; Thu, 22 Nov 2018 02:06:42 -0800 (PST) X-Google-Smtp-Source: AFSGD/W0MPQ5nXt++DF/VBl8p2yiw/DkL4DR2/hEGvLGgH2xanNM+OK7bX8v9dYX191Pi0BfY/ct X-Received: by 2002:a37:a0d5:: with SMTP id j204mr8925346qke.261.1542881202139; Thu, 22 Nov 2018 02:06:42 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1542881202; cv=none; d=google.com; s=arc-20160816; b=LDvJ3wqRR8KV2kPCd9Cl+DxdHbYw1WtEkgUJXWMv4Lc+1Fv4T+dkLtK6hE0jnjG0Tq 6YiGvAtppGEOstErgnS4ge4UApacY3/Kq+iuM7eMmwf0nxl/BVeYix2W64Lh20MM4UBU b3yrYtS5WMxQi0/4yWUn5isfyHMxI9/4DMX0rOafi6SIeFJ4LzeCpwhLtZ/AYATMH491 gdgAk6SlsapsHluRrQxtUXb9O8MCLxkPlJT/zmA/ckU65HHD5bgmhVVVH7q2gZHRrqHT O7cpD4H6utL/nGSqPOM/bS3hdHDYYpEuySdyYd2Y8jeAvikK/PpT870M+dWWRpSzdN16 z+tw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=references:in-reply-to:message-id:date:subject:cc:to:from; bh=Kd7IBInTJuE5UyuhiTUmlbkfVgxPmpI0fHwb4kJFqF0=; b=IQMWEdi+mb1hN1Yk/Ctxspige5qR+Y5wll+Y/SwM9M8UD54jO7bA/IZH6Vtdvl+6sg 8dwkfYGilB+BvBGFVaQRAaqehg21LnqbUPAILxv37BVAqBg8dkKduW2nSVCowPyhFthP Rdakw2RiIIpfSpzUaQYnny1/w/stC+6eVvWIYBzya3wg7UHtguNAdYcRYzpiAoIV+7DX zhzhQxY6NB8AY0bfBBALpzQmxfE1XltaL/hKL6zyy8UUkKVCctfA7Yyg+9beKrLY/plc PIMvgbvgrYihD3J9ksUgWYkAHqCCr5t4e2dr4X3c3Df7DAYSKKDxXZfjMFPL1ONxCIH6 tNxQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of david@redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=david@redhat.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from mx1.redhat.com (mx1.redhat.com. [209.132.183.28]) by mx.google.com with ESMTPS id n32si2080435qtd.130.2018.11.22.02.06.41 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 22 Nov 2018 02:06:42 -0800 (PST) Received-SPF: pass (google.com: domain of david@redhat.com designates 209.132.183.28 as permitted sender) client-ip=209.132.183.28; Authentication-Results: mx.google.com; spf=pass (google.com: domain of david@redhat.com designates 209.132.183.28 as permitted sender) smtp.mailfrom=david@redhat.com; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 3286D81F10; Thu, 22 Nov 2018 10:06:41 +0000 (UTC) Received: from t460s.redhat.com (ovpn-116-206.ams2.redhat.com [10.36.116.206]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1A09E16BE8; Thu, 22 Nov 2018 10:06:37 +0000 (UTC) From: David Hildenbrand To: linux-mm@kvack.org Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org, devel@linuxdriverproject.org, linux-fsdevel@vger.kernel.org, linux-pm@vger.kernel.org, xen-devel@lists.xenproject.org, kexec-ml , pv-drivers@vmware.com, David Hildenbrand , Andrew Morton , Matthew Wilcox , Michal Hocko , "Michael S. Tsirkin" Subject: [PATCH v2 1/8] mm: balloon: update comment about isolation/migration/compaction Date: Thu, 22 Nov 2018 11:06:20 +0100 Message-Id: <20181122100627.5189-2-david@redhat.com> In-Reply-To: <20181122100627.5189-1-david@redhat.com> References: <20181122100627.5189-1-david@redhat.com> X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Thu, 22 Nov 2018 10:06:41 +0000 (UTC) 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: X-Virus-Scanned: ClamAV using ClamSMTP Commit b1123ea6d3b3 ("mm: balloon: use general non-lru movable page feature") reworked balloon handling to make use of the general non-lru movable page feature. The big comment block in balloon_compaction.h contains quite some outdated information. Let's fix this. Cc: Andrew Morton Cc: Matthew Wilcox Cc: Michal Hocko Cc: "Michael S. Tsirkin" Acked-by: Michael S. Tsirkin Signed-off-by: David Hildenbrand --- include/linux/balloon_compaction.h | 26 +++++++++----------------- 1 file changed, 9 insertions(+), 17 deletions(-) diff --git a/include/linux/balloon_compaction.h b/include/linux/balloon_compaction.h index 53051f3d8f25..cbe50da5a59d 100644 --- a/include/linux/balloon_compaction.h +++ b/include/linux/balloon_compaction.h @@ -4,15 +4,18 @@ * * Common interface definitions for making balloon pages movable by compaction. * - * Despite being perfectly possible to perform ballooned pages migration, they - * make a special corner case to compaction scans because balloon pages are not - * enlisted at any LRU list like the other pages we do compact / migrate. + * Balloon page migration makes use of the general non-lru movable page + * feature. + * + * page->private is used to reference the responsible balloon device. + * page->mapping is used in context of non-lru page migration to reference + * the address space operations for page isolation/migration/compaction. * * As the page isolation scanning step a compaction thread does is a lockless * procedure (from a page standpoint), it might bring some racy situations while * performing balloon page compaction. In order to sort out these racy scenarios * and safely perform balloon's page compaction and migration we must, always, - * ensure following these three simple rules: + * ensure following these simple rules: * * i. when updating a balloon's page ->mapping element, strictly do it under * the following lock order, independently of the far superior @@ -21,19 +24,8 @@ * +--spin_lock_irq(&b_dev_info->pages_lock); * ... page->mapping updates here ... * - * ii. before isolating or dequeueing a balloon page from the balloon device - * pages list, the page reference counter must be raised by one and the - * extra refcount must be dropped when the page is enqueued back into - * the balloon device page list, thus a balloon page keeps its reference - * counter raised only while it is under our special handling; - * - * iii. after the lockless scan step have selected a potential balloon page for - * isolation, re-test the PageBalloon mark and the PagePrivate flag - * under the proper page lock, to ensure isolating a valid balloon page - * (not yet isolated, nor under release procedure) - * - * iv. isolation or dequeueing procedure must clear PagePrivate flag under - * page lock together with removing page from balloon device page list. + * ii. isolation or dequeueing procedure must remove the page from balloon + * device page list under b_dev_info->pages_lock. * * The functions provided by this interface are placed to help on coping with * the aforementioned balloon page corner case, as well as to ensure the simple