Received: by 2002:a05:6a10:9848:0:0:0:0 with SMTP id x8csp872609pxf; Wed, 7 Apr 2021 13:50:00 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx4R5rqQlNWrxyNBVWJgVt6KiPgJWw9u3EiDprgeWNaCvT70ghAT1VNAztMUnxXJTAQqR7X X-Received: by 2002:a17:906:4c96:: with SMTP id q22mr5896918eju.269.1617828600623; Wed, 07 Apr 2021 13:50:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1617828600; cv=none; d=google.com; s=arc-20160816; b=pBHk2dP8aejxk2VHlJ+54p45vEpCR9xSJ6nGw/MK+Mi+NjPE/LjzlGeKi4GFGpYW/V c29KLG0AC42HKW4dlDIKZ5fpLlwWkhs8dxvipBobYhunotonr7TSVLroLXFlCIl9UAQj iSfKdM0uzcislUUh+d5ja12uGFuXEpJwQq1vPA3/oxdl3Pv5yB2Yt3EntnpgTMsiWh+w MMMrg9ReBMCcWKdzG9Kz/rz3trgfr3TYWa1TYVTIPlVr0XGZviheMsI1J6OnGBPLZbus hj6Kx/XjimdcEpY5suKU8uRiQ/ug/H/il24Tt9yg2W0G5CET0HzUkUOVK64tq8PNERMG 1LGg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=qyvICg5bOR724dHN0jiYk0GhBKxcKHxFxhx+efkfdP0=; b=Hvvjr4RCItsC36Opnbq7j97yEh7DccgWakNTgMXCsq+8+a1/GgQcys+6klB8MqhE+/ zbg9Bf6PXoEeTmkZ4gtA5c89e0B0s4stAt1wmaWABH7KTiypqLkjCAb+QK3FuIsEzuIW gqHumhHXLIaz+jKlhhnFp8ZZMlwtLpppEhVFkdT2bRGikMDUau2S6JxHs52lamEHO8YV 029fMJWVqh7wjaj8UJMjNXeZWHRNbEN/PEpRL8cQ7qv5cW95SVQAN7fhtd1Xdp4ARmHm 7hIcUURGH4haasqyLDkrcu01J1YeB9fuCzcYHHpAou7DS7SCsu3RR+pl9lmPihmvEocR aylQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ibm.com header.s=pp1 header.b=gm4eY4fO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=ibm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a9si10298900ejb.31.2021.04.07.13.49.37; Wed, 07 Apr 2021 13:50:00 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@ibm.com header.s=pp1 header.b=gm4eY4fO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=ibm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1350793AbhDGKPn (ORCPT + 99 others); Wed, 7 Apr 2021 06:15:43 -0400 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:27222 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1350755AbhDGKPc (ORCPT ); Wed, 7 Apr 2021 06:15:32 -0400 Received: from pps.filterd (m0098413.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 137A3b4f058603; Wed, 7 Apr 2021 06:14:24 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=date : from : to : cc : subject : message-id : references : mime-version : content-type : in-reply-to; s=pp1; bh=qyvICg5bOR724dHN0jiYk0GhBKxcKHxFxhx+efkfdP0=; b=gm4eY4fO8pB91ulNkCctgOVnU/mtR5c4og1a5cvGrhbR1Cm9pHAedvD8d1FO5zA9UuT5 I7J8dz4TYgOsqFxxQ1rIRCmGXYi5okj9JLNMXs1O7P0Sr4hl6V1KB/q+X0Xfk/U60jEq 12rvuECWUyveozwWKcGxHM7aT5NmcqJVkVz+0okjBzXibYd17x9dGVcXPJY+HXkqCpOP fNTz7RBbQf0qV0mKT/GH1Zk/CO9dTOkiMGyytP27gNriFohtzwFjEcdf5fS3cKmjocfr m/t6lREXynRZUWlsZtr/+PyDg7LrnLen0c3ar+9j9sr363qZvKTOPAjCduJp3hw+0mXS tQ== Received: from pps.reinject (localhost [127.0.0.1]) by mx0b-001b2d01.pphosted.com with ESMTP id 37rw07bttq-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 07 Apr 2021 06:14:24 -0400 Received: from m0098413.ppops.net (m0098413.ppops.net [127.0.0.1]) by pps.reinject (8.16.0.43/8.16.0.43) with SMTP id 137A3wjB060848; Wed, 7 Apr 2021 06:14:23 -0400 Received: from ppma06ams.nl.ibm.com (66.31.33a9.ip4.static.sl-reverse.com [169.51.49.102]) by mx0b-001b2d01.pphosted.com with ESMTP id 37rw07btt0-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 07 Apr 2021 06:14:23 -0400 Received: from pps.filterd (ppma06ams.nl.ibm.com [127.0.0.1]) by ppma06ams.nl.ibm.com (8.16.0.43/8.16.0.43) with SMTP id 137A3BvM010573; Wed, 7 Apr 2021 10:14:21 GMT Received: from b06cxnps3075.portsmouth.uk.ibm.com (d06relay10.portsmouth.uk.ibm.com [9.149.109.195]) by ppma06ams.nl.ibm.com with ESMTP id 37rvbw8knx-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 07 Apr 2021 10:14:21 +0000 Received: from d06av22.portsmouth.uk.ibm.com (d06av22.portsmouth.uk.ibm.com [9.149.105.58]) by b06cxnps3075.portsmouth.uk.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 137AEJ1o40239448 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 7 Apr 2021 10:14:19 GMT Received: from d06av22.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 2B1014C04A; Wed, 7 Apr 2021 10:14:19 +0000 (GMT) Received: from d06av22.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id C7DAF4C044; Wed, 7 Apr 2021 10:14:18 +0000 (GMT) Received: from localhost (unknown [9.85.69.78]) by d06av22.portsmouth.uk.ibm.com (Postfix) with ESMTP; Wed, 7 Apr 2021 10:14:18 +0000 (GMT) Date: Wed, 7 Apr 2021 15:44:17 +0530 From: riteshh To: Shiyang Ruan Cc: linux-kernel@vger.kernel.org, linux-xfs@vger.kernel.org, linux-nvdimm@lists.01.org, linux-fsdevel@vger.kernel.org, darrick.wong@oracle.com, willy@infradead.org, jack@suse.cz, viro@zeniv.linux.org.uk, linux-btrfs@vger.kernel.org, david@fromorbit.com, hch@lst.de, rgoldwyn@suse.de, Ritesh Harjani Subject: Re: [PATCH 1/3] fsdax: Factor helpers to simplify dax fault code Message-ID: <20210407101417.45mu2m35hfduizpn@riteshh-domain> References: <20210407063207.676753-1-ruansy.fnst@fujitsu.com> <20210407063207.676753-2-ruansy.fnst@fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210407063207.676753-2-ruansy.fnst@fujitsu.com> X-TM-AS-GCONF: 00 X-Proofpoint-ORIG-GUID: 5kC7Gni9-yvd5Wl2UkGLEjQYyu3MvqY_ X-Proofpoint-GUID: QQcIOBDcl5-T6SjKoKh9acNasKx7P6Z1 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.369,18.0.761 definitions=2021-04-07_07:2021-04-06,2021-04-07 signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 adultscore=0 lowpriorityscore=0 suspectscore=0 phishscore=0 priorityscore=1501 bulkscore=0 malwarescore=0 clxscore=1011 impostorscore=0 mlxlogscore=999 mlxscore=0 spamscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2104060000 definitions=main-2104070070 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 21/04/07 02:32PM, Shiyang Ruan wrote: > The dax page fault code is too long and a bit difficult to read. And it > is hard to understand when we trying to add new features. Some of the > PTE/PMD codes have similar logic. So, factor them as helper functions to > simplify the code. > > Signed-off-by: Shiyang Ruan > Reviewed-by: Christoph Hellwig > Reviewed-by: Ritesh Harjani Sorry, but above email address is wrong. Either of below is ok. Reviewed-by: Ritesh Harjani OR Reviewed-by: Ritesh Harjani > > > --- > fs/dax.c | 152 ++++++++++++++++++++++++++++++------------------------- > 1 file changed, 84 insertions(+), 68 deletions(-)