[PATCH] MAINTAINERS: edac: drop bouncing email

2018-11-20 Thread Alexandre Belloni
jetztechnologies.com doesn't exist anymore and emails to Ranganathan are bouncing. Signed-off-by: Alexandre Belloni --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index a5b256b25905..424ae080b38a 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5292,7

Re: [PATCH for-4.20-fixes] cgroup: fix CSS_TASK_ITER_PROCS

2018-11-20 Thread Tejun Heo
On Thu, Nov 08, 2018 at 12:15:15PM -0800, Tejun Heo wrote: > CSS_TASK_ITER_PROCS implements process-only iteration by making > css_task_iter_advance() skip tasks which aren't threadgroup leaders; > however, when an iteration is started css_task_iter_start() calls the > inner helper function css_tas

Re: [PATCH 5/7] regulator: core: add enable_count for consumers to debug fs

2018-11-20 Thread Mark Brown
On Mon, Nov 19, 2018 at 04:26:52PM -0800, Douglas Anderson wrote: > Now that consumers all keep track of their own enable count, let's add > it into the regulator_summary. This patch series contains a bunch of random stuff that's not particularly related to each other or even dependent on each oth

Re: [PATCH v2] cpuidle: big.LITTLE: add of_node_put()

2018-11-20 Thread Frank Lee
On Wed, Nov 21, 2018 at 12:06 AM Rafael J. Wysocki wrote: > > On Tue, Nov 20, 2018 at 4:46 PM Yangtao Li wrote: > > > > The of_node_put() is missing.So we call the of_node_put() to release > > the refcount. > > I would say "of_find_node_by_path() acquires a reference to the node > returned by it

Re: [PATCH v2] cpuidle: big.LITTLE: add of_node_put()

2018-11-20 Thread Rafael J. Wysocki
On Tue, Nov 20, 2018 at 4:46 PM Yangtao Li wrote: > > The of_node_put() is missing.So we call the of_node_put() to release > the refcount. I would say "of_find_node_by_path() acquires a reference to the node returned by it and that reference needs to be dropped by its caller. bl_idle_init() doesn

Re: [PATCH 7/7] regulator: core: Remove loop disabling supplies in regulator_force_disable()

2018-11-20 Thread Doug Anderson
Hi, On Tue, Nov 20, 2018 at 7:55 AM Mark Brown wrote: > > On Mon, Nov 19, 2018 at 04:26:54PM -0800, Douglas Anderson wrote: > > In regulator_force_disable() there was a strange loop that looked like: > > > > while (rdev->open_count--) > > regulator_disable(rdev->supply); > > > > I'm not tot

Re: [PATCH v9 05/17] tpm: declare struct tpm_header

2018-11-20 Thread Jason Gunthorpe
On Tue, Nov 20, 2018 at 03:08:05PM +0200, Jarkko Sakkinen wrote: > On Mon, Nov 19, 2018 at 09:33:31PM +, Winkler, Tomas wrote: > > > > > > > > Decleare struct tpm_header that replaces struct tpm_input_header and > > Typo > > Thanks, will fix for the next version. > > > > struct tpm_output_

[PATCH v1 04/15] platform/x86: i2c-mutli-instantiate: Defer probe when no adapter found

2018-11-20 Thread Andy Shevchenko
Likewise the rest of the i2c_acpi_new_device() users, defer the probe of the i2c-multi-intantiate driver in case adapter is not yet found. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/i2c-multi-instantiate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/p

[PATCH v1 00/15] i2c-multi-instantiate: Adapt for INT3515 and alike

2018-11-20 Thread Andy Shevchenko
Currently i2c-multi-instantiate driver does not support the case of INT3515 USB PD device, where: - Interrupt() is used instead of GpioInt() - All slaves may be the same from IP point of view - There can be variadic amount of slaves This series is addressing all above. Notes: - series has been s

[PATCH v1 02/15] platform/x86: intel_cht_int33fe: Accept errors of i2c_acpi_new_device()

2018-11-20 Thread Andy Shevchenko
In the future i2c_acpi_new_device() will return error pointer in some cases. Prepare i2c-multi-instantiate driver to support that. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/intel_cht_int33fe.c | 28 +++- 1 file changed, 23 insertions(+), 5 deletions(-) diff --g

[PATCH v1 11/15] platform/x86: i2c-multi-instantiate: Introduce IOAPIC IRQ support

2018-11-20 Thread Andy Shevchenko
If ACPI table provides an Interrupt() resource we may consider to use it instead of GpioInt() one. Here we leave an error condition, when getting IRQ resource, to the driver to decide how to proceed, because some drivers may consider IRQ resource optional. Signed-off-by: Andy Shevchenko --- dri

[PATCH v1 07/15] i2c: acpi: Use ACPI_FAILURE instead of !ACPI_SUCCESS

2018-11-20 Thread Andy Shevchenko
Convert to use ACPI_FAILURE instead of !ACPI_SUCCESS. Signed-off-by: Andy Shevchenko --- drivers/i2c/i2c-core-acpi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/i2c-core-acpi.c b/drivers/i2c/i2c-core-acpi.c index 478862abb82a..99553b1469b4 100644 --- a/drivers

[PATCH v1 08/15] i2c: acpi: Introduce i2c_acpi_get_i2c_resource() helper

2018-11-20 Thread Andy Shevchenko
Besides current two users one more is coming. Definitely makes sense to introduce a helper. No functional change intended. Signed-off-by: Andy Shevchenko --- drivers/i2c/i2c-core-acpi.c | 31 +++ include/linux/acpi.h| 11 +++ 2 files changed, 30 inser

[PATCH v1 13/15] ACPI / scan: Create platform device for INT3515 ACPI nodes

2018-11-20 Thread Andy Shevchenko
The ACPI device with INT3515 _HID is representing a complex USB PD hardware infrastructure which includes several I2C slave ICs. We add an ID to the I2C multi instantiate list to enumerate all I2C slaves correctly. Signed-off-by: Andy Shevchenko --- drivers/acpi/scan.c

[PATCH v1 14/15] ACPI / scan: Create platform device for BOSC0200 ACPI nodes

2018-11-20 Thread Andy Shevchenko
On some laptops the ACPI device with BOSC0200 _HID is representing two accelerometers under one node. We add an ID to the I2C multi instantiate list to enumerate all I2C slaves correctly. For reference here is the relevant DSDT blurb from the Yoga 11e: Device (ACC) { Name (_ADR, Zero) /

[PATCH v1 09/15] platform/x86: i2c-multi-instantiate: Count I2cSerialBus() resources

2018-11-20 Thread Andy Shevchenko
Instead of relying on hard coded and thus expected number of I2C clients, count the real amount provided by firmware. This allows to support non-fixed amount of the slaves. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/i2c-multi-instantiate.c | 34 ++-- 1 file changed,

Re: [PATCH 2/7] regulator: core: Don't assume always_on when is_enabled returns err

2018-11-20 Thread Mark Brown
On Mon, Nov 19, 2018 at 04:26:49PM -0800, Douglas Anderson wrote: > At boot sometimes regulators (like qcom-rpmh-regulator) will return > -EINVAL if we don't know the enable state of the regulator. We > shouldn't take this to mean that the regulator is an always-on > regulator, but that was what

[PATCH v1 06/15] i2c: acpi: Assign fwnode for devices created via i2c_acpi_new_device()

2018-11-20 Thread Andy Shevchenko
i2c_acpi_new_device() doesn't assign fwnode like it's done, for example, in i2c_acpi_register_devices() path. Assign fwnode in i2c_acpi_new_device() as it's done elsewhere. Signed-off-by: Andy Shevchenko --- drivers/i2c/i2c-core-acpi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers

[PATCH v1 12/15] platform/x86: i2c-multi-instantiate: Allow to have same slaves

2018-11-20 Thread Andy Shevchenko
Currently the driver will not enumerate the devices where I2C slaves are of the same type. Add an instance number to make them unique. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/i2c-multi-instantiate.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/

[PATCH v1 15/15] iio: inv_mpu6050: Use i2c_acpi_get_i2c_resource() helper

2018-11-20 Thread Andy Shevchenko
ACPI provides a generic helper to get I2C Serial Bus resources. Use it instead of open coded variant. Signed-off-by: Andy Shevchenko --- drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.c | 16 ++-- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/drivers/iio/imu/inv_mpu6050/in

[PATCH v1 01/15] platform/x86: intel_cht_int33fe: Remove duplicate NULL check

2018-11-20 Thread Andy Shevchenko
Since i2c_unregister_device() became NULL-aware we may remove duplicate NULL check. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/intel_cht_int33fe.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/platform/x86/intel_cht_int33fe.c b/drivers/platform/x

[PATCH v1 05/15] i2c: acpi: Return error pointers from i2c_acpi_new_device()

2018-11-20 Thread Andy Shevchenko
The caller would like to know the reason why the i2c_acpi_new_device() fails. For example, if adapter is not available, it might be in the future and we would like to re-probe the clients again. But at the same time we would like to bail out if the error seems unrecoverable, such as out of memory c

[PATCH v1 10/15] platform/x86: i2c-multi-instantiate: Distinguish IRQ resource type

2018-11-20 Thread Andy Shevchenko
As a preparatory patch switch the driver to distinguish IRQ resource type. For now, only GpioInt() is supported. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/i2c-multi-instantiate.c | 27 +--- 1 file changed, 18 insertions(+), 9 deletions(-) diff --git a/drivers/platf

[PATCH v1 03/15] platform/x86: i2c-multi-instantiate: Accept errors of i2c_acpi_new_device()

2018-11-20 Thread Andy Shevchenko
In the future i2c_acpi_new_device() will return error pointer in some cases. Prepare i2c-multi-instantiate driver to support that. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/i2c-multi-instantiate.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/

Re: Applied "regulator: wm8994: Pass descriptor instead of GPIO number" to the regulator tree

2018-11-20 Thread Marek Szyprowski
Hi Charles, On 2018-11-20 16:36, Charles Keepax wrote: > On Tue, Nov 20, 2018 at 03:32:15PM +, Charles Keepax wrote: >> On Tue, Nov 20, 2018 at 03:58:59PM +0100, Marek Szyprowski wrote: >>> On 2018-11-20 15:47, Charles Keepax wrote: On Tue, Nov 20, 2018 at 02:43:32PM +0100, Marek Szyprows

Re: [PATCH] perf tools: Restore proper cwd on return from mnt ns

2018-11-20 Thread Arnaldo Carvalho de Melo
Em Tue, Nov 20, 2018 at 11:56:18AM +0100, Jiri Olsa escreveu: > On Mon, Nov 19, 2018 at 08:10:06AM -0800, Arnaldo Carvalho de Melo wrote: > > Em Thu, Nov 01, 2018 at 06:00:01PM +0100, Jiri Olsa escreveu: > > > Adding the current working directory to be part of the cookie > > > and restoring it in t

Re: [PATCH 7/7] regulator: core: Remove loop disabling supplies in regulator_force_disable()

2018-11-20 Thread Mark Brown
On Mon, Nov 19, 2018 at 04:26:54PM -0800, Douglas Anderson wrote: > In regulator_force_disable() there was a strange loop that looked like: > > while (rdev->open_count--) > regulator_disable(rdev->supply); > > I'm not totally sure what the goal was for this loop, but it seems > wrong to me.

Re: [PATCH v9 02/15] sched/cpufreq: Prepare schedutil for Energy Aware Scheduling

2018-11-20 Thread Quentin Perret
On Tuesday 20 Nov 2018 at 16:25:14 (+0100), Peter Zijlstra wrote: > On Tue, Nov 20, 2018 at 10:16:02AM +0530, Viresh Kumar wrote: > > On 19-11-18, 14:18, Quentin Perret wrote: > > > @@ -223,20 +222,33 @@ static unsigned long sugov_get_util(struct > > > sugov_cpu *sg_cpu) > > > > > - if ((util + c

[PATCH v2] cpuidle: big.LITTLE: add of_node_put()

2018-11-20 Thread Yangtao Li
The of_node_put() is missing.So we call the of_node_put() to release the refcount. Signed-off-by: Yangtao Li --- Changes in v2 -update changelog -do that right after its last use to also cover error handling. --- drivers/cpuidle/cpuidle-big_little.c | 6 +- 1 file changed, 5 insertions(+),

Re: [PATCH] debugobjects: scale the static pool size

2018-11-20 Thread Waiman Long
On 11/20/2018 10:12 AM, Qian Cai wrote: > >> On Nov 20, 2018, at 8:50 AM, Waiman Long wrote: >> >> On 11/20/2018 01:42 AM, Qian Cai wrote: >>> The current value of the early boot static pool size is not big enough >>> for systems with large number of CPUs with timer or/and workqueue >>> objects se

[PATCH v2 2/2] dt-bindings: iio: adc: Add docs for AD7606 ADC

2018-11-20 Thread Stefan Popa
Document support for AD7606 Analog to Digital Converter. Signed-off-by: Stefan Popa --- Changes in v2: - fixed indentation issues. - used gpios instead of gpio. - added vendor prefix for conversion-start-gpios, first-data-gpios, and range-gpios. .../devicetree/bi

Re: [PATCH cgroup/for-4.21 1/2] cpuset: Minor cgroup2 interface updates

2018-11-20 Thread Tejun Heo
Hello, Peter. On Tue, Nov 20, 2018 at 01:46:24PM +0100, Peter Zijlstra wrote: > Why though? The Changelog doesn't give rationale for the actual changes. Ah yeah, sorry about that. > And I'm not sure I agree with either one of them. > > The partition is a scheduling feature; So is everything wi

[PATCH v2 1/2] staging: iio: ad7606: Move out of staging

2018-11-20 Thread Stefan Popa
Move ad7606 ADC driver out of staging and into the mainline. Signed-off-by: Stefan Popa --- Changes in v2: - Simplified the Kconfig menu. - Added SPDX-License-Identifier. - Ordered the includes alphabetically. - Used a threaded interrupt. - Replaced ad7606_

Re: [PATCH] mtd: spi-nor: parse SFDP 4-byte Address Instruction Table

2018-11-20 Thread Tudor.Ambarus
On 11/20/2018 05:35 PM, kbuild test robot wrote: > Hi Cyrille, > > Thank you for the patch! Yet something to improve: > > [auto build test ERROR on mtd/spi-nor/next] > [also build test ERROR on v4.20-rc3 next-20181120] > [if your patch is applied to the wrong git tree,

Re: Applied "regulator: wm8994: Pass descriptor instead of GPIO number" to the regulator tree

2018-11-20 Thread Mark Brown
On Tue, Nov 20, 2018 at 03:32:15PM +, Charles Keepax wrote: > On Tue, Nov 20, 2018 at 03:58:59PM +0100, Marek Szyprowski wrote: > > No board file, everything in DT. > This is really weird, because the error in your log relates to > DBVDD1 which is an independent regulator supplied by a separa

Re: [PATCH v2] PCI: dwc: Adjust Kconfig to allow IMX6 PCIe host on IMX7

2018-11-20 Thread Lorenzo Pieralisi
On Fri, Nov 16, 2018 at 12:08:33AM +, Trent Piepho wrote: > The IMX6 PCI-e host drier also supports the IMX7d. However, the > Kconfig dependencies of the driver prevented it from being enabled > unless the kernel was built with both IMX6 and IMX7 support. It works > fine to build with only IM

Re: [PATCH] thermal: tegra: add get_trend ops

2018-11-20 Thread Thierry Reding
On Tue, Nov 20, 2018 at 05:11:17PM +0800, Wei Ni wrote: > Add support for get_trend ops that allows soctherm > sensors to be used with the step-wise governor. > > Signed-off-by: Wei Ni > --- > drivers/thermal/tegra/soctherm.c | 34 ++ > 1 file changed, 34 insertio

Re: [PATCH] mtd: spi-nor: parse SFDP 4-byte Address Instruction Table

2018-11-20 Thread kbuild test robot
Hi Cyrille, Thank you for the patch! Yet something to improve: [auto build test ERROR on mtd/spi-nor/next] [also build test ERROR on v4.20-rc3 next-20181120] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci

Re: Applied "regulator: wm8994: Pass descriptor instead of GPIO number" to the regulator tree

2018-11-20 Thread Charles Keepax
On Tue, Nov 20, 2018 at 03:32:15PM +, Charles Keepax wrote: > On Tue, Nov 20, 2018 at 03:58:59PM +0100, Marek Szyprowski wrote: > > On 2018-11-20 15:47, Charles Keepax wrote: > > > On Tue, Nov 20, 2018 at 02:43:32PM +0100, Marek Szyprowski wrote: > > >> On 2018-05-17 18:41, Mark Brown wrote: >

NULL pointer dereference in smb2_queryfs with v4.19.2

2018-11-20 Thread Stijn Tintel
Hi, My machine just rebooted after the connection to the Samba server hosting a CIFS mount was lost. Kernel version 4.19.2. The oops was recorded in pstore: <3>[533816.847894] CIFS VFS: Server store has not responded in 120 seconds. Reconnecting... <1>[533925.390079] BUG: unable to handle kernel

Re: [PATCH 3/7] doc/vm: New documentation for memory performance

2018-11-20 Thread Keith Busch
On Tue, Nov 20, 2018 at 02:51:50PM +0100, Mike Rapoport wrote: > On Wed, Nov 14, 2018 at 03:49:16PM -0700, Keith Busch wrote: > > Platforms may provide system memory where some physical address ranges > > perform differently than others. These heterogeneous memory attributes are > > common to the n

Re: [RFC PATCH 2/3] mm, memory_hotplug: deobfuscate migration part of offlining

2018-11-20 Thread osalvador
> Signed-off-by: Michal Hocko [...] > + do { > + for (pfn = start_pfn; pfn;) > + { > + /* start memory hot removal */ Should we change thAT comment? I mean, this is not really the hot- removal stage. Maybe "start memory migration" suits better? o

[PATCH v6 2/7] dmaengine: xilinx_dma: in axidma slave_sg and dma_cyclic mode align split descriptors

2018-11-20 Thread Andrea Merello
Whenever a single or cyclic transaction is prepared, the driver could eventually split it over several SG descriptors in order to deal with the HW maximum transfer length. This could end up in DMA operations starting from a misaligned address. This seems fatal for the HW if DRE (Data Realignment E

[PATCH] edac: i82975x: fix spelling mistake "reserverd" -> "reserved"

2018-11-20 Thread Alexandre Belloni
Fix a spelling mistake in a register layout description. Signed-off-by: Alexandre Belloni --- drivers/edac/i82975x_edac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/edac/i82975x_edac.c b/drivers/edac/i82975x_edac.c index 892815eaa97b..40249e9c9d26 100644 --- a/dr

Re: Applied "regulator: wm8994: Pass descriptor instead of GPIO number" to the regulator tree

2018-11-20 Thread Charles Keepax
On Tue, Nov 20, 2018 at 03:58:59PM +0100, Marek Szyprowski wrote: > Hi Charles, > > On 2018-11-20 15:47, Charles Keepax wrote: > > On Tue, Nov 20, 2018 at 02:43:32PM +0100, Marek Szyprowski wrote: > >> On 2018-05-17 18:41, Mark Brown wrote: > >>> Subject: [PATCH] regulator: wm8994: Pass descriptor

[PATCH v6 1/7] dmaengine: xilinx_dma: commonize DMA copy size calculation

2018-11-20 Thread Andrea Merello
This patch removes a bit of duplicated code by introducing a new function that implements calculations for DMA copy size, and prepares for changes to the copy size calculation that will happen in following patches. Suggested-by: Vinod Koul Signed-off-by: Andrea Merello Reviewed-by: Radhey Shyam

[PATCH v6 6/7] dt-bindings: dmaengine: xilinx_dma: drop include-sg property

2018-11-20 Thread Andrea Merello
This property is not needed anymore, because the driver now autodetects it. Delete references in documentation. Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Cc: Radhey Shyam Pandey Signed-off-by: Andrea Merello Reviewed-by: Radhey Shyam Pandey Reviewed-by: Rob Herring ---

[PATCH v6 4/7] dmaengine: xilinx_dma: program hardware supported buffer length

2018-11-20 Thread Andrea Merello
From: Radhey Shyam Pandey AXI-DMA IP supports configurable (c_sg_length_width) buffer length register width, hence read buffer length (xlnx,sg-length-width) DT property and ensure that driver doesn't program buffer length exceeding the supported limit. For VDMA and CDMA there is no change. Cc: R

[PATCH v6 5/7] dmaengine: xilinx_dma: autodetect whether the HW supports scatter-gather

2018-11-20 Thread Andrea Merello
The AXIDMA and CDMA HW can be either direct-access or scatter-gather version. These are SW incompatible. The driver can handle both versions: a DT property was used to tell the driver whether to assume the HW is in scatter-gather mode. This patch makes the driver to autodetect this information. T

[PATCH v6 3/7] dt-bindings: dmaengine: xilinx_dma: add optional xlnx,sg-length-width property

2018-11-20 Thread Andrea Merello
The width of the "length register" cannot be autodetected, and it is now specified with a DT property. Add documentation for it. Cc: Rob Herring Cc: Mark Rutland Cc: devicet...@vger.kernel.org Cc: Radhey Shyam Pandey Signed-off-by: Andrea Merello Reviewed-by: Radhey Shyam Pandey --- Changes i

[PATCH v6 7/7] dmaengine: xilinx_dma: Drop SG support for VDMA IP

2018-11-20 Thread Andrea Merello
xilinx_vdma_start_transfer() is used only for VDMA IP, still it contains conditional code on has_sg variable. has_sg is set only whenever the HW does support SG mode, that is never true for VDMA IP. This patch drops the never-taken branches. Signed-off-by: Andrea Merello Reviewed-by: Radhey Shya

Re: [RFC 0/3] Unify CPU topology across ARM64 & RISC-V

2018-11-20 Thread Jeffrey Hugo
On 11/20/2018 4:11 AM, Sudeep Holla wrote: On Thu, Nov 15, 2018 at 11:31:33AM -0700, Jeffrey Hugo wrote: [...] I was interested in testing these on QDF2400, an ARM64 platform, since this series touches core ARM64 code and I'd hate to see a regression. However, I can't figure out what baseline

Re: Re: STIBP by default.. Revert?

2018-11-20 Thread Jiri Kosina
On Mon, 19 Nov 2018, Arjan van de Ven wrote: > In the documentation, AMD officially recommends against this by default, > and I can speak for Intel that our position is that as well: this really > must not be on by default. Thanks for pointing to the AMD doc, it's indeed clearly stated there.

Re: [PATCH v9 02/15] sched/cpufreq: Prepare schedutil for Energy Aware Scheduling

2018-11-20 Thread Peter Zijlstra
On Tue, Nov 20, 2018 at 10:16:02AM +0530, Viresh Kumar wrote: > On 19-11-18, 14:18, Quentin Perret wrote: > > @@ -223,20 +222,33 @@ static unsigned long sugov_get_util(struct sugov_cpu > > *sg_cpu) > > > - if ((util + cpu_util_dl(rq)) >= max) > > - return max; > > + if (type == FREQ

Re: [PATCH] cpuidle: big.LITTLE: add of_node_put()

2018-11-20 Thread Rafael J. Wysocki
On Tue, Nov 20, 2018 at 2:00 PM Yangtao Li wrote: > > use of_node_put() to release the refcount. I gather that the of_node_put() is missing? If so, that should be stated in the changelog too. > Signed-off-by: Yangtao Li > --- > drivers/cpuidle/cpuidle-big_little.c | 8 ++-- > 1 file chang

Re: Cleaning up numbering for new x86 syscalls?

2018-11-20 Thread Andy Lutomirski
On Tue, Nov 20, 2018 at 1:03 AM Florian Weimer wrote: > > * Andy Lutomirski: > > > 5. Adjust the scripts so that we only have to wire up new syscalls > > once. They'll have a nr above 1024, and they'll have the same nr on > > all x86 variants. > > Is there a sufficiently sized gap on all other ar

Re: STIBP by default.. Revert?

2018-11-20 Thread Jiri Kosina
On Sun, 18 Nov 2018, Linus Torvalds wrote: > This was marked for stable, and honestly, nowhere in the discussion > did I see any mention of just *how* bad the performance impact of this > was. > > When performance goes down by 50% on some loads, people need to start > asking themselves whether it

Re: RFC: userspace exception fixups

2018-11-20 Thread Andy Lutomirski
On Tue, Nov 20, 2018 at 2:11 AM Jarkko Sakkinen wrote: > > On Mon, Nov 19, 2018 at 09:00:08AM -0800, Andy Lutomirski wrote: > > On Mon, Nov 19, 2018 at 8:02 AM Jarkko Sakkinen > > wrote: > > > > > > On Mon, Nov 19, 2018 at 07:29:36AM -0800, Andy Lutomirski wrote: > > > > 1. The kernel needs some

Re: [RFC PATCH 2/3] mm, memory_hotplug: deobfuscate migration part of offlining

2018-11-20 Thread Michal Hocko
On Tue 20-11-18 16:13:35, Oscar Salvador wrote: > > > Signed-off-by: Michal Hocko > [...] > > + do { > > + for (pfn = start_pfn; pfn;) > > + { > > + /* start memory hot removal */ > > Should we change thAT comment? I mean, this is not really the hot- > rem

Re: [PATCH v2 1/3] PCI: dwc: allow to limit registers set length

2018-11-20 Thread Lorenzo Pieralisi
On Tue, Nov 20, 2018 at 02:27:03PM +0100, Stefan Agner wrote: > Add length to the struct dw_pcie and check that the accessors > dw_pcie_(rd|wr)_own_conf() do not read/write beyond that point. > > Signed-off-by: Stefan Agner > --- > drivers/pci/controller/dwc/pcie-designware-host.c | 4 > dr

Re: [PATCH -next 1/2] mm/memfd: make F_SEAL_FUTURE_WRITE seal more robust

2018-11-20 Thread Andy Lutomirski
On Mon, Nov 19, 2018 at 9:21 PM Joel Fernandes (Google) wrote: > > A better way to do F_SEAL_FUTURE_WRITE seal was discussed [1] last week > where we don't need to modify core VFS structures to get the same > behavior of the seal. This solves several side-effects pointed out by > Andy [2]. > > [1]

Re: [PATCH] debugobjects: scale the static pool size

2018-11-20 Thread Qian Cai
> On Nov 20, 2018, at 8:50 AM, Waiman Long wrote: > > On 11/20/2018 01:42 AM, Qian Cai wrote: >> The current value of the early boot static pool size is not big enough >> for systems with large number of CPUs with timer or/and workqueue >> objects selected. As the results, systems have 60+ CPU

Re: [PATCH v1 1/1] arm64: Early boot time stamps

2018-11-20 Thread Pavel Tatashin
On 18-11-20 14:55:23, Marc Zyngier wrote: > > Yeah, I saw 56 is used in arm_arch_timer.c, but I could not find where > > this minimum is defined in aarch64 specs. I will change it to 56. > > See D10.1.2 (The system counter) in the ARM ARM[1], which says: > > > The Generic Timer provides a system

Re: [PATCH v2] watchdog: cpwd: add of_node_put()

2018-11-20 Thread Guenter Roeck
On 11/20/18 6:54 AM, Yangtao Li wrote: use of_node_put() to release the refcount. Signed-off-by: Yangtao Li Reviewed-by: Guenter Roeck --- Changes in v2: -do that right after its last use to also cover error handling. --- drivers/watchdog/cpwd.c | 2 ++ 1 file changed, 2 insertions(+)

Re: [RFC PATCH 1/3] mm, memory_hotplug: try to migrate full section worth of pages

2018-11-20 Thread Michal Hocko
On Tue 20-11-18 15:51:32, Oscar Salvador wrote: > On Tue, 2018-11-20 at 14:43 +0100, Michal Hocko wrote: > > From: Michal Hocko > > > > do_migrate_range has been limiting the number of pages to migrate to > > 256 > > for some reason which is not documented. > > When looking back at old memory-h

Re: Applied "regulator: wm8994: Pass descriptor instead of GPIO number" to the regulator tree

2018-11-20 Thread Marek Szyprowski
Hi Charles, On 2018-11-20 15:47, Charles Keepax wrote: > On Tue, Nov 20, 2018 at 02:43:32PM +0100, Marek Szyprowski wrote: >> On 2018-05-17 18:41, Mark Brown wrote: >>> Subject: [PATCH] regulator: wm8994: Pass descriptor instead of GPIO number >> This patch causes following kernel warning on Samsu

[PATCH 00/16] staging: vchiq: dead code removal & misc fixes

2018-11-20 Thread Nicolas Saenz Julienne
Hi All, This series was written in parallel with reading and understanding the vchiq code. So excuse me for the lack of logic in the sequence of patches. The main focus was to delete as much code as possible, I've counted around 550 lines, which is not bad. Apart from that there are some patches

[PATCH 04/16] staging: vchiq_arm: use list_for_each_entry when accessing bulk_waiter_list

2018-11-20 Thread Nicolas Saenz Julienne
The resulting code is way more readeable and intuitive compared to plain list_for_each. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 52 ++- 1 file changed, 16 insertions(+), 36 deletions(-) diff --git a/drivers/staging/vc04_services/

Re: [PATCH v1 1/1] arm64: Early boot time stamps

2018-11-20 Thread Marc Zyngier
On 20/11/2018 14:40, Pavel Tatashin wrote: >>> +static __init void sched_clock_early_init(void) >>> +{ >>> + u64 freq = arch_timer_get_cntfrq(); >>> + u64 (*read_time)(void) = arch_counter_get_cntvct; >> >> We already have arch_timer_read_counter which is exposed from >> arm_arch_timer.h. >

Re: [PATCH] mmc: core: Remove timeout when enabling cache

2018-11-20 Thread Sjoerd Simons
On Tue, 2018-11-20 at 15:24 +0100, Ulf Hansson wrote: > On 20 November 2018 at 15:00, Sjoerd Simons > wrote: > > On Tue, 2018-11-20 at 14:08 +0100, Ulf Hansson wrote: > > > + Hal Emmerich > > > > > > On 20 November 2018 at 12:38, Sjoerd Simons > > > wrote: > > > > On Tue, 2018-11-20 at 11:23 +01

[PATCH 01/16] staging: vchiq_core: rework vchiq_get_config

2018-11-20 Thread Nicolas Saenz Julienne
The function is overly complicated for what it's ultimately achieving. It's simply filling up a structure. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 12 .../interface/vchiq_arm/vchiq_core.c | 30 +-- .../interface/

[PATCH 06/16] staging: vchiq_arm: rework vchiq_ioc_copy_element_data

2018-11-20 Thread Nicolas Saenz Julienne
The function is passed to vchiq_core.c for it to go trough all the transfer elements (an array of pointers to data) and copy them into the actual transfer memory (contiguous memory). The logic in the function was "copy an element and return, except when the element is empty, in which case look for

[PATCH 09/16] staging: vchiq_core: do not initialize semaphores twice

2018-11-20 Thread Nicolas Saenz Julienne
vchiq_init_state() initialises a series of semaphores to then call remote_event_create() on the same semaphores, which initializes them again. We get rid of the second initialization. Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchiq_arm/vchiq_core.c| 11 +--

[PATCH 07/16] staging: vchiq-core: get rid of is_master distinction

2018-11-20 Thread Nicolas Saenz Julienne
VCHIQ bulk transfers are what most people call DMA transfers. The CPU sends a list of physical addresses to the VideoCore which then access the memory directly without the need for CPU interaction. With this setup we call the CPU the "slave" and the VideoCore the "master". There seems to be an op

[PATCH 05/16] staging: vchiq_arm: get rid of vchi_mh.h

2018-11-20 Thread Nicolas Saenz Julienne
The concept of VCHI_MEM_HANDLE_T is introduced by this header file and was meant to be used with bulk transfers. After a quick look in vchiq_core.c it is pretty clear that it actually accomplishes nothing nor alters the bulk transfers in any way. Signed-off-by: Nicolas Saenz Julienne --- .../vc0

[PATCH 10/16] staging: vchiq_core: don't add a wmb() before remote_event_signal()

2018-11-20 Thread Nicolas Saenz Julienne
It's the first thing remote_event_signal() does. Signed-off-by: Nicolas Saenz Julienne --- .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c| 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_core.c b/drivers/staging/vc04_s

[PATCH v2] watchdog: cpwd: add of_node_put()

2018-11-20 Thread Yangtao Li
use of_node_put() to release the refcount. Signed-off-by: Yangtao Li --- Changes in v2: -do that right after its last use to also cover error handling. --- drivers/watchdog/cpwd.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/watchdog/cpwd.c b/drivers/watchdog/cpwd.c index aee0b

[PATCH 13/16] staging: vchiq_core: fix logic redundancy in parse_open

2018-11-20 Thread Nicolas Saenz Julienne
We update sync to reflect that the firmware version is compatible with that option. We don't need to check both of them again further down the code. Signed-off-by: Nicolas Saenz Julienne --- .../staging/vc04_services/interface/vchiq_arm/vchiq_core.c| 4 +--- 1 file changed, 1 insertion(+), 3

[PATCH 08/16] staging: vchiq_core: remove unnecessary safety checks in vchiq_init_state

2018-11-20 Thread Nicolas Saenz Julienne
vchiq_init_state() checks the initial contents of slot_zero are correct. These are set in vchiq_init_slots(), using the same hard-coded defaults as the checks. Both functions are called sequentially and Video Core isn't yet aware of the slot's address. There is no way the contents of slot_zero chan

[PATCH 12/16] staging: vchiq_util: get rid of unneeded memory barriers

2018-11-20 Thread Nicolas Saenz Julienne
All the memory operations featured in this file modify/access memory that is only accessed by the CPU. So we can assume that all the memory barrier handling done by the completion routines is good enough for us. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_util.c

[PATCH 11/16] staging: vchiq: use completions instead of semaphores

2018-11-20 Thread Nicolas Saenz Julienne
It is preferred in the kernel to avoid using semaphores to wait for events, as they are optimised for the opposite situation; where the common case is that they are available and may block only occasionally. FYI see this thread: https://lkml.org/lkml/2008/4/11/323. Also completions are semanticall

[PATCH 16/16] staging: vchiq: add more tasks to the TODO list

2018-11-20 Thread Nicolas Saenz Julienne
The TODO list was missing some tasks needed before upstreaming the device. Signed-off-by: Nicolas Saenz Julienne --- .../staging/vc04_services/interface/vchi/TODO | 42 +++ 1 file changed, 42 insertions(+) diff --git a/drivers/staging/vc04_services/interface/vchi/TODO b/drivers

[PATCH 03/16] staging: vchiq_shim: delete vchi_service_create

2018-11-20 Thread Nicolas Saenz Julienne
No one is using the API neither in the actual staging tree nor in the downstream tree (https://github.com/raspberrypi/linux). Signed-off-by: Nicolas Saenz Julienne --- .../vc04_services/interface/vchi/vchi.h | 5 --- .../interface/vchiq_arm/vchiq_shim.c | 32 ---

[PATCH 14/16] staging: vchiq_arm: rework probe and init functions

2018-11-20 Thread Nicolas Saenz Julienne
Moves the allocation of a chardev region and class creation to the init function of the driver since those functions are meant to be run on a per driver basis, as opposed to the code run in the probe function which is run in a per device basis. Signed-off-by: Nicolas Saenz Julienne --- .../inter

[PATCH 02/16] staging: vchiq_arm: rework close/remove_service IOCTLS

2018-11-20 Thread Nicolas Saenz Julienne
The implementation of both IOCTLS was the same except for one function call. This joins both implementations and updates the code to avoid unneeded indentations. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vchiq_arm.c | 66 +++ 1 file changed, 24 i

[PATCH 15/16] staging: vchiq_arm: fix open/release cdev functions

2018-11-20 Thread Nicolas Saenz Julienne
Both functions checked the minor number of the cdev prior running the code. This was useless since the number of devices is already limited by alloc_chrdev_region. This removes the check and reindents the code where relevant. Signed-off-by: Nicolas Saenz Julienne --- .../interface/vchiq_arm/vch

Re: [PATCH v1 1/1] arm64: Early boot time stamps

2018-11-20 Thread Andrew Murray
On Tue, Nov 20, 2018 at 09:40:10AM -0500, Pavel Tatashin wrote: > > > +static __init void sched_clock_early_init(void) > > > +{ > > > + u64 freq = arch_timer_get_cntfrq(); > > > + u64 (*read_time)(void) = arch_counter_get_cntvct; > > > > We already have arch_timer_read_counter which is expo

Re: [RFC PATCH 1/3] mm, memory_hotplug: try to migrate full section worth of pages

2018-11-20 Thread osalvador
On Tue, 2018-11-20 at 14:43 +0100, Michal Hocko wrote: > From: Michal Hocko > > do_migrate_range has been limiting the number of pages to migrate to > 256 > for some reason which is not documented. When looking back at old memory-hotplug commits one feels pretty sad about the brevity of the cha

Re: [PATCH v3 1/2] x86/fpu: track AVX-512 usage of tasks

2018-11-20 Thread Samuel Neves
On Tue, Nov 20, 2018 at 1:32 PM Li, Aubrey wrote: > Thanks for your program, Samuel, it's very helpful. But I saw a different > output on my side, May I have your glibc version? > This system is running glibc 2.27, and kernel 4.18.7. The Xeon Gold 5120 also happens to be one of the Skylake-SP mod

Re: Applied "regulator: wm8994: Pass descriptor instead of GPIO number" to the regulator tree

2018-11-20 Thread Charles Keepax
On Tue, Nov 20, 2018 at 02:43:32PM +0100, Marek Szyprowski wrote: > On 2018-05-17 18:41, Mark Brown wrote: > > Subject: [PATCH] regulator: wm8994: Pass descriptor instead of GPIO number > > This patch causes following kernel warning on Samsung Exynos4412 based > Trats2 board: > > wm8994 4-001a: F

Re: [PATCH] watchdog: cpwd: add of_node_put()

2018-11-20 Thread Frank Lee
On Tue, Nov 20, 2018 at 10:34 PM Guenter Roeck wrote: > > On 11/20/18 5:23 AM, Yangtao Li wrote: > > use of_node_put() to release the refcount. > > > > Signed-off-by: Yangtao Li > > --- > > drivers/watchdog/cpwd.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/watchdog/

[PATCH v2 1/1] arm64: Early boot time stamps

2018-11-20 Thread Pavel Tatashin
Allow printk time stamps/sched_clock() to be available from the early boot. Signed-off-by: Pavel Tatashin --- arch/arm64/kernel/setup.c| 25 + drivers/clocksource/arm_arch_timer.c | 8 include/clocksource/arm_arch_timer.h | 3 +++ 3 files changed, 3

[PATCH v2 0/1] Early boot time stamps for arm64

2018-11-20 Thread Pavel Tatashin
Changelog: v1 - v2 Addressed comments from Marc Zyngier I made early boot time stamps available for SPARC and X86. x86: https://lore.kernel.org/lkml/20180719205545.16512-1-pasha.tatas...@oracle.com sparc: https://www.spinics.net/lists/sparclinux/msg18063.html As discussed at plumbers,

Re: [PATCH v1 1/1] arm64: Early boot time stamps

2018-11-20 Thread Pavel Tatashin
> > +static __init void sched_clock_early_init(void) > > +{ > > + u64 freq = arch_timer_get_cntfrq(); > > + u64 (*read_time)(void) = arch_counter_get_cntvct; > > We already have arch_timer_read_counter which is exposed from > arm_arch_timer.h. OK > > > + > > + /* Early clock is availa

Re: [PATCH v2 3/3] PCI: imx6: limit DBI register length

2018-11-20 Thread Lucas Stach
Am Dienstag, den 20.11.2018, 14:27 +0100 schrieb Stefan Agner: > Define the length of the DBI registers. This makes sure that > the kernel does not access registers beyond that point, avoiding > the following abort on a i.MX 6Quad: >   # cat /sys/devices/soc0/soc/1ffc000.pcie/pci\:00/\:00\:

Re: [PATCH v2 2/3] PCI: imx6: introduce drvdata

2018-11-20 Thread Lucas Stach
Am Dienstag, den 20.11.2018, 14:27 +0100 schrieb Stefan Agner: > Introduce driver data struct. This will simplify handling of device > specific differences. > > Signed-off-by: Stefan Agner Reviewed-by: Lucas Stach > --- > Changes in v2: > - Split drvdata introduction in a separate patch > - Us

Re: [PATCH v2 1/3] PCI: dwc: allow to limit registers set length

2018-11-20 Thread Lucas Stach
Am Dienstag, den 20.11.2018, 14:27 +0100 schrieb Stefan Agner: > Add length to the struct dw_pcie and check that the accessors > dw_pcie_(rd|wr)_own_conf() do not read/write beyond that point. > > Signed-off-by: Stefan Agner FWIW: Reviewed-by: Lucas Stach > --- >  drivers/pci/controller/dwc/p

Re: [RFC PATCH 2/3] mm, memory_hotplug: deobfuscate migration part of offlining

2018-11-20 Thread David Hildenbrand
On 20.11.18 15:34, Michal Hocko wrote: > On Tue 20-11-18 15:26:43, David Hildenbrand wrote: > [...] >>> + do { >>> + for (pfn = start_pfn; pfn;) >>> + { >> >> { on a new line looks weird. >> >>> + /* start memory hot removal */ >>> + ret = -

Re: [PATCH] watchdog: cpwd: add of_node_put()

2018-11-20 Thread Guenter Roeck
On 11/20/18 5:23 AM, Yangtao Li wrote: use of_node_put() to release the refcount. Signed-off-by: Yangtao Li --- drivers/watchdog/cpwd.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/watchdog/cpwd.c b/drivers/watchdog/cpwd.c index aee0b25cf10d..d3a81642ada5 100644 --- a/drivers/

[PATCH v3] usb: hub: add retry routine after intr URB submit error

2018-11-20 Thread Nicolas Saenz Julienne
The hub sends hot-plug events to the host trough it's interrupt URB. The driver takes care of completing the URB and re-submitting it. Completion errors are handled in the hub_event() work, yet submission errors are ignored, rendering the device unresponsive. All further events are lost. It is fai

<    1   2   3   4   5   6   7   8   9   >