Re: Latest xfstest results

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

 



These are good suggestions - working through 215 and 313 is turning
out to be a little more complicated than I expected though.

On Fri, Aug 29, 2014 at 12:20 PM, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> Seems like most of your issues are time stamp updates, so you
> should look into that area a lot more.  Some that I know off head what
> the issues are:
>
> On Fri, Aug 29, 2014 at 10:33:14AM -0500, Steve French wrote:
>> QA output created by 215
>>
>> creating test file
>> 4096+0 records in
>> 4096+0 records out
>>
>> writing via mmap
>> FAIL: mtime not update after mapped write
>> FAIL: ctime not update after mapped write
>
> missing file_update_time call in cifs_page_mkwrite.

Just adding that didn't fix it - but will look further into it. Maybe
server is delaying updating mtime and getting reset back to original
time stamp during revalidate.  Need to debug further.

>> QA output created by 313
>> Silence is golden
>> ctime not updated after truncate
>> mtime not updated after truncate
>
> missing update of c/mtime for truncate.  The VFS is nasty in it's
> setattr calling conventions for this.

That is very true. This is turning out to be harder to decipher than I expected.

>Check my
> "[PATCH] nfs: update time staps on truncate"
>
> post on the nfs list for details.

My first pass at using the approach suggested in the NFS patch didn't
work, and an alternative approach of skipping mtime update (set info
of the timestamps to server) completely in the ATTR_SIZE case and
simply forcing revalidate of all inode metadata from the server
(setting cifsInode->time to zero forces revalidate) actually broke
ftruncate not just truncate (reminded me of something IIRC JRA had
said about delayed updates of mtime in NTFS - need to dig up that old
note).


-- 
Thanks,

Steve
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux