On Thu, 2017-07-27 at 13:14 +1000, Michael Ellerman wrote:
> jos...@linux.vnet.ibm.com writes:
> > On Thu, Jul 20, 2017 at 10:18:18PM -0300, jos...@linux.vnet.ibm.com
> > wrote:
> > > On Thu, Jul 20, 2017 at 03:21:59PM +1000, Paul Mackerras wrote:
> > > >
> > > > Did you check the host kernel log
We use mm cpumask for serializing against lockless page table walk. Anybody
who is doing a lockless page table walk is expected to disable irq and only
cpus in mm cpumask is expected do the lockless walk. This ensure that
a THP split can send IPI to only cpus in the mm cpumask, to make sure there
a
Now that we made sure that lockless walk of linux page table is mostly limitted
to current task(current->mm->pgdir) we can update the THP update sequence to
only send IPI to cpus on which this task has run. This helps in reducing the IPI
overload on systems with large number of CPUs.
W.r.t kvm eve
Add newer helpers to make the function usage simpler. It is always recommended
to use find_current_mm_pte() for walking the page table. If we cannot use
find_current_mm_pte(), it should be documented why the said usage of
__find_linux_pte() is safe against a parallel THP split.
For now we have KVM
We now have GIGANTIC_PAGE on powerpc. Currently this is enabled only on
radix with 1G as gigantic hugepage size. Enable this with hash translation mode
too (ie, with 16G hugepage size). Depending on the total system memory we may
be able to allocate 16G hugepage size. This bring parity between radi
With commit aa888a74977a8 ("hugetlb: support larger than MAX_ORDER") we added
support for allocating gigantic hugepages via kernel command line. Switch
ppc64 arch specific code to use that.
W.r.t FSL support, we now limit our allocation range using
BOOTMEM_ALLOC_ACCESSIBLE.
We use the kernel com
For ppc64, we want to call this function when we are not running as guest.
Also, if we failed to allocate hugepages, let the user know.
Signed-off-by: Aneesh Kumar K.V
---
include/linux/hugetlb.h | 1 +
mm/hugetlb.c| 5 -
2 files changed, 5 insertions(+), 1 deletion(-)
diff --gi
Hi Jens,
I'm seeing the lockdep warning below on shutdown on a Power8 machine
using IPR.
If I'm reading it right it looks like the spin_lock() (non-irq) in
blk_mq_sched_insert_request() is the immediate cause.
Looking at blk_mq_requeue_work() (the caller), it is doing
spin_lock_irqsave(). So is
On Wed, 26 Jul 2017 18:42:14 -0700
"Paul E. McKenney" wrote:
> On Wed, Jul 26, 2017 at 04:22:00PM -0700, David Miller wrote:
> > Indeed, that really wouldn't explain how we end up with a RCU stall
> > dump listing almost all of the cpus as having missed a grace period.
>
> I have seen strange
jos...@linux.vnet.ibm.com writes:
> On Thu, Jul 20, 2017 at 10:18:18PM -0300, jos...@linux.vnet.ibm.com wrote:
>> On Thu, Jul 20, 2017 at 03:21:59PM +1000, Paul Mackerras wrote:
>> >
>> > Did you check the host kernel logs for any oops messages?
>>
>> dmesg was clean but after sometime waiting (I
Hi Michael,
At 07/27/2017 10:21 AM, Michael Ellerman wrote:
Dou Liyang writes:
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in POWERPC platform is unnecessary.
Remove it for cleanup.
Repor
Dou Liyang writes:
> Commit a7be6e5a7f8d ("mm: drop useless local parameters of
> __register_one_node()") removes the last user of parent_node().
>
> The parent_node() macro in POWERPC platform is unnecessary.
>
> Remove it for cleanup.
>
> Reported-by: Michael Ellerman
> Signed-off-by: Dou Liya
On 07/26/2017 09:36 PM, Ram Pai wrote:
On Wed, Jul 26, 2017 at 04:05:48PM +0530, Aneesh Kumar K.V wrote:
Ram Pai writes:
diff --git a/arch/powerpc/include/asm/book3s/64/hash-64k.h
b/arch/powerpc/include/asm/book3s/64/hash-64k.h
index 9732837..62e580c 100644
--- a/arch/powerpc/include/asm
On Wed, Jul 26, 2017 at 04:22:00PM -0700, David Miller wrote:
> From: "Paul E. McKenney"
> Date: Wed, 26 Jul 2017 16:15:05 -0700
>
> > On Wed, Jul 26, 2017 at 03:45:40PM -0700, David Miller wrote:
> >> Just out of curiousity, what x86 idle method is your machine using?
> >> The mwait one or the o
Segher Boessenkool writes:
> On Wed, Jul 26, 2017 at 08:03:30PM +1000, Michael Ellerman wrote:
>> Segher Boessenkool writes:
>> > A general question about these patches: some things are inside #ifdef
>> > __powerpc64__, some are not. It seems it is the wrong macro, and it
>> > should be used (o
From: "Paul E. McKenney"
Date: Wed, 26 Jul 2017 16:15:05 -0700
> On Wed, Jul 26, 2017 at 03:45:40PM -0700, David Miller wrote:
>> Just out of curiousity, what x86 idle method is your machine using?
>> The mwait one or the one which simply uses 'halt'? The mwait variant
>> might mask this bug, an
On Wed, Jul 26, 2017 at 03:45:40PM -0700, David Miller wrote:
> From: "Paul E. McKenney"
> Date: Wed, 26 Jul 2017 15:36:58 -0700
>
> > And without CONFIG_SOFTLOCKUP_DETECTOR, I see five runs of 24 with RCU
> > CPU stall warnings. So it seems likely that CONFIG_SOFTLOCKUP_DETECTOR
> > really is h
From: "Paul E. McKenney"
Date: Wed, 26 Jul 2017 15:36:58 -0700
> And without CONFIG_SOFTLOCKUP_DETECTOR, I see five runs of 24 with RCU
> CPU stall warnings. So it seems likely that CONFIG_SOFTLOCKUP_DETECTOR
> really is having an effect.
Thanks for all of the info Paul, I'll digest this and sc
On Wed, Jul 26, 2017 at 10:50:13AM -0700, Paul E. McKenney wrote:
> On Wed, Jul 26, 2017 at 09:54:32AM -0700, David Miller wrote:
> > From: "Paul E. McKenney"
> > Date: Wed, 26 Jul 2017 08:49:00 -0700
> >
> > > On Wed, Jul 26, 2017 at 04:33:40PM +0100, Jonathan Cameron wrote:
> > >> Didn't leave
Commit a7be6e5a7f8d ("mm: drop useless local parameters of
__register_one_node()") removes the last user of parent_node().
The parent_node() macro in POWERPC platform is unnecessary.
Remove it for cleanup.
Reported-by: Michael Ellerman
Signed-off-by: Dou Liyang
Cc: Benjamin Herrenschmidt
Cc:
On 07/26/2017 02:26 PM, H. Peter Anvin wrote:
\
static inline void outs##bwl(int port, const void *addr, unsigned
long count) \
{
This will clash with a fix I did to add a "memory" clobber
for the traditional imp
,Eric Biederman ,Tejun Heo
,Paolo Bonzini ,Andrew Morton
,"Kirill A . Shutemov"
,Lu Baolu
From: h...@zytor.com
Message-ID:
On July 26, 2017 9:24:45 PM GMT+02:00, Brijesh Singh
wrote:
>
>Hi Arnd and David,
>
>On 07/26/2017 05:45 AM, Arnd Bergmann wrote:
>> On Tue, Jul 25, 2017 at 11:51 AM, D
On Tue, 18 Jul 2017 14:22:20 -0300
Murilo Opsfelder Araujo wrote:
> When CONFIG_EEH=y and CONFIG_VFIO_SPAPR_EEH=n, build fails with the
> following:
>
> drivers/vfio/pci/vfio_pci.o: In function `.vfio_pci_release':
> vfio_pci.c:(.text+0xa98): undefined reference to
> `.vfio_spapr_pci_ee
Hi Arnd and David,
On 07/26/2017 05:45 AM, Arnd Bergmann wrote:
On Tue, Jul 25, 2017 at 11:51 AM, David Laight wrote:
From: Brijesh Singh
Sent: 24 July 2017 20:08
From: Tom Lendacky
Secure Encrypted Virtualization (SEV) does not support string I/O, so
unroll the string I/O operation into a
On Wed, Jul 26, 2017 at 09:54:32AM -0700, David Miller wrote:
> From: "Paul E. McKenney"
> Date: Wed, 26 Jul 2017 08:49:00 -0700
>
> > On Wed, Jul 26, 2017 at 04:33:40PM +0100, Jonathan Cameron wrote:
> >> Didn't leave it long enough. Still bad on 4.10-rc7 just took over
> >> an hour to occur.
>
On Wed, 26 Jul 2017 09:54:32 -0700
David Miller wrote:
> From: "Paul E. McKenney"
> Date: Wed, 26 Jul 2017 08:49:00 -0700
>
> > On Wed, Jul 26, 2017 at 04:33:40PM +0100, Jonathan Cameron wrote:
> >> Didn't leave it long enough. Still bad on 4.10-rc7 just took over
> >> an hour to occur.
> >
From: "Paul E. McKenney"
Date: Wed, 26 Jul 2017 08:49:00 -0700
> On Wed, Jul 26, 2017 at 04:33:40PM +0100, Jonathan Cameron wrote:
>> Didn't leave it long enough. Still bad on 4.10-rc7 just took over
>> an hour to occur.
>
> And it is quite possible that SOFTLOCKUP_DETECTOR=y and HZ_PERIODIC=y
>
From: "Paul E. McKenney"
Date: Wed, 26 Jul 2017 07:14:17 -0700
> Dave, any other ideas on what might be causing this or what might be
> tested?
I was going to go through the changes that happened between v4.12
and now to kernel/timer/tick-sched.c and see if reverting any of
those help.
But I do
On 7/25/2017 11:28 PM, Borislav Petkov wrote:
On Mon, Jul 24, 2017 at 02:07:43PM -0500, Brijesh Singh wrote:
From: Tom Lendacky
Provide support for Secure Encyrpted Virtualization (SEV). This initial
Your subject misses a verb and patch subjects should have an active verb
denoting what the p
On Wed, Jul 26, 2017 at 04:05:48PM +0530, Aneesh Kumar K.V wrote:
> Ram Pai writes:
>
> > Rearrange 64K PTE bits to free up bits 3, 4, 5 and 6,
> > in the 4K backed HPTE pages.These bits continue to be used
> > for 64K backed HPTE pages in this patch, but will be freed
> > up in the next pat
Subject: x86/realmode: ...
On Mon, Jul 24, 2017 at 02:07:45PM -0500, Brijesh Singh wrote:
> From: Tom Lendacky
>
> When SEV is active the trampoline area will need to be in encrypted
> memory so only mark the area decrypted if SME is active.
>
> Signed-off-by: Tom Lendacky
> Signed-off-by: Br
On Wed, Jul 26, 2017 at 08:03:30PM +1000, Michael Ellerman wrote:
> Segher Boessenkool writes:
> > A general question about these patches: some things are inside #ifdef
> > __powerpc64__, some are not. It seems it is the wrong macro, and it
> > should be used (or not used) consistently?
>
> Why
On Wed, Jul 26, 2017 at 04:33:40PM +0100, Jonathan Cameron wrote:
> On Wed, 26 Jul 2017 15:23:15 +0100
> Jonathan Cameron wrote:
>
> > On Wed, 26 Jul 2017 07:14:17 -0700
> > "Paul E. McKenney" wrote:
> >
> > > On Wed, Jul 26, 2017 at 01:28:01PM +0100, Jonathan Cameron wrote:
> > > > On Wed, 2
On Wed, 26 Jul 2017 15:23:15 +0100
Jonathan Cameron wrote:
> On Wed, 26 Jul 2017 07:14:17 -0700
> "Paul E. McKenney" wrote:
>
> > On Wed, Jul 26, 2017 at 01:28:01PM +0100, Jonathan Cameron wrote:
> > > On Wed, 26 Jul 2017 10:32:32 +0100
> > > Jonathan Cameron wrote:
> > >
> > > > On Wed
On Mon, Jul 24, 2017 at 02:07:44PM -0500, Brijesh Singh wrote:
> From: Tom Lendacky
>
> When SEV is active the initrd/initramfs will already have already been
> placed in memory encyrpted so do not try to encrypt it.
>
> Signed-off-by: Tom Lendacky
> Signed-off-by: Brijesh Singh
> ---
> arch/
On Wed, Jul 26, 2017 at 5:07 AM, David Laight wrote:
> From: Rob Herring
>> Sent: 25 July 2017 22:44
>> With dependencies on full_name containing the entire device node path
>> removed, stop storing the full_name in nodes created by
>> dlpar_configure_connector() and pSeries_reconfig_add_node().
>
On Wed, Jul 26, 2017 at 9:20 AM, Frank Rowand wrote:
> Hi Rob,
>
> On 07/25/17 14:44, Rob Herring wrote:
>> This series is the last steps to remove storing the full path for every
>> DT node. Instead, we can create full path strings dynamically as needed
>> with printf %pOF specifiers (commit ce4f
On Wed, 26 Jul 2017 07:14:17 -0700
"Paul E. McKenney" wrote:
> On Wed, Jul 26, 2017 at 01:28:01PM +0100, Jonathan Cameron wrote:
> > On Wed, 26 Jul 2017 10:32:32 +0100
> > Jonathan Cameron wrote:
> >
> > > On Wed, 26 Jul 2017 09:16:23 +0100
> > > Jonathan Cameron wrote:
> > >
> > > > On T
Hi Rob,
On 07/25/17 14:44, Rob Herring wrote:
> This series is the last steps to remove storing the full path for every
> DT node. Instead, we can create full path strings dynamically as needed
> with printf %pOF specifiers (commit ce4fecf1fe15). There are a number of
> remaining direct users o
On Wed, Jul 26, 2017 at 01:28:01PM +0100, Jonathan Cameron wrote:
> On Wed, 26 Jul 2017 10:32:32 +0100
> Jonathan Cameron wrote:
>
> > On Wed, 26 Jul 2017 09:16:23 +0100
> > Jonathan Cameron wrote:
> >
> > > On Tue, 25 Jul 2017 21:12:17 -0700
> > > "Paul E. McKenney" wrote:
> > >
> > > > On
On Wed, Jul 26, 2017 at 5:26 AM, Michael Ellerman wrote:
> Rob Herring writes:
>
>> With dependencies on a statically allocated full path name converted to
>> use %pOF format specifier, we can store just the basename of node, and
>> the unflattening of the FDT can be simplified.
>>
>> This commit
On Wed, 26 Jul 2017, Michael Ellerman wrote:
> Hi Thomas,
>
> I'm seeing the lockdep barf below on some bare metal Power8 machines.
>
> This seems to be caused by our smp_cpus_done(), which does:
>
> void __init smp_cpus_done(unsigned int max_cpus)
> {
> /*
>* We want the setu
Historically the boot wrapper was always built 32-bit big endian, even
for 64-bit kernels. That was because old firmwares didn't necessarily
support booting a 64-bit image. Because of that arch/powerpc/boot/Makefile
uses CROSS32CC for compilation.
However when we added 64-bit little endian support
On Thu, Jul 20, 2017 at 10:18:18PM -0300, jos...@linux.vnet.ibm.com wrote:
> On Thu, Jul 20, 2017 at 03:21:59PM +1000, Paul Mackerras wrote:
> > On Thu, Jul 20, 2017 at 12:02:23AM -0300, jos...@linux.vnet.ibm.com wrote:
> > > On Thu, Jul 20, 2017 at 09:42:50AM +1000, Benjamin Herrenschmidt wrote:
>
In commit efe0160cfd40 ("powerpc/64: Linker on-demand sfpr functions
for modules"), we added an ld version check early in the powerpc
top-level Makefile.
Because the Makefile runs before the kernel config is setup, the
checks for CONFIG_CPU_LITTLE_ENDIAN etc. all take the default case. So
we end u
On Wed, 26 Jul 2017 13:28:01 +0100
Jonathan Cameron wrote:
> On Wed, 26 Jul 2017 10:32:32 +0100
> Jonathan Cameron wrote:
>
> > On Wed, 26 Jul 2017 09:16:23 +0100
> > Jonathan Cameron wrote:
> >
> > > On Tue, 25 Jul 2017 21:12:17 -0700
> > > "Paul E. McKenney" wrote:
> > >
> > > > On
On Wed, 26 Jul 2017 10:32:32 +0100
Jonathan Cameron wrote:
> On Wed, 26 Jul 2017 09:16:23 +0100
> Jonathan Cameron wrote:
>
> > On Tue, 25 Jul 2017 21:12:17 -0700
> > "Paul E. McKenney" wrote:
> >
> > > On Tue, Jul 25, 2017 at 09:02:33PM -0700, David Miller wrote:
> > > > From: "Paul E.
Hi Thomas,
I'm seeing the lockdep barf below on some bare metal Power8 machines.
This seems to be caused by our smp_cpus_done(), which does:
void __init smp_cpus_done(unsigned int max_cpus)
{
/*
* We want the setup_cpu() here to be called on the boot CPU, but
* init
On 07/25/17 17:56, Gabriele Paoloni wrote:
> Hi Laszlo
>
> [...]
>
>>
>> Having practically zero background in gfx development (either kernel or
>> Xorg), I think the problem is that vga_default_device() /
>> vga_set_default_device(), which -- apparently -- "boot_vga" is based
>> upon, come from
On Tue, Jul 25, 2017 at 11:51 AM, David Laight wrote:
> From: Brijesh Singh
>> Sent: 24 July 2017 20:08
>> From: Tom Lendacky
>>
>> Secure Encrypted Virtualization (SEV) does not support string I/O, so
>> unroll the string I/O operation into a loop operating on one element at
>> a time.
>>
>> Sig
On Fri, 21 Jul 2017 16:31:34 +0530
"Gautham R. Shenoy" wrote:
> From: "Gautham R. Shenoy"
>
> Currently we use the stop-api provided by the firmware to program the
> SLW engine to restore the values of hypervisor resources that get lost
> on deeper idle states (such as winkle). Since the deep s
On Fri, 21 Jul 2017 16:11:37 +0530
"Gautham R. Shenoy" wrote:
> From: "Gautham R. Shenoy"
>
> The stop4 idle state on POWER9 is a deep idle state which loses
> hypervisor resources, but whose latency is low enough that it can be
> exposed via cpuidle.
>
> Until now, the deep idle states which
Ram Pai writes:
> Rearrange 64K PTE bits to free up bits 3, 4, 5 and 6,
> in the 4K backed HPTE pages.These bits continue to be used
> for 64K backed HPTE pages in this patch, but will be freed
> up in the next patch. The bit numbers are big-endian as
> defined in the ISA3.0
>
> The patch
Rob Herring writes:
> With dependencies on a statically allocated full path name converted to
> use %pOF format specifier, we can store just the basename of node, and
> the unflattening of the FDT can be simplified.
>
> This commit will affect the remaining users of full_name. After
> analyzing t
From: Rob Herring
> Sent: 25 July 2017 22:44
> With dependencies on full_name containing the entire device node path
> removed, stop storing the full_name in nodes created by
> dlpar_configure_connector() and pSeries_reconfig_add_node().
...
> dn = kzalloc(sizeof(*dn), GFP_KERNEL);
> if
Segher Boessenkool writes:
> On Tue, Jul 25, 2017 at 01:33:19PM +1000, Matt Brown wrote:
>> +static nokprobe_inline void do_prty(struct pt_regs *regs, unsigned long v,
>> +int size, int ra)
>> +{
>> +unsigned long long res = v;
>> +
>> +res = (0x000100010001000
On Wed, 26 Jul 2017 09:16:23 +0100
Jonathan Cameron wrote:
> On Tue, 25 Jul 2017 21:12:17 -0700
> "Paul E. McKenney" wrote:
>
> > On Tue, Jul 25, 2017 at 09:02:33PM -0700, David Miller wrote:
> > > From: "Paul E. McKenney"
> > > Date: Tue, 25 Jul 2017 20:55:45 -0700
> > >
> > > > On Tue
On Tue, 25 Jul 2017 21:12:17 -0700
"Paul E. McKenney" wrote:
> On Tue, Jul 25, 2017 at 09:02:33PM -0700, David Miller wrote:
> > From: "Paul E. McKenney"
> > Date: Tue, 25 Jul 2017 20:55:45 -0700
> >
> > > On Tue, Jul 25, 2017 at 02:10:29PM -0700, David Miller wrote:
> > >> Just to report,
On Tue, 25 Jul 2017 20:53:06 -0700
"Paul E. McKenney" wrote:
> On Wed, Jul 26, 2017 at 12:52:07AM +0800, Jonathan Cameron wrote:
> > On Tue, 25 Jul 2017 08:12:45 -0700
> > "Paul E. McKenney" wrote:
> >
> > > On Tue, Jul 25, 2017 at 10:42:45PM +0800, Jonathan Cameron wrote:
> > > > On Tue, 2
On Tue, Jul 25, 2017 at 06:08:05PM +1000, Balbir Singh wrote:
> On Tue, 2017-07-25 at 13:33 +1000, Matt Brown wrote:
> > This adds emulation for the prtyw and prtyd instructions.
> > Tested for logical correctness against the prtyw and prtyd instructions
> > on ppc64le.
> >
> > Signed-off-by: Matt
On Tue, Jul 25, 2017 at 01:33:17PM +1000, Matt Brown wrote:
> This adds emulations for the popcntb, popcntw, and popcntd instructions.
> Tested for correctness against the popcnt{b,w,d} instructions on ppc64le.
>
> Signed-off-by: Matt Brown
> ---
> v3:
> - optimised using the Giles-Miller m
61 matches
Mail list logo
