[{"id":1760010,"web_url":"http://patchwork.ozlabs.org/comment/1760010/","msgid":"<20170830095854.GA17353@leverpostej>","list_archive_url":null,"date":"2017-08-30T09:58:54","subject":"Re: [RFC PATCH] arm64: move THREAD_* definitions to separated header","submitter":{"id":8806,"url":"http://patchwork.ozlabs.org/api/people/8806/","name":"Mark Rutland","email":"mark.rutland@arm.com"},"content":"On Wed, Aug 30, 2017 at 12:22:49PM +0300, Yury Norov wrote:\n> Hi Mark, all.\n\nHi Yury, \n\n> In patch 'dbc9344a68e506f19f8 (\"arm64: clean up THREAD_* definitions\")'\n> you move THREAD_* definitions from arch/arm64/include/asm/thread_info.h\n> to asm/memory.h. After that asm/thread_info.h starts to depend on\n> asm/memory.h.\n> \n> When I try to apply ilp32 series on top of it [1], it causes circular\n> dependencies like this one:\n> \n> In file included from ./arch/arm64/include/asm/memory.h:30:0,\n>                  from ./arch/arm64/include/asm/thread_info.h:30,\n>                  from ./include/linux/thread_bits.h:20,\n>                  from ./include/linux/thread_info.h:13,\n>                  from ./include/asm-generic/preempt.h:4,\n>                  from ./arch/arm64/include/generated/asm/preempt.h:1,\n>                  from ./include/linux/preempt.h:80,\n>                  from ./include/linux/rcupdate.h:40,\n>                  from ./include/linux/rculist.h:10,\n>                  from ./include/linux/pid.h:4,\n>                  from ./include/linux/sched.h:13,\n>                  from arch/arm64/kernel/asm-offsets.c:21:\n> ./arch/arm64/include/asm/is_compat.h: In function ‘is_a32_compat_task’:\n> ./arch/arm64/include/asm/is_compat.h:25:9: error: implicit declaration of function ‘test_thread_flag’ [-Werror=implicit-function-declaration]\n>   return test_thread_flag(TIF_32BIT);\n>          ^~~~~~~~~~~~~~~~\n> \n> The problem is that asm/memory.h depends on asm/is_compat.h to define\n> TASK_SIZE, which in turn requires asm/thread_info.h.\n\n... and <asm/thread_info.h> include <asm/memory.h>, giving a circular\ndependency.\n\nIn other architectures, TASK_SIZE is defined in processor.h. Can we not\nmove TASK_SIZE instead of THREAD_SIZE, given that TASK_SIZE is what\ncauses the dependency?\n\nWe'd need a new __ASSEMBLY__ guard, but otherwise it looks like that\nwould solve the issue.\n\n> The most obvious solution for it is to create is_compat.c file and make\n> is_*_compat() real functions. The other option is to move THREAD_*\n> definitions to separated macro. I would prefer 2nd one because of following\n> reasons:\n>  - TASK_SIZE macro is used many times in kernel, including hot paths;\n>  - asm/memory.h is included widely, as well as asm/thread_info.h, and it's\n>    better not to make them depend one from another;\n>  - THREAD_SIZE etc are not memory-related definitions.\n\nI disagree with this last point. THREAD_SIZE is in <asm/memory.h>\nbecause it affects the kernel's memory layout, as with other definitions\nat the top of <asm/memory.h>.\n\nI would much prefer keeping those definitions together.\n\n> In this patch THREAD_* definitions moved to separated asm/thread_size.h\n> header. It's enough to resolve dependency above.\n>\n> If you find this approach useful, I can prepare other patch that moves\n> TASK_* definitions from asm/memory.h to new header to remove the dependency\n> from asm/is_compat.h.\n\nI'd prefer that we moved the TASK_SIZE* definitions first, but if\nnecessary, I'm ok with seeing the THREAD_SIZE* definitions move.\n\n[...]\n\n> diff --git a/arch/arm64/include/asm/thread_info.h b/arch/arm64/include/asm/thread_info.h\n\n> -#include <asm/memory.h>\n> +#include <asm/thread_size.h>\n>  #include <asm/stack_pointer.h>\n>  #include <asm/types.h>\n\nNit: please keep includes ordered alphabetically.\n\n[...]\n\n> diff --git a/arch/arm64/include/asm/thread_size.h b/arch/arm64/include/asm/thread_size.h\n\n> +#ifdef __KERNEL__\n\nThis can go. We haven't needed __KERNEL__ ifdefs for a long time now...\n\n[...]\n\n> diff --git a/arch/arm64/kernel/hibernate-asm.S b/arch/arm64/kernel/hibernate-asm.S\n> index e56d848b6466..46b91702ea26 100644\n> --- a/arch/arm64/kernel/hibernate-asm.S\n> +++ b/arch/arm64/kernel/hibernate-asm.S\n> @@ -22,7 +22,6 @@\n>  #include <asm/asm-offsets.h>\n>  #include <asm/assembler.h>\n>  #include <asm/cputype.h>\n> -#include <asm/memory.h>\n>  #include <asm/page.h>\n>  #include <asm/virt.h>\n\nAFAICT, we can also lose <linux/errno.h>, <asm/cputype.h>, and\n<asm/virt.h>.\n\nChanges to this file might be better as a preparatory cleanup.\n\nThanks,\nMark.","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"pNpN2pKE\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xj1HF0trNz9t0F\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 20:01:13 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dmzo5-00037h-AR; Wed, 30 Aug 2017 10:01:09 +0000","from usa-sjc-mx-foss1.foss.arm.com ([217.140.101.70]\n\thelo=foss.arm.com)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dmznX-0002Gm-0i for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 10:01:06 +0000","from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249])\n\tby usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id C8B7D13D5;\n\tWed, 30 Aug 2017 03:00:14 -0700 (PDT)","from leverpostej (usa-sjc-imap-foss1.foss.arm.com [10.72.51.249])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id\n\t34F943F483; Wed, 30 Aug 2017 03:00:13 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=pxyDMQ0fElsW+0dQ6Qv1PE3UL8IAIzUhaM/Hk3tIc3I=;\n\tb=pNpN2pKE+DMq5U\n\tvPzcafEXBoVgtk6ZxnLpUYQqlFvTzsUEKiqHgtgmNwd06lpsHc4MEAvmzQikr2Bb10sLWrvC4782/\n\t2I9omKmFd9rEG+lLtd31Ou+SIltljIGVnaWaxmha8E8Yah11iY9T2Tu7mZ97P1x8SJANoC4NlBFoc\n\tZNcfILclRs26l4hiX0i6EnlblKiI6VvHPar21+rBDacafrTdBnxsBDCtABt22onhZ5x1QE7IJH8dy\n\tfu4Ko9rYdQzFW+GWK0yHTMkV5vxVui5odkmr8q6WJ+EJX/IiGKfGSSKEvIZZtq+hIIwoeltnBZtDe\n\teRC7lGQuI8jO3eC92BiQ==;","Date":"Wed, 30 Aug 2017 10:58:54 +0100","From":"Mark Rutland <mark.rutland@arm.com>","To":"Yury Norov <ynorov@caviumnetworks.com>","Subject":"Re: [RFC PATCH] arm64: move THREAD_* definitions to separated header","Message-ID":"<20170830095854.GA17353@leverpostej>","References":"<20170830092249.20638-1-ynorov@caviumnetworks.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170830092249.20638-1-ynorov@caviumnetworks.com>","User-Agent":"Mutt/1.5.21 (2010-09-15)","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170830_030035_602316_CBC8827E ","X-CRM114-Status":"GOOD (  18.25  )","X-Spam-Score":"-6.9 (------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-6.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [217.140.101.70 listed in list.dnswl.org]\n\t-0.0 SPF_PASS               SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"Ard Biesheuvel <ard.biesheuvel@linaro.org>,\n\tCatalin Marinas <catalin.marinas@arm.com>,\n\tWill Deacon <will.deacon@arm.com>, \n\tlinux-kernel@vger.kernel.org, James Morse <james.morse@arm.com>,\n\tLaura Abbott <labbott@redhat.com>, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1760048,"web_url":"http://patchwork.ozlabs.org/comment/1760048/","msgid":"<20170830104805.cl44gwqslocnr5w4@yury-thinkpad>","list_archive_url":null,"date":"2017-08-30T10:48:05","subject":"Re: [RFC PATCH] arm64: move THREAD_* definitions to separated header","submitter":{"id":68471,"url":"http://patchwork.ozlabs.org/api/people/68471/","name":"Yury Norov","email":"ynorov@caviumnetworks.com"},"content":"On Wed, Aug 30, 2017 at 10:58:54AM +0100, Mark Rutland wrote:\n> On Wed, Aug 30, 2017 at 12:22:49PM +0300, Yury Norov wrote:\n> > Hi Mark, all.\n> \n> Hi Yury, \n> \n> > In patch 'dbc9344a68e506f19f8 (\"arm64: clean up THREAD_* definitions\")'\n> > you move THREAD_* definitions from arch/arm64/include/asm/thread_info.h\n> > to asm/memory.h. After that asm/thread_info.h starts to depend on\n> > asm/memory.h.\n> > \n> > When I try to apply ilp32 series on top of it [1], it causes circular\n> > dependencies like this one:\n> > \n> > In file included from ./arch/arm64/include/asm/memory.h:30:0,\n> >                  from ./arch/arm64/include/asm/thread_info.h:30,\n> >                  from ./include/linux/thread_bits.h:20,\n> >                  from ./include/linux/thread_info.h:13,\n> >                  from ./include/asm-generic/preempt.h:4,\n> >                  from ./arch/arm64/include/generated/asm/preempt.h:1,\n> >                  from ./include/linux/preempt.h:80,\n> >                  from ./include/linux/rcupdate.h:40,\n> >                  from ./include/linux/rculist.h:10,\n> >                  from ./include/linux/pid.h:4,\n> >                  from ./include/linux/sched.h:13,\n> >                  from arch/arm64/kernel/asm-offsets.c:21:\n> > ./arch/arm64/include/asm/is_compat.h: In function ‘is_a32_compat_task’:\n> > ./arch/arm64/include/asm/is_compat.h:25:9: error: implicit declaration of function ‘test_thread_flag’ [-Werror=implicit-function-declaration]\n> >   return test_thread_flag(TIF_32BIT);\n> >          ^~~~~~~~~~~~~~~~\n> > \n> > The problem is that asm/memory.h depends on asm/is_compat.h to define\n> > TASK_SIZE, which in turn requires asm/thread_info.h.\n> \n> ... and <asm/thread_info.h> include <asm/memory.h>, giving a circular\n> dependency.\n> \n> In other architectures, TASK_SIZE is defined in processor.h. Can we not\n> move TASK_SIZE instead of THREAD_SIZE, given that TASK_SIZE is what\n> causes the dependency?\n> \n> We'd need a new __ASSEMBLY__ guard, but otherwise it looks like that\n> would solve the issue.\n> \n> > The most obvious solution for it is to create is_compat.c file and make\n> > is_*_compat() real functions. The other option is to move THREAD_*\n> > definitions to separated macro. I would prefer 2nd one because of following\n> > reasons:\n> >  - TASK_SIZE macro is used many times in kernel, including hot paths;\n> >  - asm/memory.h is included widely, as well as asm/thread_info.h, and it's\n> >    better not to make them depend one from another;\n> >  - THREAD_SIZE etc are not memory-related definitions.\n> \n> I disagree with this last point. THREAD_SIZE is in <asm/memory.h>\n> because it affects the kernel's memory layout, as with other definitions\n> at the top of <asm/memory.h>.\n> \n> I would much prefer keeping those definitions together.\n> \n> > In this patch THREAD_* definitions moved to separated asm/thread_size.h\n> > header. It's enough to resolve dependency above.\n> >\n> > If you find this approach useful, I can prepare other patch that moves\n> > TASK_* definitions from asm/memory.h to new header to remove the dependency\n> > from asm/is_compat.h.\n> \n> I'd prefer that we moved the TASK_SIZE* definitions first, but if\n> necessary, I'm ok with seeing the THREAD_SIZE* definitions move.\n\nOK, I will try this, and if it's be enough will send new patch. If not -\nI'll resend this patch and one that moves TASK_SIZE* together in series.\n\n> [...]\n> \n> > diff --git a/arch/arm64/include/asm/thread_info.h b/arch/arm64/include/asm/thread_info.h\n> \n> > -#include <asm/memory.h>\n> > +#include <asm/thread_size.h>\n> >  #include <asm/stack_pointer.h>\n> >  #include <asm/types.h>\n> \n> Nit: please keep includes ordered alphabetically.\n\nOK\n\n> [...]\n> \n> > diff --git a/arch/arm64/include/asm/thread_size.h b/arch/arm64/include/asm/thread_size.h\n> \n> > +#ifdef __KERNEL__\n> \n> This can go. We haven't needed __KERNEL__ ifdefs for a long time now...\n\nOK\n\n> [...]\n> \n> > diff --git a/arch/arm64/kernel/hibernate-asm.S b/arch/arm64/kernel/hibernate-asm.S\n> > index e56d848b6466..46b91702ea26 100644\n> > --- a/arch/arm64/kernel/hibernate-asm.S\n> > +++ b/arch/arm64/kernel/hibernate-asm.S\n> > @@ -22,7 +22,6 @@\n> >  #include <asm/asm-offsets.h>\n> >  #include <asm/assembler.h>\n> >  #include <asm/cputype.h>\n> > -#include <asm/memory.h>\n> >  #include <asm/page.h>\n> >  #include <asm/virt.h>\n> \n> AFAICT, we can also lose <linux/errno.h>, <asm/cputype.h>, and\n> <asm/virt.h>.\n> \n> Changes to this file might be better as a preparatory cleanup.\n\nOK.\n\nYury","headers":{"Return-Path":"<linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org>","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=<UNKNOWN>)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"DsV1QcAl\"; \n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com\n\theader.i=@CAVIUMNETWORKS.onmicrosoft.com header.b=\"a5x7LlHs\"; \n\tdkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=Yuri.Norov@cavium.com; "],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xj2L60Jlhz9s8P\n\tfor <incoming-imx@patchwork.ozlabs.org>;\n\tWed, 30 Aug 2017 20:48:46 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn0Y6-0004f2-Ft; Wed, 30 Aug 2017 10:48:42 +0000","from mail-co1nam03on060c.outbound.protection.outlook.com\n\t([2a01:111:f400:fe48::60c]\n\thelo=NAM03-CO1-obe.outbound.protection.outlook.com)\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dn0Y2-0004dc-RT for linux-arm-kernel@lists.infradead.org;\n\tWed, 30 Aug 2017 10:48:40 +0000","from localhost (2a01:cb1d:41b:1500:829:7821:d3b5:bd14) by\n\tDM2PR07MB1450.namprd07.prod.outlook.com (2a01:111:e400:50c3::16) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.1.1362.18;\n\tWed, 30 Aug 2017 10:48:15 +0000"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=eS1Bsyfhp31y7jnl4cQU9tFug4ttvMW3PZfOwpjpnGk=;\n\tb=DsV1QcAltLqAPr\n\tIF4ISuvCdja8kSmX0TCTqDRKtney3fwWNMeDChCazg8vjCFc9/r4bRPDobGGCkf4uy9URelsSeWGR\n\tkerHVLhwpS9d+8gwFChZnl/lo70Z+PgyfYlZhthgfrvNG6TiBAmoJSwutGQ3YUNqlN4KbTrxeeI9W\n\tTR0LKgf0Rqkv0UhFXve6ZyLP0oQuH9K8IRljhcWGmnful/gFyMc3z7Ugd+IeBx9YubOSSccU0ii9E\n\tY6ICG1xOCjEGdAV43BoiET49IjDfESiN3WQB0cwY3N1WWbfqqiVUu0DXfReFGGUXZscv86tvOHJjb\n\tFP5W8JZ7bCcn4v7yIp0g==;","v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=eEMGGT57KdQulaA2uAXkMcFhIJE9xfuryCR692WVFho=;\n\tb=a5x7LlHslU7AxLLhPLrW1RQWdCQrtqpnnnE3sc+lBcgyChMzLPqgRQmIrB1+Goae3e4GtZSr9XBcvTbgIQkWUJ/RKrb4W6Ebajspbq9Bu5W9HOE2GGBZAAjZTnwtAA1ESjSkSFKm7LNKD0+83cDAO261G4YfxKLVcIlMrnhsOTI="],"Date":"Wed, 30 Aug 2017 13:48:05 +0300","From":"Yury Norov <ynorov@caviumnetworks.com>","To":"Mark Rutland <mark.rutland@arm.com>","Subject":"Re: [RFC PATCH] arm64: move THREAD_* definitions to separated header","Message-ID":"<20170830104805.cl44gwqslocnr5w4@yury-thinkpad>","References":"<20170830092249.20638-1-ynorov@caviumnetworks.com>\n\t<20170830095854.GA17353@leverpostej>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170830095854.GA17353@leverpostej>","User-Agent":"NeoMutt/20170113 (1.7.2)","X-Originating-IP":"[2a01:cb1d:41b:1500:829:7821:d3b5:bd14]","X-ClientProxiedBy":"VI1PR0502CA0010.eurprd05.prod.outlook.com\n\t(2603:10a6:803:1::23) To DM2PR07MB1450.namprd07.prod.outlook.com\n\t(2a01:111:e400:50c3::16)","X-MS-PublicTrafficType":"Email","X-MS-Office365-Filtering-Correlation-Id":"19bf68f4-219c-4590-6ac3-08d4ef949e63","X-Microsoft-Antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);\n\tSRVR:DM2PR07MB1450; ","X-Microsoft-Exchange-Diagnostics":["1; DM2PR07MB1450;\n\t3:AjF9pNZRM36qfn77/7VoSG3NDiLKo9ySukQkc955AbeeWg8YmFifStQqZ+7VJUQxsgD5+vOI23bbgUVZTVbtblPQkk8u5EuSoLzVHX5M1hyp/YQZKRSrhFTRdG+wZ83rkxm6dKSPlaE5ZTq7Rs9VEUduvOESKTe2W2TUZxzHw3NWB9zop+Exg9uam3J0RGquFhR9UKXcQPfBgOEzm70ms/N5DCcwDXQqgk1uGL6oaF/lasg6qBqcyk7F/eYTd/jI;\n\t25:F6zFb3trbfF2B8pRNfEpPwL03966zwRPOg8ojk9sW6/UhNtmgG39fQich0LxI+/8gVeY2eJcIImOwkV6HdllgeQesxdPRnBB1uPyFXci6rSwIdT/ZMVLD2fgEB/0HJDcQSZD+lYsJCbG1/UWEmZtUXo/VGCz7Q/OMJCwTD4JrFK/vY3/VlYgdAeje5Ys4UeC7IM7IIm1qk5TOgB81JDg5a9cmXQhrjuInGwMLBAGsCuX8TsPb+x6Ck3SEjDkn9f6FKvOWsfa4l/JuewtP/eXyJVoKoIQ0Gtb2WPvXqrGaKWiokQ6Afxr7dd0vr9MMn12OH4oFtDvkzTs7UN3lIKuJA==;\n\t31:qdsqtwmrjuav2c+n6lqJQOVhfLk00pqqHUog021zLn0kxuEvAWzHt1YiqG44iv45JE7u+BTqvfwEQBr5KdF1VOubEo8h4IDM1iZFpMmrz7b4egX35SyPHtebOQxFru8QBrX0akNE8gnw28hGhi2tSHY+X60YiaW1ODIXWuShq/Z99t+9wCPcInI98Ijsfosd7AwSrvlrK/Rxy8sdU4kTo5QCos2xOSSgwJZac5M6Gyc=","1; DM2PR07MB1450;\n\t20:FZklf5G0QwJDrgKBKXPJM4XFyDfJd33hy+/IYO0hnkQ+ieZ3wJCKrJDh1b1LVXhSW6+ZU9xirzR6ukbeBkB2ylobOThmmryPrkoiNjF2pOMJoicKrLKODjw4de7ZTCj2UKlMYGURDkbaUBnreL8drIP/GHij1o0z5eyBbIkDtZqx6HZYNRMRWdwB7DdIup5xh6Qg8mpsAWPbFR1VY92tvkJE9XarTCdg/Jum/djO7E3uFJbwsqgqEpnSOCmwe+dYqjY4Y5NvHOMYootgDGXJ/fZhmiXp8n+EqVt/cYAUrVyaLHxtNXyoqLt2RpBJVYI0ion+6YpAKilA3tH/p8/eHpJFiw6VHpZCfydlZNR96qTUOMLaJy55hX59AWgwp1Pu/7QXHiI5j4RiTry8/I9TH0B+0tJ3LubEQ8MdakZ8BkaqtKGYfWc7jknUkHMUXKKbUKRfRDcKtIao+MHzRggVrZT3Pwz/OHZ1MqWU1y07+hhqmpfocivuMzz7VClWCmgqEf0M1LbyKq7/5Uk3mZWnH2Ai55u8UH3WlbEYHkq55GdPN8M96+WrttjzvNoHyd1Dx6Hk1I4fZa7XV3BQ1/bKyVS+c6AKMlUWnEypNq/it84=;\n\t4:80wMrUr/IS11xeYjmLH1PZ7onS82XAHS3oHsdmRluGY65COIfJc5qt624ob6DjUnKLfI4bPiONo+CtsX3m7g262N++bf6Nef4h3wFSheE8g/PbtGXouQ8m7rJ0frnXasntMjSKU6JcBTtEgfUISTZ/FR1SIv9qa8bu0tI/hR9Pvf4k/DLxzczfCwY1CrLwzX1maAiijroS/OPQu7oR+VT80xbVro4N7WLlC7SJRHw6Z63b7kN0uwpalHPwdMI9wK","=?utf-8?q?1=3BDM2PR07MB1450=3B23=3AA+wF?=\n\t=?utf-8?q?dqXK/om9ndex0ZaYMcsJuH1VBTL1oFQHId1RgrX29uSdokVEL20TQfJ1?=\n\t=?utf-8?q?jmjr8d2yt2EJj3fqLyKIXxcIFrjYVh/XGRQ3oSSskgrQp6gNcEIX8DTg?=\n\t=?utf-8?q?tdcpgZudqxSA2HvM4tWEZqUjedLR/AJ82pu8iEB8rySM2AyvLaELvz58?=\n\t=?utf-8?q?PEGE2KeBtOMQ9mM4FTfZpxFFASNbs3tzIDKJqzwj1A8S69tM9epGXKdL?=\n\t=?utf-8?q?JJbwhtCPc+52S/r5Peiatpr6+BW3kXEWgncE3IuqxCxgILQ3qm6oI6Eg?=\n\t=?utf-8?q?Ua6ufvASnFi5E02P767omp/DaJfOtNMmZknhRt+a004twbG0RH+vqy3/?=\n\t=?utf-8?q?4NkZJIq6VT4FUCziwA05QAYC24reUqQodn0dB0x0OlXIXcwNbaE4aN9b?=\n\t=?utf-8?q?aTGsskExUHJtoVm9Pt1q3DYV/vovAekyG0jYeQ5hmUlz05WBjfxC8NtZ?=\n\t=?utf-8?q?JytCMoyAN86Q+rVxMHHTOt4BJApAiW0lA5UtbpmQahYU39mBmZJC5LAF?=\n\t=?utf-8?q?ASaJxh321c2dY3q3ko2Y+p9SOIRAerwZp5Nq2/AU8B36vTzDXUdunaF1?=\n\t=?utf-8?q?hK+mnGOddiBGTsN5mwP+7PCXifYpECgH9kQzOF9CQ1P6+s0sSucnW/QF?=\n\t=?utf-8?q?lE5Aq8U9YCM/T/LljWXmymCzKVqdmC5ol3NuxhMyoybetZR3ZSZ/QsnT?=\n\t=?utf-8?q?4Vd1Ms2LdmhpMKjJEghcSpF5LWCsV6cWbc3xQbRIK3c72vcyVvCY/94z?=\n\t=?utf-8?q?4t4d/HleI0zc/V4zg3NxYFPnaHIafzfw29u12fw4PIR0KHaLGGRu26JY?=\n\t=?utf-8?q?buhZ3SKPN7konDBMWXPnXR1edaspOrYOz37tKslKPhud6bz6WV5MT69N?=\n\t=?utf-8?q?ZAHFmEq1heS8OvfgAAoLwVmchQcJmR7inX2AFbt7aiBD7UNJWaLnsVPa?=\n\t=?utf-8?q?ObqGQD5dOVT+NwFIWKiaF0kVcTLnluwOt/+0REZ15Pku45eas4RrFLgR?=\n\t=?utf-8?q?5Z1mFSFw930m6IKIcMosMI7SCpBcDLg5Y5x3iOlXEBS5HcC6bFoM6A9A?=\n\t=?utf-8?q?+u/F7TBdlxOaEGw6X0IuhgBMbhXVJsRI8JA70SKlQR6a2rNUHXZrkkwH?=\n\t=?utf-8?q?VgrSb0aVJSJHRbLLw+bacAgYAiHX4Zy3uIQ2JR1nkXrPJcmqR8/YedYb?=\n\t=?utf-8?q?9v+i5lCz4tqVP6TMiXLQ6q/y+21H7pIjJnYvrobk8+fzM2gO+8ZRE3tZ?=\n\t=?utf-8?q?9uAJno4+NOE7VQY=3D?=","1; DM2PR07MB1450;\n\t6:OuJwcgAmMleVFxCngYeCgDhD3dtERxA/xGZlL6XJ311WMleU3mxIliUPMkMRtfN22yYlVfA5kR3SFdKRgURlQ3vhCohNFFOPg87KKPYK6DBcjmAT9FXgaAE11AfVzDhMQjOxtTLL3KO8QTR8trtzIYrqbbLV3KBeo9CkxzP4CZf4TlNoHq3v/hKolmlTPi4Gvj6h8kLaA3CvL9zTzApnZnFsj3+OeROvsP5EjuLNNUDqleSdN+vm/AAk2HTwxwmVTemDkLfXq0Exbc07pGVK3W8LqC/sxRx3XQrcS90GLtth3/Do6EpIjFFmi5/rMAwnWnYB5DRI69z/xvW3mH/iIQ==;\n\t5:vXFO5nKMOTj4QDMaPymFG4/nlbA8ZrAfUdcao2D+08pCYtnRul8fD1YeENAmoSbnMlE7ux+Xkmev1DTqnFm1ooWDK44OQo4V/w909UBY4Q4TigsKGQ2sssypDkFNi3MsF3Ro20zD6OR+xzaHBf0/Aw==;\n\t24:mNX+z7IvcGNCd18V2TW2M8P9mA918mXqwR6EweTyZi0pVsN6aZNXYoe0ynJtKLvbWlC3WPYzvVEt8ccpVBnsHkHvOyA8Qs4iOn73Z5M0I2M=;\n\t7:zLTnfOKuyMz+S7mvmRezFSs+GsSDxCivef59JKs4rZtczp7F/O1CBjpt4l+lRH4B+8WjK1dJU5He8b5QdqsDIaaTh5wpXecgUICGA3pnAslrF98KOUHrWSmg3C4+4stPDJu9VjQXBEBJd5phf8UCAUo+Q/yn20nEAZwQtpiyVwPQ8bwrAqYMzykTnxWxSAKospaZKpEB+rH5FFynsXzGRNxDLIG3OJ4YVn9TfhJAxBU="],"X-MS-TrafficTypeDiagnostic":"DM2PR07MB1450:","X-Exchange-Antispam-Report-Test":"UriScan:;","X-Microsoft-Antispam-PRVS":"<DM2PR07MB1450E533CE319EED26D4AF1CE39C0@DM2PR07MB1450.namprd07.prod.outlook.com>","X-Exchange-Antispam-Report-CFA-Test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(5005006)(8121501046)(3002001)(100000703101)(100105400095)(10201501046)(93006095)(6041248)(20161123558100)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123562025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:DM2PR07MB1450; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:DM2PR07MB1450; ","X-Forefront-PRVS":"041517DFAB","X-Forefront-Antispam-Report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(6069001)(24454002)(199003)(189002)(33716001)(6246003)(4001350100001)(6116002)(68736007)(105586002)(54906002)(9686003)(229853002)(83506001)(53936002)(72206003)(6496005)(47776003)(2950100002)(42882006)(42186005)(76506005)(6486002)(6916009)(6666003)(97736004)(110136004)(106356001)(50986999)(478600001)(305945005)(76176999)(23676002)(189998001)(101416001)(50466002)(25786009)(4326008)(8676002)(33646002)(81166006)(81156014)(5660300001)(1076002)(8936002)(7736002)(2906002)(2870700001)(54356999);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:DM2PR07MB1450; H:localhost; FPR:;\n\tSPF:None; \n\tPTR:InfoNoRecords; A:1; MX:1; LANG:en; ","Received-SPF":"None (protection.outlook.com: cavium.com does not designate\n\tpermitted sender hosts)","SpamDiagnosticOutput":"1:99","SpamDiagnosticMetadata":"NSPM","X-OriginatorOrg":"caviumnetworks.com","X-MS-Exchange-CrossTenant-OriginalArrivalTime":"30 Aug 2017 10:48:15.2442\n\t(UTC)","X-MS-Exchange-CrossTenant-FromEntityHeader":"Hosted","X-MS-Exchange-CrossTenant-Id":"711e4ccf-2e9b-4bcf-a551-4094005b6194","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"DM2PR07MB1450","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170830_034838_987371_0049741E ","X-CRM114-Status":"GOOD (  23.92  )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details:   (-1.9 points)\n\tpts rule name              description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno\n\ttrust [2a01:111:f400:fe48:0:0:0:60c listed in] [list.dnswl.org]\n\t0.0 HEADER_FROM_DIFFERENT_DOMAINS From and EnvelopeFrom 2nd level\n\tmail domains are different\n\t-0.0 SPF_HELO_PASS          SPF: HELO matches SPF record\n\t-1.9 BAYES_00               BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.1 DKIM_VALID Message has at least one valid DKIM or DK signature\n\t0.1 DKIM_SIGNED            Message has a DKIM or DK signature,\n\tnot necessarily valid","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":"<http://lists.infradead.org/mailman/options/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=unsubscribe>","List-Archive":"<http://lists.infradead.org/pipermail/linux-arm-kernel/>","List-Post":"<mailto:linux-arm-kernel@lists.infradead.org>","List-Help":"<mailto:linux-arm-kernel-request@lists.infradead.org?subject=help>","List-Subscribe":"<http://lists.infradead.org/mailman/listinfo/linux-arm-kernel>,\n\t<mailto:linux-arm-kernel-request@lists.infradead.org?subject=subscribe>","Cc":"Ard Biesheuvel <ard.biesheuvel@linaro.org>,\n\tCatalin Marinas <catalin.marinas@arm.com>,\n\tWill Deacon <will.deacon@arm.com>, \n\tlinux-kernel@vger.kernel.org, James Morse <james.morse@arm.com>,\n\tLaura Abbott <labbott@redhat.com>, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" <linux-arm-kernel-bounces@lists.infradead.org>","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]