Message ID | 20190322154425.3852517-1-arnd@arndb.de |
---|---|
Headers | show |
Series | candidates from spreadtrum 4.4 product kernel | expand |
On Fri, Mar 22, 2019 at 04:44:10PM +0100, Arnd Bergmann wrote: > From: Wei Qiao <wei.qiao@spreadtrum.com> > > SPRD_TIMEOUT was 256, which is too small to wait until the status > switched to workable in a while loop, so that the earlycon could > not work correctly. > > Signed-off-by: Wei Qiao <wei.qiao@spreadtrum.com> > Signed-off-by: Chunyan Zhang <chunyan.zhang@spreadtrum.com> > Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > (cherry picked from commit e1dc9b08051a2c2e694edf48d1e704f07c7c143c) > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > --- > drivers/tty/serial/sprd_serial.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Also applied to 4.9.y
On Fri, Mar 22, 2019 at 04:44:11PM +0100, Arnd Bergmann wrote: > From: Al Viro <viro@ZenIV.linux.org.uk> > > It's not hard to trigger a bunch of d_invalidate() on the same > dentry in parallel. They end up fighting each other - any > dentry picked for removal by one will be skipped by the rest > and we'll go for the next iteration through the entire > subtree, even if everything is being skipped. Morevoer, we > immediately go back to scanning the subtree. The only thing > we really need is to dissolve all mounts in the subtree and > as soon as we've nothing left to do, we can just unhash the > dentry and bugger off. > > Signed-off-by: Al Viro <viro@zeniv.linux.org.uk> > (cherry picked from commit 81be24d263dbeddaba35827036d6f6787a59c2c3) > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > --- > fs/dcache.c | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) Also added to 4.9.y
On Tue, 26 Mar 2019, Greg KH wrote: > On Fri, Mar 22, 2019 at 04:43:52PM +0100, Arnd Bergmann wrote: > > From: Julia Lawall <Julia.Lawall@lip6.fr> > > > > The mmc_pwrseq_ops structures are never modified, so declare them as const. > > > > Done with the help of Coccinelle. > > > > Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr> > > Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> > > (cherry picked from commit ffedbd2210f2f4cba490a9205adc11fd1b89a852) > > Signed-off-by: Arnd Bergmann <arnd@arndb.de> > > --- > > drivers/mmc/core/pwrseq.h | 2 +- > > drivers/mmc/core/pwrseq_emmc.c | 2 +- > > drivers/mmc/core/pwrseq_simple.c | 2 +- > > 3 files changed, 3 insertions(+), 3 deletions(-) > > > > diff --git a/drivers/mmc/core/pwrseq.h b/drivers/mmc/core/pwrseq.h > > index 096da48c6a7e..133de0426687 100644 > > --- a/drivers/mmc/core/pwrseq.h > > +++ b/drivers/mmc/core/pwrseq.h > > @@ -16,7 +16,7 @@ struct mmc_pwrseq_ops { > > }; > > > > struct mmc_pwrseq { > > - struct mmc_pwrseq_ops *ops; > > + const struct mmc_pwrseq_ops *ops; > > }; > > > > #ifdef CONFIG_OF > > diff --git a/drivers/mmc/core/pwrseq_emmc.c b/drivers/mmc/core/pwrseq_emmc.c > > index ad4f94ec7e8d..4a82bc77fe49 100644 > > --- a/drivers/mmc/core/pwrseq_emmc.c > > +++ b/drivers/mmc/core/pwrseq_emmc.c > > @@ -51,7 +51,7 @@ static void mmc_pwrseq_emmc_free(struct mmc_host *host) > > kfree(pwrseq); > > } > > > > -static struct mmc_pwrseq_ops mmc_pwrseq_emmc_ops = { > > +static const struct mmc_pwrseq_ops mmc_pwrseq_emmc_ops = { > > .post_power_on = mmc_pwrseq_emmc_reset, > > .free = mmc_pwrseq_emmc_free, > > }; > > diff --git a/drivers/mmc/core/pwrseq_simple.c b/drivers/mmc/core/pwrseq_simple.c > > index d10538bb5e07..2b16263458af 100644 > > --- a/drivers/mmc/core/pwrseq_simple.c > > +++ b/drivers/mmc/core/pwrseq_simple.c > > @@ -87,7 +87,7 @@ static void mmc_pwrseq_simple_free(struct mmc_host *host) > > kfree(pwrseq); > > } > > > > -static struct mmc_pwrseq_ops mmc_pwrseq_simple_ops = { > > +static const struct mmc_pwrseq_ops mmc_pwrseq_simple_ops = { > > .pre_power_on = mmc_pwrseq_simple_pre_power_on, > > .post_power_on = mmc_pwrseq_simple_post_power_on, > > .power_off = mmc_pwrseq_simple_power_off, > > Why is this needed for a stable patch? It doesn't fix a bug, it just > looks like it is a "nice thing" to have, right? I don't think any later > patch in this series relies it it, or am I missing something? Fine with me. julia