Re: [PATCH] fs: Initialize mountpoint list head in new_mountpoint()

From: Murilo Opsfelder Araújo
Date: Tue Aug 19 2014 - 09:50:31 EST


On Tue, Aug 19, 2014 at 4:01 AM, Thierry Reding
<thierry.reding@xxxxxxxxx> wrote:
> On Mon, Aug 18, 2014 at 11:43:54PM -0300, Murilo Opsfelder Araujo wrote:
>> The commit 89f7ca1af15bdfe7a6aed343032a84af2a69f736 makes next-20140818
>> to panic because mountpoint list head was not initialized. Initializing
>> it with NULL fixes the problem.
>>
>> commit 89f7ca1af15bdfe7a6aed343032a84af2a69f736
>> Author: Eric W. Biederman <ebiederman@xxxxxxxxxxx>
>> Date: Sun Sep 22 19:37:01 2013 -0700
>>
>> vfs: Keep a list of mounts on a mount point
>>
>> To spot any possible problems call BUG if a mountpoint
>> is put when it's list of mounts is not empty.
>>
>> AV: use hlist instead of list_head
>>
>> Signed-off-by: Murilo Opsfelder Araujo <mopsfelder@xxxxxxxxx>
>> ---
>> fs/namespace.c | 1 +
>> 1 file changed, 1 insertion(+)
>
> It's considered good etiquette and polite to credit the original author
> of a patch. This should be:
>
> Originally-by: Thierry Reding <treding@xxxxxxxxxx>

(resending to the lists)

Hello, Thierry.

I didn't know about it and you deserve all the credits, for sure.

I'm away from computer today. I'll send a v2 tomorrow night.

--
Murilo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/