Re: [RFC PATCH] sched/wait: Introduce new, more compact wait_event*() primitives

2017-03-08 Thread Ingo Molnar
* Ingo Molnar wrote: > You can find the latest (WIP, frequently rebased) version in: > > git git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.sched/core > > The first 10 patches reshape the waitqueue code to be more hackable (to me!), > because I kept bumping into confusing

[PATCH 11/26] IB/ocrdma: Delete unnecessary braces

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 7 Mar 2017 21:12:56 +0100 Do not use curly brackets at some source code places where a single statement should be sufficient. Signed-off-by: Markus Elfring --- drivers/infiniband/hw/ocrdma/ocrdma_hw.c | 17 - 1 file changed, 8 insertions(+), 9

Re: [PATCH] livepatch: make klp_mutex proper part of API

2017-03-08 Thread Miroslav Benes
On Wed, 8 Mar 2017, Josh Poimboeuf wrote: > On Wed, Mar 08, 2017 at 10:16:00AM +0100, Jiri Kosina wrote: > > From: Jiri Kosina > > > > klp_mutex is shared between core.c and transition.c, and as such would > > rather be properly located in livepatch.h so that we don't have to play > >

[PATCH 08/26] IB/ocrdma: Improve size determinations in ocrdma_mbx_rdma_stats()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 7 Mar 2017 20:16:16 +0100 Replace the specification of two data structures by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determinations a bit safer according to the Linux coding style convention. Signed-off-by:

[PATCH 13/26] IB/ocrdma: Use kcalloc() in ocrdma_alloc_resources()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 7 Mar 2017 21:55:42 +0100 * Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus reuse the corresponding function "kcalloc". This issue was detected by using the Coccinelle software.

[PATCH 09/26] IB/ocrdma: Improve another size determination in ocrdma_mq_cq_handler()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 7 Mar 2017 20:20:45 +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 convention. Signed-off-by:

Re: [RFC PATCH 3/4] xfs: map KM_MAYFAIL to __GFP_RETRY_MAYFAIL

2017-03-08 Thread Michal Hocko
On Wed 08-03-17 20:23:37, Tetsuo Handa wrote: > On 2017/03/08 0:48, Michal Hocko wrote: > > From: Michal Hocko > > > > KM_MAYFAIL didn't have any suitable GFP_FOO counterpart until recently > > so it relied on the default page allocator behavior for the given set > > of flags. This means that

Re: [PATCH] mm: move pcp and lru-pcp drainging into single wq

2017-03-08 Thread Michal Hocko
On Wed 08-03-17 20:50:45, Tetsuo Handa wrote: > Michal Hocko wrote: > > On Tue 07-03-17 22:50:48, Tetsuo Handa wrote: > > > Michal Hocko wrote: > > > > From: Michal Hocko > > > > > > > > We currently have 2 specific WQ_RECLAIM workqueues in the mm code. > > > > vmstat_wq for updating pcp stats

Re: Applied "spi: davinci: enable DMA when channels are defined in DT" to the spi tree

2017-03-08 Thread Mark Brown
On Wed, Mar 08, 2017 at 11:35:02AM +0100, Frode Isaksen wrote: > To avoid data corruption issues with UBIFS volume over SPI NOR, DMA should > not be used for vmalloc'ed buffers. The 5'th patch in my series fixes that: It's best to always try to ensure that your patch serieses are bisectable...

[PATCH 12/26] IB/ocrdma: Adjust ten checks for null pointers

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 7 Mar 2017 21:32:22 +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 fix the affected source code

Re: [PATCH 2/2] perf probe: Remove stale func add_perf_probe_events

2017-03-08 Thread Ravi Bangoria
Thanks Masami for the review, On Wednesday 08 March 2017 03:13 PM, Masami Hiramatsu wrote: > On Wed, 8 Mar 2017 12:29:08 +0530 > Ravi Bangoria wrote: > >> I don't see any user of this function. This function was being copied >> to tools/perf/builtin-probe.c by commit b02137cc6550 ("perf probe:

Re: [PATCH -mm -v6 1/9] mm, swap: Make swap cluster size same of THP size on x86_64

2017-03-08 Thread Matthew Wilcox
On Wed, Mar 08, 2017 at 03:26:05PM +0800, Huang, Ying wrote: > In this patch, the size of the swap cluster is changed to that of the > THP (Transparent Huge Page) on x86_64 architecture (512). This is for > the THP swap support on x86_64. Where one swap cluster will be used to > hold the

[PATCH 19/26] IB/ocrdma: Improve another size determination in ocrdma_alloc_mr()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 09:54:42 +0100 Replace the specification of a data type 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 convention. Signed-off-by: Markus

Re: 4.11.0-rc1 boot resulted in WARNING: CPU: 14 PID: 1722 at fs/sysfs/dir.c:31 .sysfs_warn_dup+0x78/0xb0

2017-03-08 Thread Brian Foster
On Tue, Mar 07, 2017 at 10:01:04PM +0530, Abdul Haleem wrote: > > Hi, > > Today's mainline (4.11.0-rc1) booted with warnings on Power7 LPAR. > > Issue is not reproducible all the time. > > traces: > > Found device VDASD 5. > Mounting /home... > Reached target Swap. > Found device

[PATCH 04/26] IB/ocrdma: Delete unnecessary variable initialisations in ocrdma_mbx_get_dcbx_config()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 7 Mar 2017 19:25:55 +0100 The local variables "req" and "rsp" will be set to appropriate pointers a bit later. Thus omit the explicit initialisations at the beginning. Signed-off-by: Markus Elfring --- drivers/infiniband/hw/ocrdma/ocrdma_hw.c | 5 ++--- 1 file

[PATCH 2/2] mm: Change mbind(MPOL_BIND) implementation for CDM nodes

2017-03-08 Thread Anshuman Khandual
CDM nodes need a way of explicit memory allocation mechanism from the user space. After the previous FALLBACK zonelist rebuilding process changes, the mbind(MPOL_BIND) based allocation request fails on the CDM node. This is because allocation requesting local node's FALLBACK zonelist is selected

Re: netlink: GPF in netlink_unicast

2017-03-08 Thread Richard Guy Briggs
On 2017-03-07 14:23, Paul Moore wrote: > On Tue, Mar 7, 2017 at 1:44 PM, Paul Moore wrote: > > On Tue, Mar 7, 2017 at 10:55 AM, Richard Guy Briggs wrote: > >> On 2017-03-07 09:29, Paul Moore wrote: > >>> On Mon, Mar 6, 2017 at 11:03 PM, Richard Guy Briggs > >>> wrote: > >>> > On 2017-03-06

[PATCH v2] iio: multiplexer: fix unsigned check with less than zero

2017-03-08 Thread Peter Rosin
Comparing a size_t with less than zero is always false as size_t is unsigned. So, change the type of the variable to ssize_t and replicate the size check from mux_configure_channel() into mux_write_ext_info() thus ensuring that the size will fit in the ssize_t variable. Detected by CoverityScan,

[PATCH 18/26] IB/ocrdma: Use kcalloc() in three functions

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 09:19:47 +0100 * Multiplications for the size determination of memory allocations indicated that array data structures should be processed. Thus reuse the corresponding function "kcalloc". This issue was detected by using the Coccinelle software.

[PATCH 22/26] IB/ocrdma: Returning only value constants in ocrdma_resize_cq()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 10:48:24 +0100 Return constant integer values without storing them in the local variable "status". Signed-off-by: Markus Elfring --- drivers/infiniband/hw/ocrdma/ocrdma_verbs.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff

Re: [PATCH 3/4] Documentation: dt: Add bindings documentation for CSI-2 Host Video Platform

2017-03-08 Thread Sakari Ailus
Hi Ramiro, On Tue, Mar 07, 2017 at 02:37:50PM +, Ramiro Oliveira wrote: > Create device tree bindings documentation for the CSI-2 Host Video > platform. Extra space here. > > Signed-off-by: Ramiro Oliveira > --- > .../devicetree/bindings/media/snps,plat-csi2.txt | 77 >

[PATCH v2] livepatch: make klp_mutex proper part of API

2017-03-08 Thread Jiri Kosina
From: Jiri Kosina klp_mutex is shared between core.c and transition.c, and as such would rather be properly located in a header so that we don't have to play 'extern' games from .c sources. This also silences sparse warning (wrongly) suggesting that klp_mutex should be defined static.

Re: Kernel without RTC

2017-03-08 Thread Alexandre Belloni
Hi, On 07/03/2017 at 00:48:18 +0530, Shah, Nehal-bakulchandra wrote: > Hi, > > Currently we are having hardware which does not have RTC. It is single > processor system. However it does have TSC timer. > > Now, how to use scheduler with only TSC as current kernel scheduler leverage > the RTC

[PATCH 20/26] IB/ocrdma: Delete an unnecessary variable assignment in ocrdma_alloc_mr()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 10:00:41 +0100 Delete an assignment for the local variable "status" in an if branch because the desired failure indication is already specified by a constant error code at the end. Signed-off-by: Markus Elfring ---

Re: [PATCH] virtio-pci: Remove affinity hint before freeing the interrupt

2017-03-08 Thread Michael S. Tsirkin
On Wed, Mar 08, 2017 at 08:09:27AM +, Marc Zyngier wrote: > virtio-pci registers a per-vq affinity hint when using MSIX, > but fails to remove it when freeing the interrupt, resulting > in this type of splat: > > [ 31.111202] WARNING: CPU: 0 PID: 2823 at kernel/irq/manage.c:1503 >

[PATCH 23/26] IB/ocrdma: Delete an unnecessary variable in ocrdma_dealloc_pd()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 10:58:34 +0100 1. Return zero in one case directly. 2. Return the value from a call of the function "_ocrdma_dealloc_pd" without using an extra assignment for the local variable. 3. Remove the variable "status" in this function then.

[PATCH 15/26] IB/ocrdma: Delete an error message for a failed memory allocation in ocrdma_add()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 7 Mar 2017 22:23:17 +0100 Omit an extra message for a memory allocation failure in this function. Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf Signed-off-by: Markus Elfring ---

Re: [PATCH v2] livepatch: make klp_mutex proper part of API

2017-03-08 Thread Miroslav Benes
On Wed, 8 Mar 2017, Jiri Kosina wrote: > From: Jiri Kosina > > klp_mutex is shared between core.c and transition.c, and as such would > rather be properly located in a header so that we don't have to play > 'extern' games from .c sources. > > This also silences sparse warning (wrongly)

Re: [PATCH resent] uapi libc compat: allow non-glibc to opt out of uapi definitions

2017-03-08 Thread David Woodhouse
On Fri, 2016-11-11 at 07:08 -0500, Felix Janda wrote: > Currently, libc-compat.h detects inclusion of specific glibc headers, > and defines corresponding _UAPI_DEF_* macros, which in turn are used in > uapi headers to prevent definition of conflicting structures/constants. > There is no such

[PATCH 24/26] IB/ocrdma: One jump label less in ocrdma_alloc_ucontext_pd()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 11:11:35 +0100 This issue was detected by using the Coccinelle software. 1. Return directly if a call of the function "_ocrdma_alloc_pd" failed. 2. Reduce the scope for the local variable "status" to one case of an if statement. 3. Delete the jump

Re: [PATCH v7 05/07] iommu/ipmmu-vmsa: Add new IOMMU_DOMAIN_DMA ops

2017-03-08 Thread Robin Murphy
On 07/03/17 03:17, Magnus Damm wrote: > From: Magnus Damm > > Introduce an alternative set of iommu_ops suitable for 64-bit ARM > as well as 32-bit ARM when CONFIG_IOMMU_DMA=y. Also adjust the > Kconfig to depend on ARM or IOMMU_DMA. Initialize the device > from ->xlate() when

Re: [PATCH v2] livepatch: make klp_mutex proper part of API

2017-03-08 Thread Josh Poimboeuf
On Wed, Mar 08, 2017 at 02:27:05PM +0100, Jiri Kosina wrote: > From: Jiri Kosina > > klp_mutex is shared between core.c and transition.c, and as such would > rather be properly located in a header so that we don't have to play > 'extern' games from .c sources. > > This also silences sparse

[PATCH 25/26] IB/ocrdma: Delete unnecessary braces

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 12:36:41 +0100 Do not use curly brackets at some source code places where a single statement should be sufficient. Signed-off-by: Markus Elfring --- drivers/infiniband/hw/ocrdma/ocrdma_verbs.c | 41 +++-- 1 file changed, 16

[PATCH 14/26] IB/ocrdma: Improve another size determination in ocrdma_add()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 7 Mar 2017 22:12:36 +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 convention. Signed-off-by:

Re: [PATCH] fs/9p: Compare qid.path in v9fs_test_inode

2017-03-08 Thread Tuomas Tynkkynen
On Wed, 22 Feb 2017 02:06:29 +0200 Tuomas Tynkkynen wrote: > Commit fd2421f54423 ("fs/9p: When doing inode lookup compare qid details > and inode mode bits.") transformed v9fs_qid_iget() to use iget5_locked() > instead of iget_locked(). However, the test() callback is not checking > fid.path at

Re: Kernel without RTC

2017-03-08 Thread Alexandre Belloni
On 08/03/2017 at 13:33:33 +, Hadimani, Jagadish wrote: > Hello Alexandre, > > I guess the Linux kernel uses HPET timer... > But can we can force Linux kernel to use Tsc or per core timer... > That is probably the case but your are targeting the wrong subsystem. The timekeeping is done using

Re: [PATCH 4/7] xen/9pfs: connect to the backend

2017-03-08 Thread Boris Ostrovsky
> >>> + ring->bytes = (void*)__get_free_pages(GFP_KERNEL | __GFP_ZERO, >>> XEN_9PFS_RING_ORDER); >>> + if (ring->bytes == NULL) >>> + goto error; >>> + for (i = 0; i < (1 << XEN_9PFS_RING_ORDER); i++) >>> + ring->intf->ref[i] = >>>

Re: [PATCH 1/2] x86/efi: Correct a tiny mistake in code comment

2017-03-08 Thread Borislav Petkov
On Wed, Mar 08, 2017 at 06:17:50PM +0800, Baoquan He wrote: > All right, I will just update the code comment. Just back ported kaslr > to our OS product, people reviewed and found the upper boundary of kaslr > mm region is EFI_VA_START, that's not correct, it has to be corrected > firstly in

RE: [PATCH 21/29] drivers, s390: convert fc_fcp_pkt.ref_cnt from atomic_t to refcount_t

2017-03-08 Thread Reshetova, Elena
> On 03/06/2017 03:21 PM, Elena Reshetova wrote: > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflows that might lead to use-after-free > > situations. > >

Re: [PATCH 03/26] IB/ocrdma: Improve size determinations in ocrdma_mbx_get_dcbx_config()

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 01:45:43PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 19:09:20 +0100 > > Replace the specification of data structures by references for variables > as the parameter for the operator "sizeof" to make the corresponding size >

Re: [PATCH v3 02/09] iommu/ipmmu-vmsa: Add optional root device feature

2017-03-08 Thread Geert Uytterhoeven
Hi Magnus, On Wed, Mar 8, 2017 at 12:01 PM, Magnus Damm wrote: > From: Magnus Damm > > Add root device handling to the IPMMU driver by allowing certain > DT compat strings to enable has_cache_leaf_nodes that in turn will > support both root devices with interrupts and leaf devices that > face

Re: [PATCH v3 04/09] iommu/ipmmu-vmsa: Make use of IOMMU_OF_DECLARE()

2017-03-08 Thread Geert Uytterhoeven
Hi Magnus, On Wed, Mar 8, 2017 at 12:02 PM, Magnus Damm wrote: > From: Magnus Damm > > Hook up IOMMU_OF_DECLARE() support in case CONFIG_IOMMU_DMA > is enabled. The only current supported case for 32-bit ARM > is disabled, however for 64-bit ARM usage of OF is required. > > Signed-off-by:

Re: netlink: GPF in netlink_unicast

2017-03-08 Thread Paul Moore
On Wed, Mar 8, 2017 at 8:25 AM, Richard Guy Briggs wrote: > On 2017-03-07 14:23, Paul Moore wrote: >> On Tue, Mar 7, 2017 at 1:44 PM, Paul Moore wrote: >> > On Tue, Mar 7, 2017 at 10:55 AM, Richard Guy Briggs >> > wrote: >> >> On 2017-03-07 09:29, Paul Moore wrote: >> >>> On Mon, Mar 6, 2017

Re: [PATCH 6/7] mm: convert generic code to 5-level paging

2017-03-08 Thread Michal Hocko
On Mon 06-03-17 23:45:13, Kirill A. Shutemov wrote: > Convert all non-architecture-specific code to 5-level paging. > > It's mostly mechanical adding handling one more page table level in > places where we deal with pud_t. > > Signed-off-by: Kirill A. Shutemov OK, I haven't spotted anything

Re: [PATCH 5/7] xen/9pfs: send requests to the backend

2017-03-08 Thread Boris Ostrovsky
>>> +} >>> + >>> +static int p9_xen_write_todo(struct xen_9pfs_dataring *ring, RING_IDX size) >>> +{ >>> + RING_IDX cons, prod; >>> + >>> + cons = ring->intf->out_cons; >>> + prod = ring->intf->out_prod; >>> + mb(); >>> + >>> + if (XEN_9PFS_RING_SIZE - xen_9pfs_queued(prod, cons, >>>

Re: [PATCH v3 09/09] iommu/ipmmu-vmsa: Hook up r8a7795 DT matching code

2017-03-08 Thread Geert Uytterhoeven
Hi Magnus, On Wed, Mar 8, 2017 at 12:02 PM, Magnus Damm wrote: > From: Magnus Damm > > Tie in r8a7795 features and update the IOMMU_OF_DECLARE > compat string to include the updated compat string. > > TODO: > - Consider making use of iommu_fwspec_add_ids() for uTLB handling > Needed to

Re: netlink: GPF in netlink_unicast

2017-03-08 Thread Richard Guy Briggs
On 2017-03-08 08:25, Richard Guy Briggs wrote: > On 2017-03-07 14:23, Paul Moore wrote: > > On Tue, Mar 7, 2017 at 1:44 PM, Paul Moore wrote: > > > On Tue, Mar 7, 2017 at 10:55 AM, Richard Guy Briggs > > > wrote: > > >> On 2017-03-07 09:29, Paul Moore wrote: > > >>> On Mon, Mar 6, 2017 at 11:03

Re: [PATCH v17 2/3] usb: USB Type-C connector class

2017-03-08 Thread Heikki Krogerus
On Tue, Mar 07, 2017 at 11:30:54PM +0100, Mats Karrman wrote: > > > So, both the DP controller and the USB PD phy are I2C devices, and now I > > > have to make them both > > > attach to the AM bus as well? > > The DP controller would provide the driver and the USB PD phy > > (actually, the typec

[PATCH 21/26] IB/ocrdma: Improve another size determination in ocrdma_create_srq()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 10:10:50 +0100 Replace the specification of a data type 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 convention. Signed-off-by: Markus

Re: [PATCH] virtio-pci: Remove affinity hint before freeing the interrupt

2017-03-08 Thread Marc Zyngier
On Wed, Mar 08 2017 at 1:28:13 pm GMT, "Michael S. Tsirkin" wrote: > On Wed, Mar 08, 2017 at 08:09:27AM +, Marc Zyngier wrote: >> virtio-pci registers a per-vq affinity hint when using MSIX, >> but fails to remove it when freeing the interrupt, resulting >> in this type of splat: >> >> [

RE: [Xen-devel] [PATCH 29/29] drivers, xen: convert grant_map.users from atomic_t to refcount_t

2017-03-08 Thread Reshetova, Elena
> On 03/06/2017 09:21 AM, Elena Reshetova wrote: > > refcount_t type and corresponding API should be > > used instead of atomic_t when the variable is used as > > a reference counter. This allows to avoid accidental > > refcounter overflows that might lead to use-after-free > > situations. > > >

Re: [PATCH 02/26] IB/ocrdma: Use kcalloc() in ocrdma_mbx_alloc_pd_range()

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 01:41:00PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 18:23:54 +0100 > > * Multiplications for the size determination of memory allocations > indicated that array data structures should be processed. > Thus reuse the corresponding

Re: [PATCH 2/2] reset: simple: read back to make sure changes are applied

2017-03-08 Thread Andre Przywara
Hi, On 08/03/17 09:54, Philipp Zabel wrote: > Read back the register after setting or clearing a reset bit to make > sure that the changes are applied to the reset controller hardware. > Theoretically, this avoids the write to stay stuck in a store buffer > during the delay of an

[PATCH 2/2] usb: gadget: f_uvc: Sanity check wMaxPacketSize for SuperSpeed

2017-03-08 Thread Roger Quadros
As per USB3.0 Specification "Table 9-20. Standard Endpoint Descriptor", for interrupt and isochronous endpoints, wMaxPacketSize must be set to 1024 if the endpoint defines bMaxBurst to be greater than zero. Signed-off-by: Roger Quadros --- drivers/usb/gadget/function/f_uvc.c | 8 1

[PATCH 0/2] usb: gadget: f_uvc: SuperSpeed fixes

2017-03-08 Thread Roger Quadros
Hi Laurent & Felipe, These are some fixes for SuperSpeed case. -- cheers, -roger Roger Quadros (2): usb: gadget: f_uvc: Fix SuperSpeed companion descriptor's wBytesPerInterval usb: gadget: f_uvc: Sanity check wMaxPacketSize for SuperSpeed drivers/usb/gadget/function/f_uvc.c | 10

Re: [PATCH 04/26] IB/ocrdma: Delete unnecessary variable initialisations in ocrdma_mbx_get_dcbx_config()

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 01:48:46PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 19:25:55 +0100 > > The local variables "req" and "rsp" will be set to appropriate pointers > a bit later. Thus omit the explicit initialisations at the beginning. > > Signed-off-by:

Re: [PATCH 05/26] IB/ocrdma: Improve another size determination in ocrdma_mbx_query_qp()

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 01:50:59PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 19:40:31 +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

[PATCH 17/26] IB/ocrdma: Use kmalloc_array() in ocrdma_create_srq()

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 08:45:31 +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 "kmalloc_array". This issue was detected by using the Coccinelle

Re: [PATCH 07/26] IB/ocrdma: Improve another size determination in ocrdma_mbx_get_ctrl_attribs()

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 01:55:59PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 20:07:25 +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

[PATCH v2] HID: remove initial reading of reports at connect

2017-03-08 Thread Benjamin Tissoires
It looks like a bunch of devices do not like to be polled for their reports at init time. When you look into the details, it seems that for those that are requiring the quirk HID_QUIRK_NO_INIT_REPORTS, the driver fails to retrieve part of the features/inputs while others (more generic) work. IMO,

Re: [PATCH 1/4] s390: get rid of superfluous __GFP_REPEAT

2017-03-08 Thread Michal Hocko
On Wed 08-03-17 09:23:40, Heiko Carstens wrote: > On Tue, Mar 07, 2017 at 04:48:40PM +0100, Michal Hocko wrote: > > From: Michal Hocko > > > > __GFP_REPEAT has a rather weak semantic but since it has been introduced > > around 2.6.12 it has been ignored for low order allocations. > > > >

Re: [PATCH] ideapad-laptop: Add Lenovo ideapad 310-15IKB to no_hw_rfkill dmi list

2017-03-08 Thread Andy Shevchenko
On Tue, Feb 21, 2017 at 9:53 PM, Sven Rebhan wrote: > Like other Lenovo models the Lenovo ideapad 310-15IKB does not have a > hw rfkill switch. This results in hard-blocked radios after boot, > resulting in always blocked radios rendering them unusable. > > The attached patch adds the ideapad

out of range LBA using sg_raw

2017-03-08 Thread Kashyap Desai
Hi - Need help to understand if below is something we should consider to be fixed in megaraid_sas driver or call as unreal exposure. I have created slice VD of size 10GB (raid 1) using 2 drives. Each Physical Drive size is 256GB. Last LBA of the VD and actual Physical disk associated with

Re: [PATCH 09/26] IB/ocrdma: Improve another size determination in ocrdma_mq_cq_handler()

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 02:00:56PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 20:20:45 +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

Re: [PATCH 08/26] IB/ocrdma: Improve size determinations in ocrdma_mbx_rdma_stats()

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 01:58:49PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 20:16:16 +0100 > > Replace the specification of two data structures by pointer dereferences > as the parameter for the operator "sizeof" to make the corresponding size >

Re: [PATCH] PCI: Add pci reset quirk for Cavium VNIC

2017-03-08 Thread rad
On 07.03.2017 17:16, Robin Murphy wrote: On 07/03/17 15:04, Radoslaw Biernacki wrote: From: Radoslaw Biernacki PCI reset quirk is needed for Cavium Function NIC since it does not handle a function level reset. This cause problems when VNIC is used from userspace by vfio. If application (or

Re: [v6 PATCH 00/21] x86: Enable User-Mode Instruction Prevention

2017-03-08 Thread Stas Sergeev
08.03.2017 03:32, Ricardo Neri пишет: These are the instructions covered by UMIP: * SGDT - Store Global Descriptor Table * SIDT - Store Interrupt Descriptor Table * SLDT - Store Local Descriptor Table * SMSW - Store Machine Status Word * STR - Store Task Register This patchset initially treated

Re: [PATCH 14/26] IB/ocrdma: Improve another size determination in ocrdma_add()

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 02:10:33PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 22:12:36 +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

[PATCH 1/2] usb: gadget: f_uvc: Fix SuperSpeed companion descriptor's wBytesPerInterval

2017-03-08 Thread Roger Quadros
The streaming_maxburst module parameter is 0 offset (0..15) so we must add 1 while using it for wBytesPerInterval calculation for the SuperSpeed companion descriptor. Without this host uvcvideo driver will always see the wrong wBytesPerInterval for SuperSpeed uvc gadget and may not find a

Re: [PATCH 12/26] IB/ocrdma: Adjust ten checks for null pointers

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 02:07:01PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 21:32:22 +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

Re: [PATCH 6/7] xen/9pfs: receive responses

2017-03-08 Thread Boris Ostrovsky
> >>> + >>> + if (xen_9pfs_queued(prod, cons, XEN_9PFS_RING_SIZE) < >>> sizeof(h)) { >>> + notify_remote_via_irq(ring->irq); >>> + return; >>> + } >>> + >>> + masked_prod = xen_9pfs_mask(prod, XEN_9PFS_RING_SIZE); >>> +

[PATCH 4/5] fs, xfs: convert xfs_cui_log_item.cui_refcount from atomic_t to refcount_t

2017-03-08 Thread Elena Reshetova
refcount_t type and corresponding API should be used instead of atomic_t when the variable is used as a reference counter. This allows to avoid accidental refcounter overflows that might lead to use-after-free situations. Signed-off-by: Elena Reshetova Signed-off-by: Hans Liljestrand

Re: problem with block: Move bdi_unregister() to del_gendisk() commit 165a5e22fafb127ecb5914e12e8c32a1f0d3f820

2017-03-08 Thread Jan Kara
On Wed 08-03-17 20:37:54, Arthur Marsh wrote: > > On one of my pc's I have 2 PATA disks (one, WDC below is used for booting, > the other SAMSUNG is not mounted), plus an IBM SCSI disk using a DPT 2044W > controller with eata driver and sometimes a Verbatim Storengo USB stick. > > On recent

Re: [PATCH 10/10] Bluetooth: add nokia driver

2017-03-08 Thread Rob Herring
On Tue, Mar 7, 2017 at 3:08 PM, Sebastian Reichel wrote: > Hi Rob, > > On Tue, Mar 07, 2017 at 10:30:51AM -0600, Rob Herring wrote: >> On Sat, Mar 4, 2017 at 5:58 AM, Sebastian Reichel wrote: >> > This adds a driver for the Nokia H4+ protocol, which is used >> > at least on the Nokia N9, N900 &

Re: [PATCH v5 2/5] powerpc: kretprobes: override default function entry offset

2017-03-08 Thread Arnaldo Carvalho de Melo
Em Wed, Mar 08, 2017 at 07:54:12PM +0530, Naveen N. Rao escreveu: > Hi Michael, > > On 2017/03/08 09:43PM, Michael Ellerman wrote: > > "Naveen N. Rao" writes: > > > > > With ABIv2, we offset 8 bytes into a function to get at the local entry > > > point. > > > > > > Acked-by: Ananth N

Re: [PATCH] mux-core: make it explicitly non-modular

2017-03-08 Thread Paul Gortmaker
[Re: [PATCH] mux-core: make it explicitly non-modular] On 08/03/2017 (Wed 10:38) Peter Rosin wrote: > On 2017-03-07 23:41, Paul Gortmaker wrote: > > The Kconfig currently controlling compilation of this code is: > > > > drivers/mux/Kconfig:menuconfig MULTIPLEXER > > drivers/mux/Kconfig:bool

Re: [PATCH] platform/x86: dell-laptop: Handle return error form dell_get_intensity.

2017-03-08 Thread Pali Rohár
Hi! On Wednesday 08 March 2017 17:52:27 Arvind Yadav wrote: > Here, dell_get_intensity can return an error. Right. That is truth and we should check for errors. > So we can assgine props.brightness as max_brightness. But why to max_brightness? Seems that this is incorrect handling of error

Re: [PATCH v17 2/3] usb: USB Type-C connector class

2017-03-08 Thread Guenter Roeck
On 03/07/2017 10:50 PM, Peter Chen wrote: You mean type-C trigger an ACPI event, and this ACPI event can notify related USB controller driver doing role switch? No (firmware programs the dual-role hw/registers), but never mind. That could be the case. If it is correct, there is a notifier

Re: counting file descriptors with a cgroup controller

2017-03-08 Thread Tejun Heo
Hello, On Tue, Mar 07, 2017 at 09:06:49PM +0100, Krzysztof Opasiak wrote: > Personally, I don't want to use rlimit for this as it ends up returning > error code from for example open() when we hit the limit. This may lead to > some unpredictable crashes in services (esp. those poor proprietary

Re: [PATCH v3 3/3] printk: fix double printing with earlycon

2017-03-08 Thread Aleksey Makarov
On 03/08/2017 06:33 AM, Sergey Senozhatsky wrote: Hello, sorry for the delay. On (03/07/17 15:54), Aleksey Makarov wrote: On 03/06/2017 03:59 PM, Sergey Senozhatsky wrote: On (03/03/17 18:49), Aleksey Makarov wrote: [..] +static enum { CONSOLE_MATCH, CONSOLE_MATCH_RETURN,

Re: [PATCH 10/26] IB/ocrdma: Improve another size determination in ocrdma_init_emb_mqe()

2017-03-08 Thread Devesh Sharma
Acked-by: Devesh Sharma On Wed, Mar 8, 2017 at 7:52 PM, Yuval Shaia wrote: > On Wed, Mar 08, 2017 at 02:02:46PM +0100, SF Markus Elfring wrote: >> From: Markus Elfring >> Date: Tue, 7 Mar 2017 20:33:29 +0100 >> >> Replace the specification of a data structure by a pointer dereference >> as the

Re: [PATCH] zswap: Zero-filled pages handling

2017-03-08 Thread Dan Streetman
On Wed, Mar 8, 2017 at 6:47 AM, Srividya Desireddy wrote: > > On Sat, Mar 4, 2017 at 02:55 AM, Dan Streetman wrote: >> On Sat, Feb 25, 2017 at 12:18 PM, Sarbojit Ganguly >> wrote: >>> On 25 February 2017 at 20:12, Srividya Desireddy >>> wrote: From: Srividya Desireddy Date: Thu, 23

Re: [PATCH 11/26] IB/ocrdma: Delete unnecessary braces

2017-03-08 Thread Yuval Shaia
On Wed, Mar 08, 2017 at 02:04:56PM +0100, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 21:12:56 +0100 > > Do not use curly brackets at some source code places > where a single statement should be sufficient. > > Signed-off-by: Markus Elfring > --- >

Re: [PATCH v2 0/2] patches for Innomedia INNEX GENESIS/ATARI adapter

2017-03-08 Thread Jiri Kosina
On Tue, 7 Mar 2017, Tomasz Kramkowski wrote: > I'll post it on here as a patch. I'm not sure if that would be a v3 or a > new patch. As the baseline patch is already merged and I am not rebasing hid.git, please send anything else as a followup fix. Thanks, -- Jiri Kosina SUSE Labs

Re: [patch v2] mm, vmstat: print non-populated zones in zoneinfo

2017-03-08 Thread Michal Hocko
On Fri 03-03-17 14:53:07, David Rientjes wrote: > Initscripts can use the information (protection levels) from > /proc/zoneinfo to configure vm.lowmem_reserve_ratio at boot. > > vm.lowmem_reserve_ratio is an array of ratios for each configured zone on > the system. If a zone is not populated on

Re: [PATCH 1/2] mm: use is_migrate_highatomic() to simplify the code

2017-03-08 Thread Michal Hocko
On Mon 06-03-17 12:43:18, Andrew Morton wrote: > On Mon, 6 Mar 2017 14:38:33 +0100 Michal Hocko wrote: > > > On Fri 03-03-17 15:06:19, Andrew Morton wrote: > > > On Fri, 3 Mar 2017 14:18:08 +0100 Michal Hocko wrote: > > > > > > > On Fri 03-03-17 19:10:13, Xishi Qiu wrote: > > > > > Introduce

Re: [PATCH 03/26] IB/ocrdma: Improve size determinations in ocrdma_mbx_get_dcbx_config()

2017-03-08 Thread Devesh Sharma
Acked-By: Devesh Sharma On Wed, Mar 8, 2017 at 7:24 PM, Yuval Shaia wrote: > On Wed, Mar 08, 2017 at 01:45:43PM +0100, SF Markus Elfring wrote: >> From: Markus Elfring >> Date: Tue, 7 Mar 2017 19:09:20 +0100 >> >> Replace the specification of data structures by references for variables >> as

Re: [PATCH v2 1/2] net: sched: make default fifo qdiscs appear in the dump

2017-03-08 Thread Jiri Pirko
Wed, Mar 08, 2017 at 01:03:42PM CET, ji...@kernel.org wrote: >From: Jiri Kosina > >The original reason [1] for having hidden qdiscs (potential scalability >issues in qdisc_match_from_root() with single linked list in case of large >amount of qdiscs) has been invalidated by 59cc1f61f0 ("net:

Re: Kernel without RTC

2017-03-08 Thread Thomas Gleixner
On Wed, 8 Mar 2017, Alexandre Belloni wrote: > On 08/03/2017 at 13:33:33 +, Hadimani, Jagadish wrote: > > Hello Alexandre, > > > > I guess the Linux kernel uses HPET timer... > > But can we can force Linux kernel to use Tsc or per core timer... > > > > That is probably the case but your are

Re: [PATCH 17/26] IB/ocrdma: Use kmalloc_array() in ocrdma_create_srq()

2017-03-08 Thread Devesh Sharma
On Wed, Mar 8, 2017 at 6:47 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 08:45:31 +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

Re: [PATCH v2 02/21] x86/xen: globalize have_vcpu_info_placement

2017-03-08 Thread Juergen Gross
On 02/03/17 18:53, Vitaly Kuznetsov wrote: > have_vcpu_info_placement applies to both PV and HVM and as we're going > to split the code we need to make it global. > > Rename to xen_have_vcpu_info_placement. > > Signed-off-by: Vitaly Kuznetsov Reviewed-by: Juergen Gross Juergen

Re: [RFC PATCH 3/4] xfs: map KM_MAYFAIL to __GFP_RETRY_MAYFAIL

2017-03-08 Thread Christoph Hellwig
On Tue, Mar 07, 2017 at 04:48:42PM +0100, Michal Hocko wrote: > From: Michal Hocko > > KM_MAYFAIL didn't have any suitable GFP_FOO counterpart until recently > so it relied on the default page allocator behavior for the given set > of flags. This means that small allocations actually never

Re: [PATCH v2 01/21] x86/xen: separate PV and HVM hypervisors

2017-03-08 Thread Juergen Gross
On 02/03/17 18:53, Vitaly Kuznetsov wrote: > As a preparation to splitting the code we need to untangle it: > > x86_hyper_xen -> x86_hyper_xen_hvm and x86_hyper_xen_pv > xen_platform() -> xen_platform_hvm() and xen_platform_pv() > xen_cpu_up_prepare() -> xen_cpu_up_prepare_pv() and

Re: rtlwifi: fix spelling mistake: "conuntry" -> "country"

2017-03-08 Thread Kalle Valo
Colin Ian King wrote: > From: Colin Ian King > > trivial fix to spelling mistake in RT_TRACE message > > Signed-off-by: Colin Ian King Patch applied to wireless-drivers-next.git, thanks. 3e062eb21df8 rtlwifi: fix spelling mistake: "conuntry" -> "country" --

Re: [PATCH 19/26] IB/ocrdma: Improve another size determination in ocrdma_alloc_mr()

2017-03-08 Thread Devesh Sharma
Acked-By: Devesh Sharma On Wed, Mar 8, 2017 at 6:50 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 8 Mar 2017 09:54:42 +0100 > > Replace the specification of a data type by a pointer dereference > as the parameter for the operator "sizeof" to make the corresponding size >

Re: Kernel without RTC

2017-03-08 Thread Hadimani, Jagadish
Hello Alexandre, I guess the Linux kernel uses HPET timer... But can we can force Linux kernel to use Tsc or per core timer... Sent from my iPhone > On 8 Mar 2017, at 6:47 PM, Alexandre Belloni > wrote: > > Hi, > >> On 07/03/2017 at 00:48:18 +0530, Shah, Nehal-bakulchandra wrote: >> Hi, >>

Re: [PATCH 11/26] IB/ocrdma: Delete unnecessary braces

2017-03-08 Thread Devesh Sharma
On Wed, Mar 8, 2017 at 6:34 PM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 7 Mar 2017 21:12:56 +0100 > > Do not use curly brackets at some source code places > where a single statement should be sufficient. > > Signed-off-by: Markus Elfring > --- >

Re: [RESEND PATCH] arm: assabet_defconfig: disable IDE subsystem

2017-03-08 Thread Sekhar Nori
On Tuesday 07 March 2017 11:21 PM, Bartlomiej Zolnierkiewicz wrote: > > Hi, > > On Monday, December 12, 2016 07:24:47 PM Sekhar Nori wrote: >> Hi Bartlomiej, >> >> On Monday 12 December 2016 06:15 PM, Bartlomiej Zolnierkiewicz wrote: >>> >>> Hi, >>> >>> On Monday, July 18, 2016 08:15:08 PM

[PATCH 26/26] IB/ocrdma: Adjust further ten checks for null pointers

2017-03-08 Thread SF Markus Elfring
From: Markus Elfring Date: Wed, 8 Mar 2017 12:50:14 +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 fix affected source code

Re: [PATCH v2 03/21] x86/xen: add CONFIG_XEN_PV to Kconfig

2017-03-08 Thread Juergen Gross
On 02/03/17 18:53, Vitaly Kuznetsov wrote: > All code to supprot Xen PV will get under this new option. For the s/supprot/support/ > beginning, check for it in the common code. > > Signed-off-by: Vitaly Kuznetsov > --- > arch/x86/kernel/cpu/hypervisor.c | 4 +++- >

<    7   8   9   10   11   12   13   14   15   16   >