Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread David Hildenbrand
On 06.11.19 01:08, Dan Williams wrote: On Tue, Nov 5, 2019 at 4:03 PM Sean Christopherson wrote: On Tue, Nov 05, 2019 at 03:43:29PM -0800, Dan Williams wrote: On Tue, Nov 5, 2019 at 3:30 PM Dan Williams wrote: On Tue, Nov 5, 2019 at 3:13 PM Sean Christopherson wrote: On Tue, Nov 05,

Re: [RFC] errno.h: Provide EFSCORRUPTED for everybody

2019-11-05 Thread Dave Chinner
On Tue, Nov 05, 2019 at 04:15:50PM +0100, David Sterba wrote: > On Sat, Nov 02, 2019 at 08:38:23AM +1100, Dave Chinner wrote: > > On Fri, Nov 01, 2019 at 09:57:31PM +0100, Geert Uytterhoeven wrote: > > > Hi Valdis, > > > > > > On Thu, Oct 31, 2019 at 2:11 AM Valdis Kletnieks > > > wrote: > > > >

RE: [PATCH v5 2/8] arm64: hyperv: Add hypercall and register access functions

2019-11-05 Thread Michael Kelley
From: Boqun Feng Sent: Sunday, November 3, 2019 8:37 PM > > > diff --git a/arch/arm64/Kbuild b/arch/arm64/Kbuild > > index d646582..2469421 100644 > > --- a/arch/arm64/Kbuild > > +++ b/arch/arm64/Kbuild > > @@ -3,4 +3,5 @@ obj-y += kernel/ mm/ > > obj-$(CONFIG_NET) += net/

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Dan Williams
On Tue, Nov 5, 2019 at 4:03 PM Sean Christopherson wrote: > > On Tue, Nov 05, 2019 at 03:43:29PM -0800, Dan Williams wrote: > > On Tue, Nov 5, 2019 at 3:30 PM Dan Williams > > wrote: > > > > > > On Tue, Nov 5, 2019 at 3:13 PM Sean Christopherson > > > wrote: > > > > > > > > On Tue, Nov 05,

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Sean Christopherson
On Tue, Nov 05, 2019 at 03:43:29PM -0800, Dan Williams wrote: > On Tue, Nov 5, 2019 at 3:30 PM Dan Williams wrote: > > > > On Tue, Nov 5, 2019 at 3:13 PM Sean Christopherson > > wrote: > > > > > > On Tue, Nov 05, 2019 at 03:02:40PM -0800, Dan Williams wrote: > > > > On Tue, Nov 5, 2019 at 12:31

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Dan Williams
On Tue, Nov 5, 2019 at 3:30 PM Dan Williams wrote: > > On Tue, Nov 5, 2019 at 3:13 PM Sean Christopherson > wrote: > > > > On Tue, Nov 05, 2019 at 03:02:40PM -0800, Dan Williams wrote: > > > On Tue, Nov 5, 2019 at 12:31 PM David Hildenbrand > > > wrote: > > > > > The scarier code (for me) is

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Sean Christopherson
On Tue, Nov 05, 2019 at 03:30:00PM -0800, Dan Williams wrote: > On Tue, Nov 5, 2019 at 3:13 PM Sean Christopherson > wrote: > > > > On Tue, Nov 05, 2019 at 03:02:40PM -0800, Dan Williams wrote: > > > On Tue, Nov 5, 2019 at 12:31 PM David Hildenbrand > > > wrote: > > > > > The scarier code (for

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Dan Williams
On Tue, Nov 5, 2019 at 3:13 PM Sean Christopherson wrote: > > On Tue, Nov 05, 2019 at 03:02:40PM -0800, Dan Williams wrote: > > On Tue, Nov 5, 2019 at 12:31 PM David Hildenbrand wrote: > > > > The scarier code (for me) is transparent_hugepage_adjust() and > > > > kvm_mmu_zap_collapsible_spte(),

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Sean Christopherson
On Tue, Nov 05, 2019 at 03:02:40PM -0800, Dan Williams wrote: > On Tue, Nov 5, 2019 at 12:31 PM David Hildenbrand wrote: > > > The scarier code (for me) is transparent_hugepage_adjust() and > > > kvm_mmu_zap_collapsible_spte(), as I don't at all understand the > > > interaction between THP and

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Dan Williams
On Tue, Nov 5, 2019 at 12:31 PM David Hildenbrand wrote: > > >>> I think I know what's going wrong: > >>> > >>> Pages that are pinned via gfn_to_pfn() and friends take a references, > >>> however are often released via > >>>

[PATCH] staging: vchiq: Have vchiu_queue_init() return 0 on success.

2019-11-05 Thread Marcelo Diop-Gonzalez
It could be confusing to return 1 on success and 0 on error given the style elswhere. Signed-off-by: Marcelo Diop-Gonzalez --- .../staging/vc04_services/interface/vchiq_arm/vchiq_shim.c| 2 +- .../staging/vc04_services/interface/vchiq_arm/vchiq_util.c| 4 ++-- 2 files changed, 3

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Sean Christopherson
On Tue, Nov 05, 2019 at 09:30:53PM +0100, David Hildenbrand wrote: > >>>I think I know what's going wrong: > >>> > >>>Pages that are pinned via gfn_to_pfn() and friends take a references, > >>>however are often released via >

Re: [PATCH v2 01/10] staging: exfat: Clean up return codes - FFS_FORMATERR

2019-11-05 Thread Valdis Klētnieks
On Tue, 05 Nov 2019 18:05:15 +0100, Greg Kroah-Hartman said: > This patch breaks the build: > > drivers/staging/exfat/exfat_super.c: In function ‘ffsMountVol’: > drivers/staging/exfat/exfat_super.c:387:9: error: ‘FFS_FORMATERR’ > undeclared (first use in this function) > 387 | ret =

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread David Hildenbrand
I think I know what's going wrong: Pages that are pinned via gfn_to_pfn() and friends take a references, however are often released via kvm_release_pfn_clean()/kvm_release_pfn_dirty()/kvm_release_page_clean()... E.g., in arch/x86/kvm/x86.c:reexecute_instruction() ... pfn =

Re: [PATCH v3] dt-bindings: iio: accel: add binding documentation for ADIS16240

2019-11-05 Thread Rob Herring
On Thu, Oct 31, 2019 at 09:03:01PM -0300, Rodrigo Carvalho wrote: > This patch add device tree binding documentation for ADIS16240. > > Signed-off-by: Rodrigo Ribeiro Carvalho > --- > V3: >- Remove spi-cpol and spi-cpha field. They don't seem necessary Not necessary to document or use? The

Re: [PATCH v2] hp100: remove set but not used variable val

2019-11-05 Thread Joe Perches
On Tue, 2019-11-05 at 16:50 +0100, Greg KH wrote: > On Tue, Nov 05, 2019 at 10:36:59PM +0800, Chen Wandun wrote: > > From: Chenwandun > > > > Fixes gcc '-Wunused-but-set-variable' warning: > > > > drivers/staging/hp/hp100.c: In function hp100_start_xmit: > > drivers/staging/hp/hp100.c:1629:10:

[PATCH][next] staging: vboxsf: fix dereference of pointer dentry before it is null checked

2019-11-05 Thread Colin King
From: Colin Ian King Currently the pointer dentry is being dereferenced before it is being null checked. Fix this by only dereferencing dentry once we know it is not null. Addresses-Coverity: ("Dereference before null check") Fixes: df4028658f9d ("staging: Add VirtualBox guest shared folder

Re: [PATCH v2 01/10] staging: exfat: Clean up return codes - FFS_FORMATERR

2019-11-05 Thread Greg Kroah-Hartman
On Sun, Nov 03, 2019 at 08:44:57PM -0500, Valdis Kletnieks wrote: > Convert FFS_FORMATERR to -EFSCORRUPTED > > Signed-off-by: Valdis Kletnieks > --- > drivers/staging/exfat/exfat.h | 3 ++- > drivers/staging/exfat/exfat_core.c | 8 > 2 files changed, 6 insertions(+), 5

Re: [PATCH v2 00/10] staging: exfat: Clean up return codes, revisited

2019-11-05 Thread Greg Kroah-Hartman
On Sun, Nov 03, 2019 at 08:44:56PM -0500, Valdis Kletnieks wrote: > The rest of the conversion from internal error numbers to the > standard values used in the rest of the kernel. > > Patch 10/10 is logically separate, merging multiple #defines > into one place in errno.h. It's included in the

Re: [PATCH v1 09/10] mm/memory_hotplug: Don't mark pages PG_reserved when initializing the memmap

2019-11-05 Thread Boris Ostrovsky
On 11/5/19 5:18 AM, David Hildenbrand wrote: > On 04.11.19 23:44, Boris Ostrovsky wrote: >> On 10/24/19 8:09 AM, David Hildenbrand wrote: >>> diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c >>> index 4f2e78a5e4db..af69f057913a 100644 >>> --- a/drivers/xen/balloon.c >>> +++

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread Sean Christopherson
On Tue, Nov 05, 2019 at 11:02:46AM +0100, David Hildenbrand wrote: > On 05.11.19 10:49, David Hildenbrand wrote: > >On 05.11.19 10:17, David Hildenbrand wrote: > >>On 05.11.19 05:38, Dan Williams wrote: > >>>On Thu, Oct 24, 2019 at 5:11 AM David Hildenbrand wrote: > > Right now,

Re: [PATCH v2] hp100: remove set but not used variable val

2019-11-05 Thread Greg KH
On Tue, Nov 05, 2019 at 10:36:59PM +0800, Chen Wandun wrote: > From: Chenwandun > > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/staging/hp/hp100.c: In function hp100_start_xmit: > drivers/staging/hp/hp100.c:1629:10: warning: variable val set but not used >

Re: [RFC] errno.h: Provide EFSCORRUPTED for everybody

2019-11-05 Thread David Sterba
On Sat, Nov 02, 2019 at 08:38:23AM +1100, Dave Chinner wrote: > On Fri, Nov 01, 2019 at 09:57:31PM +0100, Geert Uytterhoeven wrote: > > Hi Valdis, > > > > On Thu, Oct 31, 2019 at 2:11 AM Valdis Kletnieks > > wrote: > > > Three questions: (a) ACK/NAK on this patch, (b) should it be all in one > >

Re: [PATCH] staging: rtl8192e: fix potential use after free

2019-11-05 Thread Dan Carpenter
On Tue, Nov 05, 2019 at 10:49:11PM +0800, Pan Bian wrote: > The variable skb is released via kfree_skb() when the return value of > _rtl92e_tx is not zero. However, after that, skb is accessed again to > read its length, which may result in a use after free bug. This patch > fixes the bug by

[PATCH] staging: rtl8192e: fix potential use after free

2019-11-05 Thread Pan Bian
The variable skb is released via kfree_skb() when the return value of _rtl92e_tx is not zero. However, after that, skb is accessed again to read its length, which may result in a use after free bug. This patch fixes the bug by moving the release operation to where skb is never used later.

Re: [PATCH v2] hp100: remove set but not used variable val

2019-11-05 Thread Dan Carpenter
On Tue, Nov 05, 2019 at 10:36:59PM +0800, Chen Wandun wrote: > From: Chenwandun > > Fixes gcc '-Wunused-but-set-variable' warning: > > drivers/staging/hp/hp100.c: In function hp100_start_xmit: > drivers/staging/hp/hp100.c:1629:10: warning: variable val set but not used >

[PATCH v2] hp100: remove set but not used variable val

2019-11-05 Thread Chen Wandun
From: Chenwandun Fixes gcc '-Wunused-but-set-variable' warning: drivers/staging/hp/hp100.c: In function hp100_start_xmit: drivers/staging/hp/hp100.c:1629:10: warning: variable val set but not used [-Wunused-but-set-variable] Signed-off-by: Chenwandun --- drivers/staging/hp/hp100.c | 4

[PATCH] hp100: remove set but not used variable val

2019-11-05 Thread Chen Wandun
From: Chenwandun Fixes gcc '-Wunused-but-set-variable' warning: drivers/staging/hp/hp100.c: In function hp100_start_xmit: drivers/staging/hp/hp100.c:1629:10: warning: variable val set but not used [-Wunused-but-set-variable] Signed-off-by: Chenwandun --- drivers/staging/hp/hp100.c | 4 +---

Re: [PATCH] staging: rts5208: rewrite macro with GNU extension __auto_type

2019-11-05 Thread Jules Irenge
On Mon, 4 Nov 2019, Greg KH wrote: > On Mon, Nov 04, 2019 at 04:44:00PM +, Jules Irenge wrote: > > Rewrite macro function with GNU extension __auto_type > > to remove issue detected by checkpatch tool. > > CHECK: MACRO argument reuse - possible side-effects? > > > > Signed-off-by: Jules

Re: [PATCH v1 02/10] KVM: x86/mmu: Prepare kvm_is_mmio_pfn() for PG_reserved changes

2019-11-05 Thread David Hildenbrand
On 05.11.19 02:37, Dan Williams wrote: On Thu, Oct 24, 2019 at 5:10 AM David Hildenbrand wrote: Right now, ZONE_DEVICE memory is always set PG_reserved. We want to change that. KVM has this weird use case that you can map anything from /dev/mem into the guest. pfn_valid() is not a reliable

Re: [PATCH v2] staging: comedi: rewrite macro function with GNU extension typeof

2019-11-05 Thread Jules Irenge
On Mon, 4 Nov 2019, Greg KH wrote: > On Mon, Nov 04, 2019 at 04:33:31PM +, Jules Irenge wrote: > > Rewrite macro function with the GNU extension typeof > > to remove a possible side-effects of MACRO argument reuse "x". > > - Problem could rise if arguments have different types > > and

Re: [PATCH v2] staging: comedi: rewrite macro function with GNU extension typeof

2019-11-05 Thread Jules Irenge
On Mon, 4 Nov 2019, Ian Abbott wrote: > On 04/11/2019 16:33, Jules Irenge wrote: > > Rewrite macro function with the GNU extension typeof > > to remove a possible side-effects of MACRO argument reuse "x". > > - Problem could rise if arguments have different types > > and different use

Re: [PATCH v1 09/10] mm/memory_hotplug: Don't mark pages PG_reserved when initializing the memmap

2019-11-05 Thread David Hildenbrand
On 04.11.19 23:44, Boris Ostrovsky wrote: On 10/24/19 8:09 AM, David Hildenbrand wrote: diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c index 4f2e78a5e4db..af69f057913a 100644 --- a/drivers/xen/balloon.c +++ b/drivers/xen/balloon.c @@ -374,6 +374,13 @@ static void

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread David Hildenbrand
On 05.11.19 10:49, David Hildenbrand wrote: On 05.11.19 10:17, David Hildenbrand wrote: On 05.11.19 05:38, Dan Williams wrote: On Thu, Oct 24, 2019 at 5:11 AM David Hildenbrand wrote: Right now, ZONE_DEVICE memory is always set PG_reserved. We want to change that. KVM has this weird use

[PATCH 4/6] staging: wilc1000: avoid use of C++ style comments

2019-11-05 Thread Ajay.Kathat
From: Ajay Singh Replace C++ style comment with C style. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/hif.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/hif.c b/drivers/staging/wilc1000/hif.c index 25f035c02b10..5f6706bcedf6 100644 ---

[PATCH 3/6] staging: wilc1000: added 'WILC_' prefix in header guard macro

2019-11-05 Thread Ajay.Kathat
From: Ajay Singh Use 'WILC_' prefix in header guard to follow the proper naming convention for macro name. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/cfg80211.h | 4 ++-- drivers/staging/wilc1000/hif.h | 4 ++-- drivers/staging/wilc1000/netdev.h | 4 ++-- 3 files changed, 6

[PATCH 5/6] staging: wilc1000: added proper spacing for comments

2019-11-05 Thread Ajay.Kathat
From: Ajay Singh Added proper space for the comments and added newline before the comments inside a struct. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/mon.c | 2 +- drivers/staging/wilc1000/netdev.h | 19 ++- drivers/staging/wilc1000/wlan.h | 2 +-

[PATCH 1/6] staging: wilc1000: avoid 'bool' datatype in struct sent to firmware

2019-11-05 Thread Ajay.Kathat
From: Ajay Singh Use 'u8' instead of 'bool' datatype for struct element sent to firmware because storage of bool datatype is not guaranteed. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wilc_hif.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git

[PATCH 0/6] staging: wilc1000: changes to rename files and few other review comments

2019-11-05 Thread Ajay.Kathat
From: Ajay Singh This patch series contains changes to address some of the review comments received during full driver review [1]. Mainly the changes are related to files rename and comments formatting. 1. https://www.spinics.net/lists/linux-wireless/msg191489.html Ajay Singh (6): staging:

[PATCH 6/6] staging: wilc1000: use defines for msg types received from firmware

2019-11-05 Thread Ajay.Kathat
From: Ajay Singh Added defines for different types of messages received from firmware. Removed the unnecessary comments because after the addition of macros the message types are self-explanatory. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/wlan_cfg.c | 20 +--- 1

[PATCH 2/6] staging: wilc1000: remove 'wilc_' prefix from filenames

2019-11-05 Thread Ajay.Kathat
From: Ajay Singh Remove 'wilc_' prefix from filenames, the driver is already present inside the 'wilc1000' directory so no need to add 'wilc_' in filenames. Signed-off-by: Ajay Singh --- drivers/staging/wilc1000/Makefile | 8

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread David Hildenbrand
On 05.11.19 10:17, David Hildenbrand wrote: On 05.11.19 05:38, Dan Williams wrote: On Thu, Oct 24, 2019 at 5:11 AM David Hildenbrand wrote: Right now, ZONE_DEVICE memory is always set PG_reserved. We want to change that. KVM has this weird use case that you can map anything from /dev/mem

Re: [PATCH v1 01/10] mm/memory_hotplug: Don't allow to online/offline memory blocks with holes

2019-11-05 Thread David Hildenbrand
On 05.11.19 02:30, Dan Williams wrote: On Thu, Oct 24, 2019 at 5:10 AM David Hildenbrand wrote: Our onlining/offlining code is unnecessarily complicated. Only memory blocks added during boot can have holes (a range that is not IORESOURCE_SYSTEM_RAM). Hotplugged memory never has holes (e.g.,

Re: [PATCH v1 03/10] KVM: Prepare kvm_is_reserved_pfn() for PG_reserved changes

2019-11-05 Thread David Hildenbrand
On 05.11.19 05:38, Dan Williams wrote: On Thu, Oct 24, 2019 at 5:11 AM David Hildenbrand wrote: Right now, ZONE_DEVICE memory is always set PG_reserved. We want to change that. KVM has this weird use case that you can map anything from /dev/mem into the guest. pfn_valid() is not a reliable

Re: [PATCH 1/3] media: cedrus: Properly signal size in mode register

2019-11-05 Thread Paul Kocialkowski
Hi, On Mon 04 Nov 19, 17:33, Jernej Škrabec wrote: > Dne ponedeljek, 04. november 2019 ob 11:02:28 CET je Paul Kocialkowski > napisal(a): > > Hi Jernej, > > > > On Sat 26 Oct 19, 09:49, Jernej Skrabec wrote: > > > Mode register also holds information if video width is bigger than 2048 > > > and

Re: [PATCH 2/3] media: cedrus: Fix H264 4k support

2019-11-05 Thread Paul Kocialkowski
Hi Jernej, On Mon 04 Nov 19, 17:53, Jernej Škrabec wrote: > Dne ponedeljek, 04. november 2019 ob 11:13:19 CET je Paul Kocialkowski > napisal(a): > > Hi, > > > > On Sat 26 Oct 19, 09:49, Jernej Skrabec wrote: > > > H264 decoder needs additional or bigger buffers in order to decode 4k > > >