Re: async read patchset test results

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

 



On Thu, 13 Oct 2011 11:28:29 +0400
Pavel Shilovsky <piastryyy@xxxxxxxxx> wrote:

> I tested your current cifs-3.2 branch and faced with strange results
> (assuming Windows 7 share is mounted to /mnt/share):
> 

I've not seen this, but I'll try to reproduce it in a little while.

> piastry@adventure:~$ dd if=test1.iso of=/mnt/share/test.iso bs=100000
> 

Ok, so test1.iso is in your home directory and you're dd'ing it
to /mnt/share/test.iso (which is a cifs mount)

> piastry@adventure:~$ dd of=test.iso if=/mnt/share/test.iso bs=100000
> 

Then you copy test.iso back from the share to your homedir

> piastry@adventure:~$ md5sum test.iso
> 5eee1675f46ac594dd7800c37e3ba944  test.iso
> 

Checksum of test.iso in your homedir and end up with the above...

> piastry@adventure:~$ md5sum /mnt/share/test.iso
> 5eee1675f46ac594dd7800c37e3ba944  /mnt/HOME-share/test.iso
> 

The checksum matches the (apparent) one on the share

> piastry@adventure:~$ md5sum test1.iso
> 10d672583cab035e2b55048fbfbbf7ca  test1.iso
> 

But does not match the original.

> Then, I umounted and mounted it again with 3.1-rc9 cifs module -
> everything was fine. Then, I load previous "buggy" one and everything
> was fine too.
> 
> So, md5sums are not equal. Any ideas?
> 

Not something I've seen in my testing. Some questions:

1) where is your home directory? is it on a local disk?

2) when you say "everything was fine" what checksum did you get?

3) if this test is repeatable, is there any discrepancy in the size of
the file? IOW, when you do the md5sum, it would be good to also do a
"stat" against the file the i_size is coming up correctly after each
step.

-- 
Jeff Layton <jlayton@xxxxxxxxxx>
--
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