Re: PROBLEM: NULL pointer dereference in kernel 4.14.6

2017-12-17 Thread vcaputo
On Sun, Dec 17, 2017 at 05:49:44PM +, Bronek Kozicki wrote: > I just upgraded to 4.14.7 and tried to reproduce this error, this time under > strace. As you can see this happens when systemctl tries to read a specific > entry under /sys/fs . In case this matters, the entry is for a small virtu

Re: [PATCH v3 2/2] drm: rcar-du: calculate DPLLCR to be more small jitter

2017-12-17 Thread Kuninori Morimoto
Hi Geert > >> > From: Kuninori Morimoto > >> > In general, PLL has VCO (= Voltage controlled oscillator), > >> > one of the very important electronic feature called as "jitter" > >> > is related to this VCO. > >> > In academic generalism, VCO should be maximum to be more small jitter. > >> > In

Re: [Y2038] [PATCH v2 08/10] fix get_timespec64() for y2038 safe compat interfaces

2017-12-17 Thread Ben Hutchings
On Fri, 2017-12-15 at 13:02 +0100, Arnd Bergmann wrote: [...] > - I had an idea to handle the copying of timespec/timeval with a > one-size-fits-all >   function and multiple wrappers around it, such as > > enum user_ts_type { >   USER_TS_TIMEVAL = 1, >   USER_TS_32 = 2, >   USER_TS_CL

[PATCH 0/3] kallsyms: don't leak address

2017-12-17 Thread Tobin C. Harding
This set plugs a kernel address leak that occurs if kallsyms symbol look up fails. This set was prompted by a leaking address found using scripts/leaking_addresses.pl on a PowerPC machine in the wild. Patch set does not change behaviour when KALLSYMS is not defined (suggested by Linus). RFC has b

[PATCH 3/3] trace: print address if symbol not found

2017-12-17 Thread Tobin C. Harding
Fixes behaviour modified by: commit bd6b239cdbb2 ("kallsyms: don't leak address when symbol not found") Previous patch changed behaviour of kallsyms function sprint_symbol() to return an error code instead of printing the address if a symbol was not found. Ftrace relies on the original behaviour.

[PATCH 1/3] kallsyms: don't leak address when symbol not found

2017-12-17 Thread Tobin C. Harding
Currently if kallsyms_lookup() fails to find the symbol then the address is printed. This potentially leaks sensitive information. Instead of printing the address we can return an error, giving the calling code the option to print the address or print some sanitized message. Return error instead o

[PATCH 2/3] vsprintf: print if symbol not found

2017-12-17 Thread Tobin C. Harding
Depends on: commit bd6b239cdbb2 ("kallsyms: don't leak address when symbol not found") Currently vsprintf for specifiers %p[SsB] relies on the behaviour of kallsyms (sprint_symbol()) and prints the actual address if a symbol is not found. Previous patch changes this behaviour so tha sprint_symbol(

Re: [PATCH v2 0/3] doc: update printk documentation

2017-12-17 Thread Tobin C. Harding
On Sun, Dec 17, 2017 at 02:34:25PM -0800, Joe Perches wrote: > On Mon, 2017-12-18 at 09:30 +1100, Tobin C. Harding wrote: > > This set converts printk-formats.txt -> core-api/printk-formats.rst > > > > We also update the documentation around printing kernel addresses. > > Please also update the c

Re: [PATCH 2/3] vsprintf: print if symbol not found

2017-12-17 Thread Joe Perches
On Mon, 2017-12-18 at 10:53 +1100, Tobin C. Harding wrote: > Depends on: commit bd6b239cdbb2 ("kallsyms: don't leak address when > symbol not found") > > Currently vsprintf for specifiers %p[SsB] relies on the behaviour of > kallsyms (sprint_symbol()) and prints the actual address if a symbol is >

[trivial PATCH] treewide: Align function definition open/close braces

2017-12-17 Thread Joe Perches
Some functions definitions have either the initial open brace and/or the closing brace outside of column 1. Move those braces to column 1. This allows various function analyzers like gnu complexity to work properly for these modified functions. Miscellanea: o Remove extra trailing ; and blank l

[PATCH v4 0/2] drm: rcar-du: calculate DPLLCR to be more small jitter

2017-12-17 Thread Kuninori Morimoto
Hi Laurent, David These are v4 of DPLLCR patch for rcar-du. Mainly fixuped 2000 -> 2kHz to unambiguous Kuninori Morimoto (2): drm: rcar-du: use 1000 to avoid misunderstanding in rcar_du_dpll_divider() drm: rcar-du: calculate DPLLCR to be more small jitter drivers/gpu/drm/rcar-du/rcar_du_cr

[PATCH v4 1/2] drm: rcar-du: use 1000 to avoid misunderstanding in rcar_du_dpll_divider()

2017-12-17 Thread Kuninori Morimoto
From: Kuninori Morimoto It is difficult to understand its scale if number has many 0s. This patch uses "* 1000" to avoid it in rcar_du_dpll_divider(). Signed-off-by: Kuninori Morimoto --- v3 -> v4 - no change drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 2 +- 1 file changed, 1 insertion(+), 1 d

[PATCH v4 2/2] drm: rcar-du: calculate DPLLCR to be more small jitter

2017-12-17 Thread Kuninori Morimoto
From: Kuninori Morimoto In general, PLL has VCO (= Voltage controlled oscillator), one of the very important electronic feature called as "jitter" is related to this VCO. In academic generalism, VCO should be maximum to be more small jitter. In high frequency clock, jitter will be large impact.

Re: [PATCH v2] ARM: dts: add reset property for rk3066a-rayeager emac phy

2017-12-17 Thread Chris Zhong
Hi Heiko Thanks :) On 2017??12??17?? 04:11, Heiko Stuebner wrote: Hi Chris, Am Mittwoch, 8. November 2017, 17:50:41 CET schrieb Chris Zhong: The ethernet phy of rk3066a-rayeager has a reset pin, it controlled by GPIO1_D6, this pin should be pull down then pull up to reset the phy. Add a rese

[PATCH 01/15] staging: lustre: lnet-lib: opencode some alloc/free functions.

2017-12-17 Thread NeilBrown
These functions just call LIBCFS_ALLOC() which in-turn calls kvmalloc(). In none of these cases is the 'vmalloc' option needed. LIBCFS_ALLOC also produces a warning if NULL is returned, but that can be provided with CONFIG_SLAB_DEBUG. LIBCFS_ALLOC zeros the memory, so we need to use __GFP_ZERO to

[PATCH SERIES 1: 00/15] staging:lustre: convert most LIBCFS*ALLOC to k*malloc

2017-12-17 Thread NeilBrown
Lustre has some "convenience" macros for allocating and freeing memory. They: - warn if called from interrupt context - use kvmalloc - assume GFP_NOFS (which kvmalloc doesn't support) - print an error if the allocation fails - initialize the memory to zeroes. though the _ATOMIC version skips

[PATCH 02/15] staging: lustre: lnet: discard CFS_ALLOC_PTR

2017-12-17 Thread NeilBrown
These trivial wrappers hurt readability and as they use kvmalloc, they are overly generic. So discard them and use kmalloc/kfree as is normal in Linux. Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_private.h |3 --- drivers/staging/lustre/lnet/lnet/net_fault.c

[PATCH 05/15] staging: lustre: lnet: selftest: don't allocate small strings.

2017-12-17 Thread NeilBrown
All of the "name" buffers here are at most LST_NAME_SIZE+1 bytes, so 33 bytes at most. They are only used temporarily during the life of the function that allocates them. So it is much simpler to just allocate on the stack. Worst case is lst_tet_add_ioct(), which allocates 3 for these which 99 byte

[PATCH 04/15] staging: lustre: lnet: switch to cpumask_var_t

2017-12-17 Thread NeilBrown
So that we can use the common cpumask allocation functions, switch to cpumask_var_t. We need to be careful not the free a cpumask_var_t until the variable has been initialized, and it cannot be initialized directly. So we must be sure either that it is filled with zeros, or that zalloc_cpumask_var(

[PATCH 07/15] staging: lustre: change some LIBCFS_ALLOC calls to k?alloc(GFP_KERNEL)

2017-12-17 Thread NeilBrown
When an allocation happens from process context rather than filesystem context, it is best to use GFP_KERNEL rather than LIBCFS_ALLOC() which always uses GFP_NOFS. This include initialization during, or prior to, mount, and code run from separate worker threads. So for some of these cases, switch

[PATCH 03/15] staging: lustre: replace simple cases of LIBCFS_ALLOC with kzalloc.

2017-12-17 Thread NeilBrown
All usages of the form LIBCFS_ALLOC(variable, sizeof(variable)) or LIBCFS_ALLOC(variable, sizeof(variable's-type)) are changed to variable = kzalloc(sizeof(...), GFP_NOFS); Similarly, all LIBCFS_FREE(variable, sizeof(variable)) become kfree(variable); None of these need the vmalloc o

[PATCH 06/15] staging: lustre: lnet: use kmalloc/kvmalloc in router_proc

2017-12-17 Thread NeilBrown
The buffers allocated in router_proc are to temporarily hold strings created for procfs files. So they do not need to be zeroed and are safe to use GFP_KERNEL. So use kmalloc() directly except in two cases where it isn't trivial to confirm that the size is always small. In those cases, use kvmalloc

[PATCH 09/15] staging: lustre: more LIBCFS_ALLOC conversions to GFP_KERNEL allocations.

2017-12-17 Thread NeilBrown
None of these need GFP_NOFS so allocate directly. Change matching LIBCFS_FREE() to kfree() or kvfree(). Signed-off-by: NeilBrown --- .../staging/lustre/lnet/libcfs/linux/linux-cpu.c | 19 +++ drivers/staging/lustre/lnet/lnet/lib-eq.c |9 - drivers/staging

[PATCH 08/15] staging: lustre: Convert more LIBCFS_ALLOC allocation to direct GFP_KERNEL

2017-12-17 Thread NeilBrown
None of these need to be GFP_NOFS, so use GFP_KERNEL explicitly with kmalloc(), kvmalloc(), or kvmalloc_array(). Changing matching LIBCFS_FREE() to kfree() or kvfree() Signed-off-by: NeilBrown --- .../lustre/lnet/libcfs/linux/linux-module.c|4 +- drivers/staging/lustre/lnet/libcfs/mo

[PATCH 10/15] staging: lustre: more conversions to GFP_KERNEL allocations.

2017-12-17 Thread NeilBrown
These are not called from filesystem context, so use GFP_KERNEL, not LIBCFS_ALLOC(). Signed-off-by: NeilBrown --- drivers/staging/lustre/lnet/selftest/console.c | 57 +--- drivers/staging/lustre/lnet/selftest/framework.c |4 + drivers/staging/lustre/lnet/selftest/modu

[PATCH 11/15] staging: lustre: lnet-route: use kmalloc for small allocation

2017-12-17 Thread NeilBrown
This allocation is reasonably small. As the function is called "*_locked", it might not be safe to perform a GFP_KERNEL allocation, so be safe and use GFP_NOFS. Signed-off-by: NeilBrown --- drivers/staging/lustre/lnet/lnet/router.c |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) d

[PATCH 14/15] staging: lustre: opencode LIBCFS_ALLOC_ATOMIC calls.

2017-12-17 Thread NeilBrown
Just call kzalloc(GFP_ATOMIC) directly. We don't need the warning on failure. Signed-off-by: NeilBrown --- .../lustre/lustre/obdclass/lprocfs_status.c|7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/obdclass/lprocfs_status.c b/d

[PATCH 13/15] staging: lustre: cfs_percpt_alloc: use kvmalloc(GFP_KERNEL)

2017-12-17 Thread NeilBrown
this allocation is called from several places, but all are during initialization, so GFP_NOFS is not needed. So use kvmalloc and GFP_KERNEL. Signed-off-by: NeilBrown --- drivers/staging/lustre/lnet/libcfs/libcfs_mem.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/d

[PATCH 12/15] staging: lustre: use kmalloc for allocating ksock_tx

2017-12-17 Thread NeilBrown
The size of the data structure is primarily controlled by the iovec size, which is limited to 256. Entries in this vector are 12 bytes, so the whole will always fit in a page. So it is safe to use kmalloc (kvmalloc not needed). So replace LIBCFS_ALLOC with kmalloc. Signed-off-by: NeilBrown --- .

[PATCH 15/15] staging: lustre: remove LIBCFS_ALLOC and LIBCFS_ALLOC_ATOMIC

2017-12-17 Thread NeilBrown
These are no longer used. Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_private.h | 19 --- 1 file changed, 19 deletions(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h b/drivers/staging/lustre/include/linux/libcfs/libcfs_p

Re: [PATCH 2/3] vsprintf: print if symbol not found

2017-12-17 Thread Tobin C. Harding
On Sun, Dec 17, 2017 at 04:04:14PM -0800, Joe Perches wrote: > On Mon, 2017-12-18 at 10:53 +1100, Tobin C. Harding wrote: > > Depends on: commit bd6b239cdbb2 ("kallsyms: don't leak address when > > symbol not found") > > > > Currently vsprintf for specifiers %p[SsB] relies on the behaviour of > >

[PATCH] input/touchscreen: fix hideep.c build errors

2017-12-17 Thread Randy Dunlap
From: Randy Dunlap Fix build errors due to missing header file. Fixes these build errors: drivers/input//touchscreen/hideep.c: In function 'hideep_power_on': drivers/input//touchscreen/hideep.c:670:3: error: implicit declaration of function 'gpiod_set_value_cansleep'; did you mean 'gpio_set_val

[PATCH 1/4] staging: lustre: obdclass: discard FS_NEEDS_DEV flag.

2017-12-17 Thread NeilBrown
Lustre mounts do not need a dev, as we can see from lustre_mount() calling mount_nodev(). So remove the flag which could cause confusion elsewhere. Also format lustre_fs_types so that fields line up. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/obdclass/obd_mount.c | 10 +--

[PATCH SERIES 2: 0/4] staging:lustre: improve mounting code

2017-12-17 Thread NeilBrown
These patches make improvements to the lustre code for supporting the mount system call. There is a user-visible change: "lustre" now appears in /proc/filesystem with the "nodev" flag present. Previously it didn't have this flag, which was incorrect. There is also a bugfix: there is a possible m

[PATCH 2/4] staging: lustre: obdclass: remove pointless struct lustre_mount_data2

2017-12-17 Thread NeilBrown
This is used to pass a void* and NULL to lustre_fill_super(). It is easier just to pass the void*. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/obdclass/obd_mount.c | 19 --- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/drivers/staging/lustre/lus

[PATCH 4/4] staging: lustre: obd_mount: fix possible race with module unload.

2017-12-17 Thread NeilBrown
lustre_fill_super() calls client_fill_super() without holding a reference to the module containing client_fill_super. If that module is unloaded at a bad time, this can crash. To be able to get a reference to the module using try_get_module(), we need a pointer to the module. So replace lustre

[PATCH 3/4] staging: lustre: obdclass: remove vfsmount option from client_fill_super

2017-12-17 Thread NeilBrown
This arg is always NULL and is never used. So discard it from this and related functions. Signed-off-by: NeilBrown --- .../staging/lustre/lustre/include/lustre_disk.h|3 +-- .../staging/lustre/lustre/llite/llite_internal.h |2 +- drivers/staging/lustre/lustre/llite/llite_lib.c|

[PATCH SERIES 3: 0/4] staging:lustre: remove workitem code

2017-12-17 Thread NeilBrown
Lustre has a "workitem" subsystem with much the same functionality as the Linux workqueue subsystem. This patch converts all users of workitem to workqueue, then removes workitem. This requires that "apply_workqueue_attrs" be exported to modules, so this intro and the patch which does the EXPORT_S

[PATCH 1/4] staging: lustre: libcfs: use a workqueue for rehash work.

2017-12-17 Thread NeilBrown
lustre has a work-item queuing scheme that provides the same functionality as linux work_queues. To make the code easier for linux devs to follow, change to use work_queues. Signed-off-by: NeilBrown --- .../staging/lustre/include/linux/libcfs/libcfs.h |2 .../lustre/include/linux/libcfs/l

[PATCH 3/4] staging: lustre: lnet: convert selftest to use workqueues

2017-12-17 Thread NeilBrown
Instead of the cfs workitem library, use workqueues. As lnet wants to provide a cpu mask of allowed cpus, it needs to be a WQ_UNBOUND work queue so that tasks can run on cpus other than where they were submitted. apply_workqueue_atts needs to be exported for lustre to use it. Signed-off-by: Neil

[PATCH 2/4] staging: lustre: libcfs: remove wi_data from cfs_workitem

2017-12-17 Thread NeilBrown
In every case, the value passed via wi_data can be determined from the cfs_workitem pointer using container_of(). So use container_of(), and discard wi_data. Signed-off-by: NeilBrown --- .../lustre/include/linux/libcfs/libcfs_workitem.h |5 + drivers/staging/lustre/lnet/selftest/framew

[PATCH 4/4] staging: lustre: libcfs: remove workitem code.

2017-12-17 Thread NeilBrown
There are now no users. workqueues are doing the job that this used to do. Signed-off-by: NeilBrown --- .../staging/lustre/include/linux/libcfs/libcfs.h |1 .../lustre/include/linux/libcfs/libcfs_workitem.h | 104 drivers/staging/lustre/lnet/libcfs/Makefile|2 drivers

[PATCH SERIES 4: 0/4] staging: lustre: use standard prng

2017-12-17 Thread NeilBrown
Lustre has its own internal PRNG code. This adds nothing of value to the Linux standard prng code, so switch over to using the standard interfaces. This adds a few callers to add_device_randomness(), which helps everyone, and removes unnecessary code. Thanks, NeilBrown --- NeilBrown (4):

[PATCH 2/4] staging: lustre: replace cfs_srand() calls with add_device_randomness().

2017-12-17 Thread NeilBrown
The only places that cfs_srand is called, the random bits are mixed with bits from get_random_bytes(). So it is equally effective to add entropy to either pool. So we can replace calls to cfs_srand() with calls that add the entropy with add_device_randomness(). That function adds time-based entro

[PATCH 4/4] staging: lustre: libcfs: remove prng

2017-12-17 Thread NeilBrown
The cfs prng is no longer used, so discard it. Signed-off-by: NeilBrown --- .../staging/lustre/include/linux/libcfs/libcfs.h | 10 - drivers/staging/lustre/lnet/libcfs/Makefile|2 drivers/staging/lustre/lnet/libcfs/prng.c | 137 3 files changed, 1

[PATCH 1/4] staging: lustre: replace cfs_rand() with prandom_u32_max()

2017-12-17 Thread NeilBrown
All occurrences of cfs_rand() % X are replaced with prandom_u32_max(X) cfs_rand() is a simple Linear Congruential PRNG. prandom_u32_max() is at least as random, is seeded with more randomness, and uses cpu-local state to avoid cross-cpu issues. This is the first step is discarding the libc

[PATCH 3/4] staging: lustre: replace cfs_get_random_bytes calls with get_random_byte()

2017-12-17 Thread NeilBrown
The cfs_get_random_bytes() interface adds nothing of value to get_random_byte() (which it uses internally). So just use the standard interface. Signed-off-by: NeilBrown --- drivers/staging/lustre/lustre/include/obd_class.h |2 +- .../lustre/lustre/obdclass/lustre_handles.c|2 +-

[PATCH V1 1/1] mtd: mtk-nor: modify functions' name more generally

2017-12-17 Thread Guochun Mao
Since more and more Mediatek's SoC can use this driver to control spi-nor flash, functions' name with "mt8173_" is no longer properly. Replacing "mt8173_" with "mtk_" will be more accurate to describe these functions' usable scope. Guochun Mao (1): mtd: mtk-nor: modify functions' name more gener

[PATCH] mtd: mtk-nor: modify functions' name more generally

2017-12-17 Thread Guochun Mao
Since more and more Mediatek's SoC can use this driver to control spi-nor flash, functions' name with "mt8173_" is no longer properly. Replacing "mt8173_" with "mtk_" will be more accurate to describe these functions' usable scope. Signed-off-by: Guochun Mao --- drivers/mtd/spi-nor/mtk-quadspi.c

Re: [PATCH] PCI: designware: add a check of msi_desc in irqchip

2017-12-17 Thread Cao Zou
On 12/16/2017 01:20 AM, Marc Zyngier wrote: On 15/12/17 16:17, Lorenzo Pieralisi wrote: [+Marc] On Thu, Dec 14, 2017 at 10:21:23AM +0800, cao@windriver.com wrote: From: Zou Cao When PCIE host setup, 32 MSI irq descriptions are created, but its msi_desc is NULL, msi_desc is bound in MSI

[bugzilla-dae...@bugzilla.kernel.org: [Bug 198181] New: BUG at drivers/pci/msi.c:352 when unplugging a sysfs authorized thunderbolt 3 dock]

2017-12-17 Thread Bjorn Helgaas
- Forwarded message from bugzilla-dae...@bugzilla.kernel.org - Date: Sun, 17 Dec 2017 15:04:51 + From: bugzilla-dae...@bugzilla.kernel.org To: bugzilla@gmail.com Subject: [Bug 198181] New: BUG at drivers/pci/msi.c:352 when unplugging a sysfs authorized thunderbolt 3 dock h

Re: [PATCH v19 3/7] xbitmap: add more operations

2017-12-17 Thread Wei Wang
On 12/18/2017 06:18 AM, Matthew Wilcox wrote: On Sun, Dec 17, 2017 at 01:47:21PM +, Wang, Wei W wrote: On Saturday, December 16, 2017 3:22 AM, Matthew Wilcox wrote: On Fri, Dec 15, 2017 at 10:49:15AM -0800, Matthew Wilcox wrote: - xbit_clear() can't return an error. Neither can xbit_zero

[bugzilla-dae...@bugzilla.kernel.org: [Bug 198171] New: [AMD][X399] Inconsistent PCIe lane linking count]

2017-12-17 Thread Bjorn Helgaas
- Forwarded message from bugzilla-dae...@bugzilla.kernel.org - Date: Fri, 15 Dec 2017 22:24:36 + From: bugzilla-dae...@bugzilla.kernel.org To: bugzilla@gmail.com Subject: [Bug 198171] New: [AMD][X399] Inconsistent PCIe lane linking count https://bugzilla.kernel.org/show_bug.cgi?id

drivers/gpu/drm/i915/gvt/handlers.c:2397:1: error: the frame size of 32120 bytes is larger than 8192 bytes

2017-12-17 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 779f4e1c6c7c661db40dfebd6dd6bda7b5f88aa3 commit: d17a1d97dc208d664c91cc387ffb752c7f85dc61 x86/mm/kasan: don't use vmemmap_populate() to initialize shadow date: 5 weeks ago config: x86_64-randconfig-x001-12

Re: [PATCH 1/5] media: rc: update sunxi-ir driver to get base clock frequency from devicetree

2017-12-17 Thread Andi Shyti
Hi Philipp, just a couple of small nitpicks. > + u32 b_clk_freq; [...] > + /* Base clock frequency (optional) */ > + if (of_property_read_u32(dn, "clock-frequency", &b_clk_freq)) { > + b_clk_freq = SUNXI_IR_BASE_CLK; > + } > + how about you intialize 'b_clk_freq' to

Re: [PATCH v2 2/3] doc: update kpt_restrict documentation

2017-12-17 Thread Randy Dunlap
On 12/17/2017 02:30 PM, Tobin C. Harding wrote: > Recently the behaviour of printk specifier %pK was changed. The > documentation does not currently mirror this. > > Update documentation for sysctl kpt_restrict. > > Signed-off-by: Tobin C. Harding > --- > Documentation/sysctl/kernel.txt | 3 ++-

RE: [v2] arm: dts: ls1021a: fix the value of TMR_FIPER1

2017-12-17 Thread Y.b. Lu
Hi Shawn, Sorry for bother. I just couldn’t find this patch on your git tree. Could you help to check? Thanks a lot. Best regards, Yangbo Lu > -Original Message- > From: Shawn Guo [mailto:shawn...@kernel.org] > Sent: 2017年11月30日 8:29 > To: Y.b. Lu > Cc: devicet...@vger.kernel.org; linux

[PATCH v4 01/11] ASoC: fsl_ssi: Rename fsl_ssi_private to fsl_ssi

2017-12-17 Thread Nicolin Chen
Shorten the private data structure to save some wrapped lines. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 456 +++- 1 file changed, 220 insertions(+), 236 deletions(-) di

[PATCH v4 00/11] ASoC: fsl_ssi: Clean up - coding style level

2017-12-17 Thread Nicolin Chen
==Changelog== v3->v4 * Revised PATCH-03 "Refine all comments" by adding Timur's inputs * Rebased all other patches v2->v3 * Added Tested-by and Reviewed-by from Maciej * Revised PATCH-03 "Refine all comments" by adding Maciej's advice * Revised PATCH-05 "Refine indentations and wrappings" * R

[PATCH v4 07/11] ASoC: fsl_ssi: Rename cpu_dai parameter to dai

2017-12-17 Thread Nicolin Chen
Shortens the variable name to save space, useful for dev_err outputs. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/s

[PATCH v4 04/11] ASoC: fsl_ssi: Rename registers and fields macros

2017-12-17 Thread Nicolin Chen
This patch renames CCSR_SSI_xxx to REG_SSI_xxx and SSI_xxx_yyy style. It also slightly reduces the length of them to save some space. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 374 +-

[PATCH v4 06/11] ASoC: fsl_ssi: Refine printk outputs

2017-12-17 Thread Nicolin Chen
This patches unifies the error message in the "failed to " format. It also reduces the length of one line and adds spaces to an operator. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 16 1 file c

[PATCH v4 11/11] ASoC: fsl_ssi: Define ternary macros to simplify code

2017-12-17 Thread Nicolin Chen
Some regmap code looks redudant. So simplify it. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 27 +++ sound/soc/fsl/fsl_ssi.h | 4 2 files changed, 15 insertions(+), 16 deletions(-) diff

[PATCH v4 05/11] ASoC: fsl_ssi: Refine indentations and wrappings

2017-12-17 Thread Nicolin Chen
This patch just simply unifies the coding style. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 239 +--- sound/soc/fsl/fsl_ssi.h | 2 +- sound/soc/fsl/fsl_ssi_dbg.c |

[PATCH v4 10/11] ASoC: fsl_ssi: Rename i2smode to i2s_net

2017-12-17 Thread Nicolin Chen
Since this i2smode also includes the setting of Network mode, it should have it in the name. This patch also adds its MASK define. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 24 sound/soc/fs

[PATCH v4 02/11] ASoC: fsl_ssi: Cache pdev->dev pointer

2017-12-17 Thread Nicolin Chen
There should be no trouble to understand dev = pdev->dev. This can save some space to have more print info or save some wrapped lines. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 64 --

[PATCH v4 08/11] ASoC: fsl_ssi: Rename scr_val to scr

2017-12-17 Thread Nicolin Chen
Simplify the variable name. This reduces one over-80-character line. Signed-off-by: Nicolin Chen Tested-by: Maciej S. Szmigiero Reviewed-by: Maciej S. Szmigiero --- sound/soc/fsl/fsl_ssi.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/sound/soc/fsl/fsl_ssi.c

[PATCH v4 09/11] ASoC: fsl_ssi: Replace fsl_ssi_rxtx_reg_val with fsl_ssi_regvals

2017-12-17 Thread Nicolin Chen
The name fsl_ssi_rxtx_reg_val is too long to read comfortably. So this patch shortens it by using an array (fsl_ssi_regvals, renamed from fsl_ssi_reg_val). To do that, it also introduces two macros (TX and RX) to replace the wrapper structure. This will also help further cleanups. Meanwhile, it un

[PATCH v4 03/11] ASoC: fsl_ssi: Refine all comments

2017-12-17 Thread Nicolin Chen
This patch refines the comments by: 1) Removing all out-of-date comments 2) Removing all not-so-useful comments 3) Unifying the styles of all comments 4) Shortening comments to be more conise 5) Adding comments to improve code readablity 6) Moving all register related comments to fsl_ssi.h 7) Addin

Re: [PATCH] On ppc64le we HAVE_RELIABLE_STACKTRACE

2017-12-17 Thread Josh Poimboeuf
On Fri, Dec 15, 2017 at 07:40:09PM +1000, Nicholas Piggin wrote: > On Tue, 12 Dec 2017 08:05:01 -0600 > Josh Poimboeuf wrote: > > > On Tue, Dec 12, 2017 at 12:39:12PM +0100, Torsten Duwe wrote: > > > Hi all, > > > > > > The "Power Architecture 64-Bit ELF V2 ABI" says in section 2.3.2.3: > > > >

Re: [PATCH v19 3/7] xbitmap: add more operations

2017-12-17 Thread Matthew Wilcox
On Mon, Dec 18, 2017 at 10:33:00AM +0800, Wei Wang wrote: > > My only qualm is that I've been considering optimising the memory > > consumption when an entire 1024-bit chunk is full; instead of keeping a > > pointer to a 128-byte entry full of ones, store a special value in the > > radix tree which

Re: [PATCH v2] tools/perf/inject: fix dwarf support detection

2017-12-17 Thread Jiri Olsa
On Fri, Dec 08, 2017 at 09:02:18AM -0800, Stephane Eranian wrote: > I ran into problems trying to use the JIT support and display > source-level information. Basically, there was no dwarf debug > info generated in the jitted-XX.so files, yet I had libdw-dev > installed. > > Turns out that the feat

Re: [Xen-devel] [PATCH V3 1/2] Drivers/PCI: Export pcie_has_flr() interface

2017-12-17 Thread Alexey Kardashevskiy
On 16/12/17 05:18, Bjorn Helgaas wrote: > [+cc Russell, Sinan, Herbert, Srikanth, Derek, Satanand, Felix, Raghu] > > On Fri, Dec 15, 2017 at 09:48:02AM -0600, Govinda Tatti wrote: >> On 12/13/2017 3:24 PM, Bjorn Helgaas wrote: >>> On Wed, Dec 13, 2017 at 02:46:57PM -0600, Govinda Tatti wrote: > >

Re: AMD erratum 665 on f15h processor?

2017-12-17 Thread Andrew Randrianasulu
В сообщении от Sunday 17 December 2017 23:52:05 вы написали: > On Sun, Dec 17, 2017 at 12:04:28PM +0300, Andrew Randrianasulu wrote: > > Hello! > > > > I was trying to investigate why all my old kernels can't be booted on my > > relatively new machine. Kernels 4.10+ naturally boot - I use 4.14.3 ri

Re: [PATCH] net: phy: xgene: disable clk on error paths

2017-12-17 Thread Quan Nguyen
On Sat, Dec 16, 2017 at 4:52 AM, Alexey Khoroshilov wrote: > > There are several error paths in xgene_mdio_probe(), > where clk is left undisabled. The patch fixes them. > > Found by Linux Driver Verification project (linuxtesting.org). > > Signed-off-by: Alexey Khoroshilov > --- > drivers/net/p

Re: [PATCH v4 00/11] ASoC: fsl_ssi: Clean up - coding style level

2017-12-17 Thread Timur Tabi
On 12/17/17 8:51 PM, Nicolin Chen wrote: Nicolin Chen (11): ASoC: fsl_ssi: Rename fsl_ssi_private to fsl_ssi ASoC: fsl_ssi: Cache pdev->dev pointer ASoC: fsl_ssi: Refine all comments ASoC: fsl_ssi: Rename registers and fields macros ASoC: fsl_ssi: Refine indentations and wrappings

Re: [PATCH 3/5] arm: dts: sun8i: a83t: Add the ir pin for the A83T

2017-12-17 Thread Chen-Yu Tsai
On Mon, Dec 18, 2017 at 6:45 AM, Philipp Rossak wrote: > The CIR Pin of the A83T is located at PL12. > > Signed-off-by: Philipp Rossak > --- > arch/arm/boot/dts/sun8i-a83t.dtsi | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/arch/arm/boot/dts/sun8i-a83t.dtsi > b/arch/arm/boot/dts/

Re: [PATCH v2] perf report: Fix regression when decoding intelPT traces

2017-12-17 Thread Jiri Olsa
On Thu, Dec 14, 2017 at 10:47:00AM -0700, Mathieu Poirier wrote: > Commit (93d10af26bb7 perf tools: Optimize sample parsing for ordered > events) breaks intelPT trace decoding by invariably returning an error if > the event type isn't a PERF_SAMPLE_TIME. > > With this patch the timestamp is initia

Re: Detecting RWF_NOWAIT support

2017-12-17 Thread Goldwyn Rodrigues
On 12/16/2017 08:49 AM, Avi Kivity wrote: > > > On 12/14/2017 09:15 PM, Goldwyn Rodrigues wrote: >> >> On 12/14/2017 11:38 AM, Avi Kivity wrote: >>> I'm looking to add support for RWF_NOWAIT within a linux-aio iocb. >>> Naturally, I need to detect at runtime whether the kernel support >>> RWF_N

linux-next: build failure after merge of the tty tree

2017-12-17 Thread Stephen Rothwell
Hi Greg, After merging the tty tree, today's linux-next build (x86_64 allmodconfig) failed like this: .config:4300:warning: symbol value '' invalid for SERIAL_SH_SCI_NR_UARTS * * Restart config... * * * Serial drivers * 8250/16550 and compatible serial support (SERIAL_8250) [M/y/?] m .

Re: [PATCH 5/5] arm: dts: sun8i: a83t: bananapi-m3: Enable IR controller

2017-12-17 Thread Chen-Yu Tsai
On Mon, Dec 18, 2017 at 6:45 AM, Philipp Rossak wrote: > The Bananapi M3 has an onboard IR receiver. > This enables the onboard IR receiver subnode. > Other than the other IR receivers this one needs a base clock frequency Unlike the other... > of 300 Hz (3 MHz), to be able to work. > > Sign

Re: stack traces and zombie tasks

2017-12-17 Thread Josh Poimboeuf
On Fri, Dec 15, 2017 at 07:51:45AM -0800, Andy Lutomirski wrote: > On Fri, Dec 15, 2017 at 4:54 AM, Miroslav Benes wrote: > > Hi, > > > > commit 1959a60182f4 ("x86/dumpstack: Pin the target stack when dumping > > it") slightly changed the behaviour of stack traces dumping for zombie > > tasks. > >

Re: [PATCH 1/2] x86/stacktrace: do not fail when regs on stack for ORC

2017-12-17 Thread Josh Poimboeuf
On Thu, Dec 14, 2017 at 10:58:35PM +0100, Jiri Slaby wrote: > On 11/30/2017, 08:57 PM, Josh Poimboeuf wrote: > > So with those changes in mind, how about something like this (plus > > comments)? > > > > for (unwind_start(&state, task, NULL, NULL); !unwind_done(&state); > > unwind_next

Linux 4.15-rc4

2017-12-17 Thread Linus Torvalds
So it looks like 4.15 is finally calming down, with rc4 being about average size-wise for this time in the release process. Of course, we not only have the holiday season coming up, we *also* have some x86 entry and page table handling fixes pending. But that's not for today, and not for rc4. Let'

Re: [PATCH] On ppc64le we HAVE_RELIABLE_STACKTRACE

2017-12-17 Thread Balbir Singh
On Mon, Dec 18, 2017 at 1:58 PM, Josh Poimboeuf wrote: > On Fri, Dec 15, 2017 at 07:40:09PM +1000, Nicholas Piggin wrote: >> On Tue, 12 Dec 2017 08:05:01 -0600 >> Josh Poimboeuf wrote: >> >> > On Tue, Dec 12, 2017 at 12:39:12PM +0100, Torsten Duwe wrote: >> > > Hi all, >> > > >> > > The "Power Ar

[PATCH v4.1 2/4] kaslr: calculate the memory region in immovable node

2017-12-17 Thread Chao Fan
If there is no immovable memory region specified, use region directely. There are several conditons: 1. CONFIG_MEMORY_HOTPLUG is not specified to y. 2. immovable_mem= is not specified. Otherwise, calculate the intersecting between memmap entry and immovable memory. Rename process_mem_region to sl

Re: [PATCH net-next v6 2/2] net: ethernet: socionext: add AVE ethernet driver

2017-12-17 Thread Kunihiko Hayashi
Hello David, On Fri, 15 Dec 2017 12:57:49 -0500 David Miller wrote: > From: Kunihiko Hayashi > Date: Thu, 14 Dec 2017 19:05:10 +0900 > > > +static void ave_desc_write(struct net_device *ndev, enum desc_id id, > > + int entry, int offset, u32 val) > > +{ > > + struct ave_

Re: [RESEND PATCH v2] dmaengine: zynqmp_dma: Add runtime pm support

2017-12-17 Thread Vinod Koul
On Thu, Dec 07, 2017 at 10:59:57AM +0530, Kedareswara rao Appana wrote: > This patch adds runtime pm support in the driver. Applied, thanks -- ~Vinod

[PATCH] leaking_addresses: add generic 32-bit support

2017-12-17 Thread kaiwan . billimoria
The script attempts to detect the architecture it's running upon; as of now, we explicitly support x86_64, PPC64 and x86_32. If it's one of them, we proceed "normally". If we fail to detect the arch, we fallback to 64-bit scanning, unless the user has passed either of these option switches: "--32-b

Re: [net-next] phylib: Add device reset GPIO support causes DSA MT7530 acquires reset-gpios fails

2017-12-17 Thread Sean Wang
On Fri, 2017-12-15 at 11:10 +0100, Andrew Lunn wrote: > On Fri, Dec 15, 2017 at 02:55:03PM +0800, Sean Wang wrote: > > Hi Sergei, > > > > Recently I found the patch commit bafbdd527d56 (phylib: Add device reset > > GPIO support) would have the impact on MT7530 driver. Which causes the > > DSA MT75

Re: [PATCH] On ppc64le we HAVE_RELIABLE_STACKTRACE

2017-12-17 Thread Josh Poimboeuf
On Mon, Dec 18, 2017 at 02:39:06PM +1100, Balbir Singh wrote: > On Mon, Dec 18, 2017 at 1:58 PM, Josh Poimboeuf wrote: > > On Fri, Dec 15, 2017 at 07:40:09PM +1000, Nicholas Piggin wrote: > >> On Tue, 12 Dec 2017 08:05:01 -0600 > >> Josh Poimboeuf wrote: > >> > >> > On Tue, Dec 12, 2017 at 12:39:

Re: [PATCH v2 0/4] dmaengine: zynqmp_dma: Bug fixes

2017-12-17 Thread Vinod Koul
On Thu, Dec 07, 2017 at 10:54:24AM +0530, Kedareswara rao Appana wrote: > This patch series does the below > --> Fixes kernel doc format style issues. > --> Fixes warings in the driver. > --> Fixes issues with overflow interrupt. > --> Fixes race condition in the probe. Applied, thanks -- ~Vinod

Re: [PATCH v7 1/6] dmaengine: xilinx_dma: Check for channel idle state before submitting dma descriptor

2017-12-17 Thread Vinod Koul
On Thu, Dec 07, 2017 at 10:51:02AM +0530, Kedareswara rao Appana wrote: > @@ -2029,6 +2006,7 @@ static int xilinx_dma_terminate_all(struct dma_chan > *dchan) > > /* Remove and free all of the descriptors in the lists */ > xilinx_dma_free_descriptors(chan); > + chan->idle = true;

Re: [PATCH 13/25] arm: spear: dts: Remove leading 0x and 0s from bindings notation

2017-12-17 Thread Viresh Kumar
On 15-12-17, 13:46, Mathieu Malaterre wrote: > Improve the DTS files by removing all the leading "0x" and zeros to fix the > following dtc warnings: > > Warning (unit_address_format): Node /XXX unit name should not have leading > "0x" > > and > > Warning (unit_address_format): Node /XXX unit na

Re: [PATCH] arm64: defconfig: Select schedutil as default cpufreq governor

2017-12-17 Thread Viresh Kumar
On 15-12-17, 15:50, Catalin Marinas wrote: > On Thu, Nov 16, 2017 at 11:51:36AM +0530, Viresh Kumar wrote: > > Currently performance governor is getting selected by default, which is > > surely not a very good choice as its pretty much power hungry. > > > > Select schedutil instead. > > And why d

Re: [PATCH 2/4] sched: cpufreq: Keep track of cpufreq utilization update flags

2017-12-17 Thread Viresh Kumar
On 17-12-17, 01:19, Rafael J. Wysocki wrote: > We can do that in principle, but why should it return early? Maybe it's > a good time to update things, incidentally? > > I actually don't like the SCHED_CPUFRREQ_CLEAR flag *concept* as it is very > much specific to schedutil and blatantly ignores e

Re: [PATCH v2 0/5] Support for generalized use of make C={1,2} via a wrapper program

2017-12-17 Thread Jason Gunthorpe
On Sun, Dec 17, 2017 at 03:14:10AM +0100, Knut Omang wrote: > > I like the ability to add more checkers and keep then in the main > > upstream tree. But adding overrides for specific subsystems goes against > > the policy that all subsystems should be treated equally. > > This is a tool to enable

linux-next: manual merge of the akpm-current tree with the tip tree

2017-12-17 Thread Stephen Rothwell
Hi Andrew, Today's linux-next merge of the akpm-current tree got a conflict in: kernel/fork.c between commit: 5e28fd0b5fdb ("arch: Allow arch_dup_mmap() to fail") from the tip tree and commit: 120bd8608675 ("include/linux/sched/mm.h: uninline mmdrop_async(), etc") from the akpm-current

Re: [PATCH] cpufreq: powernv: Add support of frequency domain

2017-12-17 Thread Abhishek
On 12/14/2017 10:12 AM, Viresh Kumar wrote: + Gautham, @Gautham: Can you please help reviewing this one ? On 13-12-17, 13:49, Abhishek Goel wrote: @@ -693,6 +746,8 @@ static int powernv_cpufreq_target_index(struct cpufreq_policy *policy, { struct powernv_smp_call_data freq_data;

Re: [Y2038] [PATCH v2 08/10] fix get_timespec64() for y2038 safe compat interfaces

2017-12-17 Thread Deepa Dinamani
On Fri, Dec 15, 2017 at 4:02 AM, Arnd Bergmann wrote: > On Fri, Dec 15, 2017 at 1:21 AM, Ben Hutchings > wrote: >> On Mon, 2017-11-27 at 11:30 -0800, Deepa Dinamani wrote: >>> get/put_timespec64() interfaces will eventually be used for >>> conversions between the new y2038 safe struct __kernel_ti

<    1   2   3   4   5   >