[LU-3161] LASSERT() in lod_initialize_objects() Created: 12/Apr/13  Updated: 05/Jun/15  Resolved: 28/Oct/13

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.4.0
Fix Version/s: Lustre 2.6.0, Lustre 2.4.2, Lustre 2.5.1

Type: Bug Priority: Major
Reporter: Andreas Dilger Assignee: Zhenyu Xu
Resolution: Fixed Votes: 0
Labels: patch

Issue Links:
Related
is related to LU-3917 lod_initialize_objects()) ASSERTION( ... Resolved
is related to LU-3918 kernel NULL pointer dereference at lo... Resolved
Severity: 3
Rank (Obsolete): 7698

 Description   

Looking at the code in lod_initialize_objects() I found the following comment and code:

                /*
                 * XXX: assertion is left for testing, to make
                 * sure we never process requests till configuration
                 * is completed. to be changed to -EINVAL
                 */

                lod_getref(&md->lod_ost_descs);
                LASSERT(cfs_bitmap_check(md->lod_ost_bitmap, idx));
                LASSERT(OST_TGT(md,idx));
                LASSERTF(OST_TGT(md,idx)->ltd_ost, "idx %d\n", idx);

This should be replaced by proper error handling, in case of LOV EA corruption on disk, or OST removal.

 Comments   
Comment by Gaurav Mahajan (Inactive) [ 05/Aug/13 ]

Gerrit patch: http://review.whamcloud.com/#/c/7234/

Comment by Peter Jones [ 28/Oct/13 ]

Landed for 2.6

Comment by Zhenyu Xu [ 17/Dec/13 ]

b2_5 port at http://review.whamcloud.com/8598

Generated at Sat Feb 10 01:31:30 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.