From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-lf0-f66.google.com ([209.85.215.66]:34939 "EHLO mail-lf0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752137AbcEYVVg (ORCPT ); Wed, 25 May 2016 17:21:36 -0400 Received: by mail-lf0-f66.google.com with SMTP id p10so4288509lfb.2 for ; Wed, 25 May 2016 14:21:35 -0700 (PDT) Date: Thu, 26 May 2016 00:21:30 +0300 From: "Kirill A. Shutemov" To: neha agarwal Cc: "Kirill A. Shutemov" , Hugh Dickins , Andrea Arcangeli , Andrew Morton , Dave Hansen , Vlastimil Babka , Christoph Lameter , Naoya Horiguchi , Jerome Marchand , Yang Shi , Sasha Levin , Andres Lagar-Cavilla , Ning Qu , linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCHv8 00/32] THP-enabled tmpfs/shmem using compound pages Message-ID: <20160525212129.GB15857@node.shutemov.name> References: <1463067672-134698-1-git-send-email-kirill.shutemov@linux.intel.com> <20160525200356.GA15857@node.shutemov.name> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, May 25, 2016 at 05:11:03PM -0400, neha agarwal wrote: > On Wed, May 25, 2016 at 4:03 PM, Kirill A. Shutemov > wrote: > > > On Wed, May 25, 2016 at 03:11:55PM -0400, neha agarwal wrote: > > > Hi All, > > > > > > I have been testing Hugh's and Kirill's huge tmpfs patch sets with > > > Cassandra (NoSQL database). I am seeing significant performance gap > > between > > > these two implementations (~30%). Hugh's implementation performs better > > > than Kirill's implementation. I am surprised why I am seeing this > > > performance gap. Following is my test setup. > > > > Thanks for the report. I'll look into it. > > > > Thanks Kirill for looking into it. > > > > > Patchsets > > > ======== > > > - For Hugh's: > > > I checked out 4.6-rc3, applied Hugh's preliminary patches (01 to 10 > > > patches) from here: https://lkml.org/lkml/2016/4/5/792 and then applied > > the > > > THP patches posted on April 16 (01 to 29 patches). > > > > > > - For Kirill's: > > > I am using his branch "git:// > > > git.kernel.org/pub/scm/linux/kernel/git/kas/linux.git hugetmpfs/v8", > > which > > > is based off of 4.6-rc3, posted on May 12. > > > > > > > > > Khugepaged settings > > > ================ > > > cd /sys/kernel/mm/transparent_hugepage > > > echo 10 >khugepaged/alloc_sleep_millisecs > > > echo 10 >khugepaged/scan_sleep_millisecs > > > echo 511 >khugepaged/max_ptes_none > > > > Do you make this for both setup? > > > > It's not really nessesary for Hugh's, but it makes sense to have this > > idenatical for testing. > > > > Yeah right, Hugh's will not be impacted by these settings but for identical > testing I did that. Could you try to drop this changes and leave khugepaged with defaults. One theory is that you just create additional load on the system without any gain. As pages wasn't swapped out we have nothing to collapse back, but scanning takes CPU time. Hugh didn't change khugepaged, so it would not need to look into tmpfs mapping to check if there's something to collapse... -- Kirill A. Shutemov