Re: [PATCH 6/6] cxlflash: Fix context reference tracking on detach

2016-09-09 Thread Uma Krishnan
On 9/2/2016 3:40 PM, Uma Krishnan wrote: From: "Matthew R. Ochs" Commit 888baf069f49 ("scsi: cxlflash: Add kref to context") introduced a kref to the context. In particular, the detach routine was updated to use the kref services for managing the removal and destruction of a context. As part o

Re: [PATCH 5/6] cxlflash: Refactor WWPN setup

2016-09-09 Thread Uma Krishnan
On 9/2/2016 3:40 PM, Uma Krishnan wrote: From: "Matthew R. Ochs" Commit 964497b3bf3f ("cxlflash: Remove dual port online dependency") logically removed the ability for the WWPN setup routine afu_set_wwpn() to return a non-success value. This routine can safely be made a void to simplify the cod

Re: [PATCH 4/6] cxlflash: Improve EEH recovery time

2016-09-09 Thread Uma Krishnan
On 9/2/2016 3:40 PM, Uma Krishnan wrote: From: "Matthew R. Ochs" When an EEH occurs during device initialization, the port timeout logic can cause excessive delays as MMIO reads will fail. Depending on where they are experienced, these delays can lead to a prolonged reset, causing an unnecessar

Re: [PATCH 3/6] cxlflash: Fix to avoid EEH and host reset collisions

2016-09-09 Thread Uma Krishnan
On 9/2/2016 3:39 PM, Uma Krishnan wrote: From: "Matthew R. Ochs" The EEH reset handler is ignorant to the current state of the driver when processing a frozen event and initiating a device reset. This can be an issue if an EEH event occurs while a user or stack initiated reset is executing. M

Re: [PATCH 2/3] arch/powerpc : optprobes for powerpc core

2016-09-09 Thread Masami Hiramatsu
Hi Anju, On Fri, 9 Sep 2016 16:19:41 +0530 Anju T Sudhakar wrote: > >> +void arch_unoptimize_kprobe(struct optimized_kprobe *op) > >> +{ > >> + arch_arm_kprobe(&op->kp); > >> +} > >> + > >> +void arch_unoptimize_kprobes(struct list_head *oplist, > >> + struct list_head *don

Re: [PATCH v4 0/5] kexec_file: Add buffer hand-over for the next kernel

2016-09-09 Thread Mimi Zohar
On Thu, 2016-09-08 at 23:07 -0500, Eric W. Biederman wrote: > Thiago Jung Bauermann writes: > > > Am Mittwoch, 07 September 2016, 09:19:40 schrieb Eric W. Biederman: > >> ebied...@xmission.com (Eric W. Biederman) writes: > >> > Thiago Jung Bauermann writes: > >> >> Hello, > >> >> > >> >> The pu

Re: [PATCH 2/3] arch/powerpc : optprobes for powerpc core

2016-09-09 Thread Anju T Sudhakar
Hi Masami, Thank you for reviewing the patch. On Thursday 08 September 2016 10:17 PM, Masami Hiramatsu wrote: On Wed, 7 Sep 2016 15:03:11 +0530 Anju T Sudhakar wrote: Instructions which can be emulated are suppliants for optimization. Before optimization ensure that the address range betw

[PATCH v2 1/3] net: fs_enet: merge NAPI RX and NAPI TX

2016-09-09 Thread Christophe Leroy
Initially, a NAPI TX routine has been implemented separately from NAPI RX, as done on the freescale/gianfar driver. By merging NAPI RX and NAPI TX, we reduce the amount of TX completion interrupts. Handling of the budget in association with TX interrupts is based on indications provided at https:

[PATCH v2 3/3] net: fs_enet: make rx_copybreak value configurable

2016-09-09 Thread Christophe Leroy
Measurement shows that on a MPC8xx running at 132MHz, the optimal limit is 112: * 114 bytes packets are processed in 147 TB ticks with higher copybreak * 114 bytes packets are processed in 148 TB ticks with lower copybreak * 128 bytes packets are processed in 154 TB ticks with higher copybreak * 12

[PATCH v2 2/3] net: fs_enet: don't unmap DMA when packet len is below copybreak

2016-09-09 Thread Christophe Leroy
When the length of the packet is below the defined copybreak limit, the received packet is copied into a newly allocated skb in order to reuse the skb. This is only interesting if it allow us to avoid a new DMA mapping. We shall therefore not DMA unmap and remap the skb->data. Instead, we invalidat

[PATCH v2 0/3] Optimisation of fs_enet ethernet driver

2016-09-09 Thread Christophe Leroy
This set optimises the freescale fs_enet ethernet driver: 1/ Merge of RX and TX NAPI functions in order to limit the amount of interrupts 2/ Do not unmap DMA when packets len is below copybreak, otherwise there is no benefit in copying the skb instead of allocating a new one 3/ Make copybreak value

UBSAN: Undefined behaviour in arch/powerpc/kernel/cputable.c

2016-09-09 Thread Meelis Roos
Got this on my PowerPC G4 (533 MHz, PowerPC 7410, PowerMac3,4 platform), with gcc version 5.3.1 20160509 (Debian 5.3.1-19): [0.00] [0.00] UBSAN: Undefined behaviour in arch/powerpc/kernel/cputable.c

[GIT PULL] Please pull powerpc/linux.git powerpc-4.8-5 tag

2016-09-09 Thread Michael Ellerman
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi Linus, Please pull some more powerpc fixes for 4.8: The following changes since commit 78a3e8889b4b6b99775ed954696ff3e017f5d19b: powerpc: signals: Discard transaction state from signal frames (2016-08-29 12:48:40 +1000) are available in the g

Re: [PATCH 0/6] cxlflash: Miscellaneous fixes

2016-09-09 Thread Martin K. Petersen
> "Uma" == Uma Krishnan writes: Uma> This patch set contains miscellaneous fixes in the device cleanup Uma> and EEH recovery paths. One of the patches simplifies the WWPN Uma> assignment routine. Uma> There are a couple of fixes that resolve regressions introduced by Uma> Commit 704c4b0ddc03

Re: [PATCH -next] scsi: ibmvfc: fix typo in parameter description

2016-09-09 Thread Martin K. Petersen
> "Wei" == Wei Yongjun writes: Wei> From: Wei Yongjun Fix typo in parameter Wei> description. Applied to 4.9/scsi-queue. -- Martin K. Petersen Oracle Linux Engineering

Re: [PATCH v2 4/5] powerpc/fadump: Make ELF eflags depend on endian

2016-09-09 Thread Michael Ellerman
Mahesh Jagannath Salgaonkar writes: > On 09/08/2016 12:30 PM, Mahesh Jagannath Salgaonkar wrote: >> On 09/06/2016 11:02 AM, Daniel Axtens wrote: >>> diff --git a/arch/powerpc/kernel/fadump.c b/arch/powerpc/kernel/fadump.c >>> index 0638b82ce294..457f08e544c6 100644 >>> --- a/arch/powerpc/kernel/fa

Re: [PATCH] powerpc/ptrace: Fix cppcheck issue in gpr32_set_common/gpr32_get_common.

2016-09-09 Thread Michael Ellerman
Daniel Axtens writes: > [ Unknown signature status ] > Simon Guo writes: > >> I think keeping tm_active argument for "ifndef CONFIG_PPC_TRANSACTIONAL_MEM" >> case (with __maybe_unused prefix) will be somehow strange -- Whatever >> value is provided in the caller function for tm_active, programm

Re: Build regression introduced by 31cdd0c39c7544ced79da53aa0b7e989f3a39582

2016-09-09 Thread John Paul Adrian Glaubitz
On 09/09/2016 03:29 AM, Michael Ellerman wrote: > Interestingly it builds fine for me, even for 32-bit configs, I assume > because my toolchains are 32/64-bit they are able to cope with it. I'll > try and built a 32-bit only toolchain to catch these problems in future. Yeah, I was just asking myse

Re: [PATCH v2 1/5] powerpc/cell: drop unused iic_get_irq_host()

2016-09-09 Thread Arnd Bergmann
On Friday, September 9, 2016 10:43:33 AM CEST Daniel Axtens wrote: > Hi Arnd, > > > I'm currently experimenting with a patch that adds -Wmissing-declaration to > > the normal kernel CFLAGS for everyone, and fixing up the warnings I get with > > that on ARM. > > > > Are you looking at all the power