From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751011AbcBEE4L (ORCPT ); Thu, 4 Feb 2016 23:56:11 -0500 Received: from mail-lf0-f50.google.com ([209.85.215.50]:33360 "EHLO mail-lf0-f50.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750721AbcBEE4J (ORCPT ); Thu, 4 Feb 2016 23:56:09 -0500 MIME-Version: 1.0 In-Reply-To: <1454643648-10002-1-git-send-email-matthew.r.wilcox@intel.com> References: <1454643648-10002-1-git-send-email-matthew.r.wilcox@intel.com> Date: Fri, 5 Feb 2016 07:56:07 +0300 Message-ID: Subject: Re: [PATCH 0/2] Radix tree retry bug fix & test case From: Konstantin Khlebnikov To: Matthew Wilcox Cc: Andrew Morton , Hugh Dickins , Linux Kernel Mailing List , linux-fsdevel , "linux-mm@kvack.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Feb 5, 2016 at 6:40 AM, Matthew Wilcox wrote: > Konstantin pointed out my braino when using radix_tree_iter_retry(), > and then pointed out a second braino. I think we can fix both brainos > with one simple test (the advantage of having your braino pointed out > to you is that you know what you were expecting to happen, so you can > sometimes propose simlpy making happen what you expected to happen. > Konstantin doesn't have access to my though tprocesses.) > > Kontantin wrote a really great test ... and then didn't add it to the > test suite. That made me sad, so I added it. I haven't seen them, I wasn't in CC. And I prefer testing in vivo if possible. > > Andrew, can you drop radix-tree-fix-oops-after-radix_tree_iter_retry.patch > from your tree and add these two patches instead? If you prefer > Konstantin's fix to this one, I'll send you another patch to fix the > second problem Konstantin pointed out. Nak. Mine version generates better code. radix_tree_next_slot is a hot place. Please fix second problem in your helper separately. > > I was a bit unsure about the proper attribution here. The essentials > of the test-suite change from Konstantin are unchanged, but he didn't > have his own sign-off on it. So I made him 'From' and only added my > own sign-off. > > Konstantin Khlebnikov (1): > radix-tree tests: Add regression3 test > > Matthew Wilcox (1): > radix-tree: fix oops after radix_tree_iter_retry > > include/linux/radix-tree.h | 3 ++ > tools/testing/radix-tree/Makefile | 2 +- > tools/testing/radix-tree/linux/kernel.h | 1 + > tools/testing/radix-tree/main.c | 1 + > tools/testing/radix-tree/regression.h | 1 + > tools/testing/radix-tree/regression3.c | 86 +++++++++++++++++++++++++++++++++ > 6 files changed, 93 insertions(+), 1 deletion(-) > create mode 100644 tools/testing/radix-tree/regression3.c > > -- > 2.7.0.rc3 >