Hi Abhi,
Overall, I like the patch set. What Andreas said still applies, plus:
----- Original Message -----
> Use bio(s) to read in the journal sequentially in large chunks and
> locate the head of the journal.
> This is faster in most cases when compared to the existing bisect
> method which operates one block at a time.
>
> Signed-off-by: Abhi Das <a...@redhat.com>
> ---
(snip)
> + for (i=0; i<blocks; i++) {
Just a nit: Can you put some spaces in here like this?
for (i = 0; i < blocks; i++) {
(snip)
> + if (jd->jd_jhead.lh_sequence == 0)
> + error = 1;
Since gfs2_find_jhead is called by remount, which is called from vfs,
I think we should probably use a return code more meaningful than 1.
I'm not sure what we ought to use in this case. Perhaps -EPERM or -EACCES ?
Regards,
Bob Peterson
Red Hat File Systems
