Switch to updated coresight bindings for hw ports.
Cc: Sudeep Holla
Cc: Liviu Dudau
Cc: Mathieu Poirier
Signed-off-by: Suzuki K Poulose
---
arch/arm64/boot/dts/arm/juno-base.dtsi | 82 +-
arch/arm64/boot/dts/arm/juno.dts | 5 ++-
2 files changed, 63 inse
Coresight uses DT graph bindings to describe the connections of the
components. However we have some undocumented usage of the bindings
to describe some of the properties of the connections.
The coresight driver needs to know the hardware ports invovled
in the connection and the direction of data
When parsing the remote endpoint of an output port, we do :
rport = of_graph_get_remote_port(ep);
rparent = of_graph_get_remote_port_parent(ep);
and then parse the "remote_port" as if it was the remote endpoint,
which is wrong. The code worked fine because we used endpoint number
as the
Nobody uses the "clk" field in struct coresight_platform_data.
Remove it.
Cc: Mathieu Poirier
Signed-off-by: Suzuki K Poulose
---
include/linux/coresight.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/include/linux/coresight.h b/include/linux/coresight.h
index d950dad..32aaa1c 100644
--
So far we have relied on an undocumented property "slave-mode",
to indicate if the given port is input or not. Since we are
redefining the coresight bindings, define new property for the
"direction" of data flow for a given connection endpoint in the
device.
Each endpoint must define the following
On 06/01/2018 03:25 AM, Mark Brown wrote:
On Thu, May 24, 2018 at 12:49:21PM -0700, Guenter Roeck wrote:
From: Guenter Roeck
Commit dc31e741db49 ("ASoC: topology: ABI - Add the types for BE
DAI") introduced sound topology files version 5. Initially, this
change made the topology code incompati
Before we updat the bindings, document the current graph bindings
and usage of additional properties.
Cc: Mathieu Poirier
Signed-off-by: Suzuki K Poulose
---
.../devicetree/bindings/arm/coresight.txt | 28 ++
1 file changed, 24 insertions(+), 4 deletions(-)
diff --
If we fail to find the input / output port for a LINK component
while enabling a path, we should fail gracefully rather than
assuming port "0".
Cc: Mathieu Poirier
Signed-off-by: Suzuki K Poulose
---
drivers/hwtracing/coresight/coresight.c | 7 +--
1 file changed, 5 insertions(+), 2 deletio
The platform code parses the component connections and populates
a platform-description of the output connections in arrays of fields
(which is never freed). This is later copied in the coresight_register
to a newly allocated area, represented by coresight_connection(s).
This patch cleans up the c
The coresight drivers relied on default bindings for graph
in DT, while reusing the "reg" field of the "ports" to indicate
the actual hardware port number for the connections. However,
with the rules getting stricter w.r.t to the address mismatch
with the label, it is no longer possible to use the
From: Colin Ian King
The structure gsmi_dev is local to the source and does not need to be
in global scope, so make it static.
Cleans up sparse warning:
drivers/firmware/google/gsmi.c:96:3: warning: symbol 'gsmi_dev' was
not declared. Should it be static?
Signed-off-by: Colin Ian King
---
dri
On 01/06/18 16:11, sayali wrote:
> Hi Adrain,
>
> Updated my comments inline. Please check.
>
> Thanks,
> Sayali
> -Original Message-
> From: Adrian Hunter [mailto:adrian.hun...@intel.com]
> Sent: Friday, June 01, 2018 5:59 PM
> To: Sayali Lokhande ; subha...@codeaurora.org;
> c...@code
Em Fri, Jun 01, 2018 at 11:56:48AM +0200, Jiri Olsa escreveu:
> The compilation is broken if there's no slang support:
>
> builtin-annotate.c: In function ‘hists__find_annotations’:
> builtin-annotate.c:344:10: error: too many arguments to function
> ‘hist_entry__tui_annotate’
> key = h
Hi Stephen,
On 5/31/2018 1:11 PM, Stephen Boyd wrote:
> Quoting Sricharan R (2018-05-30 21:57:20)
>> Hi Stephen,
>>
>> On 5/30/2018 9:25 PM, Stephen Boyd wrote:
>>> Quoting Sricharan R (2018-05-24 22:40:11)
Hi Bjorn,
On 5/24/2018 11:09 PM, Bjorn Andersson wrote:
> On Tue 06 Mar
On Fri, Jun 01, 2018 at 02:57:53PM +0200, Mike Galbraith wrote:
> b956575bed91ecfb136a8300742ecbbf451471ab is the first bad commit
> commit b956575bed91ecfb136a8300742ecbbf451471ab
> Author: Andy Lutomirski
> Date: Mon Oct 9 09:50:49 2017 -0700
>
> x86/mm: Flush more aggressively in lazy TL
kill_ok_by_cred() already behaves like a boolean function. Let's actually
declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* unchanged
---
kernel/signal.c | 17 ++---
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/kernel/signal.
Hey,
This series refactors a bunch of functions in signal.c to simplify parts
of the code.
As requested, v2 drops all 3 commits that were pure coding-style
changes. The only other coding-style change is the simplification of
rt_sigaction which seemed to be worth keeping.
The greatest single change
make get_signal() already behaves like a boolean function. Let's actually
declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
include/linux/signal.h | 2 +-
kernel/signal.c| 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
d
recalc_sigpending_tsk() already behaves like a boolean function. Let's
actually declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/kernel/signal.c b/
has_pending_signals() already behaves like a boolean function. Let's
actually declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 25 ++---
1 file changed, 14 insertions(+), 11 deletions(-)
diff --git a
security_task_kill() already behaves like a boolean function. Let's
actually declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/signal.c b/kern
legacy_queue() already behaves like a boolean function. Let's actually
declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/signal.c b/kernel/signal.c
sig_ignored() already behaves like a boolean function. Let's actually
declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/signal.c b/kernel/sign
The return value of flush_sigqueue_mask() is never checked anywhere.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/kernel/signal.c b/kernel/signal.c
index a41af5a7749f..d
wants_signal() already behaves like a boolean function. Let's actually
declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 14 +-
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/kernel/signal.c b/k
unhandled_signal() already behaves like a boolean function. Let's actually
declare it as such too.
All callers treat it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
include/linux/signal.h | 2 +-
kernel/signal.c| 11 +++
2 files
Ah, that's it. Increased the limit on the mount and it works fine now.
Thanks!
- Alex.
On 01/06/18 14:05, Michal Hocko wrote:
On Fri 01-06-18 13:26:59, Alex Richman wrote:
I am using a shm MAP_SHARED, along these lines:
shm_fd = shm_open(handle, (O_RDWR | O_CREAT), (S_IRWXU | S_IRWXG |
S_IR
sig_handler_ignored() already behaves like a boolean function. Let's
actually declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/signal.c b/kerne
In case *vif* is NULL at 655: if (!vif), the execution path jumps to
label out, where *vif* is dereferenced at 679:
if (vif->sta_state == QTNF_STA_CONNECTING)
Fix this by immediately returning when *vif* is NULL instead of
jumping to label out.
Addresses-Coverity-ID: 1469567 ("Dereference after
sig_task_ignored() already behaves like a boolean function. Let's actually
declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* patch added
---
kernel/signal.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/signal.c b/kernel/s
force_sigsegv() returned 0 unconditionally so it doesn't make sense to have
it return at all. In addition, there are no callers that check
force_sigsegv()'s return value.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* unchanged
---
include/linux/sched/signal.h | 2 +-
kernel/
do_sigpending() returned 0 unconditionally so it doesn't make sense to have
it return at all. This allows us to simplify a bunch of syscall callers.
Signed-off-by: Christian Brauner
Acked-by: Al Viro
---
v1->v2:
* unchanged
v0->v1:
* unchanged
---
kernel/signal.c | 41 --
The goto is not needed and does not add any clarity. Simply return -EINVAL
on unexpected sigset_t struct size directly.
Signed-off-by: Christian Brauner
---
v1->v2:
* [Christoph Hellwig] additional cleanups
v0->v1:
* unchanged
---
kernel/signal.c | 22 ++
1 file changed, 10 i
kill_as_cred_perm() already behaves like a boolean function. Let's actually
declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* simplify to use a single return
---
kernel/signal.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --g
may_ptrace_stop() already behaves like a boolean function. Let's actually
declare it as such too.
Signed-off-by: Christian Brauner
---
v1->v2:
* unchanged
v0->v1:
* unchanged
---
kernel/signal.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/kernel/signal.c b/ke
On 5/31/2018 5:44 PM, Alex G. wrote:
> On 05/31/2018 10:30 AM, Sinan Kaya wrote:
>> On 5/31/2018 11:05 AM, Alexandru Gagniuc wrote:
>>> + if (dev_cur_speed < max_link_speed)
>>> + pci_warn(dev, "PCIe downtrain: link speed is %s (%s capable)",
>>> +pcie_bus_speed_name
From: Colin Ian King
The structure fmc_dbgfs_sdb_dump is local to the source and does not
need to be in global scope, so make it static.
Cleans up sparse warning:
drivers/fmc/fmc-debug.c:143:30: warning: symbol 'fmc_dbgfs_sdb_dump'
was not declared. Should it be static?
Signed-off-by: Colin Ian
Thanks Bjorn for the cleanup.
I validated PAS and Non- PAS ADSP PIL with the cleanup changes on SDM845.
Adding tag for ADSP PIL.
Reviewed-and-tested-by: Rohit kumar
On 5/23/2018 10:50 AM, Bjorn Andersson wrote:
With the introduction of support for the non-MSA Hexagon WCSS driver from
Srichar
On 01.06.2018 11:25, Boris Brezillon wrote:
> On Fri, 1 Jun 2018 09:26:00 +0200
> Boris Brezillon wrote:
>
>> On Fri, 1 Jun 2018 00:16:32 +0200
>> Stefan Agner wrote:
>>
>> > Add Reed-Solomon (RS) to the enumeration of ECC algorithms.
>> >
>> > Signed-off-by: Stefan Agner
>>
>> Reviewed-by: Bo
On 5/30/2018 9:08 PM, ok...@codeaurora.org wrote:
> I have seen that shutdown() is also called from remove().
>
> remove() is supposed to do a safe cleanup too. If it is leaving the hw in
> inconsistent state even though it is c lling shutdown , it is yet another bug.
Let's try to be constructiv
Hi Bjorn,
Reviewed and validated PAS and non-PAS ADSP PIL on sdm845 with the changes.
Adding tag.
On 5/23/2018 10:50 AM, Bjorn Andersson wrote:
Migrate the Hexagon V5 PAS (ADSP) driver to using the newly extracted
helper functions. The use of the handover callback does introduce latent
disabl
On Fri 01-06-18 14:24:23, Alex Richman wrote:
> Ah, that's it. Increased the limit on the mount and it works fine now.
Well, it is hidden very well. It is not until the page fault time (or
get_user_pages aka __mm_populate) when the operation fails and remasks
ENOSPC -> VM_FAULT_SIGBUS -> EFAULT -
On Fri, 01 Jun 2018 15:34:33 +0200
Stefan Agner wrote:
> On 01.06.2018 11:25, Boris Brezillon wrote:
> > On Fri, 1 Jun 2018 09:26:00 +0200
> > Boris Brezillon wrote:
> >
> >> On Fri, 1 Jun 2018 00:16:32 +0200
> >> Stefan Agner wrote:
> >>
> >> > Add Reed-Solomon (RS) to the enumeration of
From: Colin Ian King
The structure pcc_mbox_driver is local to the source and does not
need to be in global scope, so make it static.
Cleans up sparse warning:
drivers/mailbox/pcc.c:576:24: warning: symbol 'pcc_mbox_driver' was
not declared. Should it be static?
Signed-off-by: Colin Ian King
-
Kees Cook wrote:
> This is a stab at providing three new helpers for allocation size
> calculation:
>
> struct_size(), array_size(), and array3_size().
>
> These are implemented on top of Rasmus's overflow checking functions. The
> existing allocators are adjusted to use the more efficient overfl
On Fri, Jun 01, 2018 at 11:28:31AM +0200, Miklos Szeredi wrote:
> On Fri, Jun 1, 2018 at 11:24 AM, Stefan Hajnoczi wrote:
> > Ping?
> >
> > Archive link in case I broke email threading:
> > https://marc.info/?l=linux-fsdevel&m=152719324102009&w=2
>
> Thanks for the patch. Should already be fixed
Le Thursday 31 May 2018 à 15:02:04 (+0200), Vincent Guittot a écrit :
> On 31 May 2018 at 12:27, Patrick Bellasi wrote:
> >
> > Hi Vincent, Juri,
> >
> > On 28-May 18:34, Vincent Guittot wrote:
> >> On 28 May 2018 at 17:22, Juri Lelli wrote:
> >> > On 28/05/18 16:57, Vincent Guittot wrote:
> >> >
On Thu 31-05-18 20:07:28, Eric W. Biederman wrote:
> Michal Hocko writes:
>
> > On Thu 26-04-18 14:00:19, Kirill Tkhai wrote:
> >> This function searches for a new mm owner in children and siblings,
> >> and then iterates over all processes in the system in unlikely case.
> >> Despite the case is
On Thu 31-05-18 13:41:38, Eric W. Biederman wrote:
> Michal Hocko writes:
>
> > On Thu 24-05-18 14:16:35, Andrew Morton wrote:
> >> On Thu, 24 May 2018 13:10:02 +0200 Michal Hocko wrote:
> >>
> >> > I would really prefer and appreciate a repost with all the fixes folded
> >> > in.
> >>
> >> [1
Good morning Mike,
> This notion that only native NVMe multipath can be successful is utter
> bullshit. And the mere fact that I've gotten such a reaction from a
> select few speaks to some serious control issues.
Please stop making this personal.
> Imagine if XFS developers just one day impo
It can be useful to override 8250 mctrl lines with gpios, for rts on
rs485 for example, when rts is not mapped correctly to HW RTS pin.
Enable SERIAL_MCTRL_GPIO by default.
Signed-off-by: Giulio Benetti
---
drivers/tty/serial/8250/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/driv
Sometimes mctrl signals can be connected to pins different from HW ones.
User serial_mctrl_gpio helpers to align HW signals(RTS, CTS, etc.) with
gpios-rts, gpios-cts etc.
Signed-off-by: Giulio Benetti
---
drivers/tty/serial/8250/8250_core.c | 6 ++
drivers/tty/serial/8250/8250_port.c | 18
On Fri, Aug 11, 2017 at 2:44 AM, Dmitry Torokhov
wrote:
> According to Microsoft specification [1] for Precision Touchpads (and
> Touchscreens) the devices use "confidence" reports to signal accidental
> touches, or contacts that are "too large to be a finger". Instead of
> simply marking contact
* Faiz Abbas [180601 06:49]:
> Hi,
>
> On Thursday 31 May 2018 06:59 PM, Tero Kristo wrote:
> > On 31/05/18 13:14, Faiz Abbas wrote:
> >> Hi,
> >>
> >> On Thursday 31 May 2018 09:33 AM, Rob Herring wrote:
> >>> On Wed, May 30, 2018 at 07:41:30PM +0530, Faiz Abbas wrote:
> Add clkctrl data fo
On Fri, Jun 1, 2018 at 3:51 AM, Srinath Mannam
wrote:
> Hi Rob Herring,
>
> Thank you for the review.
> Please find my answers inline..
>
> On Thu, May 31, 2018 at 10:18 PM, Rob Herring wrote:
>> On Mon, May 28, 2018 at 11:11:22AM +0530, Srinath Mannam wrote:
>>> From: Pramod Kumar
>>>
>>> Add b
arch_validate_hwbkpt_settings() mixes up attribute check and commit into
a single code entity. Therefore the validation may return an error due to
incorrect atributes while still leaving halfway modified architecture
breakpoint data.
This is harmless when we deal with a new breakpoint but it becom
We can't pass the breakpoint directly on arch_check_bp_in_kernelspace()
anymore because its architecture internal datas (struct arch_hw_breakpoint)
are not yet filled by the time we call the function, and most
implementation need this backend to be up to date. So arrange the
function to take the pr
Remove the dance around old and new attributes. Just don't modify the
previous breakpoint at all until we have verified everything.
Reported-by: Linus Torvalds
Original-patch-by: Andy Lutomirski
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc:
Ingo,
Please pull the perf/breakpoint-v3 branch that can be found at:
git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git
perf/breakpoint-v3
HEAD: 3c0681cc01b39e7fac6eb810e82380165c9b56f1
Changes since v2:
* Constify the struct perf_event_attr passed in hw_breakpo
On June 1, 2018 5:08:32 AM PDT, Stephen Rothwell wrote:
>Hi Andy,
>
>On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko
> wrote:
>>
>> Oops. What is the proposed fix for that? It seems we can't rebase
>> published branches.
>
>It's unfixable without a rebase, so you could instead consider it an
This field seem to be unused, perhaps a leftover from old code...
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc: Rich Felker
Cc: Ingo Molnar
Cc: Thomas Gleixner
Cc: Will Deacon
Cc: Mark Rutland
Cc: Max Filippov
Cc: Chris Zankel
Cc: Catal
Migrate to the new API in order to remove arch_validate_hwbkpt_settings()
that clumsily mixes up architecture validation and commit.
Acked-by: Will Deacon
Acked-by: Mark Rutland
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc: Rich Felker
Cc:
We soon won't be able to rely on bp->attr anymore to get the new
type of the modifying breakpoint because the new attributes are going
to be copied only once we successfully modified the breakpoint slot.
This will fix the current misdesigned layout where the new attr are
copied to the modifying br
All architectures have implemented it, we can now remove the poor weak
version.
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc: Rich Felker
Cc: Ingo Molnar
Cc: Thomas Gleixner
Cc: Will Deacon
Cc: Mark Rutland
Cc: Max Filippov
Cc: Chris Zan
Migrate to the new API in order to remove arch_validate_hwbkpt_settings()
that clumsily mixes up architecture validation and commit
Acked-by: Michael Ellerman
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc: Rich Felker
Cc: Ingo Molnar
Cc: Tho
Michal Hocko writes:
> On Thu 31-05-18 20:07:28, Eric W. Biederman wrote:
>> Michal Hocko writes:
>>
>> > On Thu 26-04-18 14:00:19, Kirill Tkhai wrote:
>> >> This function searches for a new mm owner in children and siblings,
>> >> and then iterates over all processes in the system in unlikely
Migrate to the new API in order to remove arch_validate_hwbkpt_settings()
that clumsily mixes up architecture validation and commit.
Original-patch-by: Andy Lutomirski
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc: Rich Felker
Cc: Ingo Molnar
Migrate to the new API in order to remove arch_validate_hwbkpt_settings()
that clumsily mixes up architecture validation and commit
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc: Rich Felker
Cc: Ingo Molnar
Cc: Thomas Gleixner
Cc: Will Deaco
Migrate to the new API in order to remove arch_validate_hwbkpt_settings()
that clumsily mixes up architecture validation and commit.
Acked-by: Mark Rutland
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc: Rich Felker
Cc: Ingo Molnar
Cc: Thomas
Migrate to the new API in order to remove arch_validate_hwbkpt_settings()
that clumsily mixes up architecture validation and commit
Signed-off-by: Frederic Weisbecker
Cc: Linus Torvalds
Cc: Andy Lutomirski
Cc: Yoshinori Sato
Cc: Rich Felker
Cc: Ingo Molnar
Cc: Thomas Gleixner
Cc: Will Deaco
On Fri, Jun 1, 2018 at 3:22 AM, Johan Hovold wrote:
> Add binding for u-blox GNSS receivers.
>
> Note that the u-blox product names encodes form factor (e.g. "neo"),
> chipset (e.g. "8") and variant (e.g. "q"), but that only formfactor and
> chipset is used for the compatible strings (for now).
>
Hi Peter,
On 17 May 2018 at 22:12, Vincent Guittot wrote:
> Le Thursday 17 May 2018 à 11:32:06 (+0200), Peter Zijlstra a écrit :
>> On Thu, May 17, 2018 at 11:05:30AM +0200, Peter Zijlstra wrote:
>> > Hurm.. I think this is also wrong. See how dequeue_rt_stack() is also
>> > called from the enque
On June 1, 2018 5:08:32 AM PDT, Stephen Rothwell wrote:
>Hi Andy,
>
>On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko
> wrote:
>>
>> Oops. What is the proposed fix for that? It seems we can't rebase
>> published branches.
>
>It's unfixable without a rebase, so you could instead consider it an
On 01/06/18 12:24, Mark Rutland wrote:
> Now that all of the syscall logic works on the saved pt_regs, apply_ssbd
> can safely corrupt x0-x3 in the entry paths, and we no longer need to
> restore them. So let's remotve the logic doing so.
remove*
>
> With that logic gone, we can fold the branch
On Fri, Jun 1, 2018 at 3:08 PM, Stephen Rothwell wrote:
> Hi Andy,
>
> On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko
> wrote:
>>
>> Oops. What is the proposed fix for that? It seems we can't rebase
>> published branches.
>
> It's unfixable without a rebase, so you could instead consider it a
On Mon, May 28, 2018 at 09:31:07PM +1000, Michael Ellerman wrote:
> Frederic Weisbecker writes:
>
> > On Thu, May 24, 2018 at 12:01:52PM +1000, Michael Ellerman wrote:
> >> Frederic Weisbecker writes:
> >>
> >> > diff --git a/arch/powerpc/kernel/hw_breakpoint.c
> >> > b/arch/powerpc/kernel/hw_
This is a follow up to my last patch of switching from mm->owner to
mm->memgcg and as it might be significant enough to break userspace. I
am carefully not depending on this change.
Eric W. Biederman (2):
memcg: Ensure every task that uses an mm is in the same memory cgroup
memcgl:
>From a userspace perspective the cgroup of a mm is determined by which
the cgroup the task belongs too. As practically an mm can only belong to
a single memory cgroup having multiple tasks with the same mm in different
memory cgroups is not well defined.
Avoid the difficulties of dealing with
get_mem_cgroup_from_mm no longer needs to deal with offline memory cgroups.
mem_cgroup_can_attach no longer needs to worry about cgroup leaders
as all tasks of an mm will migrate together.
Signed-off-by: "Eric W. Biederman"
---
mm/memcontrol.c | 43 +--
From: Colin Ian King
Several helper functions are local to the source and do not
need to be in global scope, so make them static.
Cleans up sparse warnings:
symbol 'rtsx_pm_power_saving' was not declared. Should it be static?
symbol 'rtsx_set_l1off_sub_cfg_d0' was not declared. Should it be stat
On Fri, Jun 1, 2018 at 5:33 PM, wrote:
> On June 1, 2018 5:08:32 AM PDT, Stephen Rothwell
> wrote:
>>On Fri, 1 Jun 2018 14:40:35 +0300 Andy Shevchenko
>> wrote:
>>>
>>> Oops. What is the proposed fix for that? It seems we can't rebase
>>> published branches.
>>
>>It's unfixable without a rebase
Hi Giovanni,
On Fri, 2018-06-01 at 13:32 +0200, Giovanni Gherdovich wrote:
> On Thu, May 31, 2018 at 03:51:39PM -0700, Srinivas Pandruvada wrote:
> > v3
> > - Removed atomic bit operation as suggested.
> > - Added description of contention with user space.
> > - Removed hwp cache, boost utililty f
On Fri, 2018-06-01 at 14:01 +0200, Giovanni Gherdovich wrote:
> On Thu, May 31, 2018 at 03:51:43PM -0700, Srinivas Pandruvada wrote:
> > Enable HWP boost on Skylake server platform.
> >
> > Signed-off-by: Srinivas Pandruvada > .com>
> > ---
> > drivers/cpufreq/intel_pstate.c | 9 +
> > 1
Hi,
I was reading the AMD whitepaper on SSBD and noticed that they have added
two new bits in the 8000_0008 CPUID. EBX:
1) Bit[26] - similar to Intel's SSB_NO not needed anymore.
2) Bit[24] - use SPEC_CTRL MSR (0x48) instead of VIRT SPEC_CTRL MSR
(0xC001_011f).
See 124441_AMD64_SpeculativeS
The AMD document outlining the SSBD handling
124441_AMD64_SpeculativeStoreBypassDisable_Whitepaper_final.pdf
mentions that the CPUID 8000_0008.EBX[26] will mean that the
speculative store bypass disable is no longer needed.
A copy of this document is available at
https://bugzilla.kernel.org/sh
The AMD document outlining the SSBD handling
124441_AMD64_SpeculativeStoreBypassDisable_Whitepaper_final.pdf
mentions that if CPUID 8000_0008.EBX[24] is set we should be using
the SPEC_CTRL MSR (0x48) over the VIRT SPEC_CTRL MSR (0xC001_011f)
for speculative store bypass disable.
This in effect me
Both AMD and Intel can have SPEC CTRL MSR for SSBD.
However AMD also has two more other ways of doing it - which
are !SPEC_CTRL MSR ways.
Signed-off-by: Konrad Rzeszutek Wilk
---
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: Konrad Rzeszutek Wilk
Cc: Borislav Petkov
Cc: Davi
Greg Kroah-Hartman wrote:
> On Thu, May 31, 2018 at 08:56:52PM +, Nadav Amit wrote:
>> Nadav Amit wrote:
>>
>>> Nadav Amit wrote:
>>>
Ping. Please consider it for inclusion for rc4.
Nadav Amit wrote:
> From: Gil Kupfer
>
> The balloon.page field is use
PCIe downtraining happens when both the device and PCIe port are
capable of a larger bus width or higher speed than negotiated.
Downtraining might be indicative of other problems in the system, and
identifying this from userspace is neither intuitive, nor straigh
forward.
The easiest way to detect
On Fri 01-06-18 09:32:42, Eric W. Biederman wrote:
> Michal Hocko writes:
[...]
> > Group leader exiting early without tearing down the whole thread
> > group should be quite rare as well. No question that somebody might do
> > that on purpose though...
>
> The group leader exiting early is a com
From: Gil Kupfer
The balloon.page field is used for two different purposes if batching is
on or off. If batching is on, the field point to the page which is used
to communicate with with the hypervisor. If it is off, balloon.page
points to the page that is about to be (un)locked.
Unfortunately,
On 6/1/2018 11:01 AM, Alexandru Gagniuc wrote:
> + /* Multi-function PCIe share the same link/status. */
> + if (PCI_FUNC(dev->devfn) != 0)
> + return;
How about virtual functions?
--
Sinan Kaya
Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm
Technologies,
On 06/01/2018 10:03 AM, Sinan Kaya wrote:
> On 6/1/2018 11:01 AM, Alexandru Gagniuc wrote:
>> +/* Multi-function PCIe share the same link/status. */
>> +if (PCI_FUNC(dev->devfn) != 0)
>> +return;
>
> How about virtual functions?
I have almost no clue about those. Is your conce
On Fri, Jun 1, 2018 at 6:21 AM Peter Zijlstra wrote:
>
> On Fri, Jun 01, 2018 at 02:57:53PM +0200, Mike Galbraith wrote:
> > b956575bed91ecfb136a8300742ecbbf451471ab is the first bad commit
> > commit b956575bed91ecfb136a8300742ecbbf451471ab
> > Author: Andy Lutomirski
> > Date: Mon Oct 9 09:50
On 6/1/2018 11:06 AM, Alex G. wrote:
> On 06/01/2018 10:03 AM, Sinan Kaya wrote:
>> On 6/1/2018 11:01 AM, Alexandru Gagniuc wrote:
>>> + /* Multi-function PCIe share the same link/status. */
>>> + if (PCI_FUNC(dev->devfn) != 0)
>>> + return;
>>
>> How about virtual functions?
>
> I h
On Thu, May 31, 2018 at 8:04 AM, Andy Shevchenko
wrote:
> On Thu, May 31, 2018 at 12:49 AM, Arnd Bergmann wrote:
>
>> drivers/video/fbdev/omap2/omapfb/dss/hdmi5.c: In function 'hdmi_probe_of':
>> drivers/video/fbdev/omap2/omapfb/dss/hdmi5.c:584:2: error: implicit
>> declaration of function 'of_n
On Fri, Jun 1, 2018 at 5:28 AM Rik van Riel wrote:
>
> Song noticed switch_mm_irqs_off taking a lot of CPU time in recent
> kernels,using 2.4% of a 48 CPU system during a netperf to localhost run.
> Digging into the profile, we noticed that cpumask_clear_cpu and
> cpumask_set_cpu together take abo
On Fri, Jun 1, 2018 at 6:01 PM, Alexandru Gagniuc wrote:
> PCIe downtraining happens when both the device and PCIe port are
> capable of a larger bus width or higher speed than negotiated.
> Downtraining might be indicative of other problems in the system, and
> identifying this from userspace is
Hi,
On Mon, May 28, 2018 at 04:27:34PM +0530, Viresh Kumar wrote:
> On 28-05-18, 10:44, Maxime Ripard wrote:
> > Hi,
> >
> > On Fri, May 25, 2018 at 04:01:52PM +0530, Viresh Kumar wrote:
> > > The cooling device properties, like "#cooling-cells" and
> > > "dynamic-power-coefficient", should eithe
Adding a new flag that will read kpagecount for each PFN
and print out the number of time the page is mapped along
with the flags in the listing view.
Signed-off-by: Christian Hansen
---
tools/vm/page-types.c | 73 +--
1 file changed, 59 insertions
201 - 300 of 524 matches
Mail list logo