<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux/fs, branch v2.6.28</title>
<subtitle>Mirror of https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/
</subtitle>
<id>https://git.shady.money/linux/atom?h=v2.6.28</id>
<link rel='self' href='https://git.shady.money/linux/atom?h=v2.6.28'/>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/'/>
<updated>2008-12-19T22:50:22Z</updated>
<entry>
<title>fs/9p: change simple_strtol to simple_strtoul</title>
<updated>2008-12-19T22:50:22Z</updated>
<author>
<name>Julia Lawall</name>
<email>julia@diku.dk</email>
</author>
<published>2008-12-19T22:50:22Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=f1d9e4586e79e1fc7b26d1c567599a904013f199'/>
<id>urn:sha1:f1d9e4586e79e1fc7b26d1c567599a904013f199</id>
<content type='text'>
Since v9ses-&gt;uid is unsigned, it would seem better to use simple_strtoul that
simple_strtol.

A simplified version of the semantic patch that makes this change is as
follows: (http://www.emn.fr/x-info/coccinelle/)

// &lt;smpl&gt;
@r2@
long e;
position p;
@@

e = simple_strtol@p(...)

@@
position p != r2.p;
type T;
T e;
@@

e =
- simple_strtol@p
+ simple_strtoul
  (...)
// &lt;/smpl&gt;

Signed-off-by: Julia Lawall &lt;julia@diku.dk&gt;
Acked-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;

</content>
</entry>
<entry>
<title>9p: convert d_iname references to d_name.name</title>
<updated>2008-12-19T22:47:40Z</updated>
<author>
<name>Wu Fengguang</name>
<email>fengguang.wu@intel.com</email>
</author>
<published>2008-12-19T22:47:40Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=7dd0cdc51c126915e243ee3291f7b6b5ea5e5833'/>
<id>urn:sha1:7dd0cdc51c126915e243ee3291f7b6b5ea5e5833</id>
<content type='text'>
d_iname is rubbish for long file names.
Use d_name.name in printks instead.

Signed-off-by: Wu Fengguang &lt;wfg@linux.intel.com&gt;
Acked-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;

</content>
</entry>
<entry>
<title>9p: Remove potentially bad parameter from function entry debug print.</title>
<updated>2008-12-19T22:45:21Z</updated>
<author>
<name>Duane Griffin</name>
<email>duaneg@dghda.com</email>
</author>
<published>2008-12-19T22:45:21Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=6ff232070a1088807e24f657b5d6f299f1ec32f6'/>
<id>urn:sha1:6ff232070a1088807e24f657b5d6f299f1ec32f6</id>
<content type='text'>
Signed-off-by: Duane Griffin &lt;duaneg@dghda.com&gt;
Signed-off-by: Eric Van Hensbergen &lt;ericvh@gmail.com&gt;

</content>
</entry>
<entry>
<title>Merge branch 'upstream-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mfasheh/ocfs2</title>
<updated>2008-12-17T23:01:23Z</updated>
<author>
<name>Linus Torvalds</name>
<email>torvalds@linux-foundation.org</email>
</author>
<published>2008-12-17T23:01:23Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=0bc77ecbe4f69ff8ead1d2abfe84ca9ba2a7bca4'/>
<id>urn:sha1:0bc77ecbe4f69ff8ead1d2abfe84ca9ba2a7bca4</id>
<content type='text'>
* 'upstream-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/mfasheh/ocfs2:
  ocfs2: Add JBD2 compat feature bit.
  ocfs2: Always update xattr search when creating bucket.
</content>
</entry>
<entry>
<title>cifs: fix buffer overrun in parse_DFS_referrals</title>
<updated>2008-12-17T22:59:55Z</updated>
<author>
<name>Jeff Layton</name>
<email>jlayton@redhat.com</email>
</author>
<published>2008-12-17T11:31:53Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=331c31351044888916805c9cb32d8bb9e40c12e9'/>
<id>urn:sha1:331c31351044888916805c9cb32d8bb9e40c12e9</id>
<content type='text'>
While testing a kernel with memory poisoning enabled, I saw some warnings
about the redzone getting clobbered when chasing DFS referrals. The
buffer allocation for the unicode converted version of the searchName is
too small and needs to take null termination into account.

Signed-off-by: Jeff Layton &lt;jlayton@redhat.com&gt;
Acked-by: Steve French &lt;sfrench@us.ibm.com&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>ocfs2: Add JBD2 compat feature bit.</title>
<updated>2008-12-17T02:26:16Z</updated>
<author>
<name>Joel Becker</name>
<email>Joel.Becker@oracle.com</email>
</author>
<published>2008-12-17T02:10:18Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=a97721894a6dc0f9ebfe1dbaa4bb112eaf399273'/>
<id>urn:sha1:a97721894a6dc0f9ebfe1dbaa4bb112eaf399273</id>
<content type='text'>
Define the OCFS2_FEATURE_COMPAT_JBD2 bit in the filesystem header.

Signed-off-by: Joel Becker &lt;joel.becker@oracle.com&gt;
Signed-off-by: Mark Fasheh &lt;mfasheh@suse.com&gt;
</content>
</entry>
<entry>
<title>ocfs2: Always update xattr search when creating bucket.</title>
<updated>2008-12-16T22:07:37Z</updated>
<author>
<name>Tao Ma</name>
<email>tao.ma@oracle.com</email>
</author>
<published>2008-12-05T01:14:10Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=83099bc647688d816c2f7fac8e51921bdfe8db73'/>
<id>urn:sha1:83099bc647688d816c2f7fac8e51921bdfe8db73</id>
<content type='text'>
When we create xattr bucket during the process of xattr set, we always
need to update the ocfs2_xattr_search since even if the bucket size is
the same as block size, the offset will change because of the removal
of the ocfs2_xattr_block header.

Signed-off-by: Tao Ma &lt;tao.ma@oracle.com&gt;
Signed-off-by: Mark Fasheh &lt;mfasheh@suse.com&gt;
</content>
</entry>
<entry>
<title>Merge branch 'to-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/frob/linux-2.6-roland</title>
<updated>2008-12-10T22:40:21Z</updated>
<author>
<name>Linus Torvalds</name>
<email>torvalds@linux-foundation.org</email>
</author>
<published>2008-12-10T22:40:21Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=f4fd2c5b6f691ff71614acbc28461b3c1488c81b'/>
<id>urn:sha1:f4fd2c5b6f691ff71614acbc28461b3c1488c81b</id>
<content type='text'>
* 'to-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/frob/linux-2.6-roland:
  tracehook: exec double-reporting fix
</content>
</entry>
<entry>
<title>KSYM_SYMBOL_LEN fixes</title>
<updated>2008-12-10T16:01:54Z</updated>
<author>
<name>Hugh Dickins</name>
<email>hugh@veritas.com</email>
</author>
<published>2008-12-09T21:14:27Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=9c24624727f6d6c460e45762a408ca5f5b9b8ef2'/>
<id>urn:sha1:9c24624727f6d6c460e45762a408ca5f5b9b8ef2</id>
<content type='text'>
Miles Lane tailing /sys files hit a BUG which Pekka Enberg has tracked
to my 966c8c12dc9e77f931e2281ba25d2f0244b06949 sprint_symbol(): use
less stack exposing a bug in slub's list_locations() -
kallsyms_lookup() writes a 0 to namebuf[KSYM_NAME_LEN-1], but that was
beyond the end of page provided.

The 100 slop which list_locations() allows at end of page looks roughly
enough for all the other stuff it might print after the symbol before
it checks again: break out KSYM_SYMBOL_LEN earlier than before.

Latencytop and ftrace and are using KSYM_NAME_LEN buffers where they
need KSYM_SYMBOL_LEN buffers, and vmallocinfo a 2*KSYM_NAME_LEN buffer
where it wants a KSYM_SYMBOL_LEN buffer: fix those before anyone copies
them.

[akpm@linux-foundation.org: ftrace.h needs module.h]
Signed-off-by: Hugh Dickins &lt;hugh@veritas.com&gt;
Cc: Christoph Lameter &lt;cl@linux-foundation.org&gt;
Cc Miles Lane &lt;miles.lane@gmail.com&gt;
Acked-by: Pekka Enberg &lt;penberg@cs.helsinki.fi&gt;
Acked-by: Steven Rostedt &lt;srostedt@redhat.com&gt;
Acked-by: Frederic Weisbecker &lt;fweisbec@gmail.com&gt;
Cc: Rusty Russell &lt;rusty@rustcorp.com.au&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
<entry>
<title>inotify: fix IN_ONESHOT unmount event watcher</title>
<updated>2008-12-10T16:01:53Z</updated>
<author>
<name>Dmitri Monakhov</name>
<email>dmonakhov@openvz.org</email>
</author>
<published>2008-12-09T21:14:26Z</published>
<link rel='alternate' type='text/html' href='https://git.shady.money/linux/commit/?id=6ee5a399d6a92a52646836a6e10faf255c16393e'/>
<id>urn:sha1:6ee5a399d6a92a52646836a6e10faf255c16393e</id>
<content type='text'>
On umount two event will be dispatched to watcher:

1: inotify_dev_queue_event(.., IN_UNMOUNT,..)
2: remove_watch(watch, dev)
    -&gt;inotify_dev_queue_event(.., IN_IGNORED, ..)

But if watcher has IN_ONESHOT bit set then the watcher will be released
inside first event.  Which result in accessing invalid object later.  IMHO
it is not pure regression.  This bug wasn't triggered while initial
inotify interface testing phase because of another bug in IN_ONESHOT
handling logic :)

  commit ac74c00e499ed276a965e5b5600667d5dc04a84a
  Author: Ulisses Furquim &lt;ulissesf@gmail.com&gt;
  Date:   Fri Feb 8 04:18:16 2008 -0800
    inotify: fix check for one-shot watches before destroying them
    As the IN_ONESHOT bit is never set when an event is sent we must check it
    in the watch's mask and not in the event's mask.

TESTCASE:
mkdir mnt
mount -ttmpfs none mnt
mkdir mnt/d
./inotify mnt/d&amp;
umount mnt ## &lt;&lt; lockup or crash here

TESTSOURCE:
/* gcc -oinotify inotify.c */
#include &lt;stdio.h&gt;
#include &lt;stdlib.h&gt;
#include &lt;sys/inotify.h&gt;

int main(int argc, char **argv)
{
        char buf[1024];
        struct inotify_event *ie;
        char *p;
        int i;
        ssize_t l;

        p = argv[1];
        i = inotify_init();
        inotify_add_watch(i, p, ~0);

        l = read(i, buf, sizeof(buf));
        printf("read %d bytes\n", l);
        ie = (struct inotify_event *) buf;
        printf("event mask: %d\n", ie-&gt;mask);
	return 0;
}

Signed-off-by: Dmitri Monakhov &lt;dmonakhov@openvz.org&gt;
Cc: John McCutchan &lt;ttb@tentacle.dhs.org&gt;
Cc: Al Viro &lt;viro@zeniv.linux.org.uk&gt;
Cc: Robert Love &lt;rlove@google.com&gt;
Cc: Ulisses Furquim &lt;ulissesf@gmail.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
</entry>
</feed>
