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 X-Spam-Level: X-Spam-Status: No, score=-11.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 99141C433B4 for ; Tue, 27 Apr 2021 15:57:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 75E9861076 for ; Tue, 27 Apr 2021 15:57:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238008AbhD0P6C (ORCPT ); Tue, 27 Apr 2021 11:58:02 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]:35691 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236581AbhD0P6B (ORCPT ); Tue, 27 Apr 2021 11:58:01 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1619539038; h=from:from:reply-to:subject:subject: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=UeRAYJi3jVz2RcJA5cfm9/ZZeziOqivA1BEyk0ctB+8=; b=cPw5KuU5YKX57TvUkrq0W/Bm1Zp2yUw19CrCFDBBWUqu0InCoXXYyHPmCB7U4z3hTqjVb/ jJIDvQMPlki2zShs2iuAJm81/wRHJBrZev6SBDZYVo879CqQPJXgrsugNhkOXGp6U/KjKc QYmQkK4MA7xKhhZl65RZhyDexxCjqV0= Received: from mail-qt1-f197.google.com (mail-qt1-f197.google.com [209.85.160.197]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-522-vfZhkoj5NAuWuNfxHYtafw-1; Tue, 27 Apr 2021 11:57:16 -0400 X-MC-Unique: vfZhkoj5NAuWuNfxHYtafw-1 Received: by mail-qt1-f197.google.com with SMTP id b8-20020a05622a0208b02901b5b18f4f91so20738766qtx.18 for ; Tue, 27 Apr 2021 08:57:16 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=UeRAYJi3jVz2RcJA5cfm9/ZZeziOqivA1BEyk0ctB+8=; b=ksyz3lLUEKBt4IQ5bc3hkSON37VVSmzmOJubs73cLbsUwsOn4r8Twu2KhT3mTyvuHg QZJ6acGT6qju9+VzqA3mL5SQI01PYrOyYi2Wnw4/GmCkqgdj82x8nELxMLr5/AQ+7Byh jtLvdzd8KTMV9V8e5Am0XoIyiiylIepvuCrq9Fy7I9JmtZwtYe56c5pvG+S23AOV9sJ8 Q2ujgXEAmUIz0Lq1m0AtFKc6y2PGYYksJDZ7PFBNFj57f+tqvPoFMye04CEo0n8Qs4XG 385bk/ZyA8B/oMn2KBbPyvz3YqQLRzpBCTZjB2VPknI4E34XZy/39/SCdRNe+116iKzV fPRg== X-Gm-Message-State: AOAM530XfA1Dr1/JxUIkVC8DA7eRkzBWdO7s4RUlvxg2UXtD4buJxwru 6SJTZIKnbeRdNZ8KdZhzgbz2BjyCpI/MvynsdnCdSAJFF+HOX4VHFbrhVBopi5lEZLuvJbTJbhf lOwA7hj+SrhX/TOSVrJf+jSw0OA== X-Received: by 2002:a37:9305:: with SMTP id v5mr23224446qkd.223.1619539036069; Tue, 27 Apr 2021 08:57:16 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwt9C9yyPCKjs/11oWJ2ljp8cgvSKc3nf/K+fOGkHMK5svdKo+b6nP8QjYAWsUrtuDALqrP7A== X-Received: by 2002:a37:9305:: with SMTP id v5mr23224422qkd.223.1619539035849; Tue, 27 Apr 2021 08:57:15 -0700 (PDT) Received: from xz-x1 (bras-base-toroon474qw-grc-77-184-145-104-227.dsl.bell.ca. [184.145.104.227]) by smtp.gmail.com with ESMTPSA id h62sm2925543qkf.116.2021.04.27.08.57.14 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 27 Apr 2021 08:57:15 -0700 (PDT) Date: Tue, 27 Apr 2021 11:57:13 -0400 From: Peter Xu To: Hugh Dickins Cc: Axel Rasmussen , Alexander Viro , Andrea Arcangeli , Andrew Morton , Jerome Glisse , Joe Perches , Lokesh Gidra , Mike Kravetz , Mike Rapoport , Shaohua Li , Shuah Khan , Stephen Rothwell , Wang Qing , linux-api@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, linux-mm@kvack.org, Brian Geffon , "Dr . David Alan Gilbert" , Mina Almasry , Oliver Upton Subject: Re: [PATCH v4 04/10] userfaultfd/shmem: support minor fault registration for shmem Message-ID: <20210427155713.GC6820@xz-x1> References: <20210420220804.486803-1-axelrasmussen@google.com> <20210420220804.486803-5-axelrasmussen@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org On Mon, Apr 26, 2021 at 07:23:57PM -0700, Hugh Dickins wrote: > On Tue, 20 Apr 2021, Axel Rasmussen wrote: > > > This patch allows shmem-backed VMAs to be registered for minor faults. > > Minor faults are appropriately relayed to userspace in the fault path, > > for VMAs with the relevant flag. > > > > This commit doesn't hook up the UFFDIO_CONTINUE ioctl for shmem-backed > > minor faults, though, so userspace doesn't yet have a way to resolve > > such faults. > > > > Acked-by: Peter Xu > > Signed-off-by: Axel Rasmussen > > And if this "04/10" had been numbered 03/10, I would have said > Acked-by: Hugh Dickins > > Just read the comment above: "so userspace doesn't yet have a way to > resolve such faults" - if it doesn't by this stage, we're in trouble. Right, so merging the two patches might be easier. Even if we don't merge them, we'll need to touch up the commit message since at least above paragraph is not true anymore as we've already have UFFDIO_CONTINUE. Thanks, -- Peter Xu