From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752125AbbALLid (ORCPT ); Mon, 12 Jan 2015 06:38:33 -0500 Received: from cantor2.suse.de ([195.135.220.15]:45795 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751131AbbALLic (ORCPT ); Mon, 12 Jan 2015 06:38:32 -0500 Message-ID: <54B3B233.1020104@suse.cz> Date: Mon, 12 Jan 2015 12:38:27 +0100 From: Vlastimil Babka User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: "Kirill A. Shutemov" , Huang Ying CC: Stephen Rothwell , "Kirill A. Shutemov" , LKML , LKP ML , Andrew Morton Subject: Re: [LKP] [mm] Find a way to test remap_file_pages References: <1421032741.6201.162.camel@intel.com> <20150112142629.46c45532@canb.auug.org.au> <20150112111906.GB16935@node.dhcp.inet.fi> In-Reply-To: <20150112111906.GB16935@node.dhcp.inet.fi> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/12/2015 12:19 PM, Kirill A. Shutemov wrote: > On Mon, Jan 12, 2015 at 02:26:29PM +1100, Stephen Rothwell wrote: >> [Just cc'ing Andrew as this patch is in his tree] >> >> On Mon, 12 Jan 2015 11:19:01 +0800 Huang Ying wrote: >> > >> > FYI, we noticed the below changes on >> > >> > git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master >> > commit 974bb1d5d44f008d7f207437d63d46658ff41dff ("mm: replace remap_file_pages() syscall with emulation") >> > >> > That is, trinity can be used to test some aspect of remap_file_pages. Well, yeah :) I think the number of bugs trinity has already found in this area during the last year, was one of the reasons behind its deprecation and eventually removal. >> > >> > +---------------------------------------------------+------------+------------+ >> > | | d96eeaf2cb | 974bb1d5d4 | >> > +---------------------------------------------------+------------+------------+ >> > | boot_successes | 0 | 0 | >> > | boot_failures | 10 | 10 | >> > | WARNING:at_net/netlink/genetlink.c:#genl_unbind() | 10 | 5 | >> > | backtrace:netlink_setsockopt | 3 | 2 | >> > | backtrace:SyS_setsockopt | 3 | 2 | >> > | backtrace:SyS_socketcall | 3 | 2 | >> > | backtrace:do_group_exit | 10 | 5 | >> > | backtrace:SyS_exit_group | 10 | 5 | >> > | page_allocation_failure:order:#,mode | 0 | 5 | >> > | backtrace:btrfs_test_extent_io | 0 | 5 | >> > | backtrace:init_btrfs_fs | 0 | 5 | >> > | backtrace:kernel_init_freeable | 0 | 5 | >> > +---------------------------------------------------+------------+------------+ >> > >> > >> > [ 29.665081] Bits 55-60 of /proc/PID/pagemap entries are about to stop being page-shift some time soon. See the linux/Documentation/vm/pagemap.txt for details. >> > [ 29.675960] mmap: trinity-c0 (873) uses deprecated remap_file_pages() syscall. See Documentation/vm/remap_file_pages.txt. Is the warning above even related to the warning below, or just coincidence? The remap_file_pages() deprecation warning doesn't generate stacktrace AFAICS. >> > [ 29.681044] ------------[ cut here ]------------ >> > [ 29.681841] WARNING: CPU: 0 PID: 870 at net/netlink/genetlink.c:1037 genl_unbind+0x96/0xaa() >> > [ 29.683610] warning: process `trinity-c0' used the obsolete bdflush system call >> > [ 29.684911] Fix your initscripts? >> > [ 29.685683] VFS: Warning: trinity-c0 using old stat() call. Recompile your binary. >> > [ 29.686963] CPU: 0 PID: 870 Comm: trinity-main Not tainted 3.19.0-rc3-next-20150109-g06f367d #290 >> > [ 29.688399] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2011 >> > [ 29.689349] 00000000 00000000 d34f3ea4 >> > [ 29.694083] ------------[ cut here ]------------ >> > [ 29.695096] WARNING: CPU: 0 PID: 872 at net/netlink/genetlink.c:1037 genl_unbind+0x96/0xaa() >> > [ 29.696724] c15933f7 d34f3ec0 c103c7ff 0000040d c1581310 >> > [ 29.698051] 00000001 00000001 00000000 d34f3ed0 c103c89c 00000009 00000000 d34f3ee8 >> > [ 29.699904] CPU: 0 PID: 872 Comm: trinity-main Not tainted 3.19.0-rc3-next-20150109-g06f367d #290 >> > [ 29.701406] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2011 >> > [ 29.702291] 00000000 00000000 d34d9ea4 c15933f7 d34d9ec0 >> > [ 29.703253] c1581310 c18836fc 00000001 d341f840 00000000 d34f3f0c c157f810 d15b0b40 >> > [ 29.705361] Call Trace: >> > [ 29.705868] [] dump_stack+0x16/0x18 >> > [ 29.706652] [] warn_slowpath_common+0x79/0x90 >> > [ 29.707496] [] ? genl_unbind+0x96/0xaa >> > [ 29.708295] [] warn_slowpath_null+0xf/0x13 >> > [ 29.709142] [] genl_unbind+0x96/0xaa >> > [ 29.709943] [] netlink_release+0x2d7/0x2ed >> > [ 29.710776] [] sock_release+0x10/0x5a >> > [ 29.711531] [] sock_close+0xb/0xf >> > [ 29.712286] [] __fput+0xf7/0x1f0 >> > [ 29.713044] [] ____fput+0x8/0xa >> > [ 29.713794] [] task_work_run+0x4e/0x70 >> > [ 29.714566] [] do_exit+0x43a/0x9c7 >> > [ 29.715300] [] do_group_exit+0x4a/0xd6 >> > [ 29.716111] [] SyS_exit_group+0x11/0x11 >> > [ 29.716975] [] syscall_call+0x7/0x7 >> > [ 29.717722] [] ? io_submit_one+0x5b3/0x72a >> > [ 29.719207] c103c7ff 0000040d c1581310 >> > [ 29.720122] 00000001 00000001 00000000 d34d9ed0 c103c89c 00000009 00000000 d34d9ee8 >> > [ 29.721973] c1581310 c18836fc 00000004 d0ebb840 00000000 d34d9f0c c157f810 >> > [ 29.723252] ---[ end trace 28426633f87b780a ]--- >> > [ 29.724282] ------------[ cut here ]------------ > > > It doesn't make much sense to me. I don't see anything > remap_file_pages()-related here. Or even VMA related. > > And why now? The commit is in Andrew's tree for few release cycles. > > Could you elaborate on what you observe? And what the table above means? > From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============1290908293297865504==" MIME-Version: 1.0 From: Vlastimil Babka To: lkp@lists.01.org Subject: Re: [mm] Find a way to test remap_file_pages Date: Mon, 12 Jan 2015 12:38:27 +0100 Message-ID: <54B3B233.1020104@suse.cz> In-Reply-To: <20150112111906.GB16935@node.dhcp.inet.fi> List-Id: --===============1290908293297865504== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On 01/12/2015 12:19 PM, Kirill A. Shutemov wrote: > On Mon, Jan 12, 2015 at 02:26:29PM +1100, Stephen Rothwell wrote: >> [Just cc'ing Andrew as this patch is in his tree] >> = >> On Mon, 12 Jan 2015 11:19:01 +0800 Huang Ying w= rote: >> > >> > FYI, we noticed the below changes on >> > = >> > git://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git mast= er >> > commit 974bb1d5d44f008d7f207437d63d46658ff41dff ("mm: replace remap_fi= le_pages() syscall with emulation") >> > = >> > That is, trinity can be used to test some aspect of remap_file_pages. Well, yeah :) I think the number of bugs trinity has already found in this = area during the last year, was one of the reasons behind its deprecation and eventually removal. >> > = >> > +---------------------------------------------------+------------+----= --------+ >> > | | d96eeaf2cb | 974= bb1d5d4 | >> > +---------------------------------------------------+------------+----= --------+ >> > | boot_successes | 0 | 0 = | >> > | boot_failures | 10 | 10 = | >> > | WARNING:at_net/netlink/genetlink.c:#genl_unbind() | 10 | 5 = | >> > | backtrace:netlink_setsockopt | 3 | 2 = | >> > | backtrace:SyS_setsockopt | 3 | 2 = | >> > | backtrace:SyS_socketcall | 3 | 2 = | >> > | backtrace:do_group_exit | 10 | 5 = | >> > | backtrace:SyS_exit_group | 10 | 5 = | >> > | page_allocation_failure:order:#,mode | 0 | 5 = | >> > | backtrace:btrfs_test_extent_io | 0 | 5 = | >> > | backtrace:init_btrfs_fs | 0 | 5 = | >> > | backtrace:kernel_init_freeable | 0 | 5 = | >> > +---------------------------------------------------+------------+----= --------+ >> > = >> > = >> > [ 29.665081] Bits 55-60 of /proc/PID/pagemap entries are about to st= op being page-shift some time soon. See the linux/Documentation/vm/pagemap.= txt for details. >> > [ 29.675960] mmap: trinity-c0 (873) uses deprecated remap_file_pages= () syscall. See Documentation/vm/remap_file_pages.txt. Is the warning above even related to the warning below, or just coincidence= ? The remap_file_pages() deprecation warning doesn't generate stacktrace AFAICS. >> > [ 29.681044] ------------[ cut here ]------------ >> > [ 29.681841] WARNING: CPU: 0 PID: 870 at net/netlink/genetlink.c:103= 7 genl_unbind+0x96/0xaa() >> > [ 29.683610] warning: process `trinity-c0' used the obsolete bdflush= system call >> > [ 29.684911] Fix your initscripts? >> > [ 29.685683] VFS: Warning: trinity-c0 using old stat() call. Recompi= le your binary. >> > [ 29.686963] CPU: 0 PID: 870 Comm: trinity-main Not tainted 3.19.0-r= c3-next-20150109-g06f367d #290 >> > [ 29.688399] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2011 >> > [ 29.689349] 00000000 00000000 d34f3ea4 >> > [ 29.694083] ------------[ cut here ]------------ >> > [ 29.695096] WARNING: CPU: 0 PID: 872 at net/netlink/genetlink.c:103= 7 genl_unbind+0x96/0xaa() >> > [ 29.696724] c15933f7 d34f3ec0 c103c7ff 0000040d c1581310 >> > [ 29.698051] 00000001 00000001 00000000 d34f3ed0 c103c89c 00000009 = 00000000 d34f3ee8 >> > [ 29.699904] CPU: 0 PID: 872 Comm: trinity-main Not tainted 3.19.0-r= c3-next-20150109-g06f367d #290 >> > [ 29.701406] Hardware name: Bochs Bochs, BIOS Bochs 01/01/2011 >> > [ 29.702291] 00000000 00000000 d34d9ea4 c15933f7 d34d9ec0 >> > [ 29.703253] c1581310 c18836fc 00000001 d341f840 00000000 d34f3f0c = c157f810 d15b0b40 >> > [ 29.705361] Call Trace: >> > [ 29.705868] [] dump_stack+0x16/0x18 >> > [ 29.706652] [] warn_slowpath_common+0x79/0x90 >> > [ 29.707496] [] ? genl_unbind+0x96/0xaa >> > [ 29.708295] [] warn_slowpath_null+0xf/0x13 >> > [ 29.709142] [] genl_unbind+0x96/0xaa >> > [ 29.709943] [] netlink_release+0x2d7/0x2ed >> > [ 29.710776] [] sock_release+0x10/0x5a >> > [ 29.711531] [] sock_close+0xb/0xf >> > [ 29.712286] [] __fput+0xf7/0x1f0 >> > [ 29.713044] [] ____fput+0x8/0xa >> > [ 29.713794] [] task_work_run+0x4e/0x70 >> > [ 29.714566] [] do_exit+0x43a/0x9c7 >> > [ 29.715300] [] do_group_exit+0x4a/0xd6 >> > [ 29.716111] [] SyS_exit_group+0x11/0x11 >> > [ 29.716975] [] syscall_call+0x7/0x7 >> > [ 29.717722] [] ? io_submit_one+0x5b3/0x72a >> > [ 29.719207] c103c7ff 0000040d c1581310 >> > [ 29.720122] 00000001 00000001 00000000 d34d9ed0 c103c89c 00000009 = 00000000 d34d9ee8 >> > [ 29.721973] c1581310 c18836fc 00000004 d0ebb840 00000000 d34d9f0c = c157f810 >> > [ 29.723252] ---[ end trace 28426633f87b780a ]--- >> > [ 29.724282] ------------[ cut here ]------------ > = > = > It doesn't make much sense to me. I don't see anything > remap_file_pages()-related here. Or even VMA related. > = > And why now? The commit is in Andrew's tree for few release cycles. > = > Could you elaborate on what you observe? And what the table above means? > = --===============1290908293297865504==--