From mboxrd@z Thu Jan 1 00:00:00 1970 From: Milosz Tanski Subject: Re: [PATCH 0/2] Cleanup invalidate page Date: Fri, 9 Aug 2013 15:51:52 -0400 Message-ID: References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Cc: ceph-devel , "linux-fsdevel@vger.kernel.org" , "Yan, Zheng" To: Sage Weil Return-path: In-Reply-To: Sender: ceph-devel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Fri, Aug 9, 2013 at 3:44 PM, Sage Weil wrote: > On Fri, 9 Aug 2013, Milosz Tanski wrote: >> Sage, >> >> Great. Is there some automated testing system that looks for >> regressions in cephfs that I can be watching for? > > Yep, you can join the ceph-qa@ceph.com email list and watch for the > kcephfs suite results (see http://ceph.com/resources/mailing-list-irc/). I'll brace my self for the spam. > > BTW I made a few tweaks to the second patch due to a conflict (added > handling for the length arg to invalidatepage). Thanks and sorry for the oversight. Best, -Milosz > > Thanks! > sage > >> >> - Milosz >> >> On Fri, Aug 9, 2013 at 1:44 PM, Sage Weil wrote: >> > Hi Milosz, >> > >> > I pulled both these into the testing branch. Thanks! >> > >> > sage >> > >> > On Fri, 9 Aug 2013, Milosz Tanski wrote: >> > >> >> Currently ceph_invalidatepage has is overly eger with it's checks which are >> >> moot. The second change cleans up the case where offset is non zero. >> >> >> >> Please pull the from: >> >> https://bitbucket.org/adfin/linux-fs.git wip-invalidatepage >> >> >> >> This simple patchset came from the changes I made while working on fscache >> >> support for cephfs. Per Sage's request I split this up into smaller bites for >> >> testing and review. >> >> >> >> Milosz Tanski (2): >> >> ceph: Remove bogus check in invalidatepage >> >> ceph: cleanup the logic in ceph_invalidatepage >> >> >> >> fs/ceph/addr.c | 33 +++++++++++++++------------------ >> >> 1 file changed, 15 insertions(+), 18 deletions(-) >> >> >> >> -- >> >> 1.8.1.2 >> >> >> >> -- >> >> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in >> >> the body of a message to majordomo@vger.kernel.org >> >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> >> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >>