Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
drivers/gpu/drm/drm_vm.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_vm.c b/drivers/gpu/drm/drm_vm.c
index 2660543..c330104 100644
--- a/drivers/gpu/drm/drm_vm.c
+++ b/drive
Any comment for this patch ?
On Tue, Apr 17, 2018 at 7:45 PM, Souptick Joarder wrote:
> Use new return type vm_fault_t for fault handler. For
> now, this is just documenting that the function returns
> a VM_FAULT value rather than an errno. Once all instances
> are converted, vm_
On Wed, Apr 25, 2018 at 12:39 PM, David Rientjes wrote:
> On Wed, 25 Apr 2018, Souptick Joarder wrote:
>
>> Use new return type vm_fault_t for fault handler. For
>> now, this is just documenting that the function returns
>> a VM_FAULT value rather than an errno. O
> I noticed that we have the following status translation now in 4 places in 2
> files:
>
> if (err == -ENOMEM)
> return VM_FAULT_OOM;
> if (err < 0 && err != -EBUSY)
> return VM_FAULT_SIGBUS;
> return VM_FAULT_NOPAGE;
>
>
> This happens in vm
ult_t")
Previously vm_insert_page() returns err which driver
mapped into VM_FAULT_* type. The new function vmf_
insert_page() will replace this inefficiency by
returning VM_FAULT_* type.
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
v2: Updated the change log
drivers/g
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
v2: Updated the change log
drivers/misc/sgi-gru/grumain.c | 2 +-
drivers/misc/sgi-gru/grutables.h | 3 ++-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/misc/sgi-gru/grumain.c b/drivers/misc/sgi-gru/gruma
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
v2: Updated the change log
include/linux/huge_mm.h | 5 +++--
mm/huge_memory.c| 4 ++--
2 files changed, 5 insertions(+), 4 deletions(-)
diff --git a/include/linux/huge_mm.h b/include/linux/huge_mm.h
index a8a1262..d3bbf6b 10
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
v2: Updated the change log
ipc/shm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ipc/shm.c b/ipc/shm.c
index 4643865..2ba0cfc 100644
--- a/ipc/shm.c
+++ b/ipc/shm.c
@@ -378,7 +378,7 @@ void exit_shm(struct t
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
sound/core/pcm_native.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/sound/core/pcm_native.c b/sound/core/pcm_native.c
index 77ba50d..2d331f4 100644
--- a/sound/core/pcm_native.c
+++ b/sound/core/pc
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
sound/usb/usx2y/us122l.c| 2 +-
sound/usb/usx2y/usX2Yhwdep.c| 2 +-
sound/usb/usx2y/usx2yhwdeppcm.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/sound/usb/usx2y/us122l.c b/sound/usb/usx2y/us12
o convert it to vm_fault_t
type. With new vmf_insert_mixed_mkwrite() this limitation
will be addressed.
Signed-off-by: Souptick Joarder
Reviewed-by: Jan Kara
Reviewed-by: Matthew Wilcox
---
v2: vm_insert_mixed_mkwrite is replaced by new
vmf_insert_mixed_mkwrite
v3: Addressed Matthew'
On Tue, Apr 24, 2018 at 1:19 AM, Matthew Wilcox wrote:
> On Mon, Apr 23, 2018 at 11:36:25PM +0530, Souptick Joarder wrote:
>> If the insertion of PTE failed because someone else
>> already added a different entry in the mean time, we
>> treat that as success as we assu
n the mean time, we
treat that as success as we assume the same entry was
actually inserted.
vm_insert_mixed_mkwrite has inefficiency when it returns
an error value, driver has to convert it to vm_fault_t
type. With new vmf_insert_mixed_mkwrite() this limitation
will be addressed.
Signed-off-by: S
VM_FAULT_NOPAGE instead of VM_FAULT_OOM.
With new vmf_insert_mixed() this issue is addressed.
vm_insert_mixed_mkwrite has inefficiency when it returns
an error value, driver has to convert it to vm_fault_t
type. With new vmf_insert_mixed_mkwrite() this limitation
will be addressed.
Signed-off-by: S
: change return type
to vm_fault_t")
Signed-off-by: Souptick Joarder
---
drivers/android/binder.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/android/binder.c b/drivers/android/binder.c
index 15e3d3c..65be4ec 100644
--- a/drivers/android/binder.c
+++ b/dr
On Mon, Apr 23, 2018 at 7:29 PM, Jan Kara wrote:
> On Sun 22-04-18 19:25:05, Matthew Wilcox wrote:
>> On Mon, Apr 23, 2018 at 01:09:48AM +0200, Jan Kara wrote:
>> > > -int vm_insert_mixed_mkwrite(struct vm_area_struct *vma, unsigned long
>> > > addr,
>> > > - pfn_t pfn)
>> > > +vm
On Mon, Apr 23, 2018 at 2:48 PM, Greg KH wrote:
> On Mon, Apr 16, 2018 at 08:41:21PM +0530, Souptick Joarder wrote:
>> Use new return type vm_fault_t for fault handler
>> in struct vm_operations_struct. For now, this is
>> just documenting that the function returns a
>>
On Sun, Apr 22, 2018 at 3:04 AM, Matthew Wilcox wrote:
> On Sun, Apr 22, 2018 at 02:35:29AM +0530, Souptick Joarder wrote:
>> Use new return type vm_fault_t for fault handler. For
>> now, this is just documenting that the function returns
>> a VM_FAULT value rather th
o convert it to vm_fault_t
type. With new vmf_insert_mixed_mkwrite() this limitation
will be addressed.
As new function vmf_insert_mixed_mkwrite() only called
from fs/dax.c, so keeping both the changes in a single
patch.
Signed-off-by: Souptick Joarder
---
v2: vm_insert_mixed_mkwrite is
On Sun, Apr 22, 2018 at 1:04 AM, Matthew Wilcox wrote:
> On Sat, Apr 21, 2018 at 10:35:40PM +0530, Souptick Joarder wrote:
>> As of now vm_insert_mixed_mkwrite() is only getting
>> invoked from fs/dax.c, so this change has to go first
>> in linus tree before changes in da
o convert it to vm_fault_t
type. With new vmf_insert_mixed_mkwrite() this limitation
will be addressed.
Signed-off-by: Souptick Joarder
---
fs/dax.c| 78 +
include/linux/dax.h | 4 +--
2 files changed, 39 insertions(+), 43 deletions
before changes in dax.
Signed-off-by: Souptick Joarder
---
include/linux/mm.h | 4 ++--
mm/memory.c| 15 +++
2 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/include/linux/mm.h b/include/linux/mm.h
index 1ac1f06..9fe441c 100644
--- a/include/linux/mm.h
+++ b
type to
vm_fault_t")
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
v2: Modified the change log.
v3: Modified the change log
fs/kernfs/file.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/fs/kernfs/file.c b/fs/kernfs/file.c
index fd5ce88..2
> I'm fine with the patch, but shouldn't this be part of a larger series /
> cocinelle script?
>
This one is trivial but many others are not.
As part of this clean up we have fixed many
existing bugs in fault handlers of drivers/
file systems. Also we are replacing existing
vm_insert_foo() with ne
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
arch/mips/kvm/mips.c | 2 +-
arch/powerpc/kvm/powerpc.c | 2 +-
arch/s390/kvm/kvm-s390.c | 2 +-
arch/x86/kvm/x86.c | 2 +-
include/linux/kvm_host.h | 2 +-
virt/kvm/arm/arm.c | 2 +-
virt/kvm/kvm_main.c
_fault_t")
Signed-off-by: Souptick Joarder
Acked-by: Sakari Ailus
---
v2: Updated patch subject
v3: Correcting patch subject
drivers/media/v4l2-core/videobuf-dma-sg.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/v4l2-core/videobuf-dma-sg.c
b/drivers/me
_fault_t")
Signed-off-by: Souptick Joarder
Acked-by: Sakari Ailus
---
v2: Updated patch subject
drivers/media/v4l2-core/videobuf-dma-sg.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/v4l2-core/videobuf-dma-sg.c
b/drivers/media/v4l2-core/videobuf-dma-sg.c
On Wed, Apr 18, 2018 at 2:38 PM, Sakari Ailus wrote:
> On Tue, Apr 17, 2018 at 08:13:06PM +0530, Souptick Joarder wrote:
>> Use new return type vm_fault_t for fault handler. For
>> now, this is just documenting that the function returns
>> a VM_FAULT value rather than an errn
> This new function returns VM_FAULT_NOPAGE only for 0 and -EBUSY, whereas
> we used to return VM_FAULT_NOPAGE for -EAGAIN, -ERESTARTSYS and -EINTR
> as well.
Previously vm_insert_page unable to return VM_FAULT_ type due to which
different drivers
have their own mapping from err to VM_FAULT_ type.
t 8:59 PM, Jani Nikula
wrote:
> On Tue, 17 Apr 2018, Souptick Joarder wrote:
>> Use new return type vm_fault_t for fault handler. For
>> now, this is just documenting that the function returns
>> a VM_FAULT value rather than an errno. Once all instances
>> are convert
_fault_t")
Signed-off-by: Souptick Joarder
---
drivers/gpu/drm/i915/i915_drv.h | 3 ++-
drivers/gpu/drm/i915/i915_gem.c | 15 ---
2 files changed, 10 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
index a42deeb..95b0d50
_fault_t")
previously cxl_mmap_fault returns VM_FAULT_NOPAGE as
default value irrespective of vm_insert_pfn() return
value. This bug is fixed with new vmf_insert_pfn()
which will return VM_FAULT_ type based on err.
Signed-off-by: Souptick Joarder
---
drivers/misc/cxl/context.c | 7 ---
1 fi
_fault_t")
Signed-off-by: Souptick Joarder
---
drivers/media/v4l2-core/videobuf-dma-sg.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/v4l2-core/videobuf-dma-sg.c
b/drivers/media/v4l2-core/videobuf-dma-sg.c
index f412429..54257ea 100644
--- a/drivers/media/v4l2-
_fault_t")
Signed-off-by: Souptick Joarder
---
drivers/infiniband/hw/qib/qib_file_ops.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/infiniband/hw/qib/qib_file_ops.c
b/drivers/infiniband/hw/qib/qib_file_ops.c
index 52c29db3..52aea84 100644
--- a/drivers/infi
_fault_t")
Signed-off-by: Souptick Joarder
---
drivers/infiniband/hw/hfi1/file_ops.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/infiniband/hw/hfi1/file_ops.c
b/drivers/infiniband/hw/hfi1/file_ops.c
index 41fafeb..b04624d 100644
--- a/drivers/infiniband/hw/hf
_fault_t")
Signed-off-by: Souptick Joarder
---
drivers/hwtracing/intel_th/msu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hwtracing/intel_th/msu.c b/drivers/hwtracing/intel_th/msu.c
index dfb57ea..0ac84eb 100644
--- a/drivers/hwtracing/intel_th/msu.c
+++ b/driv
_fault_t")
Signed-off-by: Souptick Joarder
---
drivers/hsi/clients/cmt_speech.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/hsi/clients/cmt_speech.c b/drivers/hsi/clients/cmt_speech.c
index 324cb8e..a1d4b93 100644
--- a/drivers/hsi/clients/cmt_speech.c
+++ b/driver
_fault_t")
Previously vm_insert_page() returns err which driver
mapped into VM_FAULT_* type. The new function
vmf_insert_page() will replace this inefficiency by
returning VM_FAULT_* type.
Signed-off-by: Souptick Joarder
---
drivers/gpu/drm/tegra/gem.c | 18 ++
1 file changed,
_fault_t")
Signed-off-by: Souptick Joarder
---
drivers/gpu/drm/qxl/qxl_ttm.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/qxl/qxl_ttm.c b/drivers/gpu/drm/qxl/qxl_ttm.c
index 59cd74c..0b482b1 100644
--- a/drivers/gpu/drm/qxl/qxl_ttm.c
+++ b/driver
hange return type
to vm_fault_t")
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
v2: Modified the change log
v3: Updated the change log
drivers/uio/uio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/uio/uio.c b/drivers/uio/uio.c
index fd4848
On Mon, Apr 16, 2018 at 8:56 PM, Greg KH wrote:
> On Mon, Apr 16, 2018 at 08:24:40PM +0530, Souptick Joarder wrote:
>> Use new return type vm_fault_t for fault handler
>> in struct vm_operations_struct. For now, this is
>> just documenting that the function returns a
>>
Use new return type vm_fault_t for fault handler
in struct vm_operations_struct. For now, this is
just documenting that the function returns a VM_
FAULT value rather than an errno. Once all inst
ances are converted, vm_fault_t will become a di
stinct type.
Signed-off-by: Souptick Joarder
->
1c8f422059ae5da07db7406ab916203f9417e396
Signed-off-by: Souptick Joarder
---
drivers/android/binder.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/android/binder.c b/drivers/android/binder.c
index 15e3d3c..65be4ec 100644
--- a/drivers/android/binder.c
+++ b/drivers/android/binde
Use new return type vm_fault_t for fault handler.
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
drivers/gpu/drm/vgem/vgem_drv.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/vgem/vgem_drv.c b/drivers/gpu/drm/vgem/vgem_drv.c
index
->
1c8f422059ae5da07db7406ab916203f9417e396
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
v2: Modified the change log
drivers/uio/uio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/uio/uio.c b/drivers/uio/uio.c
index fd48483..d9235f3 100644
--- a/drivers/uio/ui
->
1c8f422059ae5da07db7406ab916203f9417e396
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
v2: Modified the change log.
fs/kernfs/file.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/fs/kernfs/file.c b/fs/kernfs/file.c
index fd5ce88..2015d8c 100644
--- a/fs/kernfs/file.c
+++ b
Use new return type vm_fault_t for fault handler
in struct vm_operations_struct.
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
kernel/relay.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/relay.c b/kernel/relay.c
index c302940..a8cdbf7 100644
Use new return type vm_fault_t for fault handler and
page_mkwrite handler in struct vm_operations_struct.
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
kernel/events/core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/events/core.c b/kernel
Use new return type vm_fault_t for fault handler.
Signed-off-by: Souptick Joarder
Reviewed-by: Matthew Wilcox
---
ipc/shm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/ipc/shm.c b/ipc/shm.c
index 4643865..2ba0cfc 100644
--- a/ipc/shm.c
+++ b/ipc/shm.c
@@ -378,7 +378,7
Use dma_pool_zalloc() instead of dma_pool_alloc + memset
Signed-off-by: Souptick Joarder
---
drivers/mailbox/bcm-flexrm-mailbox.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/mailbox/bcm-flexrm-mailbox.c
b/drivers/mailbox/bcm-flexrm-mailbox.c
index a8cf433
Hi David, Dan,
On Thu, Nov 30, 2017 at 12:50 AM, David Daney wrote:
> On 11/29/2017 08:07 AM, Souptick Joarder wrote:
>>
>> On Wed, Nov 29, 2017 at 4:00 PM, Souptick Joarder
>> wrote:
>>>
>>> On Wed, Nov 29, 2017 at 6:25 AM, David Daney
>>> wr
On Wed, Nov 29, 2017 at 4:00 PM, Souptick Joarder wrote:
> On Wed, Nov 29, 2017 at 6:25 AM, David Daney wrote:
>> From: Carlos Munoz
>>
>> The Cavium OCTEON cn78xx and cn73xx SoCs have network packet I/O
>> hardware that is significantly different from previous g
On Sat, Jan 2, 2016 at 2:02 AM, SF Markus Elfring
wrote:
> From: Markus Elfring
> Date: Fri, 1 Jan 2016 21:16:01 +0100
>
> Rename a jump label according to the current Linux coding style convention.
>
> Signed-off-by: Markus Elfring
> ---
> drivers/net/wireless/intel/iwlegacy/common.c | 4 ++--
On Wed, Dec 30, 2015 at 10:35 PM, Bjorn Andersson wrote:
> On Wed, Dec 30, 2015 at 8:47 AM, Souptick Joarder
> wrote:
>>
>> HI Bjorn,
>>
>> On Thu, Dec 24, 2015 at 2:03 PM, Bjorn Andersson wrote:
>> > Make the addr parameter const in SET_IEEE80211_PERM_
HI Bjorn,
On Thu, Dec 24, 2015 at 2:03 PM, Bjorn Andersson wrote:
> Make the addr parameter const in SET_IEEE80211_PERM_ADDR() to save
> clients from having to cast away a const qualifier.
>
> Signed-off-by: Bjorn Andersson
> ---
> include/net/mac80211.h | 2 +-
> 1 file changed, 1 insertion(+)
Brent,
On Tue, Dec 22, 2015 at 2:42 PM, Kalle Valo wrote:
> Souptick Joarder writes:
>
>> Hi Brent,
>>
>> On Tue, Dec 22, 2015 at 3:23 AM, Brent Taylor wrote:
>>> On Mon, Dec 21, 2015 at 1:23 PM, Souptick Joarder
>>> wrote:
>>>> Hi B
Hi Brent,
On Tue, Dec 22, 2015 at 3:23 AM, Brent Taylor wrote:
> On Mon, Dec 21, 2015 at 1:23 PM, Souptick Joarder
> wrote:
>> Hi Brent,
>>
>> On Tue, Dec 1, 2015 at 11:11 AM, Brent Taylor wrote:
>>> Since commit 8437754c83351d6213c1a47ff029c1126d6042a7, ar-&g
On Mon, Dec 21, 2015 at 10:17 PM, Nicholas Mc Guire wrote:
> This is an API consolidation only. The use of kmalloc + memset to 0
> is equivalent to kzalloc.
>
> Signed-off-by: Nicholas Mc Guire
> ---
>
> Found by coccinelle script (relaxed version of
> scripts/coccinelle/api/alloc/kzalloc-simple.
Hi Brent,
On Tue, Dec 1, 2015 at 11:11 AM, Brent Taylor wrote:
> Since commit 8437754c83351d6213c1a47ff029c1126d6042a7, ar->fw is expected to
> be pointing to memory allocated by vmalloc. If the api1 method (via
> ath6kl_fetch_fw_api1) is used to allocate memory for ar->fw, then kmemdup is
>
601 - 659 of 659 matches
Mail list logo