From: Markus Elfring
Date: Thu, 13 Oct 2016 14:50:05 +0200
Move the assignment for the local variable "prod" behind the source code
for a memory allocation by this function.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
di
From: Markus Elfring
Date: Thu, 13 Oct 2016 17:13:41 +0200
Move the assignment for the local variable "rr3" behind the source code
for a memory allocation by this function.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
dif
From: Markus Elfring
Date: Thu, 13 Oct 2016 15:40:33 +0200
* Use the data structure members "dev" and "udev" directly
without assigning them to intermediate variables.
Thus delete the extra variable definitions at the beginning.
* Fix indentation for the parameters of two function calls.
Si
Hi Jean,
On 13-10-2016 12:40, Jean Delvare wrote:
> Hi Carlos,
>
> On Tue, 11 Oct 2016 18:24:36 +0100, Carlos Palminha wrote:
>> This patchset fixes several coding style issues.
>>
>> Carlos Palminha (5):
>> i2c: i2c-piix4: coding style fix - '*' adjacent to data name
>> i2c: i2c-piix4: codin
Hi Stewart,
On Wed, Oct 12, 2016 at 04:35:35PM +1100, Stewart Smith wrote:
>
> What if we just treat the 0xF state from firmware as special and set it
> to DEFAULT_PSSCR_MASK in that case? That deals with old skiboot, new
> kernel, and sets a pretty small special case that's easy to track into
>
On Thu, Oct 13, 2016 at 05:55:11PM +0200, Steffen Maier wrote:
> Hm, still behaves for me like I reported for v2:
> http://marc.info/?l=linux-scsi&m=147637177902937&w=2
>
Well given what you've wrote for v2 it's kinda expected.
Byte,
Johannes
--
Johannes Thumshirn
From: Markus Elfring
Date: Thu, 13 Oct 2016 15:00:12 +0200
Adjust a jump label according to the Linux coding style convention.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/media/rc/redrat3.c b/driv
On Thu, 13 Oct 2016 18:57:14 +0800
Cheng Chao wrote:
> GIC can distribute an interrupt to more than one cpu,
> but now, gic_set_affinity sets only one cpu to handle interrupt.
What makes you think this is a good idea? What purpose does it serves?
I can only see drawbacks to this: You're waking u
On Thu, 2016-10-13 at 16:57 +0200, Greg KH wrote:
> On Thu, Oct 13, 2016 at 05:23:45PM +0300, Mikhail Golubev wrote:
> > On Thu, Oct 13, 2016 at 02:06:02PM +0200, Greg KH wrote:
> > > On Thu, Oct 13, 2016 at 02:50:18PM +0300, Mikhail Golubev wrote:
> > > > Function definitions arguments should also
From: Andi Kleen
Add a Intel event file for perf.
Signed-off-by: Andi Kleen
Cc: Jiri Olsa
Cc: Sukadev Bhattiprolu
Link: http://lkml.kernel.org/n/tip-m72axmpkxcdproq9x04zu...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo
---
.../perf/pmu-events/arch/x86/Silvermont/Cache.json | 811 +
From: Markus Elfring
Date: Thu, 13 Oct 2016 14:17:43 +0200
The local variable "ret" will be set to an appropriate value a bit later.
Thus omit the explicit initialisation at the beginning.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 2 +-
1 file changed, 1 insertion(+), 1 de
ip6_route_output() doesn't return a negative error when it fails, rather
the ->error field of the returned dst_entry struct needs to be checked.
Reported-by: Dan Carpenter
Fixes: 75b54cb57ca3 ("rxrpc: Add IPv6 support")
Signed-off-by: David Howells
---
net/rxrpc/peer_object.c |4 ++--
1 fi
On Thu, 2016-10-13 at 18:18 +0200, SF Markus Elfring wrote:
> diff --git a/drivers/media/rc/redrat3.c b/drivers/media/rc/redrat3.c
[]
> @@ -549,7 +549,7 @@ static void redrat3_get_firmware_rev(struct redrat3_dev
> *rr3)
> int rc = 0;
> char *buffer;
>
> - buffer = kzalloc(sizeof(
From: Markus Elfring
Date: Thu, 13 Oct 2016 13:23:22 +0200
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.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 2 +-
From: Markus Elfring
Date: Thu, 13 Oct 2016 13:31:17 +0200
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.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 2 +-
From: Andi Kleen
Add a Intel event file for perf.
Signed-off-by: Andi Kleen
Cc: Jiri Olsa
Cc: Sukadev Bhattiprolu
Link: http://lkml.kernel.org/n/tip-7r1wcyb5ucqxsqzcljt3i...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo
---
tools/perf/pmu-events/arch/x86/Bonnell/Cache.json | 746 +
From: Markus Elfring
Date: Thu, 13 Oct 2016 10:34:29 +0200
* Return directly after a call of the function "kcalloc" failed
at the beginning.
* Reorder two calls for the function "kfree" at the end.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 8 +++-
1 file changed, 3
From: Markus Elfring
Date: Thu, 13 Oct 2016 14:07:03 +0200
* One local variable was set to an error code before a concrete
error situation was detected. Thus move the corresponding assignment
into three if branches to indicate a memory allocation failure there.
* Adjust a jump label accordin
is BE or CPU-E within
AFS.
The patches can be found here also:
http://git.kernel.org/cgit/linux/kernel/git/dhowells/linux-fs.git/log/?h=rxrpc-rewrite
Tagged thusly:
git://git.kernel.org/pub/scm/linux/kernel/git/dhowells/linux-fs.git
rxrpc-rewrite-20161013
David
On Thu, Oct 13, 2016 at 05:15:25PM +0200, Steffen Maier wrote:
> I'm puzzled.
>
> $ git bisect start fc_bsg master
> Bisecting: 8 revisions left to test after this (roughly 3 steps)
> [005d51510eee6102636d5dbb06310531c5d46151] scsi: fc: implement kref backed
> reference counting
> $ git bisect bad
From: Namhyung Kim
Markus reported that 'perf top --hierarchy' cannot scroll down after
refresh. This was because the number of entries are not updated when
hierarchy is enabled.
Unlike normal report view, hierarchy mode needs to keep its own entry
count since it can have non-leaf entries which
From: Madhavan Srinivasan
Add support for Power8 PVR 004b0201 for tuleta and 0x004d0200 for
firestone.
Signed-off-by: Madhavan Srinivasan
CC: Andi Kleen
Cc: Jiri Olsa
Cc: Sukadev Bhattiprolu
Link: http://lkml.kernel.org/n/tip-wr6rf3d3vvggy8180ftt2...@git.kernel.org
Signed-off-by: Arnaldo Car
From: Andi Kleen
Vendor events are often specified in upper case. perf list outputs them
in lower case. Handle this case in perf-completion.sh so that completion
on the upper case events still works.
v2: Use locale aware check for upper case
v3: Use perf list json
Signed-off-by: Andi Kleen
Tes
From: Andi Kleen
Add a Intel event file for perf.
Signed-off-by: Andi Kleen
Cc: Jiri Olsa
Cc: Sukadev Bhattiprolu
Link: http://lkml.kernel.org/n/tip-q87xlo75ffonydbmvf6lr...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo
---
tools/perf/pmu-events/arch/x86/Goldmont/Cache.json | 1127
1) Fix various build warnings in tlan/qed/xen-netback drivers, from Arnd
Bergmann.
2) Propagate proper error code in strparser's strp_recv(), from Geert
Uytterhoeven.
3) Fix accidental broadcast of RTM_GETTFILTER responses, from Eric Dumazret.
4) Need to use list_for_each_entry_safe() in qe
From: Andi Kleen
Add a Intel event file for perf.
Signed-off-by: Andi Kleen
Cc: Jiri Olsa
Cc: Sukadev Bhattiprolu
Link: http://lkml.kernel.org/n/tip-q8ujoa7nzy5p7o5vgdzzx...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo
---
.../perf/pmu-events/arch/x86/IvyBridge/Cache.json | 1123
From: Anton Blanchard
When building on Ubuntu 16.04, I get the following error:
Makefile:49: *** the openjdk development package appears to me missing, install
and try again. Stop.
The problem is that update-java-alternatives has multiple spaces between
fields, and cut treats each space as a
From: Andi Kleen
Add an argument to only list json events. To be used for the command
line completion script. This requires adding a new flag to the alias
structure.
Committer notes:
Using it:
$ perf list json
Cache:
l1d.replacement
[L1D data line replacements]
l1d_pend_m
From: Jiri Olsa
Sukadev reported segfault on releasing perf env's numa data. It's due
to nr_numa_nodes being set no matter if the numa data gets parsed
properly. The perf_env__exit crash the on releasing non existed data.
Setting nr_numa_nodes only when data are parsed out properly.
Signed-off
From: Wang Nan
This patch helps with Sukadev's JSON event tree where such events can happen.
>From Andi Kleen:
Any event inclduing a .c/.o/.bpf currently triggers BPF compilation or loading
and then an error. This can happen for some Intel JSON events, which cannot
be used.
This patch fixes
From: Markus Elfring
Date: Thu, 13 Oct 2016 13:20:19 +0200
The script "checkpatch.pl" pointed information out like the following.
WARNING: Possible unnecessary 'out of memory' message
Thus remove such a logging statement in five functions.
Link:
http://events.linuxfoundation.org/sites/events/
From: Andi Kleen
Add a Intel event file for perf.
Signed-off-by: Andi Kleen
Cc: Jiri Olsa
Cc: Sukadev Bhattiprolu
Link: http://lkml.kernel.org/n/tip-0vzsmw92s0m3mcwmdh27d...@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo
---
tools/perf/pmu-events/arch/x86/Jaketown/Cache.json | 1290
From: Markus Elfring
Date: Thu, 13 Oct 2016 13:21:55 +0200
The local variable "rc" will be set to an appropriate value a bit later.
Thus omit the explicit initialisation at the beginning.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 2 +-
1 file changed, 1 insertion(+), 1 del
From: Markus Elfring
Date: Thu, 13 Oct 2016 10:50:24 +0200
The kfree() function was called in one case by the
redrat3_transmit_ir() function during error handling
even if the passed variable contained a null pointer.
* Adjust jump targets according to the Linux coding style convention.
* Move t
From: Dan Carpenter
We switched from kmap_atomic() to kmap() so the kunmap() calls need to
be updated to match.
Fixes: d001648ec7cf ('rxrpc: Don't expose skbs to in-kernel users [ver #2]')
Signed-off-by: Dan Carpenter
Signed-off-by: David Howells
---
fs/afs/fsclient.c |4 ++--
1 file cha
call->operation_ID is sometimes being used as __be32 sometimes is being
used as u32. Be consistent and settle on using as u32.
Signed-off-by: David Howells operation_ID);
+ _enter("{CB.OP %u}", call->operation_ID);
- _enter("{CB.OP %u}", operation_id);
-
- switch (operation_id
Fix the following checker warning:
net/rxrpc/call_object.c:279 rxrpc_new_client_call()
warn: passing zero to 'ERR_PTR'
where a value that's always zero is passed to ERR_PTR() so that it can be
passed to a tracepoint in an auxiliary pointer field.
Just pass NULL instead to the tra
2016-10-13 17:45+0200, Jiri Slaby:
> gcc 7 warns:
> arch/x86/kvm/ioapic.c: In function 'kvm_ioapic_reset':
> arch/x86/kvm/ioapic.c:597:2: warning: 'memset' used with length equal to
> number of elements without multiplication by element size [-Wmemset-elt-size]
>
> And it is right. Memset whole a
On Thu, Oct 13, 2016 at 09:05:57AM -0700, Nadim Almas wrote:
> Fixed coding style issue
>
> Signed-off-by: Nadim Almas
> ---
> drivers/staging/ks7010/eap_packet.h | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/ks7010/eap_packet.h
> b/drivers/stag
From: Markus Elfring
Date: Thu, 13 Oct 2016 10:25:57 +0200
Move the assignment for the data structure member "transmitting"
and the local variable "curlencheck" behind the source code
for memory allocations by this function.
Signed-off-by: Markus Elfring
---
drivers/media/rc/redrat3.c | 8
On Thu, 13 Oct 2016, Vaishali Thakkar wrote:
> Currently because of the left associativity of the operators,
> pattern IRQF_ONESHOT | flags does not match with the pattern
> when we have more than one flag after the disjunction. This
> eventually results in giving false positives by the script.
From: Markus Elfring
Date: Thu, 13 Oct 2016 08:35:57 +0200
* Multiplications for the size determination of memory allocations
indicated that array data structures should be processed.
Thus use the corresponding function "kcalloc".
This issue was detected by using the Coccinelle software.
The latter version occurs much more when running git grep.
Signed-off-by: Alexander Alemayhu
---
tools/perf/tests/backward-ring-buffer.c | 2 +-
tools/perf/tests/bpf.c | 2 +-
tools/perf/util/bpf-loader.c| 14 +++---
tools/perf/util/llvm-utils.c
On Thu, Oct 13, 2016 at 09:43:57AM -0600, David Ahern wrote:
> On 10/13/16 1:16 AM, Cyrill Gorcunov wrote:
> > On Wed, Oct 12, 2016 at 07:55:04PM -0400, David Miller wrote:
> >> From: Cyrill Gorcunov
> >> Date: Wed, 12 Oct 2016 09:53:29 +0300
> >>
> >>> I can't rename the field, neither a can use
It's not necessary to cast the result of kmalloc, since void pointers are
promoted to any other type. This also fixes following coccinelle warning:
casting value returned by memory allocation function to
(BIG_IOCTL_Command_struct *) is useless.
Signed-off-by: Javier Martinez Canillas
---
driv
It's not necessary to cast the result of kmalloc, since void pointers are
promoted to any other type. This also fixes following coccinelle warning:
casting value returned by memory allocation function to (struct
cciss_scsi_adapter_data_t *) is useless.
Signed-off-by: Javier Martinez Canillas
--
From: Dan Carpenter
Date: Thu, 13 Oct 2016 11:06:06 +0300
> We should clear out the padding and unused struct members so that we
> don't expose stack information to userspace.
>
> Fixes: fdb3accc2c15 ('tipc: add the ability to get UDP options via netlink')
> Signed-off-by: Dan Carpenter
Applie
From: Zach Brown
Date: Thu, 13 Oct 2016 10:42:46 -0500
> Do you have suggestions on how to better handle the choice of the array size
> and the speeds?
All of the speed values are simply the rate in bits per second.
There is therefore no reason you can't just print the raw value and
scale it to
Hi Loic,
On 13/10/16 14:22, loic pallardy wrote:
On 10/11/2016 03:39 PM, Matt Redfearn wrote:
Storage of the firmware name was inconsistent, either storing a pointer
to a name stored with unknown ownership, or a variable length tacked
onto the end of the struct proc allocated in rproc_alloc.
Fixed coding style issue
Signed-off-by: Nadim Almas
---
drivers/staging/ks7010/eap_packet.h | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/ks7010/eap_packet.h
b/drivers/staging/ks7010/eap_packet.h
index df7f760..90432ff 100644
--- a/drivers/staging/ks7
On Wed, 12 Oct 2016, Dave Hansen wrote:
> On 10/12/2016 06:34 AM, Thomas Gleixner wrote:
> >> > +if (c->x86 == 6 &&
> >> > +c->x86_model == INTEL_FAM6_XEON_PHI_KNL &&
> >> > +phir3mwait) {
> >> > +u64 prev;
> >> > +
> >> > +rdmsrl(MSR
On Thu, 13 Oct 2016, Grzegorz Andrejczuk wrote:
> Add hwcap2 attribute for x86.
Great value in repeating the subject line. And the subject line is crap
because this patch does way more than adding hwcap2
> Reserve 1st bit of HWCAP2 for exposing Xeon Phi ring 3 monitor/mwait.
> With this us
On Thu, 13 Oct 2016, Grzegorz Andrejczuk wrote:
> Subject: [PATCH v3 1/4] Add R3MWAIT register and bit to msr-info.h
Did you ever notice that all patches have a subsystem related prefix before
the sentence decribing the change?
See Documentation/SubmittingPatches. Also git log some/file might g
On 13/10/16 16:40, Dan Williams wrote:
> On Thu, Oct 13, 2016 at 2:08 AM, Jan Beulich wrote:
> [..]
>>> I think we can do the similar for Xen, like to lay another pseudo
>>> device on /dev/pmem and do the reservation, like 2. in my previous
>>> reply.
>> Well, my opinion certainly doesn't count mu
On Thu, Oct 13, 2016 at 8:54 AM, Toshi Kani wrote:
> ACPI Clear Uncorrectable Error DSM function may fail or may be
> unsupported on a platform. pmem_clear_poison() returns without
> clearing badblocks in such cases, which leads to a silent data
> corruption.
>
> Change pmem_do_bvec() and pmem_cl
In some rare cases, we can't retrieve the SMBus version and so we fail
binding the touchpad back. Instead of leaving a touchpad dead, try again
to reinitialize it.
Signed-off-by: Benjamin Tissoires
---
no changes in v3
no changes in v2
---
drivers/input/rmi4/rmi_smbus.c | 21 +
From: Andrew Duggan
Implements reading and setting the dribble bit in F12's control registers.
Signed-off-by: Andrew Duggan
Acked-by: Benjamin Tissoires
Signed-off-by: Benjamin Tissoires
---
new in v3
---
drivers/input/rmi4/rmi_f12.c | 73 +++-
1 fil
ACPI Clear Uncorrectable Error DSM function may fail or may be
unsupported on a platform. pmem_clear_poison() returns without
clearing badblocks in such cases, which leads to a silent data
corruption.
Change pmem_do_bvec() and pmem_clear_poison() to return -EIO
so that filesystem can log an error
From: Andrew Duggan
The rmi_f11 driver currently disables dribble packets and the palm detect
gesture for all devices. This patch creates a parameter in the 2d sensor
platform data for controlling this functionality on a per device basis.
For more information on dribble packets:
Commit 05ba999fc
From: Lyude Paul
This adds basic functionality for PS/2 passthrough on Synaptics
Touchpads using RMI4 through smbus.
Reviewed-by: Andrew Duggan
Signed-off-by: Lyude Paul
Signed-off-by: Benjamin Tissoires
---
no changes in v3
changes in v2:
- switch to use rmi_dbg instead of plain dev_dbg
-
With RMI4 over SMBus, the pass-through device can be instantiated
in a SMBus driver. However, compared to the psmouse-synaptics driver,
this pass-through PS/2 driver has no clue whether the current
serio_interrupt() is the beginning of the frame or not. Instead of
adding a protocol analysis in RMI4
If a function needs to communicate with an other, it's better to have
a way to retrieve this other.
Reviewed-by: Andrew Duggan
Signed-off-by: Benjamin Tissoires
---
no changes in v3
no changes in v2
---
drivers/input/rmi4/rmi_driver.c | 13 +
drivers/input/rmi4/rmi_driver.h | 1
On Wed 12-10-16 16:50:20, Ross Zwisler wrote:
> DAX PMDs have been disabled since Jan Kara introduced DAX radix tree based
> locking. This patch allows DAX PMDs to participate in the DAX radix tree
> based locking scheme so that they can be re-enabled using the new struct
> iomap based fault handl
From: Andrew Duggan
The rmi4 2D sensor functions report the tool type via
input_mt_report_slot_state(), but the abs parameter bit has not been
set so the tool type is not reported to userspace. This patch set
the ABS_MT_TOOL_TYPE bit.
Signed-off-by: Andrew Duggan
Signed-off-by: Benjamin Tissoir
On the latest series of ThinkPads, the button events for the TrackPoint
are reported through the touchpad itself as opposed to the TrackPoint
device. In order to report these buttons properly, we need to forward
them to the TrackPoint device and send the button presses/releases
through there instea
Code obtained from
https://raw.githubusercontent.com/mightybigcar/synaptics-rmi4/jf/drivers/input/rmi4/rmi_smbus.c
and updated to match upstream. And fixed to make it work.
Signed-off-by: Benjamin Tissoires
Signed-off-by: Andrew Duggan
---
changes in v3:
- use of irq for Host Notify, not aler
This driver is a glue between PS/2 devices that enumerate
the RMI4 device and the RMI4 SMBus driver.
We mostly use an intermediate platform device to not
add a dependency between psmouse and I2C. It also handles
the subtleties of going around the serio mutex lock by
deferring the i2c creation/dest
The SMBus Synaptics devices enumerated as PS/2 devices have
the problem of being deaf to I2C if the touchpad has been
fully initialized over PS/2 (psmouse_activate being called).
A simple PS/2 deactivate command is enough to make it back alive.
To make sure the pass-through device does not interfe
From: Andrew Duggan
Commit 5b65c2a02966 ("HID: rmi: check sanity of the incoming report") added
support for handling incomplete HID reports do to the input data being
corrupted in transit. This patch reimplements this functionality in the
function drivers so they can handle getting less valid dat
From: Bjorn Andersson
The attn IRQ is related to the chip, rather than the transport, so move
all handling of interrupts to the core driver. This also makes sure that
there are no races between interrupts and availability of the resources
used by the core driver.
Signed-off-by: Bjorn Andersson
On 10/13/2016 06:58 AM, Vincent Guittot wrote:
> Hi,
>
> On 12 October 2016 at 18:21, Joseph Salisbury
> wrote:
>> On 10/12/2016 08:20 AM, Vincent Guittot wrote:
>>> On 8 October 2016 at 13:49, Mike Galbraith wrote:
On Sat, 2016-10-08 at 13:37 +0200, Vincent Guittot wrote:
> On 8 October
From: Noam Camus
IPI_IRQ (also TIMER0_IRQ) should ack before the action->handler is called
in handle_percpu_devid_irq.
The IPI irq is edge sensitive and we might miss an IPI interrupt
if it was triggered during handling the interrupt itself.
Signed-off-by: Noam Camus
---
drivers/irqchip/irq-ez
From: Dennis Wassenberg
First check if there are data available passed by the transport device.
If data available use these data. If there are no data available
try to read the rmi block if dsata are passed this way.
This is the way the other rmi function handlers will do this.
This patch is ne
Most of the Synaptics devices are connected through PS/2 and a different
bus (SMBus or HID over I2C).
The secondary bus capability is indicated by the InterTouch bit in
extended capability 0x0C.
When we encounter such a device, we can create a platform device with
the information gathered through
From: Nick Dyer
Signed-off-by: Nick Dyer
Signed-off-by: Benjamin Tissoires
---
new in v3
---
drivers/input/rmi4/rmi_driver.c | 139 +---
1 file changed, 86 insertions(+), 53 deletions(-)
diff --git a/drivers/input/rmi4/rmi_driver.c b/drivers/input/rmi4/rm
If struct rmi_device_platform_data contains pointers to other struct,
it gets difficult to allocate a fixed size struct and copy it over between
drivers.
Change the pointers into a struct and change the code in rmi4 accordingly.
Reviewed-by: Andrew Duggan
Signed-off-by: Benjamin Tissoires
---
Hi guys,
This is the third submission of this series, with some addition of RMI4 patches
currently waiting on the list.
I integrated Bjorn's patch because in the end, it seems to be the right thing
to do. We can create an irqchip in hid-rmi and also provide an IRQ there.
Worse case, if this doesn
Typos...
Signed-off-by: Benjamin Tissoires
---
no changes in v3
new in v2
---
include/linux/rmi.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/rmi.h b/include/linux/rmi.h
index 6b22948..2fdb4fb 100644
--- a/include/linux/rmi.h
+++ b/include/linux/rmi.h
@@
On Thu, Oct 13, 2016 at 05:23:45PM +0300, Mikhail Golubev wrote:
> On Thu, Oct 13, 2016 at 02:06:02PM +0200, Greg KH wrote:
> > On Thu, Oct 13, 2016 at 02:50:18PM +0300, Mikhail Golubev wrote:
> > > Function definitions arguments should also have an identifier name as
> > > reported by checkpatch.
On 13/10/2016 17:45, Jiri Slaby wrote:
> gcc 7 warns:
> arch/x86/kvm/ioapic.c: In function 'kvm_ioapic_reset':
> arch/x86/kvm/ioapic.c:597:2: warning: 'memset' used with length equal to
> number of elements without multiplication by element size [-Wmemset-elt-size]
>
> And it is right. Memset who
On 10/13/16 03:08 -0600, Jan Beulich wrote:
On 13.10.16 at 10:53, wrote:
On 10/13/16 02:34 -0600, Jan Beulich wrote:
On 12.10.16 at 18:19, wrote:
On Wed, Oct 12, 2016 at 9:01 AM, Jan Beulich wrote:
On 12.10.16 at 17:42, wrote:
On Wed, Oct 12, 2016 at 8:39 AM, Jan Beulich wrote:
On 12.1
gcc 7 warns:
arch/x86/kvm/ioapic.c: In function 'kvm_ioapic_reset':
arch/x86/kvm/ioapic.c:597:2: warning: 'memset' used with length equal to number
of elements without multiplication by element size [-Wmemset-elt-size]
And it is right. Memset whole array using sizeof operator.
Signed-off-by: Jir
From: Josh Cartwright
The use of IRQF_ONESHOT when registering an interrupt handler with
request_irq() is non-sensical.
Not only that, it also prevents the handler from being threaded when it
otherwise should be w/ IRQ_FORCED_THREADING is enabled. This causes the
following deadlock observed by
On 10/13/16 1:16 AM, Cyrill Gorcunov wrote:
> On Wed, Oct 12, 2016 at 07:55:04PM -0400, David Miller wrote:
>> From: Cyrill Gorcunov
>> Date: Wed, 12 Oct 2016 09:53:29 +0300
>>
>>> I can't rename the field, neither a can use union.
>>
>> Remind me again what is wrong with using an anonymous union?
On Thu, 13 Oct 2016, Grzegorz Andrejczuk wrote:
> --- a/arch/x86/kernel/cpu/common.c
> +++ b/arch/x86/kernel/cpu/common.c
> @@ -1108,6 +1108,9 @@ static void identify_cpu(struct cpuinfo_x86 *c)
> #endif
> /* The boot/hotplug time assigment got cleared, restore it */
> c->logical_proc_i
On Wed 12-10-16 16:50:22, Ross Zwisler wrote:
> Now that DAX PMD faults are once again working and are now participating in
> DAX's radix tree locking scheme, allow their config option to be enabled.
>
> Signed-off-by: Ross Zwisler
Looks good. You can add:
Reviewed-by: Jan Kara
On Thu, Oct 13, 2016 at 2:08 AM, Jan Beulich wrote:
[..]
>> I think we can do the similar for Xen, like to lay another pseudo
>> device on /dev/pmem and do the reservation, like 2. in my previous
>> reply.
>
> Well, my opinion certainly doesn't count much here, but I continue to
> consider this a
Add Knights Mill (KNM) to the list of CPUIDs supported by PMU.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@kernel.org
Cc: linux-kernel@vger.kernel.org
---
This patch depends on [PATCH 03/11] x86/cpu/intel: Add Knights Mil
Add Knights Mill (KNM) to the list of CPUIDs supported by turbostat.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: linux-kernel@vger.kernel.org
---
This patch depends on [PATCH 03/11] x86/cpu/intel: Add Knights Mill
to Intel family
tools/power/x86/turbostat/turbostat.c | 7 +++
1
Add Knights Mill (KNM) to the list of CPUIDs supported by intel_rapl
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: "Rafael J. Wysocki"
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
---
This patch depends on [PATCH 03/11] x86/cpu/intel: Add Knights Mill
to Intel family
Add Knights Mill (KNM) to the list of CPUIDs supported by PMU.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@kernel.org
Cc: linux-kernel@vger.kernel.org
---
This patch depends on [PATCH 03/11] x86/cpu/intel: Add Knights Mil
Add Knights Mill (KNM) to the list of CPUIDs supported by intel_pstate.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: Srinivas Pandruvada
Cc: Len Brown
Cc: "Rafael J. Wysocki"
Cc: Viresh Kumar
Cc: linux...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
---
This patch depends on [PA
Add Knights Mill (KNM) to the list of CPUIDs supported by rapl.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@kernel.org
Cc: linux-kernel@vger.kernel.org
---
This patch depends on [PATCH 03/11] x86/cpu/intel: Add Knights Mi
Add Knights Mill (KNM) to the list of CPUIDs supported by intel_idle.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: linux...@vger.kernel.org
Cc: Len Brown
Cc: linux-kernel@vger.kernel.org
---
This patch depends on [PATCH 03/11] x86/cpu/intel: Add Knights Mill
to Intel family
drivers/
Add Knights Mill (KNM) to the list of CPUIDs supported by sb_edac.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: Mauro Carvalho Chehab
Cc: Doug Thompson
Cc: Borislav Petkov
Cc: linux-e...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: "Luck, Tony"
Cc: "Shevchenko, Andriy"
---
On Thu, 13 Oct 2016, Grzegorz Andrejczuk wrote:
>
> +static int phi_r3mwait_disabled;
__read_mostly please.
> +
> /*
> * Just in case our CPU detection goes bad, or you have a weird system,
> * allow a way to override the automatic disabling of MPX.
> @@ -61,6 +63,34 @@ void check_mpx_erra
On Mon, Oct 10, 2016 at 07:31:46AM -0700, Edward Lipinsky wrote:
>
> Thanks, that makes sense. I tried deleting the if statement and printk()
> from ddk750_help.c, and adding the following in lynxfb_pci_probe() after
> calling hw_sm750_map() and checking for errors:
>
> if (sm750_dev->revid == 0
Add CPUID of Knights Mill (KNM) processor to Intel family list.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: x...@kernel.org
Cc: linux-kernel@vger.kernel.org
---
arch/x86/include/asm/intel-family.h | 1 +
1 file changed, 1 ins
Use macros of Intel processor families instead of raw numbers.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: linux-kernel@vger.kernel.org
---
v2: Include Intel family header.
tools/power/x86/turbostat/Makefile| 3 +-
tools/power/x86/turbostat/turbostat.c | 243 +--
Use macros of Intel processor families instead of raw numbers.
Signed-off-by: Piotr Luc
Reviewed-by: Dave Hansen
Cc: Mauro Carvalho Chehab
Cc: Doug Thompson
Cc: Borislav Petkov
Cc: linux-e...@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: "Luck, Tony"
Cc: "Shevchenko, Andriy"
---
dri
301 - 400 of 723 matches
Mail list logo
