<feed xmlns='http://www.w3.org/2005/Atom'>
<title>snowball/igloo-kernel.git/fs/cifs, branch pm-android</title>
<subtitle>Igloo kernel</subtitle>
<id>https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/atom?h=pm-android</id>
<link rel='self' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/atom?h=pm-android'/>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/'/>
<updated>2012-05-16T16:26:25+00:00</updated>
<entry>
<title>cifs: fix misspelling of "forcedirectio"</title>
<updated>2012-05-16T16:26:25+00:00</updated>
<author>
<name>Jeff Layton</name>
<email>jlayton@redhat.com</email>
</author>
<published>2012-05-16T11:12:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=531c8ff0d472295f5ef5d1bd306115c81a84889e'/>
<id>urn:sha1:531c8ff0d472295f5ef5d1bd306115c81a84889e</id>
<content type='text'>
...and add a "directio" synonym since that's what the manpage has
always advertised.

Acked-by: Sachin Prabhu &lt;sprabhu@redhat.com&gt;
Signed-off-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>cifs: fix revalidation test in cifs_llseek()</title>
<updated>2012-05-09T20:16:22+00:00</updated>
<author>
<name>Dan Carpenter</name>
<email>dan.carpenter@oracle.com</email>
</author>
<published>2012-04-30T14:36:21+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=48a5730e5b71201e226ff06e245bf308feba5f10'/>
<id>urn:sha1:48a5730e5b71201e226ff06e245bf308feba5f10</id>
<content type='text'>
This test is always true so it means we revalidate the length every
time, which generates more network traffic.  When it is SEEK_SET or
SEEK_CUR, then we don't need to revalidate.

Signed-off-by: Dan Carpenter &lt;dan.carpenter@oracle.com&gt;
Reviewed-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>fs/cifs: fix parsing of dfs referrals</title>
<updated>2012-05-04T03:47:39+00:00</updated>
<author>
<name>Stefan Metzmacher</name>
<email>metze@samba.org</email>
</author>
<published>2012-05-03T22:19:28+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=d8f2799b105a24bb0bbd3380a0d56e6348484058'/>
<id>urn:sha1:d8f2799b105a24bb0bbd3380a0d56e6348484058</id>
<content type='text'>
The problem was that the first referral was parsed more than once
and so the caller tried the same referrals multiple times.

The problem was introduced partly by commit
066ce6899484d9026acd6ba3a8dbbedb33d7ae1b,
where 'ref += le16_to_cpu(ref-&gt;Size);' got lost,
but that was also wrong...

Cc: &lt;stable@vger.kernel.org&gt;
Signed-off-by: Stefan Metzmacher &lt;metze@samba.org&gt;
Tested-by: Björn Jacke &lt;bj@sernet.de&gt;
Reviewed-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>cifs: make sure we ignore the credentials= and cred= options</title>
<updated>2012-05-03T18:50:01+00:00</updated>
<author>
<name>Jeff Layton</name>
<email>jlayton@redhat.com</email>
</author>
<published>2012-05-02T18:02:40+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=a557b97616c49d81e09c8439831d4c4f13ef4050'/>
<id>urn:sha1:a557b97616c49d81e09c8439831d4c4f13ef4050</id>
<content type='text'>
Older mount.cifs programs passed this on to the kernel after parsing
the file. Make sure the kernel ignores that option.

Should fix:

    https://bugzilla.kernel.org/show_bug.cgi?id=43195

Cc: Sachin Prabhu &lt;sprabhu@redhat.com&gt;
Reported-by: Ronald &lt;ronald645@gmail.com&gt;
Signed-off-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>[CIFS] Update cifs version to 1.78</title>
<updated>2012-05-03T18:50:01+00:00</updated>
<author>
<name>Steve French</name>
<email>sfrench@us.ibm.com</email>
</author>
<published>2012-05-02T16:58:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=f966424e9935900e34cace8116d37aa70cff23d0'/>
<id>urn:sha1:f966424e9935900e34cace8116d37aa70cff23d0</id>
<content type='text'>
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>cifs - check S_AUTOMOUNT in revalidate</title>
<updated>2012-05-03T18:49:47+00:00</updated>
<author>
<name>Ian Kent</name>
<email>raven@themaw.net</email>
</author>
<published>2012-05-02T11:19:09+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=936ad9094462578953042d3395b973f1c9e6fa95'/>
<id>urn:sha1:936ad9094462578953042d3395b973f1c9e6fa95</id>
<content type='text'>
When revalidating a dentry, if the inode wasn't known to be a dfs
entry when the dentry was instantiated, such as when created via
-&gt;readdir(), the DCACHE_NEED_AUTOMOUNT flag needs to be set on the
dentry in -&gt;d_revalidate().

The false return from cifs_d_revalidate(), due to the inode now
being marked with the S_AUTOMOUNT flag, might not invalidate the
dentry if there is a concurrent unlazy path walk. This is because
the dentry reference count will be at least 2 in this case causing
d_invalidate() to return EBUSY. So the asumption that the dentry
will be discarded then correctly instantiated via -&gt;lookup() might
not hold.

Signed-off-by: Ian Kent &lt;raven@themaw.net&gt;
Reviewed-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Cc: Steve French &lt;smfrench@gmail.com&gt;
Cc: linux-cifs@vger.kernel.org
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>cifs: add missing initialization of server-&gt;req_lock</title>
<updated>2012-05-02T03:29:51+00:00</updated>
<author>
<name>Jeff Layton</name>
<email>jlayton@redhat.com</email>
</author>
<published>2012-05-01T21:41:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=58fa015f611b51e1f501b048bc5ac263c78852f0'/>
<id>urn:sha1:58fa015f611b51e1f501b048bc5ac263c78852f0</id>
<content type='text'>
Cc: Pavel Shilovsky &lt;piastryyy@gmail.com&gt;
Signed-off-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>cifs: don't cap ra_pages at the same level as default_backing_dev_info</title>
<updated>2012-05-02T03:27:54+00:00</updated>
<author>
<name>Jeff Layton</name>
<email>jlayton@redhat.com</email>
</author>
<published>2012-05-01T21:41:49+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=8f71465c19ffefbfd0da3c1f5dc172b4bce05e93'/>
<id>urn:sha1:8f71465c19ffefbfd0da3c1f5dc172b4bce05e93</id>
<content type='text'>
While testing, I've found that even when we are able to negotiate a
much larger rsize with the server, on-the-wire reads often end up being
capped at 128k because of ra_pages being capped at that level.

Lifting this restriction gave almost a twofold increase in sequential
read performance on my craptactular KVM test rig with a 1M rsize.

I think this is safe since the actual ra_pages that the VM requests
is run through max_sane_readahead() prior to submitting the I/O. Under
memory pressure we should end up with large readahead requests being
suppressed anyway.

Signed-off-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>CIFS: Fix indentation in cifs_show_options</title>
<updated>2012-05-02T03:19:43+00:00</updated>
<author>
<name>Sachin Prabhu</name>
<email>sprabhu@redhat.com</email>
</author>
<published>2012-04-25T11:10:14+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=156d17905e783d057061b3b56a9b3befec064e47'/>
<id>urn:sha1:156d17905e783d057061b3b56a9b3befec064e47</id>
<content type='text'>
Trivial patch which fixes a misplaced tab in cifs_show_options().

Signed-off-by: Sachin Prabhu &lt;sprabhu@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
<entry>
<title>Use correct conversion specifiers in cifs_show_options</title>
<updated>2012-04-24T16:36:25+00:00</updated>
<author>
<name>Sachin Prabhu</name>
<email>sprabhu@redhat.com</email>
</author>
<published>2012-04-24T14:28:30+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/snowball/igloo-kernel.git/commit/?id=28f8881023c9713c303c0feda270929f9384c019'/>
<id>urn:sha1:28f8881023c9713c303c0feda270929f9384c019</id>
<content type='text'>
cifs_show_options uses the wrong conversion specifier for uid, gid,
rsize &amp; wsize. Correct this to %u to match it to the variable type
'unsigned integer'.

Signed-off-by: Sachin Prabhu &lt;sprabhu@redhat.com&gt;
Reviewed-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Signed-off-by: Steve French &lt;sfrench@us.ibm.com&gt;
</content>
</entry>
</feed>
