All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] fs: 9p: Adding new return type vm_fault_t
@ 2018-07-02 15:49 Souptick Joarder
  2018-07-04  8:46 ` [V9fs-developer] " piaojun
  0 siblings, 1 reply; 6+ messages in thread
From: Souptick Joarder @ 2018-07-02 15:49 UTC (permalink / raw)
  To: akpm; +Cc: ericvh, rminnich, lucho, v9fs-developer, linux-kernel, willy

Use new return type vm_fault_t for page_mkwrite
handler.

see commit 1c8f422059ae ("mm: change return type to
vm_fault_t") for reference.

Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com>
---
 fs/9p/vfs_file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
index 03c9e32..5f2e48d 100644
--- a/fs/9p/vfs_file.c
+++ b/fs/9p/vfs_file.c
@@ -533,7 +533,7 @@ int v9fs_file_fsync_dotl(struct file *filp, loff_t start, loff_t end,
 	return retval;
 }

-static int
+static vm_fault_t
 v9fs_vm_page_mkwrite(struct vm_fault *vmf)
 {
 	struct v9fs_inode *v9inode;
--
1.9.1


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [V9fs-developer] [PATCH] fs: 9p: Adding new return type vm_fault_t
  2018-07-02 15:49 [PATCH] fs: 9p: Adding new return type vm_fault_t Souptick Joarder
@ 2018-07-04  8:46 ` piaojun
  0 siblings, 0 replies; 6+ messages in thread
From: piaojun @ 2018-07-04  8:46 UTC (permalink / raw)
  To: Souptick Joarder, akpm
  Cc: lucho, ericvh, linux-kernel, willy, v9fs-developer, rminnich

LGTM

On 2018/7/2 23:49, Souptick Joarder wrote:
> Use new return type vm_fault_t for page_mkwrite
> handler.
> 
> see commit 1c8f422059ae ("mm: change return type to
> vm_fault_t") for reference.
> 
> Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
> Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com>
Acked-by: Jun Piao <piaojun@huawei.com>
> ---
>  fs/9p/vfs_file.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
> index 03c9e32..5f2e48d 100644
> --- a/fs/9p/vfs_file.c
> +++ b/fs/9p/vfs_file.c
> @@ -533,7 +533,7 @@ int v9fs_file_fsync_dotl(struct file *filp, loff_t start, loff_t end,
>  	return retval;
>  }
> 
> -static int
> +static vm_fault_t
>  v9fs_vm_page_mkwrite(struct vm_fault *vmf)
>  {
>  	struct v9fs_inode *v9inode;
> --
> 1.9.1
> 
> 
> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> _______________________________________________
> V9fs-developer mailing list
> V9fs-developer@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/v9fs-developer
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] fs: 9p: Adding new return type vm_fault_t
  2018-06-10  9:56 ` Souptick Joarder
@ 2018-06-19  5:07   ` Souptick Joarder
  0 siblings, 0 replies; 6+ messages in thread
From: Souptick Joarder @ 2018-06-19  5:07 UTC (permalink / raw)
  To: Eric Van Hensbergen, rminnich, Latchesar Ionkov
  Cc: v9fs-developer, Matthew Wilcox, netdev, linux-kernel

On Sun, Jun 10, 2018 at 3:26 PM, Souptick Joarder <jrdr.linux@gmail.com> wrote:
> On Sun, Jun 10, 2018 at 3:26 PM, Souptick Joarder <jrdr.linux@gmail.com> wrote:
>> Use new return type vm_fault_t for page_mkwrite
>> handler.
>>
>> Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
>> Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com>
>> ---
>>  fs/9p/vfs_file.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
>> index 03c9e32..5f2e48d 100644
>> --- a/fs/9p/vfs_file.c
>> +++ b/fs/9p/vfs_file.c
>> @@ -533,7 +533,7 @@ int v9fs_file_fsync_dotl(struct file *filp, loff_t start, loff_t end,
>>         return retval;
>>  }
>>
>> -static int
>> +static vm_fault_t
>>  v9fs_vm_page_mkwrite(struct vm_fault *vmf)
>>  {
>>         struct v9fs_inode *v9inode;
>> --
>> 1.9.1
>>
>
> Eric, as requested, posted it in net-dev and lkml mailing list.

Eric, any comment on this patch ?

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] fs: 9p: Adding new return type vm_fault_t
  2018-06-10  9:56 ` Souptick Joarder
  (?)
@ 2018-06-10  9:56 ` Souptick Joarder
  2018-06-19  5:07   ` Souptick Joarder
  -1 siblings, 1 reply; 6+ messages in thread
From: Souptick Joarder @ 2018-06-10  9:56 UTC (permalink / raw)
  To: Eric Van Hensbergen, rminnich, Latchesar Ionkov
  Cc: v9fs-developer, Matthew Wilcox, netdev, linux-kernel

On Sun, Jun 10, 2018 at 3:26 PM, Souptick Joarder <jrdr.linux@gmail.com> wrote:
> Use new return type vm_fault_t for page_mkwrite
> handler.
>
> Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
> Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com>
> ---
>  fs/9p/vfs_file.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
> index 03c9e32..5f2e48d 100644
> --- a/fs/9p/vfs_file.c
> +++ b/fs/9p/vfs_file.c
> @@ -533,7 +533,7 @@ int v9fs_file_fsync_dotl(struct file *filp, loff_t start, loff_t end,
>         return retval;
>  }
>
> -static int
> +static vm_fault_t
>  v9fs_vm_page_mkwrite(struct vm_fault *vmf)
>  {
>         struct v9fs_inode *v9inode;
> --
> 1.9.1
>

Eric, as requested, posted it in net-dev and lkml mailing list.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] fs: 9p: Adding new return type vm_fault_t
@ 2018-06-10  9:56 ` Souptick Joarder
  0 siblings, 0 replies; 6+ messages in thread
From: Souptick Joarder @ 2018-06-10  9:56 UTC (permalink / raw)
  To: ericvh, rminnich, lucho; +Cc: v9fs-developer, willy, netdev, linux-kernel

Use new return type vm_fault_t for page_mkwrite
handler.

Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com>
---
 fs/9p/vfs_file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
index 03c9e32..5f2e48d 100644
--- a/fs/9p/vfs_file.c
+++ b/fs/9p/vfs_file.c
@@ -533,7 +533,7 @@ int v9fs_file_fsync_dotl(struct file *filp, loff_t start, loff_t end,
 	return retval;
 }

-static int
+static vm_fault_t
 v9fs_vm_page_mkwrite(struct vm_fault *vmf)
 {
 	struct v9fs_inode *v9inode;
--
1.9.1

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] fs: 9p: Adding new return type vm_fault_t
@ 2018-06-10  9:56 ` Souptick Joarder
  0 siblings, 0 replies; 6+ messages in thread
From: Souptick Joarder @ 2018-06-10  9:56 UTC (permalink / raw)
  To: ericvh, rminnich, lucho; +Cc: v9fs-developer, willy, netdev, linux-kernel

Use new return type vm_fault_t for page_mkwrite
handler.

Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
Reviewed-by: Matthew Wilcox <mawilcox@microsoft.com>
---
 fs/9p/vfs_file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
index 03c9e32..5f2e48d 100644
--- a/fs/9p/vfs_file.c
+++ b/fs/9p/vfs_file.c
@@ -533,7 +533,7 @@ int v9fs_file_fsync_dotl(struct file *filp, loff_t start, loff_t end,
 	return retval;
 }

-static int
+static vm_fault_t
 v9fs_vm_page_mkwrite(struct vm_fault *vmf)
 {
 	struct v9fs_inode *v9inode;

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-07-04  8:47 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-02 15:49 [PATCH] fs: 9p: Adding new return type vm_fault_t Souptick Joarder
2018-07-04  8:46 ` [V9fs-developer] " piaojun
  -- strict thread matches above, loose matches on Subject: below --
2018-06-10  9:56 Souptick Joarder
2018-06-10  9:56 ` Souptick Joarder
2018-06-10  9:56 ` Souptick Joarder
2018-06-19  5:07   ` Souptick Joarder

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.