From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boaz Harrosh Subject: Re: [PATCH 1/3 v6] mm(v4.1): New pfn_mkwrite same as page_mkwrite for VM_PFNMAP Date: Tue, 07 Apr 2015 17:09:16 +0300 Message-ID: <5523E50C.4060706@plexistor.com> References: <55239645.9000507@plexistor.com> <552397E6.5030506@plexistor.com> <5523D43C.1060708@plexistor.com> <20150407131700.GA13946@node.dhcp.inet.fi> <20150407132601.GA14252@node.dhcp.inet.fi> <5523DD83.4050609@plexistor.com> <20150407134759.GB14252@node.dhcp.inet.fi> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Dave Chinner , Matthew Wilcox , Andrew Morton , "Kirill A. Shutemov" , Jan Kara , Hugh Dickins , Mel Gorman , linux-mm@kvack.org, linux-nvdimm , linux-fsdevel , Eryu Guan , Christoph Hellwig , Stable Tree To: "Kirill A. Shutemov" Return-path: In-Reply-To: <20150407134759.GB14252@node.dhcp.inet.fi> Sender: stable-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On 04/07/2015 04:47 PM, Kirill A. Shutemov wrote: <> >> I did not understand if you want that I keep it "return ret". > > I think "return 0;" is right way to go. It matches wp_page_shared() > behaviour. > Ok I sent a v7 with "return 0;" as is in wp_page_shared(). I guess it is better. It survived of course an xfstests quick but I'll let it smoke for the night as well. Thanks for all your help Boaz