On 2015/5/20 19:09, Junxiao Bi wrote:
> On 05/20/2015 06:25 PM, Joseph Qi wrote:
>> On 2015/5/20 16:22, Junxiao Bi wrote:
>>> On 05/12/2015 09:53 AM, Joseph Qi wrote:
>>>> jbd2_journal_dirty_metadata may fail. Currently it cannot take care of
>>>> non zero return value and just BUG in ocfs2_journal_dirty.
>>>> This patch is aborting the handle and journal instead of BUG.
>>>>
>>>> Signed-off-by: Joseph Qi <joseph...@huawei.com>
>>>> Cc: joyce.xue <xuejiu...@huawei.com>
>>>> ---
>>>> fs/ocfs2/journal.c | 12 +++++++++++-
>>>> 1 file changed, 11 insertions(+), 1 deletion(-)
>>>>
>>>> diff --git a/fs/ocfs2/journal.c b/fs/ocfs2/journal.c
>>>> index ff53192..eefca1e 100644
>>>> --- a/fs/ocfs2/journal.c
>>>> +++ b/fs/ocfs2/journal.c
>>>> @@ -775,7 +775,17 @@ void ocfs2_journal_dirty(handle_t *handle, struct
>>>> buffer_head *bh)
>>>> trace_ocfs2_journal_dirty((unsigned long long)bh->b_blocknr);
>>>>
>>>> status = jbd2_journal_dirty_metadata(handle, bh);
>>>> - BUG_ON(status);
>>>> + if (status) {
>>>> + mlog_errno(status);
>>>> + if (!is_handle_aborted(handle)) {
>>>> + journal_t *journal = handle->h_transaction->t_journal;
>>>> +
>>>> + mlog(ML_ERROR, "jbd2_journal_dirty_metadata failed. "
>>>> + "Aborting transaction and journal.");
>>>> + handle->h_err = status;
>>>> + jbd2_journal_abort_handle(handle);
>>>> + jbd2_journal_abort(journal, status);
>>> Let fs go after journal lose affect seemed not safe, may we set fs
>>> read-only here?
>>>
>>> Thanks,
>>> Junxiao.
>>>
>>>
>> Do you mean journal can still be updated even if it is aborted?
> No, journal will not be updated. After abort, journal api will return
> error, at last fs will be set read-only, but there is also api which
> didn't return error like ocfs2_journal_dirty(), so why not stop at the
> first time?
>
> Thanks,
> Junxiao.
Agree with you.
But here bh can be anything like di, gd, ... As Joyce Xue reported, the
problem also exists in ocfs2_abort_trigger. So I don't think we can get
sb we needed.
Is there any other way except changing prototype of ocfs2_journal_dirty?
>>
>>>> + }
>>>> + }
>>>> }
>>>>
>>>> #define OCFS2_DEFAULT_COMMIT_INTERVAL (HZ *
>>>> JBD2_DEFAULT_MAX_COMMIT_AGE)
>>>>
>>>
>>>
>>>
>>
>>
>
>
> .
>
_______________________________________________
Ocfs2-devel mailing list
Ocfs2-devel@oss.oracle.com
https://oss.oracle.com/mailman/listinfo/ocfs2-devel
