[PATCH 08/12] staging: lustre: llite: debugging for ll_file_open LASSERT

[James Simmons]

From: Andreas Dilger 

Add debugging for LASSERTF(it_disposition(it, DISP_ENQ_OPEN_REF)
in ll_file_open(), since this is a rarely hit failure under racer,
and it would be useful to get more information if this is hit
again.  Print the full intent disposition, as well as the status,
in case Oleg's earlier comment about bailing out on any error is
actually the case here.

Signed-off-by: Andreas Dilger 
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-1993
Reviewed-on: http://review.whamcloud.com/6250
Reviewed-by: Bob Glossman 
Reviewed-by: John L. Hammond 
Reviewed-by: Oleg Drokin 
Signed-off-by: James Simmons 
---
 drivers/staging/lustre/lustre/llite/file.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/llite/file.c 
b/drivers/staging/lustre/lustre/llite/file.c
index a8de10e..f47f2ac 100644
--- a/drivers/staging/lustre/lustre/llite/file.c
+++ b/drivers/staging/lustre/lustre/llite/file.c
@@ -679,7 +679,9 @@ restart:
if (rc)
goto out_och_free;
 
-   LASSERT(it_disposition(it, DISP_ENQ_OPEN_REF));
+   LASSERTF(it_disposition(it, DISP_ENQ_OPEN_REF),
+"inode %p: disposition %x, status %d\n", inode,
+it_disposition(it, ~0), it->d.lustre.it_status);
 
rc = ll_local_open(file, it, fd, *och_p);
if (rc)
-- 
2.7.4

[PATCH 08/12] staging: lustre: llite: debugging for ll_file_open LASSERT

[James Simmons]

From: Andreas Dilger 

Add debugging for LASSERTF(it_disposition(it, DISP_ENQ_OPEN_REF)
in ll_file_open(), since this is a rarely hit failure under racer,
and it would be useful to get more information if this is hit
again.  Print the full intent disposition, as well as the status,
in case Oleg's earlier comment about bailing out on any error is
actually the case here.

Signed-off-by: Andreas Dilger 
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-1993
Reviewed-on: http://review.whamcloud.com/6250
Reviewed-by: Bob Glossman 
Reviewed-by: John L. Hammond 
Reviewed-by: Oleg Drokin 
Signed-off-by: James Simmons 
---
 drivers/staging/lustre/lustre/llite/file.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/lustre/lustre/llite/file.c 
b/drivers/staging/lustre/lustre/llite/file.c
index a8de10e..f47f2ac 100644
--- a/drivers/staging/lustre/lustre/llite/file.c
+++ b/drivers/staging/lustre/lustre/llite/file.c
@@ -679,7 +679,9 @@ restart:
if (rc)
goto out_och_free;
 
-   LASSERT(it_disposition(it, DISP_ENQ_OPEN_REF));
+   LASSERTF(it_disposition(it, DISP_ENQ_OPEN_REF),
+"inode %p: disposition %x, status %d\n", inode,
+it_disposition(it, ~0), it->d.lustre.it_status);
 
rc = ll_local_open(file, it, fd, *och_p);
if (rc)
-- 
2.7.4