From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60676 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239394AbiCYGub (ORCPT ); Fri, 25 Mar 2022 02:50:31 -0400 Received: from mx0b-00069f02.pphosted.com (mx0b-00069f02.pphosted.com [205.220.177.32]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BBF7A31DE4 for ; Thu, 24 Mar 2022 23:48:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=date : from : to : cc : subject : message-id : references : content-type : in-reply-to : mime-version; s=corp-2021-07-09; bh=sWHHi9UneYDhmWCvNHY+iZuto61sg+74sBKDr2ZE7BI=; b=uKWBMNTNAz4VsqPvp3bKW3CIAXN6hcyLAr5yfD3heiKlyl7I16FHKdmJuYldzFYa2FZp bnnQk3Xb88o1Ba1hf7fJejVt7Gn64uQ/bHk8hIiDv9adYBRuxR3rA4VxNrquVizaBchJ 0ndMPMA1G7ssLbxTdnZAgzVh91/gH2s0Jj9OCrn6l0Klg107J/K3jkOnoLdfxcSshjHY 7r2qoKYzkWmDDydbCBwlyIaD57S/1dfVowpFtQsa4Q+E+mOrjk31tCOthkN2+4ytRTqA mCJMGZ3WrPTftHfxXwgbGaQD2GTHhnDvbaEwLBM/zL2jYRtTqgc4Jh51379+bcz0uNQ9 kQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.onmicrosoft.com; s=selector2-oracle-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=sWHHi9UneYDhmWCvNHY+iZuto61sg+74sBKDr2ZE7BI=; b=e4ZllljzoHlciFaI2DHhr7Q6oHOGmJJGTMR5SspPyAaaTmpoZfEMdlsB/HS3flWDpWDFchonON1pj614ot+FltjDc4rEt2I2Gz09c2+UhJS3/AnbEBAJX2Li7mcmpfth0fFM4pB2dyFaVpthP0se1kQEIPrmFHAA99MxOZcavn8= Date: Fri, 25 Mar 2022 09:48:41 +0300 From: Dan Carpenter Subject: Re: alloc_page leaks tracing Message-ID: <20220325064841.GH12805@kadam> References: <1E9B599B-743A-4327-A36A-9DB2AE9E6E51@linuxhacker.ru> <20220310094235.GH3293@kadam> <82B7079C-AD17-4E5E-A245-71E9ED53E5A3@linuxhacker.ru> <20220311211215.GN3315@kadam> <20220322080513.GS336@kadam> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220322080513.GS336@kadam> MIME-Version: 1.0 List-ID: To: Oleg Drokin Cc: smatch@vger.kernel.org Another option to free the page is put_page() right? To me it looks like r5l_recovery_verify_data_checksum_for_mb() should use free_page() but put_page() also works I think. My understanding is that you could swap all the free_page() calls for put_page() but not the other way round. regards, dan carpenter