-
Notifications
You must be signed in to change notification settings - Fork 126
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Hangs or crashes when using Ver 0.11.10 with XFS #402
Comments
Hi @yito24, thank you for the The issue is currently under investigation. Hope to fix it soon. |
I have merged the elastio-snap code and am testing it, and no crashes have occurred. bio_helper.c if (PageAnon(pg)) return NULL; if (!pg->mapping) return NULL; if (!virt_addr_valid(pg->mapping)) return NULL; return pg->mapping->host; int bio_needs_cow(struct bio *bio, struct inode *inode) { bio_iter_t iter; bio_iter_bvec_t bvec; /*if (!test_bit(COW_ON_BDEV, &dev->sd_cow_state)) { return 1; }*/ #if (defined HAVE_ENUM_REQ_OPF) || \ (defined HAVE_ENUM_REQ_OP && defined HAVE_WRITE_ZEROES) // HAVE_ENUM_REQ_OPF: KERNEL_VERSION >= 4.10 && KERNEL_VERSION <= 5.19 // HAVE_ENUM_REQ_OP: KERNEL_VERSION < 4.10 && KERNEL_VERSION >= 6.0 // HAVE_WRITE_ZEROES: KERNEL_VERSION >= 4.10 if(bio_op(bio) == REQ_OP_WRITE_ZEROES) return 1; #endif //check the inode of each page return true if it does not match our cow file bio_for_each_segment(bvec, bio, iter) { if (page_get_inode(bio_iter_page(bio, iter)) != inode){ return 1; } } return 0; |
Thanks, will take a look soon) |
Hello, please recheck with newer version 0.12.0. Thanks. |
This has been fixed in 0.12.0. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I tested 0.11.10 on the following operating systems:
Using XFS on this OS causes crashes.
No crashes with EXT4.
This occurs in this function.
[ bio_needs_cow ]
I've attached the dmesg.txt from Ubuntu 24.04.
dmesg.202412231756.txt
Already reported for RHEL9.5.
#396 (comment)
The text was updated successfully, but these errors were encountered: