The following commit has been merged into the perf/core branch of tip:
Commit-ID: 7720804a2ae46c90265a32c81c45fb6f8d2f4e8b
Gitweb:
https://git.kernel.org/tip/7720804a2ae46c90265a32c81c45fb6f8d2f4e8b
Author:Masami Hiramatsu
AuthorDate:Sun, 01 Sep 2019 12:03:08 +09:00
Commit
Hi Stephen,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on ]
url:
https://github.com/0day-ci/linux/commits/Stephen-Boyd/tpm-Add-driver-for-cr50/20190901-100447
base:
config: x86_64-randconfig-s1-09021303 (attached as .config)
compiler: gcc-7 (Debian 7.4.0-11
Hi,
On Tue, Jul 30, 2019 at 06:10:03PM +0800, Richard Tresidder wrote:
> Add device tree binding documentation for addition of force_load boolean value
> to allow loading a battery during boot even if notpresent at that time.
>
> Signed-off-by: Richard Tresidder
> ---
> Documentation/devicetree
From: Colin Ian King
Variable port_rate is being initialized with a value that is never read
and is being re-assigned a little later on. The assignment is redundant
and hence can be removed.
Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King
---
net/sched/sch_cbs.c | 2 +-
1 fi
From: Branden Bonaby Sent: Wednesday, August 28,
2019 9:24 PM
>
> Introduce user specified latency in the packet reception path
> By exposing the test parameters as part of the debugfs channel
> attributes. We will control the testing state via these attributes.
>
> Signed-off-by: Branden Bonab
Hi,
Looks good to me, but needs to be split (one patch adding STATUS_EMPTY
to the power-supply core and one patch fixing sbs-battery driver).
-- Sebastian
On Thu, Aug 01, 2019 at 12:34:18PM -0600, Nick Crews wrote:
> Thanks Richard, I still would like some more opinions
> on this changing the us
On 02.09.2019 13:52, Hayes Wang wrote:
> First, for AUTONEG_DISABLE, we only need to modify MII_BMCR.
>
> Second, add advertising parameter for rtl8152_set_speed(). Add
> RTL_ADVERTISED_xxx for advertising parameter of rtl8152_set_speed().
> Then, the advertising settings from ethtool could be sav
Hi,
On Mon, Sep 02, 2019 at 03:00:40PM +0200, Michal Simek wrote:
> Trivial patch which just corrects error message.
>
> Fixes: 371bb20d6927 ("power: Add simple gpio-restart driver")
> Signed-off-by: Michal Simek
> ---
Thanks, queued.
-- Sebastian
>
> drivers/power/reset/gpio-restart.c | 2
From: Colin Ian King
In the case where an allocation fails for priv_ep ret is
assigned -ENOMEM and the code exits via the exit path 'err'.
The exit path returns -ENOMEM without using variable ret, so
the assingment is redundant and can be removed.
Addresses-Coverity: ("Unused value")
Signed-off-
On Mon, Sep 02, 2019 at 11:11:45AM -0700, Joe Perches wrote:
> On Mon, 2019-09-02 at 11:07 -0700, Randy Dunlap wrote:
> > On 9/2/19 10:46 AM, Greg KH wrote:
> > > On Mon, Sep 02, 2019 at 10:39:39AM -0700, Randy Dunlap wrote:
> > > > On 9/2/19 5:43 AM, Stephen Rothwell wrote:
> > > > > Hi all,
> > >
Mon, Sep 02, 2019 at 08:05:20PM CEST, allan.niel...@microchip.com wrote:
>The 09/02/2019 19:51, Jiri Pirko wrote:
>> External E-Mail
>>
>>
>> Mon, Sep 02, 2019 at 07:42:31PM CEST, allan.niel...@microchip.com wrote:
>> >Hi Jiri,
>> >
>> >Sorry for joining the discussion this late, but I have been
On 9/2/19 8:12 PM, Greg KH wrote:
> On Sun, Sep 01, 2019 at 06:53:34PM +0200, Jacek Anaszewski wrote:
>> Hi Akinobu,
>>
>> Thank you for the patch.
>>
>> I have one nit below but in general it looks good to me.
>> I've tested it with 2000 mtd triggers (~14kB file size)
>> and it worked flawlessly.
On Sun, Sep 01, 2019 at 07:08:29AM -0700, Guenter Roeck wrote:
> On 9/1/19 1:03 AM, Kalle Valo wrote:
> > Guenter Roeck writes:
> >
> > > On Mon, Aug 19, 2019 at 06:02:29PM -0400, Hui Peng wrote:
> > > > `dev` (struct rsi_91x_usbdev *) field of adapter
> > > > (struct rsi_91x_usbdev *) is allocat
From: Yizhuo
Date: Fri, 30 Aug 2019 19:00:48 -0700
> In function sun8i_dwmac_set_syscon(), local variable "val" could
> be uninitialized if function regmap_field_read() returns -EINVAL.
> However, it will be used directly in the if statement, which
> is potentially unsafe.
>
> Signed-off-by: Yiz
Fixes checkpatch.pl warning:
CHECK: Prefer using the BIT macro
Signed-off-by: Valentin Vidic
---
drivers/staging/exfat/exfat.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/exfat/exfat.h b/drivers/staging/exfat/exfat.h
index bae180e10609..f71d4e8c0c
Hi,
On Mon, Jul 22, 2019 at 10:45:54PM -0400, Paul Cercueil wrote:
> The ADC in the JZ4740 can work either in high-precision mode with a 2.5V
> range, or in low-precision mode with a 7.5V range. The code in place in
> this driver will select the proper scale according to the maximum
> voltage of t
On Mon, 02 Sep 2019 17:25:24 +0200, Greg Kroah-Hartman said:
> I dug up my old discussion with the current vfat maintainer and he said
> something to the affect of, "leave the existing code alone, make a new
> filesystem, I don't want anything to do with exfat".
>
> And I don't blame them, vfat is
Hi,
On Sun, Jul 21, 2019 at 06:03:28PM +0200, Christophe JAILLET wrote:
> It is likely that 'max10742_[un]lock_model()' functions should be
> 'max17042_[un]lock_model()'
> (0 and 7 switched in 10742)
>
> Signed-off-by: Christophe JAILLET
> ---
Thanks, queued.
-- Sebastian
> drivers/power/sup
From: Ben Wei
Date: Mon, 2 Sep 2019 02:46:52 +
> Update NC-SI command handler (both standard and OEM) to take into
> account of payload paddings in allocating skb (in case of payload
> size is not 32-bit aligned).
>
> The checksum field follows payload field, without taking payload
> padding
Fixes checkpatch.pl warnings:
CHECK: Please don't use multiple blank lines
CHECK: Blank lines aren't necessary after an open brace '{'
CHECK: Please use a blank line after function/struct/union/enum
declarations
Signed-off-by: Valentin Vidic
---
drivers/staging/exfat/exfat.h
Hi,
On Wed, Jul 17, 2019 at 10:18:48PM +0800, YueHaibing wrote:
> Fixes gcc '-Wunused-but-set-variable' warnings:
>
> drivers/power/supply/ab8500_charger.c:
> In function ab8500_charger_init_hw_registers:
> drivers/power/supply/ab8500_charger.c:3013:24: warning:
> variable vbup33_vrtcn set but
On Mon, Sep 02, 2019 at 03:00:17PM -0400, Valdis Klētnieks wrote:
> On Mon, 02 Sep 2019 17:25:24 +0200, Greg Kroah-Hartman said:
>
> > I dug up my old discussion with the current vfat maintainer and he said
> > something to the affect of, "leave the existing code alone, make a new
> > filesystem,
Hi,
On Sat, Jul 13, 2019 at 06:42:48PM +0300, Matwey V. Kornilov wrote:
> Closing angle bracket was missing.
>
> Signed-off-by: Matwey V. Kornilov
> ---
Thanks, queued.
-- Sebastian
> drivers/power/reset/reboot-mode.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/
From: Matteo Croce
Date: Mon, 2 Sep 2019 12:21:35 +0200
> This patchset workarounds an PP2 HW limitation which prevents to use
> per-cpu rx buffers.
> The first patch is just a refactor to prepare for the second one.
> The second one allocates percpu buffers if the following conditions are met:
On Mon, Sep 02, 2019 at 08:47:02PM +0200, Jacek Anaszewski wrote:
> On 9/2/19 8:12 PM, Greg KH wrote:
> > On Sun, Sep 01, 2019 at 06:53:34PM +0200, Jacek Anaszewski wrote:
> >> Hi Akinobu,
> >>
> >> Thank you for the patch.
> >>
> >> I have one nit below but in general it looks good to me.
> >> I'v
Hi,
On Mon, Jul 08, 2019 at 08:32:51PM +0800, Fuqian Huang wrote:
> Introduce local variable 'struct device *dev' and use it instead of
> dereferencing it repeatly.
>
> Signed-off-by: Fuqian Huang
> ---
Thanks, queued.
-- Sebastian
> drivers/power/supply/sc27xx_fuel_gauge.c | 47
> +
Hi,
On Mon, Jul 08, 2019 at 08:32:59PM +0800, Fuqian Huang wrote:
> devm_add_action_or_reset() is introduced as a helper function which
> internally calls devm_add_action(). If devm_add_action() fails
> then it will execute the action mentioned and return the error code.
> This reduce source cod
Hi,
On Fri, Jul 05, 2019 at 10:36:12AM +0100, Colin King wrote:
> From: Colin Ian King
>
> The variable ret is being assigned with a value that is never
> read and it is being updated later with a new value. The
> assignment is redundant and can be removed.
>
> Addresses-Coverity: ("Unused valu
Hi,
On Fri, Jul 05, 2019 at 05:37:51AM -0600, Angus Ainslie (Purism) wrote:
> The BQ25895 is almost identical to the BQ25890.
>
> Signed-off-by: Angus Ainslie (Purism)
> ---
Thanks, queued.
-- Sebastian
> drivers/power/supply/bq25890_charger.c | 12
> 1 file changed, 8 insertion
Hi,
On Wed, Jul 03, 2019 at 10:48:10AM +0200, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski
>
> Define a MODULE_ALIAS() in the charger sub-driver for max77650 so that
> the appropriate module gets loaded together with the core mfd driver.
>
> Signed-off-by: Bartosz Golaszewski
> ---
Fill in the appropriate limits to avoid inconsistencies
in the vfs cached inode times when timestamps are
outside the permitted range.
Note that the min timestamp is assumed to be
01 Jan 1970 00:00:00 (Unix epoch). This is consistent
with the way we convert timestamps in adfs_adfs2unix_time().
Si
Hello Mark,
Thanks a lot for your comments.
On 2019/09/02 21:02, Mark Brown wrote:
On Sun, Sep 01, 2019 at 01:26:48AM +0900, Katsuhiro Suzuki wrote:
This patch change the judge timing about playing/capturing PCM rate.
Original code set constraints list of PCM rate limits at set_sysclk.
This s
On Monday, September 2, 2019 8:10:10 PM CEST Ingo Molnar wrote:
> * Federico Vaga wrote:
> > On Saturday, August 31, 2019 4:43:44 PM CEST Jonathan Corbet wrote:
> > > On Sat, 31 Aug 2019 15:41:16 +0200
> > >
> > > Federico Vaga wrote:
> > > > several CPU's and you want to use spinlocks you can
On Mon, 2019-09-02 at 21:06 +0200, Sebastian Reichel wrote:
> Hi,
>
> On Sat, Jul 13, 2019 at 06:42:48PM +0300, Matwey V. Kornilov wrote:
> > Closing angle bracket was missing.
> >
> > Signed-off-by: Matwey V. Kornilov
> > ---
>
> Thanks, queued.
This is relatively widespread.
There are 80 mo
On 9/2/19 11:47 AM, Greg KH wrote:
On Sun, Sep 01, 2019 at 07:08:29AM -0700, Guenter Roeck wrote:
On 9/1/19 1:03 AM, Kalle Valo wrote:
Guenter Roeck writes:
On Mon, Aug 19, 2019 at 06:02:29PM -0400, Hui Peng wrote:
`dev` (struct rsi_91x_usbdev *) field of adapter
(struct rsi_91x_usbdev *) i
On Fri, Aug 30, 2019 at 11:14:39AM -0700, Sagi Grimberg wrote:
>
> > > > > > > You are correct that this information can be derived from sysfs,
> > > > > > > but the
> > > > > > > main reason why we add these here, is because in udev rule we
> > > > > > > can't
> > > > > > > just go ahead and st
Tentatively merged into cifs-2.6.git pending additional testing
Kicked off buildbot with rc7+patches in cifs for-next
See
http://smb3-test-rhel-75.southcentralus.cloudapp.azure.com/#/builders/2/builds/247
On Mon, Sep 2, 2019 at 10:33 AM Colin King wrote:
>
> From: Colin Ian King
>
> The assig
In function ec_read_u8(), variable "value" could be uninitialized
if ec_read() fails. However, "value" is returned directly and used
in its callers. This is potentially unsafe.
Signed-off-by: Yizhuo
---
drivers/platform/x86/compal-laptop.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
d
Dear Friend,
Please forgive me for stressing you with my predicaments as I know that this
letter may come to you as big surprise. Actually, as my pastor advised me to
reject earthly reward and thanks by handing the project to someone I have never
seen or met for a greater reward in heaven aw
On Mon, Sep 02, 2019 at 12:32:37PM -0700, Guenter Roeck wrote:
> On 9/2/19 11:47 AM, Greg KH wrote:
> > On Sun, Sep 01, 2019 at 07:08:29AM -0700, Guenter Roeck wrote:
> > > On 9/1/19 1:03 AM, Kalle Valo wrote:
> > > > Guenter Roeck writes:
> > > >
> > > > > On Mon, Aug 19, 2019 at 06:02:29PM -040
Hi!
> >
> > And if there is a meaningful way to make the kernel behave better, that
> > would
> > obviously be of huge value too.
> >
> > Thanks
> > Daniel
>
> Hi,
>
> Is there a way for an application to be told that there is a memory
> pressure situation?
> For example, say I do a "make -j32"
Hi!
> > Without knowing the internal design, but having more infor now, looks to me
> > that
> > should be modelled more as a kind of power supply? Maybe something similar
> > to
> > UCS1002-2 device (drivers/power/supply/ucs1002_power.c) but behind the EC?
>
> This would work, the problem woul
Em Mon, 2 Sep 2019 12:33:38 +
Robert Richter escreveu:
> A bunch of cleanups and fixes for issues found while working with the
> code. Changes are individual and independent from each other. They can
> be applied separately (only #4 depends on #3).
>
> Also updating the reviewer's entry as I
On Mon 2019-09-02 16:45:38, Anson Huang wrote:
> i.MX8MN DDR4 EVK board has a GPIO LED to indicate status,
> add support for it.
LED maintainers want to be on the cc list...
> @@ -15,6 +15,18 @@
> stdout-path = &uart2;
> };
>
> + leds {
> + compatible = "gpio
On Mon, Sep 2, 2019 at 12:43 PM Joao Martins wrote:
>
> When cpus != maxcpus cpuidle-haltpoll will fail to register all vcpus
> past the online ones and thus fail to register the idle driver.
> This is because cpuidle_add_sysfs() will return with -ENODEV as a
> consequence from get_cpu_device() re
Hi Arnd and Nandor,
On Mon, Aug 05, 2019 at 09:57:15AM +0200, Arnd Bergmann wrote:
> Nandor Han points out that there might be drivers that can use
> the reboot-mode interfaces but might also be usable on configurations
> without device tree.
>
> Move the 'depends on OF' dependency into CONFIG_RE
On Sun, Sep 1, 2019 at 12:29 PM Martin Blumenstingl
wrote:
>
> On Sun, Sep 1, 2019 at 12:45 PM Remi Pommarel wrote:
> >
> > meson_saradc's irq handler uses priv->regmap so make sure that it is
> > allocated before the irq get enabled.
> >
> > This also fixes crash when CONFIG_DEBUG_SHIRQ is enabl
Hi Luca,
> + * Topology:
> + *
> + * Slave X @ 0x10
> + * .-. |
> + * .-. | |---+ B
> + * | CPU |--A--| ATR |
> + * `-' | |---+ C
> + * `-' |
> + * Slave Y @ 0x10
> + *
> + * A
> + - i2c-alias-pool: list of I2C addresses that are known to be available on
> the
> + "local" (SoC-to-deser) I2C bus; they will be picked at
> +runtime and used as aliases to reach remove I2C chips
After some internal discussion, I have been kinda convinced th
On Thu, Jul 11, 2019 at 12:35 PM Chuanhua Han wrote:
>
> Enable NXP i2c controller to boot with ACPI
>
> Signed-off-by: Meenakshi Aggarwal
> Signed-off-by: Udit Kumar
> Signed-off-by: Chuanhua Han
Wolfram, any objections to this from the i2c side?
> ---
> drivers/acpi/acpi_apd.c | 6 ++
On Mon, Sep 02, 2019 at 10:56:22PM +0200, Rafael J. Wysocki wrote:
> On Thu, Jul 11, 2019 at 12:35 PM Chuanhua Han wrote:
> >
> > Enable NXP i2c controller to boot with ACPI
> >
> > Signed-off-by: Meenakshi Aggarwal
> > Signed-off-by: Udit Kumar
> > Signed-off-by: Chuanhua Han
>
> Wolfram, any
Hi Luca,
[...]
> Thanks for your patch! Though we already have this change in our
> internal tree (submitted by YueHaibing) and it will reach the mainline
> soon.
Thank you for letting me know. I am glad it's fixed. :)
Krzysztof
Hi,
On Wed, Jul 31, 2019 at 06:00:22PM +0800, Baolin Wang wrote:
> This patch set adds new attributes for userspace, and fixes the the
> accuracy issue of coulomb counter calculation, as well as optimizing
> the battery capacity calibration in some abnormal scenarios.
>
> Any comments are welcome
Sorry for the delayed reply.
On Mon, Jul 22, 2019 at 8:29 PM Bjorn Helgaas wrote:
>
> [+cc Tony (original author), Borislav (merged original patch)]
>
> On Mon, Jul 22, 2019 at 10:31:11AM +0200, Rafael J. Wysocki wrote:
> > On Mon, Jul 22, 2019 at 4:36 AM Kelsey Skunberg
> > wrote:
> > >
> > > a
Hi all,
Commit
5eae593422da ("vfs: Convert fuse to use the new mount API")
is missing a Signed-off-by from its author.
Commit
c08428c528bc ("vfs: Create fs_context-aware mount_bdev() replacement")
is missing a Signed-off-by from its committer.
--
Cheers,
Stephen Rothwell
pgpuBikdtQxMY
On Mon, Sep 2, 2019 at 11:58 PM Rafael J. Wysocki wrote:
>
> On Thu, Jul 11, 2019 at 12:35 PM Chuanhua Han wrote:
> >
> > Enable NXP i2c controller to boot with ACPI
> >
> > Signed-off-by: Meenakshi Aggarwal
> > Signed-off-by: Udit Kumar
> > Signed-off-by: Chuanhua Han
>
> Wolfram, any objecti
On Mon, Sep 2, 2019 at 10:39 PM Sebastian Reichel wrote:
>
> This patch does not look good to me. Better patch would be to
> allow compiling CONFIG_REBOOT_MODE without CONFIG_OF. Obviously
> the configuration would not be useful for anything except compile
> testing, but that is also true for this
Hi all,
Commits
f12dc871e3ec ("kgdb: fix comment regarding static function")
c60d67aad7b7 ("kdb: Replace strncmp with str_has_prefix")
are missing a Signed-off-by from their committer.
--
Cheers,
Stephen Rothwell
pgp1tiNEWoLiD.pgp
Description: OpenPGP digital signature
Here's a patch series that adds interconnect support for the Qualcomm
MSM8974, which is needed so that the GPU can be supported upstream. I'm
having an issue with getting this to work fully and included a detailed
write up in patch 2 with what I'm seeing. I think I am close to
getting this working
Add driver for the Qualcomm MSM8974 interconnect providers that support
setting system bandwidth requirements between various network-on-chip
fabrics.
I marked this as a PATCH RFC since I'm not able to write to all of the
master IDs with qcom_icc_rpm_smd_send(). I included tables below that
shows
Add device tree bindings for the Qualcomm MSM8974 interconnect providers
that support setting system bandwidth requirements between various
network-on-chip fabrics.
Signed-off-by: Brian Masney
---
.../bindings/interconnect/qcom,msm8974.yaml | 163 ++
.../dt-bindings/interconnec
On Wed, Aug 21, 2019 at 5:44 AM Wenwen Wang wrote:
>
> In acpi_pci_irq_enable(), 'entry' is allocated by kzalloc() in
> acpi_pci_irq_check_entry() (invoked from acpi_pci_irq_lookup()). However,
> it is not deallocated if acpi_pci_irq_valid() returns false, leading to a
> memory leak. To fix this i
On Wed, Aug 21, 2019 at 4:53 PM Jonathan Cameron
wrote:
>
> Generic Initiators are a new ACPI concept that allows for the
> description of proximity domains that contain a device which
> performs memory access (such as a network card) but neither
> host CPU nor Memory.
>
> This patch has the parsi
2019년 8월 31일 (토) 오전 5:50, Andrew Morton 님이 작성:
>
> On Fri, 30 Aug 2019 12:57:16 +0900 Austin Kim wrote:
>
> > If !area->pages statement is true where memory allocation fails,
> > area is freed.
> >
> > In this case 'area->pages = pages' should not executed.
> > So move 'area->pages = pages' after
On Tue, Aug 6, 2019 at 6:24 AM Daniel Black wrote:
>
> ACPI-6.3 corresponds to when hmat revision was bumped from
> 1 to 2. In this version ACPI_HMAT_MEMORY_PD_VALID was
> deprecated and made reserved.
>
> As such in revision 2+ we shouldn't be testing this flag.
>
> This is as per ACPI-6.3, 5.2.2
On Sat, Aug 31, 2019 at 10:34 AM tiantao wrote:
>
> for if else statements having single block no braces are needed fixed
> the following checkpatch warning
>
> WARNING: braces {} are not necessary for any arm of this statement
> + if (!prev_node) {
> [...]
> + } else {
Hi Rui,
On 9/2/19 2:38 AM, Rui Miguel Silva wrote:
Hi Steve,
On Tue 06 Aug 2019 at 00:34, Steve Longerbeam wrote:
Implement a notifier bound op to register media links from the remote
sub-device's source pad(s) to the CSI sink pad.
Signed-off-by: Steve Longerbeam
---
drivers/staging/media/i
On Thu, Aug 29, 2019 at 7:24 PM Marcelo Tosatti wrote:
>
> On Thu, Aug 29, 2019 at 06:16:05PM +0100, Joao Martins wrote:
> > On 8/29/19 4:10 PM, Joao Martins wrote:
> > > When cpus != maxcpus cpuidle-haltpoll will fail to register all vcpus
> > > past the online ones and thus fail to register the
On Fri, Aug 30, 2019 at 1:09 PM Joao Martins wrote:
>
> On 8/29/19 10:51 PM, Daniel Lezcano wrote:
> > On 29/08/2019 23:12, Joao Martins wrote:
> >
> > [ ... ]
> >
> Say you wanted to have a kvm specific config, you would still see the
> same
> problem if you happen to compile inte
Hi,
On Mon, Sep 2, 2019 at 11:45 AM Chuan Hua, Lei
wrote:
>
> Hi Martin,
>
>
> On 9/2/2019 5:38 AM, Martin Blumenstingl wrote:
> > Hi,
> >
> > On Fri, Aug 30, 2019 at 5:02 AM Chuan Hua, Lei
> > wrote:
> >> Hi Martin,
> >>
> >> On 8/30/2019 5:40 AM, Martin Blumenstingl wrote:
> >>> Hi,
> >>>
> >>
On Fri, Aug 30, 2019 at 5:28 AM Ben Chuang wrote:
>
> From: Ben Chuang
>
> Add support for the GL9750 and GL9755 chipsets.
>
> Enable v4 mode and wait 5ms after set 1.8V signal enable for GL9750/
> GL9755. Fix the value of SDHCI_MAX_CURRENT register and use the vendor
> tuning flow for GL9750.
>
In function pfuze100_regulator_probe(), variable "val" could be
initialized if regmap_read() fails. However, "val" is used to
decide the control flow later in the if statement, which is
potentially unsafe.
Signed-off-by: Yizhuo
---
drivers/regulator/pfuze100-regulator.c | 2 +-
1 file changed, 1
On Mon, Sep 02, 2019 at 10:34:07PM +0200, Rafael J. Wysocki wrote:
> On Mon, Sep 2, 2019 at 12:43 PM Joao Martins
> wrote:
> >
> > When cpus != maxcpus cpuidle-haltpoll will fail to register all vcpus
> > past the online ones and thus fail to register the idle driver.
> > This is because cpuidle_
Hello,
I only noticed this patchset today and I don't have much time left.
Here's my initial impressions without going through the code in detail.
I'll continue my review in the next days (as time permits).
As with all other Intel LGM patches: I don't have access to the
datasheets, so it's possib
On Wed, Aug 28, 2019 at 07:02:31PM -0700, Ira Weiny wrote:
> On Mon, Aug 26, 2019 at 03:55:10PM +1000, Dave Chinner wrote:
> > On Fri, Aug 23, 2019 at 10:08:36PM -0700, Ira Weiny wrote:
> > > On Sat, Aug 24, 2019 at 10:11:24AM +1000, Dave Chinner wrote:
> > > > On Fri, Aug 23, 2019 at 09:04:29AM -0
Inside function at91rm9200_timer_interrupt(), variable sr could
be uninitialized if regmap_read() fails. However, sr is used
to decide the control flow later in the if statement, which is
potentially unsafe. We could check the return value of
regmap_read() and print an error here.
Signed-off-by: Y
On Mon, Sep 02, 2019 at 11:19:58PM +0200, Rafael J. Wysocki wrote:
> On Wed, Aug 21, 2019 at 5:44 AM Wenwen Wang wrote:
> >
> > In acpi_pci_irq_enable(), 'entry' is allocated by kzalloc() in
> > acpi_pci_irq_check_entry() (invoked from acpi_pci_irq_lookup()). However,
> > it is not deallocated if
On 02/09/2019 15:29:46-0700, Yizhuo wrote:
> Inside function at91rm9200_timer_interrupt(), variable sr could
> be uninitialized if regmap_read() fails. However, sr is used
Could you elaborate on how this could fail?
> to decide the control flow later in the if statement, which is
> potentially un
In function axp288_extcon_log_rsi(), variable "val" could be
uninitialized if regmap_read() fails. However, it's ued to
decide the control flow later in the if statement, which is
potentially unsafe.
Signed-off-by: Yizhuo
---
drivers/extcon/extcon-axp288.c | 5 +
1 file changed, 5 insertions
>
> On Fri, Aug 30, 2019 at 05:58:39PM +, Safford, David (GE Global Research,
> US) wrote:
> > > Thank you for your advice. We also discussed earlier and concluded that
> > > checking and raw remapping are enough to work around this. The link is
> > > here, https://lkml.org/lkml/2019/8/29/962
Hi Patrick,
On Thu, Aug 22, 2019 at 6:28 AM Patrick Bellasi wrote:
>
> The cgroup CPU bandwidth controller allows to assign a specified
> (maximum) bandwidth to the tasks of a group. However this bandwidth is
> defined and enforced only on a temporal base, without considering the
> actual frequen
Hi David,
On Mon, 02 Sep 2019 17:01:19 +0100 David Howells wrote:
>
> Stephen Rothwell wrote:
>
> > The forward declararion doesn't seem to work (at laste for the
> > !CONFIG_USB_NOTIFICATIONS case.
>
> In the !CONFIG_USB_NOTIFICATIONS case, the argument is to a stub inline
> function. Even
On 09/02, Chao Yu wrote:
> On 2019/9/1 15:25, Jaegeuk Kim wrote:
> > On 08/31, Chao Yu wrote:
> >> On 2019/8/30 23:34, Jaegeuk Kim wrote:
> >>> This can guarantee inline_data has smaller i_size.
> >>
> >> So I guess "f2fs: fix to avoid corruption during inline conversion" didn't
> >> fix
> >> such
In function mdio_sc_cfg_reg_write(), variable reg_value could be
uninitialized if regmap_read() fails. However, this variable is
used later in the if statement, which is potentially unsafe.
Signed-off-by: Yizhuo
---
drivers/net/ethernet/hisilicon/hns_mdio.c | 8 +++-
1 file changed, 7 insert
Recently device pass-through stops working for Linux VM running on Hyper-V.
git-bisect shows the regression is caused by the recent commit
467a3bb97432 ("PCI: hv: Allocate a named fwnode ..."), but the root cause
is that the commit d59f6617eef0 forgets to set the domain->fwnode for
IRQCHIP_FWNOD
Stephen Rothwell wrote:
> > static inline void post_usb_device_notification(const struct usb_device
> > *udev,
> > unsigned int subtype, u32
> > error) {}
>
> But not published, yet?
I was hoping to get the smack patch reviewed by Casey first, but I
Hi David,
On Tue, 3 Sep 2019 09:07:22 +1000 Stephen Rothwell
wrote:
>
> > > I then discovered that I needed to install libkeyutils-dev :-( but it
> > > built OK after that.
> >
> > ? The kernel shouldn't require that to build.
>
> I was doing an x86_64 allmodconfig build which seems to
Sorry for the inconvenience. I made some mistake here, please ignore
this patch and I will submit a new one.
On Mon, Sep 2, 2019 at 4:14 PM Yizhuo wrote:
>
> In function mdio_sc_cfg_reg_write(), variable reg_value could be
> uninitialized if regmap_read() fails. However, this variable is
> used l
Thanks Andy, sorry for the inconvenience, I will check the log more
carefully next time.
On Mon, Sep 2, 2019 at 3:48 PM Andy Shevchenko
wrote:
>
>
>
> On Tuesday, September 3, 2019, Yizhuo wrote:
>>
>> In function axp288_extcon_log_rsi(), variable "val" could be
>> uninitialized if regmap_read()
Hi Al,
On Mon, 2 Sep 2019 16:30:04 +0100 Al Viro wrote:
>
> ... originals cheerfully dropped; will be gone in today's push to
> vfs.git#for-next.
Not pushed out yet?
--
Cheers,
Stephen Rothwell
pgpC6lmOgb0FJ.pgp
Description: OpenPGP digital signature
On Tue, Sep 03, 2019 at 09:23:17AM +1000, Stephen Rothwell wrote:
> Hi Al,
>
> On Mon, 2 Sep 2019 16:30:04 +0100 Al Viro wrote:
> >
> > ... originals cheerfully dropped; will be gone in today's push to
> > vfs.git#for-next.
>
> Not pushed out yet?
Give me about fifteen minutes...
Hi David,
On Tue, 03 Sep 2019 00:20:08 +0100 David Howells wrote:
>
> Ah, yes - the sample demonstrates the key/keyring notifications as well as USB
> and block notifications and requires a constant from the keyutils-devel
> package. Maybe I should get it from the kernel UAPI headers instead, bu
> > Update NC-SI command handler (both standard and OEM) to take into
> > account of payload paddings in allocating skb (in case of payload
> > size is not 32-bit aligned).
> >
> > The checksum field follows payload field, without taking payload
> > padding into account can cause checksum being tr
Hi Jason,
On Mon, 2 Sep 2019 10:51:41 + Jason Gunthorpe wrote:
>
> On Mon, Sep 02, 2019 at 08:50:17PM +1000, Stephen Rothwell wrote:
> > Hi all,
>
> > ERROR: "nd_region_provider_data" [drivers/acpi/nfit/nfit.ko] undefined!
> > ERROR: "to_nd_blk_region" [drivers/acpi/nfit/nfit.ko] undefined
Michal Suchánek writes:
> On Mon, 02 Sep 2019 14:01:17 +1000
> Michael Ellerman wrote:
>> Michael Ellerman writes:
>> > Michal Suchanek writes:
>> ...
>> >> @@ -295,6 +279,12 @@ static inline int current_is_64bit(void)
>> >> }
>> >>
>> >> #else /* CONFIG_PPC64 */
>> >> +static int read_u
Stephen Rothwell wrote:
> > Ah, yes - the sample demonstrates the key/keyring notifications as well as
> > USB and block notifications and requires a constant from the
> > keyutils-devel package. Maybe I should get it from the kernel UAPI
> > headers instead, but that risks generating a collisio
Stephen Rothwell wrote:
> > I was doing an x86_64 allmodconfig build which seems to build (all of?)
> > the samples.
>
> Of course, this breaks our crossbuilds :-(
>
> e.g. S390 allyesconfig build:
>
> /usr/bin/ld: cannot find -lkeyutils
Is it a requirement that stuff in samples/ should be ab
Segher Boessenkool writes:
> On Mon, Sep 02, 2019 at 12:03:12PM +1000, Michael Ellerman wrote:
>> Michal Suchanek writes:
>> > On bigendian ppc64 it is common to have 32bit legacy binaries but much
>> > less so on littleendian.
>>
>> I think the toolchain people will tell you that there is no 32
sb_getblk does not guarantee that buffer_head is uptodate. If there is
async read running in parallel for same buffer_head, it can overwrite
just initialized msdos_dir_entry, leading to corruption:
FAT-fs (loop0): error, corrupted directory (invalid entries)
FAT-fs (loop0): Filesystem has been
Michal Suchánek writes:
> On Mon, 02 Sep 2019 12:03:12 +1000
> Michael Ellerman wrote:
>
>> Michal Suchanek writes:
>> > On bigendian ppc64 it is common to have 32bit legacy binaries but much
>> > less so on littleendian.
>>
>> I think the toolchain people will tell you that there is no 32-bi
601 - 700 of 874 matches
Mail list logo