[RESEND PATCH 3/7] mm/gup: Change GUP fast to use flags rather than a write 'bool'

2019-02-19 Thread ira . weiny
From: Ira Weiny To facilitate additional options to get_user_pages_fast() change the singular write parameter to be gup_flags. This patch does not change any functionality. New functionality will follow in subsequent patches. Some of the get_user_pages_fast() call sites were unchanged because

[RESEND PATCH 6/7] IB/qib: Use the new FOLL_LONGTERM flag to get_user_pages_fast()

2019-02-19 Thread ira . weiny
From: Ira Weiny Use the new FOLL_LONGTERM to get_user_pages_fast() to protect against FS DAX pages being mapped. Signed-off-by: Ira Weiny --- drivers/infiniband/hw/qib/qib_user_sdma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/qib/qib_user_sdma.c

[RESEND PATCH 2/7] mm/gup: Change write parameter to flags in fast walk

2019-02-19 Thread ira . weiny
From: Ira Weiny In order to support more options in the GUP fast walk, change the write parameter to flags throughout the call stack. This patch does not change functionality and passes FOLL_WRITE where write was previously used. Signed-off-by: Ira Weiny --- mm/gup.c | 52

[RESEND PATCH 5/7] IB/hfi1: Use the new FOLL_LONGTERM flag to get_user_pages_fast()

2019-02-19 Thread ira . weiny
From: Ira Weiny Use the new FOLL_LONGTERM to get_user_pages_fast() to protect against FS DAX pages being mapped. Signed-off-by: Ira Weiny --- drivers/infiniband/hw/hfi1/user_pages.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/infiniband/hw/hfi1/user_pages.

[RESEND PATCH 7/7] IB/mthca: Use the new FOLL_LONGTERM flag to get_user_pages_fast()

2019-02-19 Thread ira . weiny
From: Ira Weiny Use the new FOLL_LONGTERM to get_user_pages_fast() to protect against FS DAX pages being mapped. Signed-off-by: Ira Weiny --- drivers/infiniband/hw/mthca/mthca_memfree.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/infiniband/hw/mthca/mthca_memf

[RESEND PATCH 1/7] mm/gup: Replace get_user_pages_longterm() with FOLL_LONGTERM

2019-02-19 Thread ira . weiny
From: Ira Weiny Rather than have a separate get_user_pages_longterm() call, introduce FOLL_LONGTERM and change the longterm callers to use it. This patch does not change any functionality. FOLL_LONGTERM can only be supported with get_user_pages() as it requires vmas to determine if DAX is in us

[RESEND PATCH 0/7] Add FOLL_LONGTERM to GUP fast and use it

2019-02-19 Thread ira . weiny
From: Ira Weiny Resending these as I had only 1 minor comment which I believe we have covered in this series. I was anticipating these going through the mm tree as they depend on a cleanup patch there and the IB changes are very minor. But they could just as well go through the IB tree. NOTE:

[RESEND PATCH 4/7] mm/gup: Add FOLL_LONGTERM capability to GUP fast

2019-02-19 Thread ira . weiny
From: Ira Weiny DAX pages were previously unprotected from longterm pins when users called get_user_pages_fast(). Use the new FOLL_LONGTERM flag to check for DEVMAP pages and fall back to regular GUP processing if a DEVMAP page is encountered. Signed-off-by: Ira Weiny --- mm/gup.c | 24 ++

Re: [PATCH] staging: erofs: remove rcu_read_lock() in erofs_try_to_free_cached_page

2019-02-19 Thread Gao Xiang
Hi Greg, Could you consider take some time on these two patches for linux-5.1-rc1? 1) staging: erofs: fix race of initializing xattrs of a inode at the same time 2) staging: erofs: remove rcu_read_lock() in erofs_try_to_free_cached_page It will be of great help, especially patch 1 (it could cau

Re: [PATCH] staging: erofs: remove rcu_read_lock() in erofs_try_to_free_cached_page

2019-02-19 Thread Chao Yu
On 2019/2/16 16:46, Gao Xiang wrote: > page_private(page) cannot be changed if page lock is taken. > > Besides, the corresponding workgroup won't be freed > if the page is already protected by page lock, therefore > no need to take rcu read lock. > > Signed-off-by: Gao Xiang Reviewed-by: Chao Y

[PATCH] media: imx: vdic: Restore default case to prepare_vdi_in_buffers()

2019-02-19 Thread Steve Longerbeam
Restore a default case to prepare_vdi_in_buffers() to fix the following smatch errors: drivers/staging/media/imx/imx-media-vdic.c:236 prepare_vdi_in_buffers() error: uninitialized symbol 'prev_phys'. drivers/staging/media/imx/imx-media-vdic.c:237 prepare_vdi_in_buffers() error: uninitialized sym

[PATCH v5 7/7] media: imx: Allow BT.709 encoding for IC routes

2019-02-19 Thread Steve Longerbeam
The IC now supports BT.709 Y'CbCr encoding, in addition to existing BT.601 encoding, so allow both, for pipelines that route through the IC. Reported-by: Tim Harvey Signed-off-by: Steve Longerbeam --- Changes in v5: - rebased this patch on top of repurposing the function to imx_media_try_color

[PATCH v5 6/7] media: imx: Try colorimetry at both sink and source pads

2019-02-19 Thread Steve Longerbeam
The colorimetry parameters need to be tested at both sink and source pads. Specifically, for prpencvf, the IC only supports RGB full-range quantization at input and output. Fix this by cleaning up imx_media_fill_default_mbus_fields(), renaming to imx_media_try_colorimetry(), and call it at both si

[PATCH v5 3/7] gpu: ipu-v3: ipu-ic: Fully describe colorspace conversions

2019-02-19 Thread Steve Longerbeam
Only providing the input and output RGB/YUV space to the IC task init functions is not sufficient. To fully characterize a colorspace conversion, the colorspace (chromaticities), Y'CbCr encoding standard, and quantization also need to be specified. Define a 'struct ipu_ic_colorspace' that includes

Re: [PATCH 13/14] staging: android: ion: Do not sync CPU cache on map/unmap

2019-02-19 Thread Laura Abbott
On 1/24/19 8:44 AM, Brian Starkey wrote: On Thu, Jan 24, 2019 at 10:04:46AM -0600, Andrew F. Davis wrote: On 1/23/19 11:11 AM, Brian Starkey wrote: [snip] I'm very new to all this, so any pointers to history in this area are appreciated. [snip] In case you didn't come across it alrea

Re: [PATCH] staging: android: ion: fix sys heap pool's gfp_flags

2019-02-19 Thread Laura Abbott
On 1/31/19 10:59 PM, Qing Xia wrote: In the first loop, gfp_flags will be modified to high_order_gfp_flags, and there will be no chance to change back to low_order_gfp_flags. Fixes: e7f63771 ("ION: Sys_heap: Add cached pool to spead up cached buffer alloc") Signed-off-by: Qing Xia --- driver

Re: [PATCH v2] staging: android: ion: Allocate from heap ID directly without mask

2019-02-19 Thread Laura Abbott
On 2/15/19 11:01 AM, John Stultz wrote: On Fri, Feb 15, 2019 at 2:51 AM Brian Starkey wrote: Hi John, On Thu, Feb 14, 2019 at 09:38:29AM -0800, John Stultz wrote: [snip] Some thoughts, as this ABI break has the potential to be pretty painful. 1) Unfortunately, this ABI is exposed *throu

Re: [PATCH] net: dsa: add missing of_node_put

2019-02-19 Thread Julia Lawall
On Wed, 20 Feb 2019, Himadri Pandya wrote: > Decrement the reference count on port while returning out of the loop. I don't think you sent this to the right people. You need to send it to the people indicated in get_maintainer.pl. This is not updating a staging driver. julia > > Signed-off

[PATCH] net: dsa: add missing of_node_put

2019-02-19 Thread Himadri Pandya
Decrement the reference count on port while returning out of the loop. Signed-off-by: Himadri Pandya --- net/dsa/dsa2.c | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/net/dsa/dsa2.c b/net/dsa/dsa2.c index a1917025e155..396e7433dd8f 100644 --- a/net/dsa/dsa2.c

Re: [PATCH 1/2] media: ipu3: shut up warnings produced with W=1

2019-02-19 Thread Mauro Carvalho Chehab
Em Tue, 19 Feb 2019 16:03:17 +0200 Sakari Ailus escreveu: > Hi Mauro, > > On Tue, Feb 19, 2019 at 09:00:29AM -0500, Mauro Carvalho Chehab wrote: > > There are lots of warnings produced by this driver. It is not > > as much as atomisp, but it is still a lot. > > > > So, use the same solution to

Re: [PATCH 2/2] ipu3-mmu: fix some kernel-doc macros

2019-02-19 Thread Sakari Ailus
On Tue, Feb 19, 2019 at 09:00:30AM -0500, Mauro Carvalho Chehab wrote: > Some kernel-doc markups are wrong. fix them. > > Signed-off-by: Mauro Carvalho Chehab Acked-by: Sakari Ailus -- Sakari Ailus sakari.ai...@linux.intel.com ___ devel mailing list

Re: [PATCH 1/2] media: ipu3: shut up warnings produced with W=1

2019-02-19 Thread Sakari Ailus
Hi Mauro, On Tue, Feb 19, 2019 at 09:00:29AM -0500, Mauro Carvalho Chehab wrote: > There are lots of warnings produced by this driver. It is not > as much as atomisp, but it is still a lot. > > So, use the same solution to hide most of them. > Those need to be fixed before promoting it out of sta

[PATCH 2/2] ipu3-mmu: fix some kernel-doc macros

2019-02-19 Thread Mauro Carvalho Chehab
Some kernel-doc markups are wrong. fix them. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/media/ipu3/ipu3-mmu.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/media/ipu3/ipu3-mmu.c b/drivers/staging/media/ipu3/ipu3-mmu.c index cd2038b22b55..

[PATCH 1/2] media: ipu3: shut up warnings produced with W=1

2019-02-19 Thread Mauro Carvalho Chehab
There are lots of warnings produced by this driver. It is not as much as atomisp, but it is still a lot. So, use the same solution to hide most of them. Those need to be fixed before promoting it out of staging, so add it at the TODO list. Signed-off-by: Mauro Carvalho Chehab --- drivers/stagin

[PATCH 1/2] media: imx7-media-csi: don't store a floating pointer

2019-02-19 Thread Mauro Carvalho Chehab
if imx7_csi_try_fmt() fails, outcc variable won't be initialized and csi->cc[IMX7_CSI_PAD_SRC] would be pointing to a random location. Signed-off-by: Mauro Carvalho Chehab --- drivers/staging/media/imx/imx7-media-csi.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) di

[PATCH 2/2] media: imx7-media-csi: get rid of unused var

2019-02-19 Thread Mauro Carvalho Chehab
drivers/staging/media/imx/imx7-media-csi.c: In function 'imx7_csi_enum_mbus_code': drivers/staging/media/imx/imx7-media-csi.c:926:33: warning: variable 'in_cc' set but not used [-Wunused-but-set-variable] const struct imx_media_pixfmt *in_cc;

Re: [Outreachy Kernel][PATCH v2] staging: erofs: match alignment with open parentheses

2019-02-19 Thread Chao Yu
On 2019/2/19 18:24, Gao Xiang wrote: > [+cc linux-erofs and staging mailing list] > > On 2019/2/19 17:25, Bhanusree Pola wrote: >> Align code with open parantheses to improve the readability. >> Issue found using checkpatch.pl >> >> Signed-off-by: Bhanusree Pola Reviewed-by: Chao Yu Thanks, _

Re: [Outreachy Kernel][PATCH v2] staging: erofs: match alignment with open parentheses

2019-02-19 Thread Gao Xiang
[+cc linux-erofs and staging mailing list] On 2019/2/19 17:25, Bhanusree Pola wrote: > Align code with open parantheses to improve the readability. > Issue found using checkpatch.pl > > Signed-off-by: Bhanusree Pola Reviewed-by: Gao Xiang Thanks, Gao Xiang > --- > > v2: -modified log messag

Re: [PATCH v4] staging: nrf24: add new driver for 2.4GHz radio transceiver

2019-02-19 Thread Greg Kroah-Hartman
On Wed, Feb 13, 2019 at 08:40:35PM +0100, Marcin Ciupak wrote: > This patch adds driver for Nordic Semiconductor nRF24L01+ radio > transceiver. > > Signed-off-by: Marcin Ciupak > --- > Changes in v2: > - add terminating newlines to all logging formats > Changes in v3: > - patch subject > -

[PATCH] media: staging/imx: Allow driver to build if COMPILE_TEST is enabled

2019-02-19 Thread Javier Martinez Canillas
The driver has runtime but no build time dependency with IMX_IPUV3_CORE, so can be built for testing purposes if COMPILE_TEST option is enabled. This is useful to have more build coverage and make sure that the driver is not affected by changes that could cause build regressions. Signed-off-by: J