Re: [PATCH 09/11] KVM: PPC: Book3S: Improve a size determination in two functions

Hi Markus, [auto build test ERROR on next-20170119] [also build test ERROR on v4.10-rc4] [cannot apply to v4.9-rc8 v4.9-rc7 v4.9-rc6] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 09/11] KVM: PPC: Book3S: Improve a size determination in two functions

Hi Markus, [auto build test ERROR on next-20170119] [also build test ERROR on v4.10-rc4] [cannot apply to v4.9-rc8 v4.9-rc7 v4.9-rc6] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

BUILD_BUG_ON(!__builtin_constant_p(feature)) breaks bcc trace tool

Hi, We added: BUILD_BUG_ON(!__builtin_constant_p(feature)) to cpu_has_feature() and mmu_has_feature() in order to catch usage issues (such as cpu_has_feature(cpu_has_feature(X)). Unfortunately LLVM isn't smart enough to resolve this, and it errors out. I work around it in my clang/LLVM builds

[PATCH v3 03/37] treewide: Consolidate set_dma_ops() implementations

Now that all set_dma_ops() implementations are identical (ignoring BUG_ON() statements), remove the architecture specific definitions and add a definition in . Signed-off-by: Bart Van Assche Cc: Benjamin Herrenschmidt Cc: Chris Metcalf

Re: powerpc/nvram: Move an assignment for the variable "ret" in dev_nvram_write()

On 01/19/2017 11:08 PM, SF Markus Elfring wrote: >> I think you really could have squashed patches 1-3 into a single patch >> that returns directly after any failure. > > Thanks for your constructive feedback. > > I have got software development concerns around such patch squashing. > > >> At

Re: [PATCH v8 2/3] PCI: Make sure the driver could get correct BAR size from pci_resource_len()

Hi Yongji, [auto build test WARNING on pci/next] [also build test WARNING on v4.10-rc4 next-20170120] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Yongji-Xie/PCI-Ignore-requested-alignment

Re: [PATCH] powerpc/pasemi/nemo: Fix low memory values for boot.

On Fri, Jan 20, 2017 at 05:46:13PM +, Darren Stevens wrote: > --- a/arch/powerpc/kernel/head_64.S > +++ b/arch/powerpc/kernel/head_64.S > @@ -79,6 +79,13 @@ _GLOBAL(__start) > /* NOP this out unconditionally */ > BEGIN_FTR_SECTION > FIXUP_ENDIAN > +#ifdef CONFIG_PPC_PASEMI_NEMO > +

Re: [PATCH v8 2/3] PCI: Make sure the driver could get correct BAR size from pci_resource_len()

Hi Yongji, [auto build test WARNING on pci/next] [also build test WARNING on v4.10-rc4 next-20170120] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Yongji-Xie/PCI-Ignore-requested-alignment

[PATCH 11/11] KVM: PPC: Return directly after a failed copy_from_user() in two functions

From: Markus Elfring Date: Fri, 20 Jan 2017 18:00:35 +0100 * Return directly after a call of the function "copy_from_user" (or two other checks) failed in a case block. This issue was detected by using the Coccinelle software. * Delete the jump label "out"

Re: [PATCH 37/37] ARM: dts: DRA7: Add pcie1 dt node for EP mode

* Kishon Vijay Abraham I [170112 02:34]: > Add pcie1 dt node in order for the controller to operate in > endpoint mode. However since none of the dra7 based boards have > slots configured to operate in endpoint mode, keep EP mode > disabled. Can this be merged separately later on

[PATCH 10/11] KVM: PPC: e500: Use kcalloc() in e500_mmu_host_init()

From: Markus Elfring Date: Fri, 20 Jan 2017 16:30:18 +0100 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kcalloc". This issue was detected

[PATCH 09/11] KVM: PPC: Book3S: Improve a size determination in two functions

From: Markus Elfring Date: Fri, 20 Jan 2017 16:28:43 +0100 Replace the specification of data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

Re: [PATCH 36/37] ARM: DRA7: clockdomain: Change the CLKTRCTRL of CM_PCIE_CLKSTCTRL to SW_WKUP

* Kishon Vijay Abraham I [170115 22:06]: > Hi Tony, > > On Friday 13 January 2017 10:45 PM, Tony Lindgren wrote: > > * Kishon Vijay Abraham I [170112 02:35]: > >> The PCIe programming sequence in TRM suggests CLKSTCTRL of PCIe should > >> be set to SW_WKUP. There

[PATCH 08/11] KVM: PPC: Book3S: Use seq_puts() in xics_debug_show()

From: Markus Elfring Date: Fri, 20 Jan 2017 16:26:51 +0100 A string which did not contain data format specifications should be put into a sequence. Thus use the corresponding function "seq_puts". This issue was detected by using the Coccinelle software.

[PATCH 07/11] KVM: PPC: Book3S HV: Improve size determinations in five functions

From: Markus Elfring Date: Fri, 20 Jan 2017 16:23:30 +0100 Replace the specification of data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 06/11] KVM: PPC: Book3S HV: Use kcalloc() in kvmppc_alloc_host_rm_ops()

From: Markus Elfring Date: Fri, 20 Jan 2017 16:20:43 +0100 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kcalloc". This issue was detected

[PATCH 05/11] KVM: PPC: Book3S HV: Adjust nine checks for null pointers

From: Markus Elfring Date: Fri, 20 Jan 2017 11:25:48 +0100 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script "checkpatch.pl" pointed information out like the following. Comparison to NULL could be written … Thus

[PATCH 04/11] KVM: PPC: Book3S HV: Use common error handling code in kvmppc_clr_passthru_irq()

From: Markus Elfring Date: Fri, 20 Jan 2017 11:00:08 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. Signed-off-by: Markus Elfring --- arch/powerpc/kvm/book3s_hv.c | 8

[PATCH 03/11] KVM: PPC: Book3S HV: Move error code assignments in two functions

From: Markus Elfring Date: Fri, 20 Jan 2017 10:30:26 +0100 A local variable was set to an error code in a few cases before a concrete error situation was detected. Thus move the corresponding assignments into if branches to indicate a software failure there. This

[PATCH 02/11] KVM: PPC: Book3S HV: Improve a size determination in kvmppc_alloc_hpt()

From: Markus Elfring Date: Fri, 20 Jan 2017 10:00:13 +0100 Replace the specification of a data structure by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 01/11] KVM: PPC: Book3S HV: Move assignments for the variable "err" in kvm_htab_write()

From: Markus Elfring Date: Thu, 19 Jan 2017 22:45:56 +0100 * A local variable was set to an error code in five cases before a concrete error situation was detected. Thus move the corresponding assignments into if branches to indicate a software failure there.

[PATCH 00/11] PowerPC-KVM: Fine-tuning for some function implementations

From: Markus Elfring Date: Fri, 20 Jan 2017 19:07:21 +0100 Some update suggestions were taken into account from static source code analysis. Markus Elfring (11): Move assignments for the variable "err" in kvm_htab_write() Improve a size determination in

[RFT PATCH v3] modversions: redefine kcrctab entries as relative CRC pointers

The modversion symbol CRCs are emitted as ELF symbols, which allows us to easily populate the kcrctab sections by relying on the linker to associate each kcrctab slot with the correct value. This has a couple of downsides: - On architectures that support runtime relocation, a R__RELATIVE

[PATCH v2] powerpc/pasemi: Remove hardcoded bus numbers on smbus

The pasemi smbus controller uses PCI_FUNC(dev->devfn) to define which number bus to attach to, however this fails when something else is probed first, for example an ATI Radeon graphics card will claim 9 or 10 busses, including the ones the pasemi wants. Patch the driver to call i2c_add_adapter

[PATCH] powerpc/pasemi/nemo: Fix low memory values for boot.

Commit 5c0484e25ec0 ('powerpc: Endian safe trampoline') added a macro 'FIXUP_ENDIAN' to head_64.S. The CFE on Nemo has a bug that shows up when this is included, the system hangs right after printing the initial memory map, before the CPU's are started. Clearing the long long at addr 8 has been

Re: [PATCH v4 3/3] modversions: treat symbol CRCs as 32 bit quantities on 64 bit archs

On 19 January 2017 at 17:24, Linus Torvalds wrote: > On Thu, Jan 19, 2017 at 1:22 AM, Ard Biesheuvel > wrote: >>> >>> Your genksyms.c change is not exactly obvious. I looked at it, and my >>> brain just shut down. Why both the >>> >>>