Re: XFS: bad clientid on recovery on MIPS (VIPT cache)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Yes I implemented these definitions for MIPS, without them this
problem will continue to occur.

On Mon, Jan 9, 2012 at 12:52 PM, Ajeet Yadav <ajeet.yadav.77@xxxxxxxxx> wrote:
> Hi,
> You are using which kernel version ? because this is fixed in XFS in
> kernel 2.6.34
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-stable.git;a=commit;h=73c77e2ccc14413c232c3e0b3aa43a0c4b72ec70
>
> but for this patch to work correctly, yer you need to provide the
> architecture dependent definition of invalidate_kernel_vmap_range() /
> flush_kernel_vmap_range() and ARCH_HAS_FLUSH_KERNEL_DCACHE_PAGE,
> similar to this patch
>
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-stable.git;a=commit;h=252a9afff76097667429b583e8b5b170b47665a4
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-stable.git;a=commit;h=73be1591579084a8103a7005dd3172f3e9dd7362
>
> Note: include/linux/highmem.h provide dummy definitions if
> architecture does not define them
>
> #ifndef ARCH_HAS_FLUSH_KERNEL_DCACHE_PAGE
> static inline void flush_kernel_dcache_page(struct page *page)
> {
> }
> static inline void flush_kernel_vmap_range(void *vaddr, int size)
> {
> }
> static inline void invalidate_kernel_vmap_range(void *vaddr, int size)
> {
> }
> #endi
>
>
>
>
> On Fri, Jan 6, 2012 at 3:44 PM, Tony Lu <zlu@xxxxxxxxxx> wrote:
>> Hi, Ajeet
>>
>> Sorry to bother you. By googling, I found that I am encountering the same XFS umount issue on TILE architecture. And I noticed that somebody suggested that other architectures like ARM, MIPS need to implement the following functions to fix this issue.
>>
>> flush_kernel_dcache_page()
>> flush_kernel_vmap_range()
>> void invalidate_kernel_vmap_range()
>>
>> I am not sure if these functions can really fix this issue, since I implemented these functions for TILE architecture, but did not fix it.
>>
>> So I really want to know that if you implemented these functions for MIPS to fix this issue. Or is there other method to fix this issue?
>>
>> I would be very appreciated for any feedback from you.
>>
>> Thanks
>> -Tony
>>
>>>Dear member,
>>>If I remove USB flash storage from PC/ARM/MIPS target (i.e without umount) while >copying file to XFS partition and connect it to MIPS target I always get
>>>XFS: xlog_recover_process_data: bad clientid
>>>XFS: log mount/recovery failed: error 5
>>>XFS: log mount failed
>>>This problem does not occur, if I connect the same USB flash to PC/ARM target.
>>>I have tested this on 2.6.30 - 2.6.34 (It everywhere when concerend with MIPS)
>>>
>>>I had two workaround to avoid this problem on MIPS, but I wish to know the exact >reason of this problme on MIPS.
>>>I am really tired and giving up, so finally thought of taking your help, your >guidence can help me.
>>>
>>>1. __flush_cache_all() at the begining of each iteration of while loop in >function xlog_recover_process_data().
>>>
>>>     while ((dp < lp) && num_logops) {
>>>+       __flush_cache_all() ;
>>>         ASSERT(dp + sizeof(xlog_op_header_t) <= lp);
>>>         ohead = (xlog_op_header_t *)dp;
>>>         dp += sizeof(xlog_op_header_t);
>>>         if (ohead->oh_clientid != XFS_TRANSACTION &&
>>>             ohead->oh_clientid != XFS_LOG) {
>>>             xlog_warn(
>>>             "XFS: xlog_recover_process_data: bad clientid")
>>>
>>>OR
>>>
>>>2. Provide new buffer in each call to xlog_bread in function >xlog_do_recovery_pass().
>>>
>>>          if (error)
>>>              goto bread_err2;
>>>          /* blocks in data section */
>>>          bblks = (int)BTOBB(be32_to_cpu(rhead->h_len));
>>>+        xlog_put_bp(dbp);
>>>+        dbp = xlog_get_bp(log, BTOBB(h_size));
>>>          error = xlog_bread(log, blk_no + hblks, bblks, dbp,
>>>                                     &offset);
>>>          if (error)
>>>              goto bread_err2;
>>>
>>>WIth Regards
>>>Ajeet Yadav

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs



[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux