> + if (ret == -ENOSPC && !enospc) {
> + ssize_t ret2 = xfs_flush_pages(ip, 0, -1, 0, FI_NONE);
> + if (ret2) {
> + ret = ret2;
> goto out_unlock_internal;
> + }
> enospc = 1;
> goto write_retry;
> }
I don't think there is any reason for the ret2 variale here. If it's
non-zero we assign it to ret, and if it's zero we overwrite it again
after jumping to the write_retry label.
_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs
