From: David Decotigny
Signed-off-by: David Decotigny
---
net/bridge/br_if.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/net/bridge/br_if.c b/net/bridge/br_if.c
index c367b3e..b37a1cc 100644
--- a/net/bridge/br_if.c
+++ b/net/bridge/br_if.c
@@ -36,10 +36,10 @@
*/
From: David Decotigny
Signed-off-by: David Decotigny
---
drivers/net/team/team.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/team/team.c b/drivers/net/team/team.c
index 00558e1..2769835 100644
--- a/drivers/net/team/team.c
+++ b/drivers/net/team/team.
On 02/23/2016 02:37 PM, Kees Cook wrote:
On Tue, Feb 23, 2016 at 1:25 PM, Kees Cook wrote:
zero poison only:
DEBUG_PAGEALLOC=n
PAGE_POISONING=y
PAGE_POISONING_NO_SANITY=y
PAGE_POISONING_ZERO=y
page_poison=on
This combo (in next-20160223) results in an unusable system. :(
[1.754183] rand
From: David Decotigny
Signed-off-by: David Decotigny
---
net/core/net-sysfs.c | 15 +--
net/packet/af_packet.c | 11 +--
2 files changed, 14 insertions(+), 12 deletions(-)
diff --git a/net/core/net-sysfs.c b/net/core/net-sysfs.c
index 4ae17c3..2b3f76f 100644
--- a/net/cor
From: David Decotigny
Signed-off-by: David Decotigny
---
drivers/net/ethernet/mellanox/mlx4/en_ethtool.c | 357 +---
drivers/net/ethernet/mellanox/mlx4/en_main.c| 1 +
drivers/net/ethernet/mellanox/mlx4/mlx4_en.h| 1 +
3 files changed, 189 insertions(+), 170 dele
From: David Decotigny
Signed-off-by: David Decotigny
---
net/8021q/vlan_dev.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/net/8021q/vlan_dev.c b/net/8021q/vlan_dev.c
index 055f0e9..e416a40 100644
--- a/net/8021q/vlan_dev.c
+++ b/net/8021q/vlan_dev.c
@@ -621,12 +6
From: David Decotigny
replaced by __ethtool_get_link_ksettings.
Signed-off-by: David Decotigny
---
include/linux/ethtool.h | 4
net/core/ethtool.c | 45 ++---
2 files changed, 14 insertions(+), 35 deletions(-)
diff --git a/include/linux/ethto
From: David Decotigny
Signed-off-by: David Decotigny
---
drivers/infiniband/hw/usnic/usnic_ib_verbs.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/infiniband/hw/usnic/usnic_ib_verbs.c
b/drivers/infiniband/hw/usnic/usnic_ib_verbs.c
index 6cdb4d2..ea003ec 100644
--- a/drivers/inf
From: David Decotigny
History:
v9
- add 'link' in macro, struct and function names
- rename ethtool_link_ksettings::parent -> ::base
- remove un-needed mlx4 en_dbg_enabled() companion patch
- note: bitmap u32[] API patches were merged separately by Kan Liang
v8
- bitmap u32 API returns nu
From: Jake Oshins
This patch fixes a race condition in this driver. Using the
function pci_domain_nr() only works if the PCI bus has already
been fully created. This patch just deletes one call site,
as it was in debug prints which aren't strictly necessary.
Another call site is changed to look
On Wed, Feb 24, 2016 at 8:36 AM, Andy Lutomirski wrote:
>>
>> So the EFI runtime crap should not change once it is mapped. And those
>> should be global. It is only natural.
>
> Why is it natural?
>
> Long-term, I'd rather see EFI runtime services use an actual mm_struct
> and use_mm.
Definitely.
On Wed, Feb 24, 2016 at 07:59:05AM -0800, Andi Kleen wrote:
> On Wed, Feb 24, 2016 at 09:46:55AM +0100, Jiri Olsa wrote:
> > Adding DATALA events into Haaswell events sysfs
> > directory, so it's easier to use them.
> >
> > Adding all DATALA events as specified by Intel SDM
> > manual 18.10.3 PEBS
On Wed, Feb 24, 2016 at 7:58 AM, Kirill A. Shutemov
wrote:
> Sebastian Ott and Gerald Schaefer reported random crashes on s390.
> It was bisected to my THP refcounting patchset.
>
> The problem is that pmdp_invalidated() called with wrong virtual
> address. It got offset up by HPAGE_PMD_SIZE by lo
On Wed, Feb 24, 2016 at 9:12 AM, Will Deacon wrote:
> On Wed, Feb 24, 2016 at 08:58:14AM -0800, Olof Johansson wrote:
>> On Tue, Feb 23, 2016 at 01:55:51PM +, Suzuki K. Poulose wrote:
>> > On 23/02/16 11:40, Will Deacon wrote:
>> > >On Tue, Feb 23, 2016 at 10:49:42AM +, Suzuki K Poulose wr
> -Original Message-
> From: linux-pci-ow...@vger.kernel.org [mailto:linux-pci-
> ow...@vger.kernel.org] On Behalf Of Zytaruk, Kelly
> Sent: Tuesday, February 23, 2016 12:47 PM
> To: Bjorn Helgaas
> Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org;
> bhelg...@google.com; Marsan
Allow users to have address fields displayed as symbols in the output
by appending '.sym' or 'sym-offset' to field names:
# echo hist:keys=aaa.sym,bbb.sym-offset ... \
[ if filter] > event/trigger
Signed-off-by: Tom Zanussi
Tested-by: Masami Hiramatsu
Reviewed-by: Namhyung Kim
Allow users to have common_pid field values displayed as program names
in the output by appending '.execname' to a common_pid field name:
# echo hist:keys=common_pid.execname ... \
[ if filter] > event/trigger
Signed-off-by: Tom Zanussi
Tested-by: Masami Hiramatsu
Reviewed-by:
This is v14 of the 'hist triggers' patchset, which incorporates
suggestions from Steve, and drops the first 8 from v13.
Patch 1 can be merged into patch 8 from v13, 'tracing: Add lock-free
tracing_map'. Following the v13 suggestions, I made a couple changes
to the tracing map constants to clarify
Allow users to have numeric fields displayed as hex values in the
output by appending '.hex' to field names:
# echo hist:keys=aaa,bbb.hex:vals=ccc.hex ... \
[ if filter] > event/trigger
Signed-off-by: Tom Zanussi
Tested-by: Masami Hiramatsu
Reviewed-by: Namhyung Kim
---
kerne
It's often useful to be able to use a stacktrace as a hash key, for
keeping a count of the number of times a particular call path resulted
in a trace event, for instance. Add a special key named 'stacktrace'
which can be used as key in a 'keys=' param for this purpose:
# echo hist:keys=stackt
From: Namhyung Kim
The string in a trace event is usually recorded as dynamic array which
is variable length. But current hist code only support fixed length
array so it cannot support most strings.
This patch fixes it by checking filter_type of the field and get
proper pointer with it. With t
Allow users to specify keys and/or values to sort on. With this
addition, keys and values specified using the 'keys=' and 'vals='
keywords can be used to sort the hist trigger output via a new 'sort='
keyword. If multiple sort keys are specified, the output will be
sorted using the second key as
Some syscalls are declared conditionally, so corresponding wrappers
are conditional accordingly.
Signed-off-by: Yury Norov
---
arch/s390/kernel/Makefile | 2 +-
arch/s390/kernel/compat_wrapper.c | 129 -
kernel/Makefile | 1 +
kernel/comp
Allow users to append 'clear' to an existing trigger in order to have
the hash table cleared.
This expands the hist trigger syntax from this:
# echo hist:keys=xxx:vals=yyy:sort=zzz.descending:pause/cont \
[ if filter] >> event/trigger
to this:
# echo hist:keys=xxx:vals=yyy:sor
As generic unistd syscall table is written in C, syscall
prototypes declaration is needed. It's added to compat header.
Signed-off-by: Yury Norov
---
include/linux/compat.h| 225 +
include/uapi/asm-generic/unistd.h | 227 +++
__SC_COMPAT_CAST for s390 is too specific due to 31-bit pointer length, so it's
moved to arch/s390/include/asm/compat.h. Generic declaration assumes that long,
unsigned long and pointer types are all 32-bit length.
linux/syscalls_structs.h header is introduced, because from now (see next patch)
st
Signed-off-by: Yury Norov
---
arch/Kconfig | 4
arch/s390/Kconfig | 1 +
2 files changed, 5 insertions(+)
diff --git a/arch/Kconfig b/arch/Kconfig
index f6b649d..6393093 100644
--- a/arch/Kconfig
+++ b/arch/Kconfig
@@ -583,6 +583,10 @@ config HAVE_COPY_THREAD_TLS
normal C par
Allow users to have syscall id fields displayed as syscall names in
the output by appending '.syscall' to field names:
# echo hist:keys=aaa.syscall ... \
[ if filter] > event/trigger
Signed-off-by: Tom Zanussi
Tested-by: Masami Hiramatsu
Reviewed-by: Namhyung Kim
---
kernel/t
On 2/24/2016 5:15 AM, Laxman Dewangan wrote:
Use devm_pinctrl_register() for pin control registration.
Signed-off-by: Laxman Dewangan
Cc: Ray Jui
Cc: Scott Branden
Cc: Jon Mason
---
drivers/pinctrl/bcm/pinctrl-nsp-gpio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/
Allow users to define any number of hist triggers per trace event.
Any number of hist triggers may be added for a given event, which may
differ by key, value, or filter.
Reading the event's 'hist' file will display the output of all the
hist triggers defined on an event concatenated in the order t
Allow users to define 'named' hist triggers. All triggers created
with the same 'name=xxx' option will update the same shared histogram
data.
This expands the hist trigger syntax from this:
# echo hist:keys=xxx ... [ if filter] > event/trigger
to this:
# echo hist:name=xxx:keys=xxx ...
On Feb 24, 2016 09:38, "Tony Luck" wrote:
>
> 2) But if we want to use this for copy_from_user() as part of the
> write(2) call stack (and I *do* want to do that)
I don't think that is even remotely an option.
If doing a rep movs can cause the chip to crash, we cannot possibly
allow user space t
Named triggers are sets of triggers that share a common set of trigger
data. An example of functionality that could benefit from this type
of capability would be a set of inlined probes that would each
contribute event counts, for example, to a shared counter data
structure.
The first named trigg
Signed-off-by: Yury Norov
---
Documentation/adding-syscalls.txt | 32
1 file changed, 32 insertions(+)
diff --git a/Documentation/adding-syscalls.txt
b/Documentation/adding-syscalls.txt
index cc2d4ac..1db880c 100644
--- a/Documentation/adding-syscalls.txt
+++ b/
The problem that makes us use wrappers is that some compat
architectures allows user code to access top halves of registers.
This is not a problem for syscalls that are already handled by compat
code, or for that who has types of the same size in kernel and
userspace. In case of s390 and lp64/ilp32
From: Namhyung Kim
Allow users to have numeric fields displayed as log2 values in case
value range is very wide by appending '.log2' to field names.
For example,
# echo 'hist:key=bytes_req' > kmalloc/trigger
# cat kmalloc/hist
{ bytes_req:504 } hitcount: 1
{ bytes_req:
Fengguang Wu writes:
>> This starts to sound more like something you would want to write in
>> the cover letter, or the trailer block next to Signed-off-by: at the
>> end of the first patch in the series.
>
> Yes, that's roughly what the current patch does, except in the latter
> case we add new
From: Masami Hiramatsu
This adds simple event trigger testcases for ftracetest,
which covers following triggers.
- traceon-traceoff trigger
- enable/disable_event trigger
- snapshot trigger
- stacktrace trigger
- trigger filters
Here is the test result.
# ./ftracetest test.d/trigg
From: Masami Hiramatsu
Add the hist trigger testcases for ftracetest.
This checks the basic histogram trigger behaviors like as;
- Histogram trigger itself
- Histogram with string key
- Histogram with compound keys
- Histogram with sort key
- Histogram trigger modifiers (execname, hex, sysca
Add documentation and usage examples for 'hist' triggers.
Signed-off-by: Tom Zanussi
Tested-by: Masami Hiramatsu
Reviewed-by: Namhyung Kim
---
Documentation/trace/events.txt | 1155
1 file changed, 1155 insertions(+)
diff --git a/Documentation/trace/ev
Similar to enable_event/disable_event triggers, these triggers enable
and disable the aggregation of events into maps rather than enabling
and disabling their writing into the trace buffer.
They can be used to automatically start and stop hist triggers based
on a matching filter condition.
If the
If we assume the maximum size for a string field, we don't have to
worry about its position. Since we only allow two keys in a compound
key and having more than one string key in a given compound key
doesn't make much sense anyway, trading a bit of extra space instead
of introducing an arbitrary r
Allow users to specify multiple trace event fields to use in keys by
allowing multiple fields in the 'keys=' keyword. With this addition,
any unique combination of any of the fields named in the 'keys'
keyword will result in a new entry being added to the hash table.
Signed-off-by: Tom Zanussi
T
Allow users to append 'pause' or 'continue' to an existing trigger in
order to have it paused or to have a paused trace continue.
This expands the hist trigger syntax from this:
# echo hist:keys=xxx:vals=yyy:sort=zzz.descending \
[ if filter] >> event/trigger
to this:
# echo hi
'hist' triggers allow users to continually aggregate trace events,
which can then be viewed afterwards by simply reading a 'hist' file
containing the aggregation in a human-readable format.
The basic idea is very simple and boils down to a mechanism whereby
trace events, rather than being exhausti
Allow users to specify trace event fields to use in aggregated sums
via a new 'vals=' keyword. Before this addition, the only aggregated
sum supported was the implied value 'hitcount'. With this addition,
'hitcount' is also supported as an explicit value field, as is any
numeric trace event field
Make it clear exactly how many keys and values are supported through
better defines, and add 1 to the vals count, since normally clients
want support for at least a hitcount and two other values.
Also, note the error return value for tracing_map_add_key/val_field()
in the comments.
Signed-off-by:
I've checked next next-20160218 tag and confirm that it works.
18.02.2016 19:03, Bjorn Helgaas пишет:
On Thu, Feb 18, 2016 at 08:13:41AM +0300, Мороз Олег wrote:
On 17.02.2016 21:36, Bjorn Helgaas wrote:
These are clean reverts of 991de2e59090 ("PCI, x86: Implement
pcibios_alloc_irq() and pcib
On 02/24/2016 03:05 AM, Jiri Slaby wrote:
> This is the start of the stable review cycle for the 3.12.55 release.
> There are 142 patches in this series, all will be posted as a response
> to this one. If anyone has any issues with these being applied, please
> let me know.
>
> Responses should b
Hi Laxman,
[auto build test ERROR on pinctrl/for-next]
[also build test ERROR on v4.5-rc5 next-20160224]
[if your patch is applied to the wrong git tree, please drop us a note to help
improving the system]
url:
https://github.com/0day-ci/linux/commits/Laxman-Dewangan/pinctrl-Add-and-use
On 2/24/2016 5:15 AM, Laxman Dewangan wrote:
Use devm_pinctrl_register() for pin control registration and clean
the error path.
Signed-off-by: Laxman Dewangan
Cc: Ray Jui
Cc: Scott Branden
Cc: Jon Mason
---
drivers/pinctrl/bcm/pinctrl-iproc-gpio.c | 12 ++--
1 file changed, 2 inse
On 02/23/2016 08:33 PM, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 3.10.98 release.
> There are 54 patches in this series, all will be posted as a response
> to this one. If anyone has any issues with these being applied, please
> let me know.
>
> Responses s
On Sat, Feb 20, 2016 at 11:52:09PM +0800, Leo Yan wrote:
> With profiling Hi6220's power modeling so get dynamic coefficient and
> sustainable power. So pass these parameters from DT.
>
> Now enable power allocator wit only one actor for CPU part, so directly
> use cluster0's thermal sensor for mo
On 02/23/2016 08:33 PM, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 3.14.62 release.
> There are 70 patches in this series, all will be posted as a response
> to this one. If anyone has any issues with these being applied, please
> let me know.
>
> Responses s
On 02/23/2016 08:32 PM, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.4.3 release.
> There are 137 patches in this series, all will be posted as a response
> to this one. If anyone has any issues with these being applied, please
> let me know.
>
> Responses sh
On 2/24/2016 5:15 AM, Laxman Dewangan wrote:
Use devm_pinctrl_register() for pin control registration.
Signed-off-by: Laxman Dewangan
Cc: Ray Jui
---
drivers/pinctrl/bcm/pinctrl-bcm2835.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/pinctrl/bcm/pinctrl-bcm28
On 02/23/2016 06:17 PM, Steven Rostedt wrote:
On Thu, 17 Dec 2015 12:01:52 -0500
Josef Bacik wrote:
I noticed while using the streaming infrastructure in trace-cmd that I was
seemingly missing events. Using other tracing methods I got these events and
record->missed_events was never being set
On 2/24/2016 5:15 AM, Laxman Dewangan wrote:
Use devm_pinctrl_register() for pin control registration.
Signed-off-by: Laxman Dewangan
Cc: Ray Jui
Cc: Scott Branden
Cc: Jon Mason
---
drivers/pinctrl/bcm/pinctrl-cygnus-mux.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On 2/23/2016 6:35 AM, Borislav Petkov wrote:
On Tue, Feb 16, 2016 at 03:45:11PM -0600, Aravind Gopalakrishnan wrote:
/*
+ * Set the error_count and interrupt_enable sysfs attributes here.
+ * This function gets called during the init phase and when someone
+ * makes changes to either of the
On 02/23/2016 05:26 PM, Steven Rostedt wrote:
On Tue, 15 Dec 2015 11:13:24 -0500
Josef Bacik wrote:
Sometimes we want to FIFO identical entries in the hash table, so add
trace_hash_find_reverse, which will cycle through the hash list in reverse.
This is helpful in our case where we are monitor
On 2/24/2016 5:15 AM, Laxman Dewangan wrote:
Use devm_pinctrl_register() for pin control registration.
Signed-off-by: Laxman Dewangan
Cc: Ray Jui
---
drivers/pinctrl/bcm/pinctrl-bcm281xx.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/pinctrl/bcm/pinctrl-bcm
On 24/02/16 18:10, Aleksey Makarov wrote:
From: Leif Lindholm
In order to support selecting earlycon via either ACPI or DT, move
the decision on whether to attempt ACPI configuration into the
early_param handling. Then make acpi_boot_table_init() bail out if
acpi_disabled.
Signed-off-by: Lei
On Wed 24 Feb 05:15 PST 2016, Laxman Dewangan wrote:
> Add device managed APIs devm_pinctrl_register() and
> devm_pinctrl_unregister() for the APIs pinctrl_register()
> and pinctrl_unregister().
>
> This helps in reducing code in error path and sometimes
> removal of .remove callback for driver u
Hi,
On Wed, Feb 24, 2016 at 09:33:13AM +, Yang, Wenyou wrote:
> > -Original Message-
> > From: Wenyou Yang [mailto:wenyou.y...@atmel.com]
> > Sent: 2016年2月15日 16:22
> > To: Sebastian Reichel ; Dmitry Eremin-Solenikov
> > ; David Woodhouse ; Rob
> > Herring ; Pawel Moll ; Mark Brown
> >
On Wed, 24 Feb 2016, Thomas Gleixner wrote:
On Wed, 10 Feb 2016, Vikas Shivappa wrote:
+static enum hrtimer_restart mbm_hrtimer_handle(struct hrtimer *hrtimer)
+{
+ if (list_empty(&cache_groups))
+ goto out;
+
+ list_for_each_entry(iter, &cache_groups, hw.cqm_groups_
On Wed, 24 Feb 2016, Thomas Gleixner wrote:
On Wed, 10 Feb 2016, Vikas Shivappa wrote:ar
+static int intel_mbm_init(void)
+{
+ int ret = 0, array_size, maxid = cqm_max_rmid + 1;
+
+ mbm_socket_max = cpumask_weight(&cqm_cpumask);
This should use the new topology_max_packages() fu
On Wed 24 Feb 05:15 PST 2016, Laxman Dewangan wrote:
> Use devm_pinctrl_register() for pin control registration and clean
> the error path.
>
> Signed-off-by: Laxman Dewangan
> Cc: Bjorn Andersson
> Cc: Thomas Gleixner
> Cc: Lee Jones
> Cc: Stephen Boyd
> ---
Reviewed-by: Bjorn Andersson
On Wed 24 Feb 05:15 PST 2016, Laxman Dewangan wrote:
> Use devm_pinctrl_register() for pin control registration and clean
> the error path.
>
> Signed-off-by: Laxman Dewangan
> Cc: Bjorn Andersson
> ---
Reviewed-by: Bjorn Andersson
Regards,
Bjorn
> drivers/pinctrl/qcom/pinctrl-ssbi-mpp.c |
On 02/24/2016 06:22 AM, Paolo Bonzini wrote:
On 15/02/2016 19:44, Laura Abbott wrote:
-static inline struct kmem_cache_node *free_debug_processing(
+static inline int free_debug_processing(
struct kmem_cache *s, struct page *page,
void *head, void *tail, int bulk_cnt,
u
On Wed 24 Feb 05:15 PST 2016, Laxman Dewangan wrote:
> Use devm_pinctrl_register() for pin control registration and clean
> the error path.
>
> Signed-off-by: Laxman Dewangan
> Cc: Bjorn Andersson
> ---
Reviewed-by: Bjorn Andersson
Regards,
Bjorn
> drivers/pinctrl/qcom/pinctrl-ssbi-gpio.c
On 02/24/2016 10:06 AM, Will Deacon wrote:
On Wed, Feb 24, 2016 at 10:03:02AM -0800, David Daney wrote:
On 02/24/2016 06:07 AM, Marc Zyngier wrote:
On 24/02/16 13:40, Will Deacon wrote:
[...]
(and Catalin, if you pick this up, watch out for the conflicts in
cpufeature.h)
Yup, that one is a
On 2/24/2016 5:37 AM, Borislav Petkov wrote:
On Tue, Feb 23, 2016 at 05:02:40PM -0600, Aravind Gopalakrishnan wrote:
On 2/23/16 6:11 AM, Borislav Petkov wrote:
On Tue, Feb 16, 2016 at 03:45:10PM -0600, Aravind Gopalakrishnan wrote:
/* AMD-specific bits */
-#define MCI_STATUS_DEFERRED(1UL
On Wed 24 Feb 05:15 PST 2016, Laxman Dewangan wrote:
> Use devm_pinctrl_register() for pin control registration and clean
> the error path.
>
> Signed-off-by: Laxman Dewangan
> Cc: Ivan T. Ivanov
> Cc: Lee Jones
> Cc: Stephen Boyd
> Cc: Jonas Gorski
Reviewed-by: Bjorn Andersson
Regards,
B
On Wed 24 Feb 05:15 PST 2016, Laxman Dewangan wrote:
> Use devm_pinctrl_register() for pin control registration and clean
> the error path.
>
> Signed-off-by: Laxman Dewangan
> Cc: "Ivan T. Ivanov"
> Cc: Lee Jones
> Cc: Stephen Boyd
> Cc: Jonas Gorski
> ---
Reviewed-by Bjorn Andersson
Reg
On Wed, Feb 24, 2016 at 10:03:02AM -0800, David Daney wrote:
> On 02/24/2016 06:07 AM, Marc Zyngier wrote:
> >On 24/02/16 13:40, Will Deacon wrote:
> >
> >[...]
> >
> >>(and Catalin, if you pick this up, watch out for the conflicts in
> >>cpufeature.h)
> >
> >Yup, that one is about to become a mine
On 02/24/2016 06:07 AM, Marc Zyngier wrote:
On 24/02/16 13:40, Will Deacon wrote:
[...]
(and Catalin, if you pick this up, watch out for the conflicts in
cpufeature.h)
Yup, that one is about to become a minefield (ARM64_HAS_NO_HW_PREFETCH,
ARM64_HAS_UAO, ARM64_ALT_PAN_NOT_UAO and ARM64_HAS_V
Ubsan reports the following warning due to a typo in
update_accessed_dirty_bits template, the patch fixes
the typo:
[ 168.791851]
[ 168.791862] UBSAN: Undefined behaviour in arch/x86/kvm/paging_tmpl.h:252:15
[ 168
On 2/24/2016 5:33 AM, Borislav Petkov wrote:
On Tue, Feb 23, 2016 at 04:56:38PM -0600, Aravind Gopalakrishnan wrote:
I think MSR_AMD64_SMCA_MC0_MISC0 would be required in mce.c as well.
So might be better to retain it here.
Actually, I'm thinking, these all are - even if used in multiple files
Signed-off-by: Adam Buchbinder
---
tools/perf/tests/openat-syscall-all-cpus.c | 2 +-
tools/perf/util/evsel.c| 2 +-
tools/perf/util/machine.c | 2 +-
tools/perf/util/parse-events.c | 2 +-
tools/perf/util/session.c | 2 +-
5 files
Signed-off-by: Adam Buchbinder
---
tools/testing/selftests/memfd/fuse_test.c | 2 +-
tools/testing/selftests/net/psock_tpacket.c | 2 +-
tools/testing/selftests/powerpc/tm/tm-signal-msr-resv.c | 2 +-
tools/testing/selftests/powerpc/vphn/test-vphn.c| 2 +-
4 file
On Wed, Feb 24, 2016 at 06:45:53PM +0530, Laxman Dewangan wrote:
> Use devm_pinctrl_register() for pin control registration and clean
> the error path.
>
> Signed-off-by: Laxman Dewangan
> Cc: Maxime Ripard
> Cc: Chen-Yu Tsai
Acked-by: Maxime Ripard
Thanks!
Maxime
--
Maxime Ripard, Free El
On 2/24/2016 5:28 AM, Borislav Petkov wrote:
On Tue, Feb 23, 2016 at 04:50:37PM -0600, Aravind Gopalakrishnan wrote:
Sorry about that. Looks like this pair is not defined in spelling.txt. So,
might be worth adding there as well?
Oh geez, we have a spelling.txt! I think we can declare the kernel
The following scenario:
CPU0 CPU1
ctx = find_get_ctx();
perf_event_exit_task_context()
mutex_lock(&ctx->mutex);
perf_install_in_context(ctx, ...);
/* NO-OP */
mutex_unlock(&ctx->mutex);
...
perf_release
Alexander reported that when the 'original' context gets destroyed, no
new clones happen.
This can happen irrespective of the ctx switch optimization, any task
can die, even the parent, and we want to continue monitoring the task
hierarchy until we either close the event or no tasks are left in th
In case of: err_file: fput(event_file), we'll end up calling
perf_release() which in turn will free the event.
Do not then free the event _again_.
Reviewed-by: Alexander Shishkin
Tested-by: Alexander Shishkin
Signed-off-by: Peter Zijlstra (Intel)
---
kernel/events/core.c |7 ++-
1 fil
The recent commit 3e349507d12d ("perf: Fix perf_enable_on_exec() event
scheduling") caused this by moving task_ctx_sched_out() from before
__perf_event_mask_enable() to after it.
The overlooked concequence of that change is that task_ctx_sched_out()
would update the ctx time fields, and now __perf
Because event_sched_out() checks event->pending_disable _before_
actually disabling the event, it can happen that the event fires after
it checks but before it gets disabled.
This would leave event->pending_disable set and the queued irq_work
will try and process it.
However, if the event trigger
Completely reworks perf_install_in_context() (again!) in order to
ensure that there will be no ctx time hole between add_event_to_ctx()
and any potential ctx_sched_in().
Signed-off-by: Peter Zijlstra (Intel)
---
kernel/events/core.c | 115 +++
1 f
Since there is no serialization between task_function_call() doing
task_curr() and the other CPU doing context switches, we could end
up not sending an IPI even if we had to.
And I'm not sure I still buy my own argument we're OK.
Signed-off-by: Peter Zijlstra (Intel)
---
kernel/events/core.c |
About half of these patches are syz-kaller inspired, the rest is reported
fallout from the last round of fixes.
With these patches syz-kaller can still trigger some fail; most notably some
NMI watchdog triggers and a very sporadic unthrottle bug (much like last time).
But stability is otherwise gr
Signed-off-by: Peter Zijlstra (Intel)
---
kernel/events/core.c | 18 --
1 file changed, 12 insertions(+), 6 deletions(-)
--- a/kernel/events/core.c
+++ b/kernel/events/core.c
@@ -2170,12 +2170,12 @@ perf_install_in_context(struct perf_even
raw_spin_unlock_irq(&c
Currently any ctx_sched_in() call will re-start the ctx time tracking,
this means that calls like:
ctx_sched_in(.event_type = EVENT_PINNED);
ctx_sched_in(.event_type = EVENT_FLEXIBLE);
will have a hole in their ctx time tracking. This is likely harmless
but can confuse things a li
In the err_file: fput(event_file) case, the event will not yet have
been attached to a context. However perf_release() does assume it has
been. Cure this.
Reviewed-by: Alexander Shishkin
Tested-by: Alexander Shishkin
Signed-off-by: Peter Zijlstra (Intel)
---
kernel/events/core.c | 16 +++
Similar to the perf_enable_on_exec(), ensure that event timings are
consistent across perf_event_enable().
Signed-off-by: Peter Zijlstra (Intel)
---
kernel/events/core.c | 42 +++---
1 file changed, 23 insertions(+), 19 deletions(-)
--- a/kernel/events/core
perf_install_in_context() relies upon the context switch hooks to have
scheduled in events when the IPI misses its target -- after all, if
the task has moved from the CPU (or wasn't running at all), it will
have to context switch to run elsewhere.
This however doesn't appear to be happening.
It i
On 02/24/2016 12:26 PM, Andrew Cooper wrote:
On 24/02/16 15:19, Boris Ostrovsky wrote:
+ /* Clear .bss */
+ xor REG(ax),REG(ax)
If we are nitpicking,
This should be xor %eax, %eax even in 64bit. Functionally identical,
and shorter to encode.
Right, Brian Gerst pointed this out t
On Wed, Feb 24, 2016 at 04:36:33PM +, Liang, Kan wrote:
> >
> > Thanks, that fixes it.
> >
> > Note: your patch appears to be against linux-next. I had to change
> > "arch/x86/events/intel/uncore_snbep.c" to
> > "arch/x86/kernel/cpu/perf_event_intel_uncore_snbep.c" for the patch to
> > apply
On 24/02/16 04:08, Mark Brown wrote:
On Tue, Feb 23, 2016 at 02:16:32PM +, Richard Fitzgerald wrote:
The debug logging of FLL calculations was confusing. Values were
printed in hex without indicating this by a leading 0x, and
despite these normally being required in decimal. Also where the
r
Group of probably overly rigorous whitespace and code cleanups.
- Alphabetize includes
- Assign to variables in the order they are defined
- Alignment issues
- Group alike statements together
- Use helper macros
Signed-off-by: Andrew F. Davis
---
Removed some unneeded changes in v1 at Jonath
> There are few things I see missing in this patch in your updates made to
> actbl1.h, I list them below:
>
> 1) Adding time stamp as you did breaks backwards compatibility (something
> we discovered during our testing). We tried to remedy this by adding a
> new version of the acpi_hest_generi
601 - 700 of 1339 matches
Mail list logo