Re: commit cfafcd117 "futex: Rework futex_lock_pi() to use rt_mutex_*_proxy_lock()" causes glibc nptl/tst-robustpi8 failure

2017-05-17 Thread Peter Zijlstra
On Thu, May 18, 2017 at 08:46:17AM +0200, Peter Zijlstra wrote: > On Wed, May 17, 2017 at 07:50:31PM +0200, Florian Weimer wrote: > > On 05/17/2017 07:36 PM, Markus Trippelsdorf wrote: > > > Since: > > > commit cfafcd117da0216520568c195cb2f6cd1980c4bb > > > Author: Peter Zijlstra > > > Date: Wed

Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-17 Thread Christoph Hellwig
Looks good, Reviewed-by: Christoph Hellwig

Re: [PATCH] scsi: zero per-cmd driver data for each MQ I/O

2017-05-17 Thread Christoph Hellwig
On Wed, May 17, 2017 at 11:05:18PM +, Bart Van Assche wrote: > Thank you for the feedback. I'm working on a patch series that merges the > scsi-sq > and scsi-mq code paths for command initialization and that should fix the bug > you > encountered. While that sounds great (I tried it a while

Re: [PATCH] clk: sunxi-ng: enable SUNXI_CCU_MP for PRCM

2017-05-17 Thread Maxime Ripard
On Thu, May 18, 2017 at 10:25:23AM +0800, Chen-Yu Tsai wrote: > On Thu, May 18, 2017 at 5:19 AM, Arnd Bergmann wrote: > > The newly added PRCM CCU driver uses SUNXI_CCU_MP_WITH_MUX_GATE, which > > causes > > a link error when no other driver enables SUNXI_CCU_MP: > > > > drivers/clk/built-in.o:(.

Re: tty: closing n_gsm line discipline always times out

2017-05-17 Thread Sascha Hauer
On Wed, May 17, 2017 at 04:23:58PM +0100, Alan Cox wrote: > On Wed, 17 May 2017 15:44:56 +0200 > Sascha Hauer wrote: > > > Hi All, > > > > When the n_gsm line discipline is closed it wants to shutdown the line > > discipline properly and asks the link partner to end the mux protocol. This > > i

Re: [PATCH v2] iio: adc: sun4i-gpadc-iio: fix parent device being used in devm function

2017-05-17 Thread Maxime Ripard
On Thu, May 18, 2017 at 08:36:07AM +0200, Quentin Schulz wrote: > For the sake of DT binding stability, this IIO driver is a child of an > MFD driver for Allwinner A10, A13 and A31 because there already exists a > DT binding for this IP. The MFD driver has a DT node but the IIO driver > does not. >

[PATCH] x86/mm/KASLR: Do not adapt the size of the direct mapping section for SGI UV system

2017-05-17 Thread Baoquan He
On SGI UV system, kernel casually hang with kaslr enabled. The back trace is: kernel BUG at arch/x86/mm/init_64.c:311! invalid opcode: [#1] SMP [...] RIP: 0010:__init_extra_mapping+0x188/0x196 [...] Call Trace: init_extra_mapping_uc+0x13/0x15 map_high+0x67/0x75 map_mmioh_high_uv3+0x20a/0x

Re: [PATCH v2 1/2] dt-bindings: mfd: Add DT bindings documentation for the max7360 mfd driver

2017-05-17 Thread Lee Jones
On Sun, 14 May 2017, Valentin Sitdikov wrote: > This patch adds documentation for the max7360 bindings. > The max7360 is multifunctional device containing gpio, > keypad, pwm and rotary encoder submodules. > > Signed-off-by: Valentin Sitdikov > Signed-off-by: Andrei Dranitca > --- > Documentat

Re: [PATCH] drm/amd: include instead of "linux/delay.h"

2017-05-17 Thread Christian König
Am 18.05.2017 um 06:43 schrieb Masahiro Yamada: Use <...> notation to include headers located in include/linux. While we are here, tweak the includes order a bit to sort them alphabetically. Signed-off-by: Masahiro Yamada Reviewed-by: Christian König --- drivers/gpu/drm/amd/powerplay/hw

Re: commit cfafcd117 "futex: Rework futex_lock_pi() to use rt_mutex_*_proxy_lock()" causes glibc nptl/tst-robustpi8 failure

2017-05-17 Thread Peter Zijlstra
On Wed, May 17, 2017 at 07:50:31PM +0200, Florian Weimer wrote: > On 05/17/2017 07:36 PM, Markus Trippelsdorf wrote: > > Since: > > commit cfafcd117da0216520568c195cb2f6cd1980c4bb > > Author: Peter Zijlstra > > Date: Wed Mar 22 11:35:58 2017 +0100 > > > > futex: Rework futex_lock_pi() to us

[PATCH v1] mm: drop NULL return check of pte_offset_map_lock()

2017-05-17 Thread Naoya Horiguchi
pte_offset_map_lock() finds and takes ptl, and returns pte. But some callers return without unlocking the ptl when pte == NULL, which seems weird. Git history said that !pte check in change_pte_range() was introduced in commit 1ad9f620c3a2 ("mm: numa: recheck for transhuge pages under lock during

[PATCH v4] x86/efi: Correct ident mapping of efi old_map when kalsr enabled

2017-05-17 Thread Baoquan He
For EFI with 'efi=old_map' kernel option specified, Kernel will panic when kaslr is enabled. The back trace is: BUG: unable to handle kernel paging request at 7febd57e IP: 0x7febd57e PGD 1025a067 PUD 0 Oops: 0010 [#1] SMP [ ... ] Call Trace: ? efi_call+0x58/0x90 ? printk+0x58/0x6f efi

Re: [infiniband-hw-i40iw] question about identical code for different branches

2017-05-17 Thread Leon Romanovsky
On Wed, May 17, 2017 at 05:06:54PM -0500, Gustavo A. R. Silva wrote: > > Hello everybody, > > While looking into Coverity ID 1362263 I ran into the following piece of > code at drivers/infiniband/hw/i40iw/i40iw_virtchnl.c:445: > > 445if (vchnl_msg->iw_op_code == I40IW_VCHNL_OP_GET_VER) { >

[PATCH] nvme/pci: remap BAR0 to cover admin CQ doorbell for large stride

2017-05-17 Thread Xu Yu
The existing driver initially maps 8192 bytes of BAR0 which is intended to cover doorbells of admin SQ and CQ. However, if a large stride, e.g. 10, is used, the doorbell of admin CQ will be out of 8192 bytes. Consequently, a page fault will be raised when the admin CQ doorbell is accessed in nvme_c

[PATCH v2] iio: adc: sun4i-gpadc-iio: fix parent device being used in devm function

2017-05-17 Thread Quentin Schulz
For the sake of DT binding stability, this IIO driver is a child of an MFD driver for Allwinner A10, A13 and A31 because there already exists a DT binding for this IP. The MFD driver has a DT node but the IIO driver does not. The IIO device registers the temperature sensor in the thermal framework

[PATCH] ocfs2: give an obvious tip for dismatch cluster names

2017-05-17 Thread Gang He
This patch is used to add an obvious error message, due to dismatch cluster names between on-disk and in the current cluster. We can meet this case during OCFS2 cluster migration, if we can give the user an obvious tip for why they can not mount the file system after migration, they can quickly fix

[PATCH 05/23] uuid: don't export guid_index and uuid_index

2017-05-17 Thread Christoph Hellwig
These are only used in uuid.c and vsprintf.c and aren't something modules should use directly. Signed-off-by: Christoph Hellwig --- lib/uuid.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/lib/uuid.c b/lib/uuid.c index f80dc63f6ca8..003bf6823003 100644 --- a/lib/uuid.c +++ b/lib/uuid.c @@

[PATCH 02/23] xfs: use uuid_be to implement the uuid_t type

2017-05-17 Thread Christoph Hellwig
Use the generic Linux definition to implement our UUID type, this will allow using more generic infrastructure in the future. Signed-off-by: Christoph Hellwig --- fs/xfs/uuid.h | 4 fs/xfs/xfs_linux.h | 3 +++ 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/fs/xfs/uuid.h

[PATCH 06/23] uuid: add the v1 layout to uuid_t

2017-05-17 Thread Christoph Hellwig
Turn the content of uuid_t into a union and add the fields for the v1 interpretation to it. Signed-off-by: Christoph Hellwig Reviewed-by: David Howells --- include/linux/uuid.h | 59 1 file changed, 32 insertions(+), 27 deletions(-) diff --g

[PATCH 08/23] uuid: hoist uuid_is_null() helper from libnvdimm

2017-05-17 Thread Christoph Hellwig
Hoist the libnvdimm helper as an inline helper to linux/uuid.h using an auxiliary const variable uuid_null in lib/uuid.c. [hch: also add the guid variant. Both do the same but I'd like to keep casts to a minimum] The common helper uses the new abstract type uuid_t * instead of u8 *. Suggested-b

[PATCH 11/23] uuid: remove struct uuid_v1

2017-05-17 Thread Christoph Hellwig
Unused now. Signed-off-by: Christoph Hellwig --- include/linux/uuid.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/uuid.h b/include/linux/uuid.h index 47ca06d846d4..e8a065035113 100644 --- a/include/linux/uuid.h +++ b/include/linux/uuid.h @@ -40,7 +40,7 @@ ty

[PATCH 10/23] afs: switch to use uuid_t and uuid_gen

2017-05-17 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig Reviewed-by: David Howells --- fs/afs/cmservice.c | 46 +++--- fs/afs/internal.h | 2 +- fs/afs/main.c | 4 ++-- 3 files changed, 26 insertions(+), 26 deletions(-) diff --git a/fs/afs/cmservice.c b/fs/afs/cmservice

[PATCH 13/23] md: namespace private helper names

2017-05-17 Thread Christoph Hellwig
From: Amir Goldstein The md private helper uuid_equal() collides with a generic helper of the same name. Rename the md private helper to md_uuid_equal() and do the same for md_sb_equal(). Cc: Shaohua Li Signed-off-by: Amir Goldstein Signed-off-by: Christoph Hellwig --- drivers/md/md.c | 8 +

[PATCH 16/23] ima/policy: switch to use uuid_t

2017-05-17 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig --- security/integrity/ima/ima_policy.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/security/integrity/ima/ima_policy.c b/security/integrity/ima/ima_policy.c index 49fbc3e8f012..da3e7d50e0d7 100644 --- a/security/integrity/ima/

[PATCH 12/23] xfs: remove uuid_getnodeuniq and xfs_uu_t

2017-05-17 Thread Christoph Hellwig
Directly use the v1 intepretation of uuid_t instead. Signed-off-by: Christoph Hellwig --- fs/xfs/uuid.c | 25 - fs/xfs/uuid.h | 1 - fs/xfs/xfs_mount.c | 6 +- 3 files changed, 5 insertions(+), 27 deletions(-) diff --git a/fs/xfs/uuid.c b/fs/xfs/uuid.c in

[PATCH 17/23] fs: switch ->s_uuid to uuid_t

2017-05-17 Thread Christoph Hellwig
For some file systems we still memcpy into it, but in various places this already allows us to use the proper uuid helpers. More to come.. Signed-off-by: Christoph Hellwig --- drivers/xen/tmem.c | 6 +++--- fs/ext4/super.c | 2 +- fs/f2fs/super.c

[PATCH 19/23] partitions/ldm: switch to use uuid_t

2017-05-17 Thread Christoph Hellwig
And the uuid helpers. Signed-off-by: Christoph Hellwig --- block/partitions/ldm.c | 10 +- block/partitions/ldm.h | 6 ++ 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/block/partitions/ldm.c b/block/partitions/ldm.c index edcea70674c9..bbf929e80b94 100644 --- a/bloc

[PATCH 14/23] xfs: use the common helper uuid_is_null()

2017-05-17 Thread Christoph Hellwig
From: Amir Goldstein Use the common helper uuid_is_null() and remove the xfs specific helper uuid_is_nil(). The common helper does not check for the NULL pointer value as xfs helper did, but xfs code never calls the helper with a pointer that can be NULL. Conform comments and warning strings to

[PATCH 20/23] sysctl: switch to use uuid_t

2017-05-17 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig --- kernel/sysctl_binary.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kernel/sysctl_binary.c b/kernel/sysctl_binary.c index ece4b177052b..1ae520eed53c 100644 --- a/kernel/sysctl_binary.c +++ b/kernel/sysctl_binary.c @@ -1119,7 +1119,7

[PATCH 23/23] uuid: remove uuid_be

2017-05-17 Thread Christoph Hellwig
Everything uses uuid_t now. Signed-off-by: Christoph Hellwig --- include/linux/uuid.h | 14 -- 1 file changed, 14 deletions(-) diff --git a/include/linux/uuid.h b/include/linux/uuid.h index e8a065035113..6b9a05570565 100644 --- a/include/linux/uuid.h +++ b/include/linux/uuid.h @@ -1

[PATCH 22/23] scsi_debug: switch to uuid_t

2017-05-17 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig --- drivers/scsi/scsi_debug.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/scsi/scsi_debug.c b/drivers/scsi/scsi_debug.c index 17249c3650fe..35ee09644cfb 100644 --- a/drivers/scsi/scsi_debug.c +++ b/drivers/scsi/scsi_debug

[PATCH 21/23] nvme: switch to uuid_t

2017-05-17 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig --- drivers/nvme/host/fabrics.c | 8 drivers/nvme/host/fabrics.h | 2 +- drivers/nvme/host/fc.c | 3 +-- drivers/nvme/target/nvmet.h | 1 + include/linux/nvme-fc.h | 3 +-- include/linux/nvme.h| 2 +- 6 files changed, 9 insertions(+), 10

[PATCH 18/23] overlayfs: use uuid_t instead of uuid_be

2017-05-17 Thread Christoph Hellwig
Signed-off-by: Christoph Hellwig --- fs/overlayfs/copy_up.c | 2 +- fs/overlayfs/overlayfs.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/overlayfs/copy_up.c b/fs/overlayfs/copy_up.c index 5b795873f7fa..2a67e8c13098 100644 --- a/fs/overlayfs/copy_up.c +++ b/fs/overl

[PATCH 15/23] block: remove blk_part_pack_uuid

2017-05-17 Thread Christoph Hellwig
This helper was only used by IMA of all things, which would get spurious errors if CONFIG_BLOCK is disabled. Just opencode the call there. Signed-off-by: Christoph Hellwig --- include/linux/genhd.h | 11 --- security/integrity/ima/ima_policy.c | 3 +-- 2 files changed, 1

[PATCH 04/23] uuid: rename uuid types

2017-05-17 Thread Christoph Hellwig
Our "little endian" UUID really is a Wintel GUID, so rename it and its helpers such (guid_t). The big endian UUID is the only true one, so give it the name uuid_t. The uuid_le and uuid_be names are retained for now, but will hopefully go away soon. The exception to that are the _cmp helpers that

[PATCH 07/23] uuid: hoist helpers uuid_equal() and uuid_copy() from xfs

2017-05-17 Thread Christoph Hellwig
These helper are used to compare and copy two uuid_t type objects. Signed-off-by: Amir Goldstein [hch: also provide the respective guid_ versions] Signed-off-by: Christoph Hellwig --- fs/xfs/uuid.c| 6 -- fs/xfs/uuid.h| 7 --- include/linux/uuid.h | 20

[PATCH 09/23] S390/sysinfo: use uuid_is_null instead of opencoding it

2017-05-17 Thread Christoph Hellwig
.. and switch to use uuid_t instead of the old uuid_be type while we're at it. Signed-off-by: Christoph Hellwig --- arch/s390/include/asm/sysinfo.h | 4 ++-- arch/s390/kernel/sysinfo.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/s390/include/asm/sysinfo.h b/a

Re: [PATCH 1/2] mtd: nand: add generic helpers to check, match, maximize ECC settings

2017-05-17 Thread Masahiro Yamada
Hi Boris, 2017-05-15 20:54 GMT+09:00 Boris Brezillon : > Hi Masahiro, > > Sorry for the late reply. > > On Mon, 8 May 2017 12:40:47 +0900 > Masahiro Yamada wrote: > >> Hi Boris, >> >> >> 2017-04-29 1:32 GMT+09:00 Boris Brezillon >> : >> >> >> + for (setting = caps->ecc_settings; setting->s

cleanup UUID types V3

2017-05-17 Thread Christoph Hellwig
Hi all, this series, which is a combined effort from Amir, Andy and me introduces new uuid_t and guid_t type names that are less confusing than the existing types, adds new helpers for them and starts switching the fs code over to it. Andy has additional patches on top to convert many of the user

[PATCH 03/23] uuid: remove uuid_be defintions from the uapi header

2017-05-17 Thread Christoph Hellwig
We don't use uuid_be and the UUID_BE constants in any uapi headers, so make them private to the kernel. Signed-off-by: Christoph Hellwig --- include/linux/uuid.h | 15 +++ include/uapi/linux/uuid.h | 16 2 files changed, 15 insertions(+), 16 deletions(-) diff -

[PATCH 01/23] xfs: use uuid_copy() helper to abstract uuid_t

2017-05-17 Thread Christoph Hellwig
From: Amir Goldstein uuid_t definition is about to change. Signed-off-by: Amir Goldstein Signed-off-by: Christoph Hellwig --- fs/xfs/xfs_inode_item.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/fs/xfs/xfs_inode_item.c b/fs/xfs/xfs_inode_item.c index 08cb7d1a4a3

Re: 答复: [PATCH 1/2] PCI: dwc: kirin: add PCIe Driver for HiSilicon Kirin SoC

2017-05-17 Thread Guodong Xu
On Thu, May 18, 2017 at 9:50 AM, songxiaowei wrote: > Hi Arnd, > > > On Fri, May 12, 2017 at 3:51 AM, Song Xiaowei > wrote: >> From: songxiaowei > > Looks good overall, just a few details: > > Please fix your ~/.gitconfig to contain the same real name ("Song Xiaowei" > instead of "songxiaowei")

Re: [RFC PATCH 6/6] drm/i915/gvt: support QEMU getting the dmabuf

2017-05-17 Thread Gerd Hoffmann
Hi, > > +static long intel_vgpu_dmabuf_mgr_fd_ioctl(struct file *filp, > > + unsigned int ioctl, unsigned long arg) > > +{ > > + struct intel_vgpu *vgpu = filp->private_data; > > + int minsz; > > + struct intel_vgpu_dmabuf dmabuf; > > + int ret; > > +

Re: [PATCH v6 05/15] lockdep: Implement crossrelease feature

2017-05-17 Thread Byungchul Park
On Tue, May 16, 2017 at 04:18:46PM +0200, Peter Zijlstra wrote: > On Tue, Apr 25, 2017 at 02:40:44PM +0900, Byungchul Park wrote: > > On Mon, Apr 24, 2017 at 12:17:47PM +0200, Peter Zijlstra wrote: > > > > My complaint is mostly about naming.. and "hist_gen_id" might be a > > > better name. > > >

RE: [PATCH] kexec/kdump: Minor Documentation updates for arm64 and Image

2017-05-17 Thread Bharat Bhushan
> -Original Message- > From: Dave Young [mailto:dyo...@redhat.com] > Sent: Thursday, May 18, 2017 11:29 AM > To: Bharat Bhushan > Cc: b...@redhat.com; vgo...@redhat.com; cor...@lwn.net; > ke...@lists.infradead.org; linux-...@vger.kernel.org; linux- > ker...@vger.kernel.org; takahiro.aka.

Re: [PATCH 1/3] ARM: davinci: da8xx: Create DSP device only when assigned memory

2017-05-17 Thread Sekhar Nori
On Wednesday 17 May 2017 03:43 AM, Suman Anna wrote: > The DSP device on Davinci platforms does not have an MMU and requires > specific DDR memory to boot. This memory is reserved using the rproc_mem > kernel boot parameter and is assigned to the device on non-DT boots. > The remoteproc core uses t

Re: [PATCH v3 2/2] powerpc/mm/hugetlb: Add support for 1G huge pages

2017-05-17 Thread Aneesh Kumar K.V
On Thursday 18 May 2017 10:51 AM, Michael Ellerman wrote: "Aneesh Kumar K.V" writes: POWER9 supports hugepages of size 2M and 1G in radix MMU mode. This patch enables the usage of 1G page size for hugetlbfs. This also update the helper such we can do 1G page allocation at runtime. We still

Re: [infiniband-hw-i40iw] question about identical code for different branches

2017-05-17 Thread Yuval Shaia
On Wed, May 17, 2017 at 05:06:54PM -0500, Gustavo A. R. Silva wrote: > > Hello everybody, > > While looking into Coverity ID 1362263 I ran into the following piece of > code at drivers/infiniband/hw/i40iw/i40iw_virtchnl.c:445: > > 445if (vchnl_msg->iw_op_code == I40IW_VCHNL_OP_GET_VER) {

Re: [PATCH] kexec/kdump: Minor Documentation updates for arm64 and Image

2017-05-17 Thread Dave Young
Add Takahiro and Pratyush, they should be able to review the arm64 part. On 05/18/17 at 11:03am, Bharat Bhushan wrote: > This patch have minor updates in Documentation for arm64i as > relocatable kernel. > Also this patch updates documentation for using uncompressed > image "Image" which is used f

Re: [PATCH v1] staging: ccree: fix all whitespace errors

2017-05-17 Thread Dhiru Kholia
On Sun, May 14, 2017 at 08:53:02AM +0300, Gilad Ben-Yossef wrote: > Hi Dhiru, > > On Sat, May 13, 2017 at 5:52 PM, Dhiru Kholia wrote: > > These whitespace issues were found by the checkpatch.pl script. This > > patch helps in making the staging tree a bit cleaner. > > > > Signed-off-by: Dhiru Kh

Re: Microchip USB Hub Driver Harmonization

2017-05-17 Thread Richard Leitner
On 05/17/2017 06:01 PM, Krzysztof Kozlowski wrote: On Wed, May 17, 2017 at 12:58:38PM +0200, Richard Leitner wrote: ... 1. Currently usb251xb uses i2c_smbus_*, usb3503 uses regmap_* and usb4604 uses i2c_master_* functions for the hub configuration. What would be the preferred solution? regmap

Re: [net-intel-i40e] question about assignment overwrite

2017-05-17 Thread Jeff Kirsher
On Wed, 2017-05-17 at 15:48 -0500, Gustavo A. R. Silva wrote: > While looking into Coverity ID 1408956 I ran into the following > piece   > of code at drivers/net/ethernet/intel/i40e/i40e_main.c:8807: > > 8807    if (pf->hw.mac.type == I40E_MAC_X722) { > 8808    pf->flags |= I40E_F

[PATCH v1] drivers/staging/ccree: fix coding style issues

2017-05-17 Thread Dhiru Kholia
This patch fixes the "please, no space before tabs" warnings emitted by checkpatch.pl script. Signed-off-by: Dhiru Kholia --- drivers/staging/ccree/cc_crypto_ctx.h | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/staging/ccree/cc_crypto_ctx.h b/driv

Re: [PATCH] kexec/kdump: Minor Documentation updates for arm64 and Image

2017-05-17 Thread Baoquan He
On 05/18/17 at 11:03am, Bharat Bhushan wrote: > This patch have minor updates in Documentation for arm64i as > relocatable kernel. > Also this patch updates documentation for using uncompressed > image "Image" which is used for ARM64. > > Signed-off-by: Bharat Bhushan > --- > Documentation/kdump

Re: [PATCH v1 2/2] hw_random: timeriomem_rng: Allow setting RNG quality from platform data

2017-05-17 Thread PrasannaKumar Muralidharan
Hi Rick, Minor comment below. On 18 May 2017 at 03:59, Rick Altherr wrote: > When a hw_random device's quality is non-zero, it will automatically be > used to fill the kernel's entropy pool. Since timeriomem_rng is used by > many different devices, the quality needs to be provided by platform >

Re: [PATCH v2 00/29] gpu/drm: remove -Iinclude/drm compiler flags from Makefile

2017-05-17 Thread Daniel Vetter
On Mon, Apr 24, 2017 at 01:50:18PM +0900, Masahiro Yamada wrote: > Many Makefiles needed to add -Iinclude/drm as an include path, > but the right thing to do is to include headers in the form > #include > > This series fixes the source files, then rip off -Iinclude/drm flags. > > > Masahiro Y

Re: [PATCH 3/3] EDAC: mv64x60: replace in_le32/out_le32 with ioread32/iowrite32

2017-05-17 Thread Michael Ellerman
Borislav Petkov writes: > Top-posting so that the PPC list can see the whole patch below. > > Since I don't know PPC, let me add PPC ML to CC for a confirmation this > change is correct. > > Which brings me to the tangential: this driver is from 2006-ish and > is for some "Marvell MV64x60 Memory

[PATCH] kexec/kdump: Minor Documentation updates for arm64 and Image

2017-05-17 Thread Bharat Bhushan
This patch have minor updates in Documentation for arm64i as relocatable kernel. Also this patch updates documentation for using uncompressed image "Image" which is used for ARM64. Signed-off-by: Bharat Bhushan --- Documentation/kdump/kdump.txt | 10 -- 1 file changed, 8 insertions(+), 2

Re: [PATCH 0/7] crypto: aesni: provide generic gcm(aes)

2017-05-17 Thread Herbert Xu
On Fri, Apr 28, 2017 at 06:11:55PM +0200, Sabrina Dubroca wrote: > The current aesni AES-GCM implementation only offers support for > rfc4106(gcm(aes)). This makes some things a little bit simpler > (handling of associated data and authentication tag), but it means > that non-IPsec users of gcm(ae

[4.9.28] vmscan: shrink_slab: ext4_es_scan+0x0/0x150 negative objects to delete nr=-2147483624

2017-05-17 Thread Marc Burkhardt
Hi everybody, tonight my dmesg was flooded with mesages like vmscan: shrink_slab: ext4_es_scan+0x0/0x150 negative objects to delete nr=-2147483624 Is that an integer overflow happening in ext4? It's the first time I see this message. Any help on how to debug/reprocude this are appreciated. P

Re: [PATCH v5] of/pci: Fix memory leak in of_pci_get_host_bridge_resources

2017-05-17 Thread jeffy
Hi Bjorn, On 05/18/2017 05:08 AM, Bjorn Helgaas wrote: On Wed, Apr 05, 2017 at 10:28:22AM +0800, Jeffy Chen wrote: Currently we only free the allocated resource struct when error. This would cause memory leak after pci_free_resource_list. I think I see the problem here. The release path is t

Re: [PATCH v3 06/16] drm/i2c: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Archit Taneja
On 05/18/2017 09:59 AM, Masahiro Yamada wrote: With the include directives under include/drm/ fixed, this flag is no longer needed. Acked-by: Archit Taneja Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/i2c/Makefile | 2 -- 1 file changed, 2 deletions(-) diff

Re: [PATCH] crypto: tcrypt: don't disable irqs and wait

2017-05-17 Thread Herbert Xu
On Tue, Apr 25, 2017 at 11:25:40AM +0300, Gilad Ben-Yossef wrote: > The tcrypt AEAD cycles speed tests disables irqs during the test, which is > broken at the very least since commit > '1425d2d17f7309c6 ("crypto: tcrypt - Fix AEAD speed tests")' > adds a wait for completion as part of the test and

Re: [PATCH v8 06/08] iommu/ipmmu-vmsa: Use fwspec iommu_priv on ARM64

2017-05-17 Thread Magnus Damm
Hi Robin, On Wed, May 17, 2017 at 11:29 PM, Robin Murphy wrote: > Hi Magnus, > > On 17/05/17 11:07, Magnus Damm wrote: >> From: Magnus Damm >> >> Convert from archdata to iommu_priv via iommu_fwspec on ARM64 but >> let 32-bit ARM keep on using archdata for now. >> >> Once the 32-bit ARM code and

Re: [PATCH v3 2/2] powerpc/mm/hugetlb: Add support for 1G huge pages

2017-05-17 Thread Michael Ellerman
"Aneesh Kumar K.V" writes: > POWER9 supports hugepages of size 2M and 1G in radix MMU mode. This patch > enables the usage of 1G page size for hugetlbfs. This also update the helper > such we can do 1G page allocation at runtime. > > We still don't enable 1G page size on DD1 version. This is to a

[scsi] 66de5f1b19: BUG:unable_to_handle_kernel

2017-05-17 Thread kernel test robot
FYI, we noticed the following commit: commit: 66de5f1b197e175840a132f7ee8927bfc3ff92b3 ("scsi: Move scsi_device.cmd_list manipulation code") https://github.com/bvanassche/linux for-next in testcase: boot on test machine: qemu-system-x86_64 -enable-kvm -cpu kvm64,+ssse3 -smp 2 -m 8G caused bel

Re: [PATCH 5/8] gpio: exar: Fix reading of directions and values

2017-05-17 Thread Jan Kiszka
On 2017-05-13 15:36, Andy Shevchenko wrote: > On Sat, May 13, 2017 at 10:29 AM, Jan Kiszka wrote: >> First, the logic for translating a register bit to the return code of >> exar_get_direction and exar_get_value were wrong. And second, there was >> a flip regarding the register bank in exar_get_di

Re: [PATCH 0/7] perf tool: add meta-data header support for pipe-mode

2017-05-17 Thread Andi Kleen
> The difficulty in pipe mode is that information needs to be written > sequentially to the pipe. Meta data headers are usually generated > (and also expected) at the beginning of the file (or piped output). > To solve this problem, we introduce new synthetic record types, > one for each meta-data

Re: [PATCH 2/4] crypto: drbg wait for crypto op not signal safe

2017-05-17 Thread Herbert Xu
On Thu, May 11, 2017 at 02:53:43PM +0300, Gilad Ben-Yossef wrote: > drbg_kcapi_sym_ctr() was using wait_for_completion_interruptible() to > wait for completion of async crypto op but if a signal occurs it > may return before DMA ops of HW crypto provider finish, thus > corrupting the output buffer.

Re: [PATCH 8/8] serial: exar: Add support for IOT2040 device

2017-05-17 Thread Jan Kiszka
On 2017-05-13 15:54, Andy Shevchenko wrote: > On Sat, May 13, 2017 at 10:29 AM, Jan Kiszka wrote: >> This implements the setup of RS232 and the switch-over to RS485 or RS422 >> for the Siemens IOT2040. That uses an EXAR XR17V352 with external logic >> to switch between the different modes. The ext

Re: [PATCH 1/2] wcn36xx: Pass used skb to ieee80211_tx_status()

2017-05-17 Thread Bjorn Andersson
On Wed 17 May 06:14 PDT 2017, Johannes Berg wrote: > On Thu, 2017-05-04 at 13:13 +, Kalle Valo wrote: > > > > > > This code intentionally checked if TX status was requested, and > > > > if not then it doesn't go to the effort of building it. > > > > > > > > > > What I'm finding puzzling is

Re: [PATCH] perf/x86/intel/cqm: Make sure the head event of cache_groups always has valid RMID

2017-05-17 Thread David Carrillo-Cisneros
On Tue, May 16, 2017 at 7:38 AM, Peter Zijlstra wrote: > On Thu, May 04, 2017 at 10:31:43AM +0800, Zefan Li wrote: >> It is assumed that the head of cache_groups always has valid RMID, >> which isn't true. >> >> When we deallocate RMID from conflicting events currently we don't >> move them to the

Re: [PATCH RFC 0/3] Improve stability of system clock

2017-05-17 Thread Richard Cochran
On Wed, May 17, 2017 at 04:06:07PM -0700, John Stultz wrote: > On Wed, May 17, 2017 at 10:22 AM, Miroslav Lichvar > wrote: > > Is there a better way to run the timekeeping code in an userspace > > application? I suspect it would need something like the Linux Kernel > > Library project. > > I dun

Re: [PATCH 2/2] zram: do not count duplicated pages as compressed

2017-05-17 Thread Minchan Kim
Hi Sergey, On Wed, May 17, 2017 at 06:14:23PM +0900, Sergey Senozhatsky wrote: > Hello Minchan, > > On (05/17/17 17:32), Minchan Kim wrote: > [..] > > > what we can return now is a `partially updated' data, with some new > > > and some stale pages. this is quite unlikely to end up anywhere good.

[PATCH] drm/stm: add COMPILE_TEST to Kconfig

2017-05-17 Thread Masahiro Yamada
Add COMPILE_TEST for the compilation test coverage. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/stm/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/stm/Kconfig b/drivers/gpu/drm/stm/Kconfig index 2c4817f..722ad11 100644 --- a/drivers/gpu/drm/st

[PATCH] drm/amd: include instead of "linux/delay.h"

2017-05-17 Thread Masahiro Yamada
Use <...> notation to include headers located in include/linux. While we are here, tweak the includes order a bit to sort them alphabetically. Signed-off-by: Masahiro Yamada --- drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c| 4 ++-- drivers/gpu/drm/amd/powerplay/hwmgr/pp_acpi.c | 2 +

Re: [PATCH 0/3] staging: android: ion: Minor clean ups and fixes

2017-05-17 Thread Sumit Semwal
Hi Archit, On 17 May 2017 at 13:45, Archit Taneja wrote: > The recent ION clean ups introduced some leftover code that can be > removed, and a bug that comes up if the call to dma_buf_map_attachment() > from an importer fails. Fix these. > > Archit Taneja (3): > staging: android: ion: Remove un

Re: [PATCH] scsi: remove useless variable assignment

2017-05-17 Thread James Bottomley
On Wed, 2017-05-17 at 19:30 -0500, Gustavo A. R. Silva wrote: > Remove this assignment once the value stored in variable _k_ is > overwritten after a few lines. > > Addresses-Coverity-ID: 1226927 > Signed-off-by: Gustavo A. R. Silva > --- > drivers/scsi/qlogicfas408.c | 1 - > 1 file changed, 1

Re: [PATCH 4/4] mtd: mchp23k256: Add support for mchp23lcv1024

2017-05-17 Thread Chris Packham
On 17/05/17 17:39, Chris Packham wrote: > The mchp23lcv1024 is software compatible with the mchp23k256, the > only difference (from a software point of view) is the size. There > is no way to detect the size so we must be told via a Device Tree. I've just noticed that another difference between t

Re: [PATCH v2 15/29] drm/vc4: fix include notation and remove -Iinclude/drm flag

2017-05-17 Thread Masahiro Yamada
Hi Daniel, 2017-05-17 21:40 GMT+09:00 Daniel Vetter : > On Mon, Apr 24, 2017 at 01:50:33PM +0900, Masahiro Yamada wrote: >> Include instead of relative path from include/drm, then >> remove the -Iinclude/drm compiler flag. >> >> While we are here, use <...> instead of "..." for include/linux/*.h

[PATCH v3 12/16] drm/stm: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/stm/Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/stm/Makefile b/drivers/gpu/drm/stm/Makefile index e114d4

[PATCH v3 07/16] drm/mga: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/mga/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/mga/Makefile b/drivers/gpu/drm/mga/Makefile index 6068478.

[PATCH v3 05/16] drm/i810: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/i810/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i810/Makefile b/drivers/gpu/drm/i810/Makefile index 43844

[PATCH v3 08/16] drm/omap: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/omapdrm/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/omapdrm/Makefile b/drivers/gpu/drm/omapdrm/Makefile in

[PATCH v3 16/16] drm/via: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/via/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/via/Makefile b/drivers/gpu/drm/via/Makefile index d59e258.

[PATCH v3 09/16] drm/r128: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/r128/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/r128/Makefile b/drivers/gpu/drm/r128/Makefile index 1cc72

[PATCH v3 02/16] drm/virtio: fix include notation and remove -Iinclude/drm flag

2017-05-17 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/virtio/Makefile | 2 -- drivers/gpu/drm/virtio/virtgpu_debugfs.c | 2 +- drivers/gpu/drm/virtio/virtgpu_drv.c |

[PATCH v3 14/16] drm/udl: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/udl/Makefile | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/udl/Makefile b/drivers/gpu/drm/udl/Makefile index 195bc

[PATCH v3 06/16] drm/i2c: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/i2c/Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/Makefile b/drivers/gpu/drm/i2c/Makefile index 43aa33

[PATCH v3 04/16] drm/gma500: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/gma500/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/Makefile b/drivers/gpu/drm/gma500/Makefile index

[PATCH v3 15/16] drm/vgem: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/vgem/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/vgem/Makefile b/drivers/gpu/drm/vgem/Makefile index bfcde

[PATCH v3 10/16] drm/savage: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/savage/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/savage/Makefile b/drivers/gpu/drm/savage/Makefile index

[PATCH v3 00/16] gpu/drm: remove -Iinclude/drm compiler flags from Makefile

2017-05-17 Thread Masahiro Yamada
Many Makefiles needed to add -Iinclude/drm as an include path, but the right thing to do is to include headers in the form #include This series fixes the source files, then rip off -Iinclude/drm flags. V3: rebased on commit bb2af9bda33 (drm-misc-next) Masahiro Yamada (16): drm/vc4: fix in

[PATCH v3 01/16] drm/vc4: fix include notation and remove -Iinclude/drm flag

2017-05-17 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. While we are here, use <...> instead of "..." for include/linux/*.h and include/sound/*.h headers too. Signed-off-by: Masahiro Yamada --- Changes in v3: - Rebase drivers/gpu/drm/vc4/Makefile

[PATCH v3 13/16] drm/tdfx: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/tdfx/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/tdfx/Makefile b/drivers/gpu/drm/tdfx/Makefile index 0379f

[PATCH v3 11/16] drm/sis: remove unneeded -Iinclude/drm compiler flag

2017-05-17 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/sis/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/sis/Makefile b/drivers/gpu/drm/sis/Makefile index 441c061.

[PATCH v3 03/16] drm/vmwgfx: fix include notation and remove -Iinclude/drm flag

2017-05-17 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v3: None drivers/gpu/drm/vmwgfx/Makefile | 3 --- drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 3 ++- drivers/gpu/drm/vmwgfx/vmwgfx_context.c | 3

[PATCH 0/7] perf tool: add meta-data header support for pipe-mode

2017-05-17 Thread David Carrillo-Cisneros
(This is a rebased and updated version of Stephane Eranian's version in https://patchwork.kernel.org/patch/1499081/) Up until now, meta-data was only available when perf record was used in "regular" mode, i.e., generating a perf.data file. For users depending on pipe mode, neither host or event

[PATCH 3/7] perf header: use struct feat_fd for write

2017-05-17 Thread David Carrillo-Cisneros
As preparation for using header records in pipe mode, replace int fd with struct feat_fd fd in write functions for all header record types. Record types that are not used in pipe-mode (such as auxtrace) will never be called with fd->buf set and fail if so. Signed-off-by: David Carrillo-Cisneros

[PATCH 2/7] perf util: add const modifier to buf in "writen" function

2017-05-17 Thread David Carrillo-Cisneros
Make buf in helper function "writen" constant to simplify the life of its callers. This requires to hack a cast of buf prior to passing it to "ion" which is simpler than the alternative of reworking the "ion" function to provide a read and a write paths, the latter with constant buf argument. Sig

  1   2   3   4   5   6   7   8   9   10   >