Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 12 Sep 2018 23:06:30 +0200
From:   Ingo Molnar <mingo@kernel.org>
To:     Richard Henderson <rth@twiddle.net>
Cc:     Matthew Wilcox <willy@infradead.org>,
        Waiman Long <longman@redhat.com>, x86@kernel.org,
        linux-kernel@vger.kernel.org, Thomas Gleixner <tglx@linutronix.de>,
        Peter Zijlstra <a.p.zijlstra@chello.nl>
Subject: Re: [PATCH 0/2] Use named address spaces for percpu data
Message-ID: <20180912210630.GA2958@gmail.com>
References: <20180912144442.29271-1-willy@infradead.org>
 <988cf77c-e0cc-25fb-dcd6-945e6f2ba65e@twiddle.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <988cf77c-e0cc-25fb-dcd6-945e6f2ba65e@twiddle.net>
User-Agent: Mutt/1.9.4 (2018-02-28)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


* Richard Henderson <rth@twiddle.net> wrote:

> On 09/12/2018 07:44 AM, Matthew Wilcox wrote:
> > rth wrote a patch back in 2016 that uses gcc's address space machinery
> > to improve code generation for percpu accesses.  Ingo asked for some
> > minor changes to be made, but Richard didn't respond.  While looking at
> > something related, I came across them, and asked Richard's permission
> > to resurrect and resend the patch, with the changes Ingo asked for at
> > the time.
> > 
> > The important information about why we want these patch is in 1/2.
> > 
> > Richard Henderson (2):
> >   x86: Use named address spaces in asm/percpu.h
> >   x86: Enable named address spaces for percpu data
> > 
> >  arch/x86/include/asm/percpu.h | 147 ++++++++++++++++++++--------------
> >  1 file changed, 88 insertions(+), 59 deletions(-)
> 
> Thanks for resurrecting this, Willy.  I'd totally forgotten about it.
> The adjustments you made look fine.

Not all GCC versions seem to like this series:

 In file included from arch/x86/mm/tlb.c:12:0:
 arch/x86/mm/tlb.c: In function ‘switch_mm_irqs_off’:
 ./arch/x86/include/asm/mmu_context.h:210:28: internal compiler error: Segmentation fault
  #define switch_mm_irqs_off switch_mm_irqs_off
 arch/x86/mm/tlb.c:183:6: note: in expansion of macro ‘switch_mm_irqs_off’
  void switch_mm_irqs_off(struct mm_struct *prev, struct mm_struct *next,
 Please submit a full bug report,
 with preprocessed source if appropriate.
 See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.

gcc version 7.3.0 (Ubuntu 7.3.0-16ubuntu3) 

Thanks,

	Ingo