The pstore backend lock wasn't being used during pstore_unregister().
Add sanity check and locking.
Signed-off-by: Kees Cook
---
fs/pstore/platform.c | 18 --
1 file changed, 16 insertions(+), 2 deletions(-)
diff --git a/fs/pstore/platform.c b/fs/pstore/platform.c
index
Currently pstore can only have a single backend attached at a time, and it
tracks the active backend via "psinfo", under a lock. The locking for this
does not need to be a spinlock, and in order to avoid may_sleep() issues
during future changes to pstore_unregister(), switch to a mutex instead.
The pstorefs internal list lock doesn't need to be a spinlock and will
create problems when trying to access the list in the subsequent patch
that will walk the pstorefs records during pstore_unregister(). Change
this to a mutex to avoid may_sleep() warnings when unregistering devices.
Hi,
This fixes a long-standing problem[1] with pstore where the filesystem
view of backend records was not updated when the backend was unloaded
(in a modular build) through pstore_unregister(). This series is
mostly refactoring and improvements to the various locking semantics
around management
The pstore.update_ms value was being disabled during pstore_unregister(),
which would cause any prior value to go unnoticed on the next
pstore_register(). Instead, just let del_timer() stop the timer, which
was always sufficient. This additionally refactors the timer reset code
and allows the
On Wed, 6 May 2020 16:40:19 +0800
Po Liu wrote:
> } else if (matches(*argv, "base-time") == 0) {
> + NEXT_ARG();
> + if (get_u64(_time, *argv, 10)) {
> + invalidarg = "base-time";
> +
randconfig-b001-20200430
i386 randconfig-b002-20200430
x86_64 randconfig-b001-20200430
i386 randconfig-b003-20200430
x86_64 randconfig-b002-20200430
x86_64 randconfig-b003-20200430
i386 randconfig-b003-20200506
i386
The name "pstore_lock" sounds very global, but it is only supposed to be
used for managing changes to "psinfo", so rename it accordingly.
Signed-off-by: Kees Cook
---
fs/pstore/platform.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/fs/pstore/platform.c
On Wed, May 06, 2020 at 02:55:17PM +, Michael Kelley wrote:
[...]
> > Hv_pci_bus_exit() calls hv_send_resources_released() to release all child
> > resources.
> > These child resources were allocated in hv_send_resources_allocated().
> > Hv_send_resources_allocated() could fail in the
On Wed, 6 May 2020 07:41:51 -0700 "Paul E. McKenney" wrote:
> On Wed, May 06, 2020 at 02:59:26PM +0200, SeongJae Park wrote:
> > TL; DR: It was not kernel's fault, but the benchmark program.
> >
> > So, the problem is reproducible using the lebench[1] only. I carefully read
> > it's code
On 2020-05-05 16:12, Rob Herring wrote:
> On Mon, Apr 27, 2020 at 03:49:30PM +0300, amirmi...@gmail.com wrote:
>> From: Amir Mizinski
>>
>> Added a YAML schema to support tpm tis i2c related dt-bindings for the I2c
>> PTP based physical layer.
>>
>> This patch adds the documentation for
Hi Eizan,
Thank you for the patch.
On 6/5/20 7:49, Eizan Miyamoto wrote:
> These fields are not used and can be removed.
>
> Signed-off-by: ei...@chromium.org
You need to drop the above line, for this and the other patches.
> Reviewed-by: Enric Balletbo I Serra
> Signed-off-by: Eizan
Vivek Goyal writes:
> On Wed, Apr 29, 2020 at 11:36:29AM +0200, Vitaly Kuznetsov wrote:
>> Commit 9a6e7c39810e (""KVM: async_pf: Fix #DF due to inject "Page not
>> Present" and "Page Ready" exceptions simultaneously") added a protection
>> against 'page ready' notification coming before 'page
Hi,
I updated my old Sony Vaio a few days ago and discovered that a few Fn-keys on
my old Sony Vaio had stopped working.
The bisection points at the commit in the subject and reverting it makes
everything work again even with current
Linus' git.
Full bug report below. Please CC: me as I'm not
On Tue, 5 May 2020 14:31:44 +0200
Joerg Roedel wrote:
> On Mon, May 04, 2020 at 03:10:06PM -0400, Steven Rostedt wrote:
> > I'm fine with adding it to the tracing code (with that ridiculous
> > comment! ;-)
> >
> > I'll even tag is as stable, but again, it's uncertain what commit that it
> >
On 5/5/20 3:16 PM, Thomas Gleixner wrote:
From: Thomas Gleixner
While working on the entry consolidation I stumbled over the KVM async page
fault handler and kvm_async_pf_task_wait() in particular. It took me a
while to realize that the randomly sprinkled around rcu_irq_enter()/exit()
Add Kishon Vijay Abraham I as MAINTAINER for TI J721E SoC PCIe.
Signed-off-by: Kishon Vijay Abraham I
---
MAINTAINERS | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 2926327e4976..9d40e1318f7c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@
Add support for PCIe controller in J721E SoC. The controller uses the
Cadence PCIe core programmed by pcie-cadence*.c. The PCIe controller
will work in both host mode and device mode.
Some of the features of the controller are:
*) Supports both RC mode and EP mode
*) Supports MSI and MSI-X
Add J721E in pci_device_id table so that pci-epf-test can be used
for testing PCIe EP in J721E.
Signed-off-by: Kishon Vijay Abraham I
---
drivers/misc/pci_endpoint_test.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers/misc/pci_endpoint_test.c
Cadence driver uses "mem" memory resource to obtain the offset of
configuration space address region, memory space address region and
message space address region. The obtained offset is used to program
the Address Translation Unit (ATU). However certain platforms like TI's
J721E SoC require the
On Sat, May 02, 2020 at 12:09:13PM +0200, Takashi Iwai wrote:
> On Sat, 02 May 2020 09:27:31 +0200,
> Takashi Iwai wrote:
> >
> > On Sat, 02 May 2020 09:17:28 +0200,
> > Lukas Wunner wrote:
> > >
> > > On Sat, May 02, 2020 at 09:11:58AM +0200, Takashi Iwai wrote:
> > > > ---
Commit 1b79c5284439 ("PCI: cadence: Add host driver for Cadence PCIe
controller") in order to update Vendor ID, directly wrote to
PCI_VENDOR_ID register. However PCI_VENDOR_ID in root port configuration
space is read-only register and writing to it will have no effect.
Use local management
Add PCIe EP mode dt-bindings for TI's J721E SoC.
Signed-off-by: Kishon Vijay Abraham I
Reviewed-by: Rob Herring
---
.../bindings/pci/ti,j721e-pci-ep.yaml | 89 +++
1 file changed, 89 insertions(+)
create mode 100644
Add host mode dt-bindings for TI's J721E SoC.
Signed-off-by: Kishon Vijay Abraham I
Reviewed-by: Rob Herring
---
.../bindings/pci/ti,j721e-pci-host.yaml | 113 ++
1 file changed, 113 insertions(+)
create mode 100644
From: Alan Douglas
Implement ->set_msix() and ->get_msix() callback functions in order
to configure MSIX capability in the PCIe endpoint controller.
Add cdns_pcie_ep_send_msix_irq() to send MSIX interrupts to Host.
cdns_pcie_ep_send_msix_irq() gets the MSIX table address (virtual
address) from
Certain platforms like TI's J721E allows only 32-bit configuration
space access. In such cases pci_generic_config_read and
pci_generic_config_write cannot be used. Add support in Cadence core
to let pci_host_bridge have custom pci_ops.
Signed-off-by: Kishon Vijay Abraham I
---
Add cdns_pcie_ops to start link and verify link status. The registers
to start link and to check link status is in Platform specific PCIe
wrapper. Add support for platform specific drivers to add callback
functions for the PCIe Cadence core to start link and verify link status.
Signed-off-by:
Add support to use custom read and write accessors. Platforms that
don't support half word or byte access or any other constraint
while accessing registers can use this feature to populate custom
read and write accessors. These custom accessors are used for both
standard register access and
On Wed, 2020-05-06 at 16:51 +0300, Luciano Coelho wrote:
> On Tue, 2020-05-05 at 20:19 -0700, Joe Perches wrote:
> > On Wed, 2020-05-06 at 11:07 +0800, Samuel Zou wrote:
> > > This silences the following coccinelle warning:
> > >
> > > "WARNING: sum of probable bitmasks, consider |"
> >
> > I
commit bd22885aa188 ("PCI: cadence: Refactor driver to use as a core
library") while refactoring the Cadence PCIe driver to be used as
library, removed pm_runtime_get_sync() from cdns_pcie_ep_setup()
and cdns_pcie_host_setup() but missed to remove the corresponding
pm_runtime_put_sync() in the
Certain platforms like TI's J721E allow only 32-bit register accesses.
Add read and write accessors to perform only 32-bit accesses in order to
support platforms like TI's J721E.
Signed-off-by: Kishon Vijay Abraham I
---
drivers/pci/controller/cadence/pcie-cadence.c | 40 +++
TI's J721E SoC uses Cadence PCIe core to implement both RC mode
and EP mode.
The high level features are:
*) Supports Legacy, MSI and MSI-X interrupt
*) Supports upto GEN4 speed mode
*) Supports SR-IOV
*) Supports multiple physical function
*) Ability to route all transactions via SMMU
Add a macro for aligning down a pointer. This is useful to get an
aligned register address when a device allows only word access and
doesn't allow half word or byte access.
Signed-off-by: Kishon Vijay Abraham I
---
include/linux/kernel.h | 1 +
1 file changed, 1 insertion(+)
diff --git
> There is a spurious 'put_device()' in the remove function.
Do you find differences in the clean-up of system resources suspicious
between the implementations of the functions “puv3_i2c_remove” and
“puv3_i2c_probe”?
Regards,
Markus
Hi Vishal,
Thank you for the patch.
There are a few questions for Hans below.
On Wed, Apr 29, 2020 at 07:47:04PM +0530, Vishal Sagar wrote:
> The Xilinx UHD-SDI Rx subsystem soft IP is used to capture native SDI
> streams from SDI sources like SDI broadcast equipment like cameras and
> mixers.
On 06/05/2020 14:37, Juri Lelli wrote:
> On 06/05/20 12:54, Dietmar Eggemann wrote:
>> On 27/04/2020 10:37, Dietmar Eggemann wrote:
[...]
>> There is an issue w/ excl. cpusets and cpuset.sched_load_balance=0. The
>> latter is needed to demonstrate the problem since DL task affinity can't
>> be
> > Hi Oleksij
> >
> > reg is normally 0 to 31, since that is the address range for MDIO.
> > Did you use 14 here because of what strapping allows?
>
> Yes. Only BITs 1:3 are configurable. BIT(0) is always 0 for the PHY0 and 1
> for the PHY1
O.K. good.
> > > +required:
> > > + - compatible
>
From: Aishwarya Ramakrishnan
On Tue, May 5, 2020 at 11:49 PM Greg Kroah-Hartman
wrote:
> On Tue, May 05, 2020 at 09:03:24PM +0530, Aishwarya Ramakrishnan wrote:
>> It is more clear to use DEFINE_DEBUGFS_ATTRIBUTE to define debugfs file
>> operation rather than DEFINE_SIMPLE_ATTRIBUTE.
> No it
On Tue, May 05, 2020 at 04:09:16PM +0100, Will Deacon wrote:
> On Tue, May 05, 2020 at 03:15:29PM +0100, Daniel Thompson wrote:
> > On Mon, May 04, 2020 at 09:48:04PM +0100, Will Deacon wrote:
> > > On Mon, May 04, 2020 at 06:05:18PM +0100, Daniel Thompson wrote:
> > > > diff --git
On Tue, May 05, 2020 at 08:35:06AM +0200, Oleksij Rempel wrote:
> The TJA11xx PHYs have a vendor specific Master/Slave configuration bit,
> which is not compatible with IEEE 803.2-2018 spec for 100Base-T1
> devices. So, provide a custom config_ange call back to solve this
> problem.
>
>
On Tue, May 05, 2020 at 08:35:05AM +0200, Oleksij Rempel wrote:
> This UAPI is needed for BroadR-Reach 100BASE-T1 devices. Due to lack of
> auto-negotiation support, we needed to be able to configure the
> MASTER-SLAVE role of the port manually or from an application in user
> space.
>
> The same
On 05/06, Chao Yu wrote:
> In f2fs_try_to_free_nids(), .nid_list_lock spinlock critical region will
> increase as expected shrink number increase, to avoid spining other CPUs
> for long time, it's better to implement like extent cache and nats
> shrinker.
>
> Signed-off-by: Chao Yu
> ---
> v2:
>
On Wed 06 May 2020 at 16:54, Philipp Zabel wrote:
> Hi Jerome,
>
> On Wed, 2020-05-06 at 15:50 +0200, Jerome Brunet wrote:
>> On Tue 14 Apr 2020 at 10:28, Jerome Brunet wrote:
>>
>> > On Thu 23 Jan 2020 at 11:13, Philipp Zabel wrote:
>> >
>> > > On Wed, 2020-01-22 at 10:25 +0100, Jerome
Alexandre Chartre writes:
> On 5/5/20 3:16 PM, Thomas Gleixner wrote:
>> @@ -10009,12 +10009,11 @@ static void kick_ilb(unsigned int flags)
>> return;
>>
>> /*
>> - * Use smp_send_reschedule() instead of resched_cpu().
>> - * This way we generate a sched IPI on the
+1
--mark
On Wed, May 06, 2020 at 09:15:14AM +0200, Borislav Petkov wrote:
> From: Mark Gross
>
> Intel uses the same family/model for several CPUs. Sometimes the
> stepping must be checked to tell them apart.
>
> On x86 there can be at most 16 steppings. Add a steppings bitmask to
>
Reviewed-by: Mark Gross
On Wed, May 06, 2020 at 09:15:15AM +0200, Borislav Petkov wrote:
> From: Borislav Petkov
>
> ... to match Intel family 6 CPUs with steppings.
>
> Signed-off-by: Borislav Petkov
> ---
> arch/x86/include/asm/cpu_device_id.h | 4
> 1 file changed, 4 insertions(+)
Reviewed-by: Mark Gross
On Wed, May 06, 2020 at 09:15:16AM +0200, Borislav Petkov wrote:
> From: Borislav Petkov
>
> ... and get rid of the function pointers which would spit out the
> microcode revision based on the CPU stepping.
>
> Signed-off-by: Borislav Petkov
> Cc: Peter Zijlstra
Kees Cook writes:
> On Tue, May 05, 2020 at 02:45:33PM -0500, Eric W. Biederman wrote:
>>
>> The current idiom for the callers is:
>>
>> flush_old_exec(bprm);
>> set_personality(...);
>> setup_new_exec(bprm);
>>
>> In 2010 Linus split flush_old_exec into flush_old_exec and
>> setup_new_exec.
Hi,
> From: Thanu Rangarajan
> Sent: Wednesday, May 6, 2020 1:58 PM
>
> Hi,
> ACPI CPPC already supports the notion of boost. Not sure we need any
> enhancements there.
>
> Regards,
> Thanu
>
> On 06/05/2020, 18:19, "Sudeep Holla" wrote:
>
> + Thanu, Souvik who work with ASWG
>
> On
On 05/06, Chao Yu wrote:
> On 2020/5/6 7:05, Jaegeuk Kim wrote:
> > On 05/05, Chao Yu wrote:
> >> On 2020-5-4 22:30, Jaegeuk Kim wrote:
> >>> From: Daeho Jeong
> >>>
> >>> Current zstd compression buffer size is one page and header size less
> >>> than cluster size. By this, zstd compression
Introduce the concept of a shared PHY storage which can be used by some
QSGMII PHYs to ease initialization and access to global per-package
registers.
Changes since v2:
- restore page to standard after reading the base address in the mscc
driver, thanks Antoine.
Changes since v1:
- fix
Use the new phy_package_shared common storage to ease the package
initialization and to access the global registers.
Signed-off-by: Michael Walle
Tested-by: Vladimir Oltean
Reviewed-by: Andrew Lunn
Reviewed-by: Florian Fainelli
---
drivers/net/phy/mscc/mscc.h | 1 -
Hi Jerome,
On Wed, 2020-05-06 at 15:50 +0200, Jerome Brunet wrote:
> On Tue 14 Apr 2020 at 10:28, Jerome Brunet wrote:
>
> > On Thu 23 Jan 2020 at 11:13, Philipp Zabel wrote:
> >
> > > On Wed, 2020-01-22 at 10:25 +0100, Jerome Brunet wrote:
> > > > Add the reset line of the internal DAC found
Use the new phy_package_shared common storage to ease the package
initialization and to access the global registers.
Signed-off-by: Michael Walle
Reviewed-by: Andrew Lunn
Reviewed-by: Florian Fainelli
---
drivers/net/phy/bcm54140.c | 57 --
1 file changed,
There are packages which contain multiple PHY devices, eg. a quad PHY
transceiver. Provide functions to allocate and free shared storage.
Usually, a quad PHY contains global registers, which don't belong to any
PHY. Provide convenience functions to access these registers.
Signed-off-by: Michael
From: Wei Hu Sent: Wednesday, May 6, 2020 6:22 AM
> > -Original Message-
> > From: Lorenzo Pieralisi
> > Sent: Wednesday, May 6, 2020 7:10 PM
> > To: Wei Hu
> > Cc: KY Srinivasan ; Haiyang Zhang
> > ; Stephen Hemminger ;
> > wei@kernel.org; r...@kernel.org; bhelg...@google.com;
On Wed, 6 May 2020 14:19:11 +0800, Jason Yan wrote:
> Fix the following coccicheck warning:
>
> drivers/spi/spi-armada-3700.c:283:8-11: Unneeded variable: "ret". Return
> "0" on line 315
>
> Signed-off-by: Jason Yan
>
> [...]
Applied to
Hello,
On my Lenovo T460p I cannot shutdown and reboot when the iommu is
enabled. This is using linux 5.2.7 as provided by Debian, 5.6.4 has the
same problem. Suspend/resume also fails; I suspect this is the same
issue.
When requesting power off the kernel messages just end with:
sd
Hi,
I discovered a bug while preparing my next series, which also made me
realize I had never tested the extended mode feature of netconsole. :-/
The only other user of extended output is /dev/kmsg, and it is doing it
correctly.
Explanation and patch below.
On 2020-05-01, John Ogness wrote:
>
Hey Marco,
On Tue, 5 May 2020 at 12:17, Marco Felsch wrote:
>
> Hi Robert,
>
> On 20-05-05 12:01, Robert Foss wrote:
> > Query the sensor for its module revision, and compare it
> > to known revisions.
> >
> > Currently 2A and 1B revision indentification is supported.
> >
> > Signed-off-by:
On Wed 06 May 2020 at 16:12, Sameer Pujar wrote:
>>>
>>> | Front End PCMs | SoC DSP | Back End DAIs|
>>>
>>> *
>>> ADMAIF<0> <> * * -> I2S
>>> * *
>>> ADMAIF<1>
HI, all
When reviewing function mei_me_cl_rm_by_uuid, I notice that function
__mei_me_cl_by_uuid increases me_cl refcount
and function _mei_me_cl_del delete a list node with decrement me_cl
refcount , actually here cause a refcount leak as we haven't release
the refcount
by __mei_me_cl_by_uuid ?
* Pavel Machek [200506 10:12]:
> Hi!
>
> So... I found out that USB networking works way better when I don't
> attempt to charge the phone at the same. Yes, green light was
> blinking.
OK yes we don't have much of a charger detection currently and the
charger tries to reconnect with the LED
On Tue, 5 May 2020 at 12:16, Marco Felsch wrote:
>
> Hi Robert,
>
> On 20-05-05 12:01, Robert Foss wrote:
> > Add match table, enable ov8856_probe() to support
> > both ACPI and DT modes.
> >
> > ACPI and DT modes are primarily distinguished from
> > by checking for ACPI mode and by having
On Tue, May 5, 2020 at 5:59 PM Kees Cook wrote:
>
> On Tue, May 05, 2020 at 11:45:07AM -0400, Pavel Tatashin wrote:
> > Add a new field to pstore_info that passes information about kmesg dump
> > maximum reason.
> >
> > This allows a finer control of what kmesg dumps are stored on pstore
> >
From: John Mathew
Add documentation for
-scheduler overview
-scheduler state transtion
-CFS overview
-scheduler data structs
Add rst for scheduler APIs and modify sched/core.c
to add kernel-doc comments.
Suggested-by: Lukas Bulwahn
Co-developed-by: Mostafa Chamanara
Signed-off-by:
Hey Amit,
Thanks for taking time to review
the series!
On 2020-05-06 18:08, Amit Kucheria wrote:
On Tue, May 5, 2020 at 1:54 AM Sibi Sankar
wrote:
This patch series aims to extend cpu based scaling support to L3/DDR
on
SDM845 and SC7180 SoCs.
Patches [1-3] - Blacklist SDM845 and SC7180 in
Hi Paul,
Cheers for the quick reply!
On Wed, May 06, 2020 at 07:36:16AM -0700, Paul E. McKenney wrote:
> On Wed, May 06, 2020 at 02:28:17PM +0100, Will Deacon wrote:
> > I'm looking to rebase my READ_ONCE() series [1] on top of the KCSAN patches
> > so that we can get them in for 5.8. However,
From: John Mathew
Add documentation for introduction to
-context-switch
-x86 context-switch
-MIPS context switch
Suggested-by: Lukas Bulwahn
Co-developed-by: Mostafa Chamanara
Signed-off-by: Mostafa Chamanara
Co-developed-by: Oleg Tsymbal
Signed-off-by: Oleg Tsymbal
Signed-off-by: John
On Wed, May 06, 2020 at 04:47:30PM +0800, Huacai Chen wrote:
> > For the above reasons, I think what you are concerned is not a
> > big deal.
> I don't think so, this is obviously a regression. If we can accept a
> regression of RS780E, why we still maintain Loongson-2EF rather than
> simply drop
On Wed, May 06, 2020 at 02:59:26PM +0200, SeongJae Park wrote:
> TL; DR: It was not kernel's fault, but the benchmark program.
>
> So, the problem is reproducible using the lebench[1] only. I carefully read
> it's code again.
>
> Before running the problem occurred "poll big" sub test, lebench
From: John Mathew
Add new sections to enable addition of new documentation on
the scheduler. Existing documentation is moved under the related
new sections. The sections are
- overview
- sched-features
- arch-specific.rst
- sched-debugging.rst
Suggested-by: Lukas Bulwahn
Signed-off-by:
On Mon, May 04, 2020 at 06:54:57PM +0300, Konstantin Khlebnikov wrote:
> For some reason NOWAIT currently is passed only for writes.
>
> Signed-off-by: Konstantin Khlebnikov
> Fixes: 03a07c92a9ed ("block: return on congested block device")
Looks good,
Reviewed-by: Christoph Hellwig
On Mon, May 04, 2020 at 06:54:53PM +0300, Konstantin Khlebnikov wrote:
> This is required to avoid waiting in lower layers.
>
> Signed-off-by: Konstantin Khlebnikov
Looks good,
Reviewed-by: Christoph Hellwig
On Mon, May 04, 2020 at 07:23:50PM +0300, Konstantin Khlebnikov wrote:
> On 04/05/2020 19.00, Christoph Hellwig wrote:
> > On Mon, May 04, 2020 at 06:54:53PM +0300, Konstantin Khlebnikov wrote:
> > > This is required to avoid waiting in lower layers.
> > >
> > > Signed-off-by: Konstantin
This patch series updates the scheduler documentation to add more topics
wrt to scheduler overview. New sections are added to provide a brief
overview of the kernel structs used by the scheduler, scheduler invocation
and context switch. First version of this patch series was reviewed at
Link:
On Wed, May 6, 2020 at 1:45 PM Florian Fainelli wrote:
>
> Hi Masahiro, Michal,
>
> While updating our systems from 4.9 to 5.4, we noticed that one of the
> kernel modules that we build, which is done by linking an object that we
> pre-compile out of Kbuild stopped working.
>
> I bisected it down
On Wed, May 06, 2020 at 02:28:17PM +0100, Will Deacon wrote:
> Hi TIP folks,
>
> I'm looking to rebase my READ_ONCE() series [1] on top of the KCSAN patches
> so that we can get them in for 5.8. However, tip/locking/kcsan seems to be
> missing some bits:
>
> * An update to checkpatch.pl to
From: Sarthak Garg
Consider the following stack trace
-001|raw_spin_lock_irqsave
-002|mmc_blk_cqe_complete_rq
-003|__blk_mq_complete_request(inline)
-003|blk_mq_complete_request(rq)
-004|mmc_cqe_timed_out(inline)
-004|mmc_mq_timed_out
mmc_mq_timed_out acquires the queue_lock for the first
Fixes for a couple of issues observed with CQE. One with CQE completion
path and the other one is with CQE recovery path.
Sarthak Garg (1):
mmc: core: Fix recursive locking issue in CQE recovery path
Veerabhadrarao Badiganti (1):
mmc: core: Check request type before completing the request
Sorry for being verbose; I've been procrastinating replying, and in
doing so the things I wanted to say kept growing.
On Fri, Apr 24, 2020 at 10:24:43PM +0800, Aaron Lu wrote:
> To make this work, the root level sched entities' vruntime of the two
> threads must be directly comparable. So one
In the request completion path with CQE, request type is being checked
after the request is getting completed. This is resulting in returning
the wrong request type and leading to the IO hang issue.
ASYNC request type is getting returned for DCMD type requests.
Because of this mismatch,
If the function platform_get_irq() failed, the negative value
returned will not be detected here. So fix error handling in
mt6797_afe_pcm_dev_probe(). And when get irq failed, the function
platform_get_irq() logs an error message, so remove redundant
message here.
Signed-off-by: Zhang Shengju
On Wed, May 6, 2020 at 5:59 AM SeongJae Park wrote:
>
> TL; DR: It was not kernel's fault, but the benchmark program.
>
> So, the problem is reproducible using the lebench[1] only. I carefully read
> it's code again.
>
> Before running the problem occurred "poll big" sub test, lebench executes
>
On Wed, May 6, 2020 at 9:52 AM Steven Rostedt wrote:
>
> On Tue, 5 May 2020 14:59:37 -0700
> Kees Cook wrote:
>
> > > @@ -97,6 +97,8 @@ struct pstore_record {
> > > * @read_mutex:serializes @open, @read, @close, and @erase callbacks
> > > * @flags: bitfield of frontends the backend can
From: "Steven Rostedt (VMware)"
Running on a slower machine, it is possible that the preempt delay kernel
thread may still be executing if the module was immediately removed after
added, and this can cause the kernel to crash as the kernel thread might be
executing after its code has been
Since the commit 6a13a0d7b4d1 ("ftrace/kprobe: Show the
maxactive number on kprobe_events") introduced to show the
instance number of kretprobe events, the length of the 1st
format of the kprobe event will not 1, but it can be longer.
This caused a parser error in perf-probe.
Skip the length
Matthew Wilcox wrote:
> > Won't that screw up ITER_MAPPING? Does that mean that ITER_MAPPING isn't
> > viable?
>
> Can you remind me why ITER_MAPPING needs:
>
> "The caller must guarantee that the pages are all present and they must be
> locked using PG_locked, PG_writeback or PG_fscache to
On Wed, May 6, 2020 at 4:44 AM Nathan Chancellor
wrote:
>
> On Tue, May 05, 2020 at 04:19:17PM +0200, Arnd Bergmann wrote:
> > clang points out that doing arithmetic between diffent enums is usually
> ^ different
> > a mistake:
> >
> >
Typo in $SUBJECT, should be "add_memory_driver_managed" ...
--
Thanks,
David / dhildenb
Fantastic. Thanks!
Reviewed-by: Dan Carpenter
regards,
dan carpenter
With the release of Linux 5.1 has been added a new syscall,
clock_gettime64, that provided a 64 bit time value for a specified
clock_ID to make the kernel Y2038 safe on 32 bit architectures.
Extend the vdso correctness test to cover the newly exposed vdso
function.
Cc: Shuah Khan
Signed-off-by:
sched/fair: Fix enqueue_task_fair warning some more
The recent patch, fe61468b2cb (sched/fair: Fix enqueue_task_fair warning)
did not fully resolve the issues with the (rq->tmp_alone_branch !=
>leaf_cfs_rq_list) warning in enqueue_task_fair. There is a case where
the first for_each_sched_entity
The current version of the multiarch vDSO selftest verifies only
gettimeofday.
Extend the vDSO selftest to clock_getres, to verify that the
syscall and the vDSO library function return the same information.
The extension has been used to verify the hrtimer_resoltion fix.
Cc: Shuah Khan
Currently the vDSO tests are built only on x86 platforms and cannot be
cross compiled.
Enable vDSO TARGET for all the platforms.
Future patches will extend the tests.
Cc: Shuah Khan
Signed-off-by: Vincenzo Frascino
---
tools/testing/selftests/Makefile | 1 +
Move test_vdso from x86 to the vDSO test suite.
Suggested-by: Andy Lutomirski
Cc: Shuah Khan
Signed-off-by: Vincenzo Frascino
---
tools/testing/selftests/vDSO/Makefile | 10 --
.../{x86/test_vdso.c => vDSO/vdso_correctness_test.c} | 0
This series extends the kselftests for the vDSO library making sure: that
they compile correctly on non x86 platforms, that they can be cross
compiled and introducing new tests that verify the correctness of the
library.
The so extended vDSO kselftests have been verified on all the platforms
On May 6, 2020 7:03:52 AM PDT, Jason Yan wrote:
>The '==' expression itself is bool, no need to convert it to bool
>again.
>This fixes the following coccicheck warning:
>
>arch/x86/net/bpf_jit_comp32.c:1478:50-55: WARNING: conversion to bool
>not needed here
The current version of the multiarch vDSO selftest verifies only
gettimeofday.
Extend the vDSO selftest to the other library functions:
- time
- clock_getres
- clock_gettime
The extension has been used to verify the unified vdso library on the
supported architectures.
Cc: Shuah Khan
On Wed, 6 May 2020 14:17:26 +0800, Jason Yan wrote:
> Fix the following coccicheck warning:
>
> drivers/regulator/db8500-prcmu.c:184:1-17: WARNING: Assignment of 0/1 to
> bool variable
>
> Signed-off-by: Jason Yan
>
> [...]
Applied to
901 - 1000 of 1543 matches
Mail list logo