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;
From:   Jane Chu <jane.chu@oracle.com>
To:     david@fromorbit.com, djwong@kernel.org, dan.j.williams@intel.com,
        hch@infradead.org, vishal.l.verma@intel.com, dave.jiang@intel.com,
        agk@redhat.com, snitzer@redhat.com, dm-devel@redhat.com,
        ira.weiny@intel.com, willy@infradead.org, vgoyal@redhat.com,
        linux-fsdevel@vger.kernel.org, nvdimm@lists.linux.dev,
        linux-kernel@vger.kernel.org, linux-xfs@vger.kernel.org
Subject: [PATCH v4 2/7] dax: introduce dax device flag DAXDEV_RECOVERY
Date:   Wed, 26 Jan 2022 14:11:11 -0700
Message-Id: <20220126211116.860012-3-jane.chu@oracle.com>
In-Reply-To: <20220126211116.860012-1-jane.chu@oracle.com>
References: <20220126211116.860012-1-jane.chu@oracle.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?hLH8/cBwAlhBTcGUFXV7YiS5k2/Ieqh8JU1oK7uxkbiOAs0dwzfDz/SGk48m?=
 =?us-ascii?Q?JkStB+9RPR75Dozl3CXUYtvxCev/1Pq7foDQG+i3VXgGWxTmV3nExUjbCV/r?=
 =?us-ascii?Q?JIBWs8NnolppWZmvJTqQiDk8Gw3MGIZMn7oJlATB3xQ4ak63rsnCNVZcBCJp?=
 =?us-ascii?Q?H8n2kq0KpIhYONkakwFG7PqGEAQ5phAqyREEvts+g4sVPzNQe2gOdKq6DoL+?=
 =?us-ascii?Q?d/BvSJVXdKjr8+wY4zZOHdyJHFJAZe9HMrXi7TPRyC3g15vEP2LhHxElOrNO?=
 =?us-ascii?Q?+hBahsaNF4JHuUPiNUYCN5vIkMp1SAZ15nX29EYHMDfYnj00aMfH7cWIw49C?=
 =?us-ascii?Q?yh2jCEXeN8/Y/JDv/2aHrUXvmrc/ses5I9hpPaIM/Q3uwJ508OYE7o/2OUeL?=
 =?us-ascii?Q?plQfQQveGnXpXOlAsn/OW2dt3JHhd0P9lFAmKosyVWXfPi3LJ03z6/9qPf78?=
 =?us-ascii?Q?7H1eAI2wIQPJOPLglL8WfOgTz+CKM6NoNWKobvsSiSHV1CKsCogDyma/TcQv?=
 =?us-ascii?Q?xTl80WLuF2RiCRvDp8p/b2Tvl0tQaHvAAS0S4r3/ETvdtGJpC4L7oOf/6Zd4?=
 =?us-ascii?Q?Uhq/21Do44k8Hl+M0mChIN3f3B3vumY85O0nvDNYsCxEzkwNJNzCIh0wFZ/m?=
 =?us-ascii?Q?4EQ9BTHHI14zC/62xi9ejeQGMeFocnuDTdKm07nC4WGN5td7aQNELSnBCE+4?=
 =?us-ascii?Q?JGpxPi0uLj+bOPWmdObu+SyUEvgmecfiYXMhIuUL8ppRshPLbdwvVtE6g+Lf?=
 =?us-ascii?Q?iYCcE7T/h3pjKG0NdpCCbzcjlvN7ocMbFg4Tvf/8Bxps09lTbMIMyx/0p+PF?=
 =?us-ascii?Q?gRhoD8sXcYt+Byu6ojTTN0jEsUYEGpO8bDYpmGAyZ2Pb73C/C1WYaMvhHtKf?=
 =?us-ascii?Q?7M7qV4A6/m1ZiT/5bgafBMuGDUO7twVsRTr7A7iaeHEoC0PjeukwfXDI6qGP?=
 =?us-ascii?Q?gO/LsJAj28DXSHjQdVzeCdu1G7avlW6ArQPB8ltu1VUOP4kPDYgLiXe1Oozh?=
 =?us-ascii?Q?qkzkAI+qhH/3GNr436zXxIK93KsOb8LfkAnlHGlpklV/ktD9rzLGRBF6n5l6?=
 =?us-ascii?Q?hM5gzqG1sLuziRfgy6FdfFE8zKN6spXfR0VapZTKJa8WOKPqSQZUBPiT2s0l?=
 =?us-ascii?Q?9p9RF3oNdHbzKS309xjrrwWJT2c6pNS5sI0fBzz5DBXFWWYCY+Xobu5oUDvH?=
 =?us-ascii?Q?qaZBD3He6HfU7e2ZPLlAt7+Sm4UIkls2snIhVze/VB0KnKmQgKeZKHDD6D4U?=
 =?us-ascii?Q?3skw1ydDI5wAnwSvce5a74ofnffWQblO72LupimGfgY2gD3Y1xnVh6Guubf/?=
 =?us-ascii?Q?ZNx14FSqa4njVoSDAsaMnd2yMqVadJlsUwA+KlC5a4kucICptEW/0oHEhu5R?=
 =?us-ascii?Q?V8f3OtP0PE2Uxo/RDs6qvm3+q/0v0w9b7OzLmCtxWkTdT+cPiLGikek+lWqW?=
 =?us-ascii?Q?bH9pS9WOyGof2EafxO7aglSlfDqFCMDJZOo8SXLI1RZR0YI4cFaFAlrlU4d6?=
 =?us-ascii?Q?PcZgRli5/XTASbq2/dHOiMhjJhbSJiIyVEk60Yf5b+EJEViyDduf80RhngkP?=
 =?us-ascii?Q?RX7iBTT/mIS7ekwiiUdRJV8rwKNbPsmAm2Ene7S++l4/Pm2gRilzOKwxkZJB?=
 =?us-ascii?Q?Mu8tLeD4taNwi5zUZ28lcNBIT49PHaHqmSZDsi4lg4Ac?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6b17c1e7-c810-4cff-a655-08d9e110726a
X-MS-Exchange-CrossTenant-AuthSource: SJ0PR10MB4429.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jan 2022 21:11:40.6203
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: qvlNmQ64RxRCUzCPiwBPfStL44xNXhX3pQts/X/vm5G3pIXjkWky7xDs1gYYdpxlOEGCYAG/ZWc1OMmIEodjEA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM8PR10MB5494
X-Proofpoint-Virus-Version: vendor=nai engine=6300 definitions=10239 signatures=668683
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 mlxscore=0 malwarescore=0
 spamscore=0 phishscore=0 suspectscore=0 adultscore=0 mlxlogscore=999
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2201110000
 definitions=main-2201260122
X-Proofpoint-GUID: W1FyW1TOqGE73TG2ekhmDeOX9a5ByR_K
X-Proofpoint-ORIG-GUID: W1FyW1TOqGE73TG2ekhmDeOX9a5ByR_K
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Introduce dax device flag DAXDEV_RECOVERY to indicate a device
that is capable of recoverying from media poison.
For MD raid DAX devices, the capability is allowed for partial
device as oppose to the entire device. And the final poison
detection and repair rely on the provisioning base drivers.

Signed-off-by: Jane Chu <jane.chu@oracle.com>
---
 drivers/dax/super.c   | 24 ++++++++++++++++++++++++
 drivers/nvdimm/pmem.c |  1 +
 include/linux/dax.h   | 15 +++++++++++++++
 3 files changed, 40 insertions(+)

diff --git a/drivers/dax/super.c b/drivers/dax/super.c
index e3029389d809..f4f607d9698b 100644
--- a/drivers/dax/super.c
+++ b/drivers/dax/super.c
@@ -109,6 +109,8 @@ enum dax_device_flags {
 	DAXDEV_NOCACHE,
 	/* handle CPU fetch exceptions during reads */
 	DAXDEV_NOMC,
+	/* flag to indicate device capable of poison recovery */
+	DAXDEV_RECOVERY,
 };
 
 /**
@@ -311,6 +313,28 @@ static void dax_destroy_inode(struct inode *inode)
 			"kill_dax() must be called before final iput()\n");
 }
 
+void set_dax_recovery(struct dax_device *dax_dev)
+{
+	set_bit(DAXDEV_RECOVERY, &dax_dev->flags);
+}
+EXPORT_SYMBOL_GPL(set_dax_recovery);
+
+bool dax_recovery_capable(struct dax_device *dax_dev)
+{
+	return test_bit(DAXDEV_RECOVERY, &dax_dev->flags);
+}
+EXPORT_SYMBOL_GPL(dax_recovery_capable);
+
+int dax_prep_recovery(struct dax_device *dax_dev, void **kaddr)
+{
+	if (dax_recovery_capable(dax_dev)) {
+		set_bit(DAXDEV_RECOVERY, (unsigned long *)kaddr);
+		return 0;
+	}
+	return -EINVAL;
+}
+EXPORT_SYMBOL_GPL(dax_prep_recovery);
+
 static const struct super_operations dax_sops = {
 	.statfs = simple_statfs,
 	.alloc_inode = dax_alloc_inode,
diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c
index 58d95242a836..e8823813a8df 100644
--- a/drivers/nvdimm/pmem.c
+++ b/drivers/nvdimm/pmem.c
@@ -487,6 +487,7 @@ static int pmem_attach_disk(struct device *dev,
 	if (rc)
 		goto out_cleanup_dax;
 	dax_write_cache(dax_dev, nvdimm_has_cache(nd_region));
+	set_dax_recovery(dax_dev);
 	pmem->dax_dev = dax_dev;
 
 	rc = device_add_disk(dev, disk, pmem_attribute_groups);
diff --git a/include/linux/dax.h b/include/linux/dax.h
index 9fc5f99a0ae2..5449a0a81c83 100644
--- a/include/linux/dax.h
+++ b/include/linux/dax.h
@@ -40,6 +40,8 @@ void dax_write_cache(struct dax_device *dax_dev, bool wc);
 bool dax_write_cache_enabled(struct dax_device *dax_dev);
 bool dax_synchronous(struct dax_device *dax_dev);
 void set_dax_synchronous(struct dax_device *dax_dev);
+void set_dax_recovery(struct dax_device *dax_dev);
+bool dax_recovery_capable(struct dax_device *dax_dev);
 /*
  * Check if given mapping is supported by the file / underlying device.
  */
@@ -87,6 +89,13 @@ static inline bool daxdev_mapping_supported(struct vm_area_struct *vma,
 {
 	return !(vma->vm_flags & VM_SYNC);
 }
+static inline void set_dax_recovery(struct dax_device *dax_dev);
+{
+}
+static inline bool dax_recovery_capable(struct dax_device *dax_dev)
+{
+	return false;
+}
 #endif
 
 void set_dax_nocache(struct dax_device *dax_dev);
@@ -128,6 +137,7 @@ struct page *dax_layout_busy_page(struct address_space *mapping);
 struct page *dax_layout_busy_page_range(struct address_space *mapping, loff_t start, loff_t end);
 dax_entry_t dax_lock_page(struct page *page);
 void dax_unlock_page(struct page *page, dax_entry_t cookie);
+int dax_prep_recovery(struct dax_device *dax_dev, void **kaddr);
 #else
 static inline struct page *dax_layout_busy_page(struct address_space *mapping)
 {
@@ -155,6 +165,11 @@ static inline dax_entry_t dax_lock_page(struct page *page)
 static inline void dax_unlock_page(struct page *page, dax_entry_t cookie)
 {
 }
+
+static inline int dax_prep_recovery(struct dax_device *dax_dev, void **kaddr)
+{
+	return -EINVAL;
+}
 #endif
 
 int dax_zero_range(struct inode *inode, loff_t pos, loff_t len, bool *did_zero,
-- 
2.18.4