diff mbox

[PTACH] SQUASHME: pnfs-obj: Important fallout from the last rebase

Message ID 4DDA3AA4.8000205@panasas.com (mailing list archive)
State New, archived
Headers show

Commit Message

Boaz Harrosh May 23, 2011, 10:44 a.m. UTC
On the last rebase I dropped this hunk it needs to go into  the:
	[PATCH v5 31/38] pnfs-obj: osd raid engine read/write implementation

With out this the code does not work

Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
---
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Benny Halevy May 23, 2011, 1:53 p.m. UTC | #1
On 2011-05-23 13:44, Boaz Harrosh wrote:
> 
> On the last rebase I dropped this hunk it needs to go into  the:
> 	[PATCH v5 31/38] pnfs-obj: osd raid engine read/write implementation
> 
> With out this the code does not work

I agree that in the current implementation the ld must implement
pg_test for pnfs to work.
This patch is reverse though...

Benny

> 
> Signed-off-by: Boaz Harrosh <bharrosh@panasas.com>
> ---
> diff --git a/fs/nfs/objlayout/objio_osd.c b/fs/nfs/objlayout/objio_osd.c
> index 105d8a6..74b5d3f 100644
> --- a/fs/nfs/objlayout/objio_osd.c
> +++ b/fs/nfs/objlayout/objio_osd.c
> @@ -989,19 +990,6 @@ ssize_t objio_write_pagelist(struct objlayout_io_state *ol_state, bool stable)
>  	return _write_exec(ios);
>  }
>  
> -/*
> - * objlayout_pg_test(). Called by nfs_can_coalesce_requests()
> - *
> - * return 1 :  coalesce page
> - * return 0 :  don't coalesce page
> - */
> -int
> -objlayout_pg_test(struct nfs_pageio_descriptor *pgio, struct nfs_page *prev,
> -		   struct nfs_page *req)
> -{
> -	return 1;
> -}
> -
>  static struct pnfs_layoutdriver_type objlayout_type = {
>  	.id = LAYOUT_OSD2_OBJECTS,
>  	.name = "LAYOUT_OSD2_OBJECTS",
> @@ -1021,8 +1009,6 @@ static struct pnfs_layoutdriver_type objlayout_type = {
>  
>  	.encode_layoutcommit	 = objlayout_encode_layoutcommit,
>  	.encode_layoutreturn     = objlayout_encode_layoutreturn,
> -
> -	.pg_test		= objlayout_pg_test,
>  };
>  
>  void *objio_init_mt(void)

--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/fs/nfs/objlayout/objio_osd.c b/fs/nfs/objlayout/objio_osd.c
index 105d8a6..74b5d3f 100644
--- a/fs/nfs/objlayout/objio_osd.c
+++ b/fs/nfs/objlayout/objio_osd.c
@@ -989,19 +990,6 @@  ssize_t objio_write_pagelist(struct objlayout_io_state *ol_state, bool stable)
 	return _write_exec(ios);
 }
 
-/*
- * objlayout_pg_test(). Called by nfs_can_coalesce_requests()
- *
- * return 1 :  coalesce page
- * return 0 :  don't coalesce page
- */
-int
-objlayout_pg_test(struct nfs_pageio_descriptor *pgio, struct nfs_page *prev,
-		   struct nfs_page *req)
-{
-	return 1;
-}
-
 static struct pnfs_layoutdriver_type objlayout_type = {
 	.id = LAYOUT_OSD2_OBJECTS,
 	.name = "LAYOUT_OSD2_OBJECTS",
@@ -1021,8 +1009,6 @@  static struct pnfs_layoutdriver_type objlayout_type = {
 
 	.encode_layoutcommit	 = objlayout_encode_layoutcommit,
 	.encode_layoutreturn     = objlayout_encode_layoutreturn,
-
-	.pg_test		= objlayout_pg_test,
 };
 
 void *objio_init_mt(void)