Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Fri, Jul 28, 2017 at 11:41:29AM -0700, Paul E. McKenney wrote: > On Fri, Jul 28, 2017 at 07:55:30AM -0700, Paul E. McKenney wrote: > > On Fri, Jul 28, 2017 at 08:54:16PM +0800, Boqun Feng wrote: > > > Hi Jonathan, > > > > > > FWIW, there is wakeup-missing issue in swake_up() and

Re: blk_mq_sched_insert_request: inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage

Jens Axboe wrote on 07/28/2017 09:25:48 AM: > Can you try the below fix? Should be more palatable than the previous > one. Brian, maybe you can take a look at the IRQ issue mentioned above? Working on an ipr change to address the IRQ issue. -Brian

Re: [RFC v6 21/62] powerpc: introduce execute-only pkey

Ram Pai writes: > --- a/arch/powerpc/mm/pkeys.c > +++ b/arch/powerpc/mm/pkeys.c > @@ -97,3 +97,60 @@ int __arch_set_user_pkey_access(struct task_struct *tsk, > int pkey, > init_iamr(pkey, new_iamr_bits); > return 0; > } > + > +static inline bool

Re: [RFC v6 27/62] powerpc: helper to validate key-access permissions of a pte

Ram Pai writes: > --- a/arch/powerpc/mm/pkeys.c > +++ b/arch/powerpc/mm/pkeys.c > @@ -201,3 +201,36 @@ int __arch_override_mprotect_pkey(struct vm_area_struct > *vma, int prot, >*/ > return vma_pkey(vma); > } > + > +static bool pkey_access_permitted(int pkey,

[PATCH v6 2/2] powerpc/fadump: update documentation about 'fadump_extra_args=' parameter

With the introduction of 'fadump_extra_args=' parameter to pass additional parameters to fadump (capture) kernel, update documentation about it. Signed-off-by: Hari Bathini --- Changes from v5: * Changed subject line to have 'fadump_extra_args='

[PATCH v6 1/2] powerpc/fadump: reduce memory consumption for capture kernel

With fadump (dump capture) kernel booting like a regular kernel, it almost needs the same amount of memory to boot as the production kernel, which is unwarranted for a dump capture kernel. But with no option to disable some of the unnecessary subsystems in fadump kernel, that much memory is wasted

Re: blk_mq_sched_insert_request: inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage

On 07/28/2017 10:17 AM, Brian J King wrote: > Jens Axboe wrote on 07/28/2017 09:25:48 AM: > >> Can you try the below fix? Should be more palatable than the previous >> one. Brian, maybe you can take a look at the IRQ issue mentioned above? Michael, Does this address the issue

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Fri, Jul 28, 2017 at 11:41:29AM -0700, Paul E. McKenney wrote: > On Fri, Jul 28, 2017 at 07:55:30AM -0700, Paul E. McKenney wrote: > > On Fri, Jul 28, 2017 at 08:54:16PM +0800, Boqun Feng wrote: [ . . . ] > > Even though Jonathan's testing indicates that it didn't fix this > > particular

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Fri, Jul 28, 2017 at 06:27:05PM +0100, Jonathan Cameron wrote: > On Fri, 28 Jul 2017 09:55:29 -0700 > "Paul E. McKenney" wrote: > > > On Fri, Jul 28, 2017 at 02:24:03PM +0100, Jonathan Cameron wrote: > > > On Fri, 28 Jul 2017 08:44:11 +0100 > > > Jonathan Cameron

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

From: Boqun Feng Date: Fri, 28 Jul 2017 20:54:16 +0800 > Hi Jonathan, > > FWIW, there is wakeup-missing issue in swake_up() and swake_up_all(): > > https://marc.info/?l=linux-kernel=149750022019663 > > and RCU begins to use swait/wake last year, so I thought this

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Fri, Jul 28, 2017 at 07:55:30AM -0700, Paul E. McKenney wrote: > On Fri, Jul 28, 2017 at 08:54:16PM +0800, Boqun Feng wrote: > > Hi Jonathan, > > > > FWIW, there is wakeup-missing issue in swake_up() and swake_up_all(): > > > > https://marc.info/?l=linux-kernel=149750022019663 > > > >

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Fri, 28 Jul 2017 09:55:29 -0700 "Paul E. McKenney" wrote: > On Fri, Jul 28, 2017 at 02:24:03PM +0100, Jonathan Cameron wrote: > > On Fri, 28 Jul 2017 08:44:11 +0100 > > Jonathan Cameron wrote: > > [ . . . ] > > > Ok. Some info. I

[PATCH] powerpc: Avoid reserving hugepages twice on PPC32

On PPC32, reserve_hugetlb_gpages() is already called in MMU_init(). Signed-off-by: John Ogness --- arch/powerpc/kernel/setup-common.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/powerpc/kernel/setup-common.c b/arch/powerpc/kernel/setup-common.c

Re: [RFC Part1 PATCH v3 09/17] resource: Consolidate resource walking code

On Mon, Jul 24, 2017 at 02:07:49PM -0500, Brijesh Singh wrote: > From: Tom Lendacky > > The walk_iomem_res_desc(), walk_system_ram_res() and walk_system_ram_range() > functions each have much of the same code. Create a new function that > consolidates the common code

Re: blk_mq_sched_insert_request: inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage

On 07/28/2017 09:13 AM, Bart Van Assche wrote: > On Fri, 2017-07-28 at 08:25 -0600, Jens Axboe wrote: >> On 07/28/2017 12:19 AM, Michael Ellerman wrote: >>> OK, so the resolution is "fix it in IPR" ? >> >> I'll leave that to the SCSI crew. But at least one bug is in IPR, if you >> look at the call

Re: blk_mq_sched_insert_request: inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage

On Fri, 2017-07-28 at 08:25 -0600, Jens Axboe wrote: > On 07/28/2017 12:19 AM, Michael Ellerman wrote: > > OK, so the resolution is "fix it in IPR" ? > > I'll leave that to the SCSI crew. But at least one bug is in IPR, if you > look at the call trace: > > - timer function triggers, runs

[PATCH] fs: convert a pile of fsync routines to errseq_t based reporting

From: Jeff Layton This patch converts most of the in-kernel filesystems that do writeback out of the pagecache to report errors using the errseq_t-based infrastructure that was recently added. This allows them to report errors once for each open file description. Most

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Fri, Jul 28, 2017 at 08:54:16PM +0800, Boqun Feng wrote: > Hi Jonathan, > > FWIW, there is wakeup-missing issue in swake_up() and swake_up_all(): > > https://marc.info/?l=linux-kernel=149750022019663 > > and RCU begins to use swait/wake last year, so I thought this could be > relevant.

Re: blk_mq_sched_insert_request: inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage

On 07/28/2017 12:19 AM, Michael Ellerman wrote: > Jens Axboe writes: >> On 07/27/2017 08:47 AM, Bart Van Assche wrote: >>> On Thu, 2017-07-27 at 08:02 -0600, Jens Axboe wrote: The bug looks like SCSI running the queue inline from IRQ context, that's not a good idea. >

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Fri, 28 Jul 2017 08:44:11 +0100 Jonathan Cameron wrote: > On Thu, 27 Jul 2017 09:52:45 -0700 > "Paul E. McKenney" wrote: > > > On Thu, Jul 27, 2017 at 05:39:23PM +0100, Jonathan Cameron wrote: > > > On Thu, 27 Jul 2017 14:49:03

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Fri, 28 Jul 2017 20:54:16 +0800 Boqun Feng wrote: > Hi Jonathan, > > FWIW, there is wakeup-missing issue in swake_up() and swake_up_all(): > > https://marc.info/?l=linux-kernel=149750022019663 > > and RCU begins to use swait/wake last year, so I thought this

[GIT PULL] Please pull powerpc/linux.git powerpc-4.13-4 tag

Hi Linus, Please pull some more powerpc fixes for 4.13: The following changes since commit 029d9252b116fa52a95150819e62af1f6e420fe5: powerpc/mm: Mark __init memory no-execute when STRICT_KERNEL_RWX=y (2017-07-18 19:54:24 +1000) are available in the git repository at:

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

Hi Jonathan, FWIW, there is wakeup-missing issue in swake_up() and swake_up_all(): https://marc.info/?l=linux-kernel=149750022019663 and RCU begins to use swait/wake last year, so I thought this could be relevant. Could you try the following patch and see if it works? Thanks. Regards,

Re: [PATCH v3 1/7] integrity: Introduce struct evm_hmac_xattr

Hi Thiago, On Thu, 2017-07-06 at 19:17 -0300, Thiago Jung Bauermann wrote: > Even though struct evm_ima_xattr_data includes a fixed-size array to hold a > SHA1 digest, most of the code ignores the array and uses the struct to mean > "type indicator followed by data of unspecified size" and tracks

Re: [RFC Part1 PATCH v3 08/17] x86/efi: Access EFI data as encrypted when SEV is active

On Mon, Jul 24, 2017 at 02:07:48PM -0500, Brijesh Singh wrote: > From: Tom Lendacky > > EFI data is encrypted when the kernel is run under SEV. Update the > page table references to be sure the EFI memory areas are accessed > encrypted. > > Signed-off-by: Tom Lendacky

Re: [v2] powerpc/mm: Check for _PAGE_PTE in *_devmap()

On Thu, 2017-07-27 at 15:35:53 UTC, Oliver O'Halloran wrote: > The ISA radix translation tree contains two different types of entry, > directories and leaves. The formats of the two entries are different > with the directory entries containing no spare bits for use by software. > As a result we

RE: [RFC Part1 PATCH v3 07/17] x86/mm: Include SEV for encryption memory attribute changes

From: Borislav Petkov > Sent: 27 July 2017 15:59 > On Mon, Jul 24, 2017 at 02:07:47PM -0500, Brijesh Singh wrote: > > From: Tom Lendacky > > > > The current code checks only for sme_active() when determining whether > > to perform the encryption attribute change. Include

Re: RCU lockup issues when CONFIG_SOFTLOCKUP_DETECTOR=n - any one else seeing this?

On Thu, 27 Jul 2017 09:52:45 -0700 "Paul E. McKenney" wrote: > On Thu, Jul 27, 2017 at 05:39:23PM +0100, Jonathan Cameron wrote: > > On Thu, 27 Jul 2017 14:49:03 +0100 > > Jonathan Cameron wrote: > > > > > On Thu, 27 Jul 2017 05:49:13

Re: [PATCH V8 1/3] powernv: powercap: Add support for powercap framework

Cyril Bur writes: > This is more for MPE and Stewart: > If there is some mutual exclusion that needs to happen, it should be > done in skiboot. We've had problems in the past with double entering > skiboot and hard to interpret errors ending up in userspace, this > solves that

RE: [PATCH v2] qe: fix compile issue for arm64

Fri 7/28/2017 2:14 PM, Michael Ellerman wrote: > -Original Message- > From: Michael Ellerman [mailto:m...@ellerman.id.au] > Sent: Friday, July 28, 2017 2:14 PM > To: Qiang Zhao ; o...@buserror.net > Cc: valentin.longch...@keymile.com;

Re: [PATCH v3 1/3] mm/hugetlb: Allow arch to override and call the weak function

On Thu 27-07-17 21:50:35, Aneesh Kumar K.V wrote: > > > On 07/27/2017 06:31 PM, Michal Hocko wrote: > >On Thu 27-07-17 11:48:26, Aneesh Kumar K.V wrote: > >>For ppc64, we want to call this function when we are not running as guest. > > > >What does this mean? > > > > ppc64 guest (aka LPAR)

Re: [RFC PATCH 3/3] mm/hugetlb: Remove pmd_huge_split_prepare

On Thu 27-07-17 21:27:37, Aneesh Kumar K.V wrote: > > > On 07/27/2017 06:27 PM, Michal Hocko wrote: > >On Thu 27-07-17 14:07:56, Aneesh Kumar K.V wrote: > >>Instead of marking the pmd ready for split, invalidate the pmd. This should > >>take care of powerpc requirement. > > > >which is? > > I

Re: [RFC PATCH 1/3] powerpc/mm: update pmdp_invalidate to return old pmd value

On Thu 27-07-17 21:18:35, Aneesh Kumar K.V wrote: > > > On 07/27/2017 06:24 PM, Michal Hocko wrote: > >EMISSING_CHANGELOG > > > >besides that no user actually uses the return value. Please fold this > >into the patch which uses the new functionality. > > > The patch series was suppose to help

Re: [PATCH] powerpc/boot: Fix 64-bit boot wrapper build with non-biarch compiler

On Wed, 2017-07-26 at 23:19 +1000, Michael Ellerman wrote: > 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

Re: blk_mq_sched_insert_request: inconsistent {SOFTIRQ-ON-W} -> {IN-SOFTIRQ-W} usage

Jens Axboe writes: > On 07/27/2017 08:47 AM, Bart Van Assche wrote: >> On Thu, 2017-07-27 at 08:02 -0600, Jens Axboe wrote: >>> The bug looks like SCSI running the queue inline from IRQ >>> context, that's not a good idea. ... >> >> scsi_run_queue() works fine if no scheduler is

Re: [PATCH v2] qe: fix compile issue for arm64

Zhao Qiang writes: > Signed-off-by: Zhao Qiang > --- > Changes for v2: > - include all Errata QE_General4 in #ifdef > > drivers/soc/fsl/qe/qe.c | 2 ++ > 1 file changed, 2 insertions(+) AFAICS this driver can only be built on PPC, what am I

Re: [PATCH] powerpc/configs: Add a powernv_be_defconfig

On Mon, 2017-07-24 at 22:50 +1000, Michael Ellerman wrote: > Although pretty much everyone using powernv is running little endian, > we should still test we can build for big endian. So add a > powernv_be_defconfig, which is autogenerated by flipping the endian > symbol in powernv_defconfig. > >