From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_PASS,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3D032C43387 for ; Mon, 17 Dec 2018 19:08:40 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 039B6214C6 for ; Mon, 17 Dec 2018 19:08:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1545073720; bh=cr1ptpSKwY/dz1KiayisNq6Hh6//Cex6kdBnmnrZ+mc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=UZRqcDOzV/ng8HarcGX+/A6blZxKtD+8MGLAmf44jMJomW9T0wd2kkiQXm6vFLtDQ p9DpEnPqAz15zLTK4bz3uUj8A3JT8BcQp3VuW+W/HtMecLk5MqXn5LYxWDR+JF3tqt 9Y+GVqKBpXXiTLHlTW3Wv2anMY9EwNSw+AdzfWnk= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2387571AbeLQTIj (ORCPT ); Mon, 17 Dec 2018 14:08:39 -0500 Received: from mail.kernel.org ([198.145.29.99]:54526 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726399AbeLQTIi (ORCPT ); Mon, 17 Dec 2018 14:08:38 -0500 Received: from localhost (5356596B.cm-6-7b.dynamic.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id ABB9221473; Mon, 17 Dec 2018 19:08:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1545073717; bh=cr1ptpSKwY/dz1KiayisNq6Hh6//Cex6kdBnmnrZ+mc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=xBflS5p3HI9InaoMn4Pjrd9N0tNkc+ALkMP4Qqa3ZXKSNXnm+wa81xEC5eiTpPPQ9 drRS2KWyaef9/MTrh25KOEVsvA/uDwgL8hGQIF0HPiF4JNSJBKs0cAthzpP7VLtdwu AXV86bc+NGMNvXC/WRsc79WXDNa3+rjFHErS5XSY= Date: Mon, 17 Dec 2018 20:08:34 +0100 From: Greg Kroah-Hartman To: Guenter Roeck Cc: Ben Hutchings , linux-kernel@vger.kernel.org, torvalds@linux-foundation.org, akpm@linux-foundation.org, shuah@kernel.org, patches@kernelci.org, lkft-triage@lists.linaro.org, stable@vger.kernel.org Subject: Re: [PATCH 4.4 00/88] 4.4.168-stable review Message-ID: <20181217190834.GB13773@kroah.com> References: <20181214115702.151309521@linuxfoundation.org> <20181215080729.GG10074@kroah.com> <1545004693.19315.66.camel@codethink.co.uk> <20181217090533.GA12938@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.11.1 (2018-12-01) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Dec 17, 2018 at 05:46:01AM -0800, Guenter Roeck wrote: > On 12/17/18 1:05 AM, Greg Kroah-Hartman wrote: > > On Sun, Dec 16, 2018 at 11:58:13PM +0000, Ben Hutchings wrote: > > > On Sat, 2018-12-15 at 07:45 -0800, Guenter Roeck wrote: > > > > On 12/15/18 12:07 AM, Greg Kroah-Hartman wrote: > > > > > On Fri, Dec 14, 2018 at 06:10:29PM -0800, Guenter Roeck wrote: > > > > > > On 12/14/18 3:59 AM, Greg Kroah-Hartman wrote: > > > > > > > This is the start of the stable review cycle for the 4.4.168 release. > > > > > > > There are 88 patches in this series, all will be posted as a response > > > > > > > to this one.  If anyone has any issues with these being applied, please > > > > > > > let me know. > > > > > > > > > > > > > > Responses should be made by Sun Dec 16 11:56:41 UTC 2018. > > > > > > > Anything received after that time might be too late. > > > > > > > > > > > > Build results: > > > > > > total: 170 pass: 145 fail: 25 > > > > > > Failed builds: > > > > > > arm:allnoconfig > > > > > > arm:tinyconfig > > > > > > arm:efm32_defconfig > > > > > > blackfin:defconfig > > > > > > blackfin:BF561-EZKIT-SMP_defconfig > > > > > > c6x:dsk6455_defconfig > > > > > > c6x:evmc6457_defconfig > > > > > > c6x:evmc6678_defconfig > > > > > > h8300:allnoconfig > > > > > > h8300:tinyconfig > > > > > > h8300:edosk2674_defconfig > > > > > > h8300:h8300h-sim_defconfig > > > > > > h8300:h8s-sim_defconfig > > > > > > m68k:allnoconfig > > > > > > m68k:tinyconfig > > > > > > m68k:m5272c3_defconfig > > > > > > m68k:m5307c3_defconfig > > > > > > m68k:m5249evb_defconfig > > > > > > m68k:m5407c3_defconfig > > > > > > microblaze:nommu_defconfig > > > > > > microblaze:allnoconfig > > > > > > microblaze:tinyconfig > > > > > > sh:defconfig > > > > > > sh:allnoconfig > > > > > > sh:tinyconfig > > > > > > Qemu test results: > > > > > > total: 288 pass: 288 fail: 0 > > > > > > > > > > > > mm/nommu.c: In function '__get_user_pages_unlocked': > > > > > > mm/nommu.c:211:49: error: 'write' undeclared (first use in this function) > > > > > > mm/nommu.c:211:56: error: 'force' undeclared (first use in this function) > > > > > > mm/nommu.c:212:9: warning: passing argument 7 of 'get_user_pages' from incompatible pointer type [enabled by default] > > > > > > mm/nommu.c:185:6: note: expected 'struct vm_area_struct **' but argument is of type 'struct page **' > > > > > > mm/nommu.c:212:9: error: too many arguments to function 'get_user_pages' > > > > > > mm/nommu.c:185:6: note: declared here > > > > > > > > > > > > Details are available at https://kerneltests.org/builders/. > > > > > > > > > > Ugh, I'll dig through this later on today, we must be missing something > > > > > with those backports that Ben did... > > > > > > > > > > > > > 69ce144e5c3a ("mm: replace get_user_pages() write/force parameters > > > > with gup_flags") seems to have missed converting a call of > > > > get_user_pages(). > > > > > > Right. This was changed earlier upstream in commit cde70140fed8 > > > "mm/gup: Overload get_user_pages() functions", but I don't think it > > > makes sense to apply all of that. I'm attaching a minimal patch > > > (tested with an arm allnoconfig build) which should ideally be inserted > > > before mm-replace-get_user_pages-write-force-parameters-with- > > > gup_flags.patch. > > > > > > Ben. > > > > > > -- > > > Ben Hutchings, Software Developer   Codethink Ltd > > > https://www.codethink.co.uk/ Dale House, 35 Dale Street > > > Manchester, M1 2HF, United Kingdom > > > > > From 0d0afe933f60f5736c984e9171214aa34b18764c Mon Sep 17 00:00:00 2001 > > > From: Ben Hutchings > > > Date: Sun, 16 Dec 2018 23:50:08 +0000 > > > Subject: [PATCH] mm/nommu.c: Switch __get_user_pages_unlocked() to use > > > __get_user_pages() > > > > > > Extracted from commit cde70140fed8 "mm/gup: Overload get_user_pages() > > > functions". This is needed before picking commit 768ae309a961 > > > "mm: replace get_user_pages() write/force parameters with gup_flags". > > > > > > Signed-off-by: Ben Hutchings > > > --- > > > mm/nommu.c | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/mm/nommu.c b/mm/nommu.c > > > index fa1560c218d5..2360546db065 100644 > > > --- a/mm/nommu.c > > > +++ b/mm/nommu.c > > > @@ -208,8 +208,8 @@ long __get_user_pages_unlocked(struct task_struct *tsk, struct mm_struct *mm, > > > { > > > long ret; > > > down_read(&mm->mmap_sem); > > > - ret = get_user_pages(tsk, mm, start, nr_pages, write, force, > > > - pages, NULL); > > > + ret = __get_user_pages(tsk, mm, start, nr_pages, gup_flags, pages, > > > + NULL, NULL); > > > up_read(&mm->mmap_sem); > > > return ret; > > > } > > > > Thanks for the patch. I've added it to the queue and pushed out a -rc2 > > with this in it. > > > > Let's see what the builders say :) > > > > v4.4.167-89-g9c558d7fe359 seemed to be happy. v4.4.167-89-g50a0280f2f7e > replaced it and will take a while. If I read your site right, it passed everything except one qemu test? Is that normal? thanks, greg k-h From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.kernel.org ([198.145.29.99]:54526 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726399AbeLQTIi (ORCPT ); Mon, 17 Dec 2018 14:08:38 -0500 Date: Mon, 17 Dec 2018 20:08:34 +0100 From: Greg Kroah-Hartman To: Guenter Roeck Cc: Ben Hutchings , linux-kernel@vger.kernel.org, torvalds@linux-foundation.org, akpm@linux-foundation.org, shuah@kernel.org, patches@kernelci.org, lkft-triage@lists.linaro.org, stable@vger.kernel.org Subject: Re: [PATCH 4.4 00/88] 4.4.168-stable review Message-ID: <20181217190834.GB13773@kroah.com> References: <20181214115702.151309521@linuxfoundation.org> <20181215080729.GG10074@kroah.com> <1545004693.19315.66.camel@codethink.co.uk> <20181217090533.GA12938@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Sender: stable-owner@vger.kernel.org List-ID: On Mon, Dec 17, 2018 at 05:46:01AM -0800, Guenter Roeck wrote: > On 12/17/18 1:05 AM, Greg Kroah-Hartman wrote: > > On Sun, Dec 16, 2018 at 11:58:13PM +0000, Ben Hutchings wrote: > > > On Sat, 2018-12-15 at 07:45 -0800, Guenter Roeck wrote: > > > > On 12/15/18 12:07 AM, Greg Kroah-Hartman wrote: > > > > > On Fri, Dec 14, 2018 at 06:10:29PM -0800, Guenter Roeck wrote: > > > > > > On 12/14/18 3:59 AM, Greg Kroah-Hartman wrote: > > > > > > > This is the start of the stable review cycle for the 4.4.168 release. > > > > > > > There are 88 patches in this series, all will be posted as a response > > > > > > > to this one.��If anyone has any issues with these being applied, please > > > > > > > let me know. > > > > > > > > > > > > > > Responses should be made by Sun Dec 16 11:56:41 UTC 2018. > > > > > > > Anything received after that time might be too late. > > > > > > > > > > > > Build results: > > > > > > total: 170 pass: 145 fail: 25 > > > > > > Failed builds: > > > > > > arm:allnoconfig > > > > > > arm:tinyconfig > > > > > > arm:efm32_defconfig > > > > > > blackfin:defconfig > > > > > > blackfin:BF561-EZKIT-SMP_defconfig > > > > > > c6x:dsk6455_defconfig > > > > > > c6x:evmc6457_defconfig > > > > > > c6x:evmc6678_defconfig > > > > > > h8300:allnoconfig > > > > > > h8300:tinyconfig > > > > > > h8300:edosk2674_defconfig > > > > > > h8300:h8300h-sim_defconfig > > > > > > h8300:h8s-sim_defconfig > > > > > > m68k:allnoconfig > > > > > > m68k:tinyconfig > > > > > > m68k:m5272c3_defconfig > > > > > > m68k:m5307c3_defconfig > > > > > > m68k:m5249evb_defconfig > > > > > > m68k:m5407c3_defconfig > > > > > > microblaze:nommu_defconfig > > > > > > microblaze:allnoconfig > > > > > > microblaze:tinyconfig > > > > > > sh:defconfig > > > > > > sh:allnoconfig > > > > > > sh:tinyconfig > > > > > > Qemu test results: > > > > > > total: 288 pass: 288 fail: 0 > > > > > > > > > > > > mm/nommu.c: In function '__get_user_pages_unlocked': > > > > > > mm/nommu.c:211:49: error: 'write' undeclared (first use in this function) > > > > > > mm/nommu.c:211:56: error: 'force' undeclared (first use in this function) > > > > > > mm/nommu.c:212:9: warning: passing argument 7 of 'get_user_pages' from incompatible pointer type [enabled by default] > > > > > > mm/nommu.c:185:6: note: expected 'struct vm_area_struct **' but argument is of type 'struct page **' > > > > > > mm/nommu.c:212:9: error: too many arguments to function 'get_user_pages' > > > > > > mm/nommu.c:185:6: note: declared here > > > > > > > > > > > > Details are available at https://kerneltests.org/builders/. > > > > > > > > > > Ugh, I'll dig through this later on today, we must be missing something > > > > > with those backports that Ben did... > > > > > > > > > > > > > 69ce144e5c3a ("mm: replace get_user_pages() write/force parameters > > > > with gup_flags") seems to have missed converting a call of > > > > get_user_pages(). > > > > > > Right. This was changed earlier upstream in commit cde70140fed8 > > > "mm/gup: Overload get_user_pages() functions", but I don't think it > > > makes sense to apply all of that. I'm attaching a minimal patch > > > (tested with an arm allnoconfig build) which should ideally be inserted > > > before mm-replace-get_user_pages-write-force-parameters-with- > > > gup_flags.patch. > > > > > > Ben. > > > > > > -- > > > Ben Hutchings, Software Developer � Codethink Ltd > > > https://www.codethink.co.uk/ Dale House, 35 Dale Street > > > Manchester, M1 2HF, United Kingdom > > > > > From 0d0afe933f60f5736c984e9171214aa34b18764c Mon Sep 17 00:00:00 2001 > > > From: Ben Hutchings > > > Date: Sun, 16 Dec 2018 23:50:08 +0000 > > > Subject: [PATCH] mm/nommu.c: Switch __get_user_pages_unlocked() to use > > > __get_user_pages() > > > > > > Extracted from commit cde70140fed8 "mm/gup: Overload get_user_pages() > > > functions". This is needed before picking commit 768ae309a961 > > > "mm: replace get_user_pages() write/force parameters with gup_flags". > > > > > > Signed-off-by: Ben Hutchings > > > --- > > > mm/nommu.c | 4 ++-- > > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/mm/nommu.c b/mm/nommu.c > > > index fa1560c218d5..2360546db065 100644 > > > --- a/mm/nommu.c > > > +++ b/mm/nommu.c > > > @@ -208,8 +208,8 @@ long __get_user_pages_unlocked(struct task_struct *tsk, struct mm_struct *mm, > > > { > > > long ret; > > > down_read(&mm->mmap_sem); > > > - ret = get_user_pages(tsk, mm, start, nr_pages, write, force, > > > - pages, NULL); > > > + ret = __get_user_pages(tsk, mm, start, nr_pages, gup_flags, pages, > > > + NULL, NULL); > > > up_read(&mm->mmap_sem); > > > return ret; > > > } > > > > Thanks for the patch. I've added it to the queue and pushed out a -rc2 > > with this in it. > > > > Let's see what the builders say :) > > > > v4.4.167-89-g9c558d7fe359 seemed to be happy. v4.4.167-89-g50a0280f2f7e > replaced it and will take a while. If I read your site right, it passed everything except one qemu test? Is that normal? thanks, greg k-h