* + mips-vpe-remove-leftover-unlock_kernel.patch added to -mm tree
@ 2009-11-03 7:01 akpm
[not found] ` <20091103140953.GA23611@linux-mips.org>
0 siblings, 1 reply; 4+ messages in thread
From: akpm @ 2009-11-03 7:01 UTC (permalink / raw)
To: mm-commits; +Cc: jkacur, ralf
The patch titled
mips: vpe: remove leftover unlock_kernel()
has been added to the -mm tree. Its filename is
mips-vpe-remove-leftover-unlock_kernel.patch
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/SubmitChecklist when testing your code ***
See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find
out what to do about this
The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
------------------------------------------------------
Subject: mips: vpe: remove leftover unlock_kernel()
From: John Kacur <jkacur@redhat.com>
commit 1bbfc20d0161cd94b1b8111566be2fa41b41b608 ("MIPS: VPE: Get rid of
BKL.") was supposed to have removed the BKL, but there is a leftover and
unpaired unlock_kernel call in the function vpe_open. Remove it.
Signed-off-by: John Kacur <jkacur@redhat.com>
Cc: Ralf Baechle <ralf@linux-mips.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---
arch/mips/kernel/vpe.c | 3 ---
1 file changed, 3 deletions(-)
diff -puN arch/mips/kernel/vpe.c~mips-vpe-remove-leftover-unlock_kernel arch/mips/kernel/vpe.c
--- a/arch/mips/kernel/vpe.c~mips-vpe-remove-leftover-unlock_kernel
+++ a/arch/mips/kernel/vpe.c
@@ -38,7 +38,6 @@
#include <linux/vmalloc.h>
#include <linux/elf.h>
#include <linux/seq_file.h>
-#include <linux/smp_lock.h>
#include <linux/syscalls.h>
#include <linux/moduleloader.h>
#include <linux/interrupt.h>
@@ -1116,8 +1115,6 @@ static int vpe_open(struct inode *inode,
v->shared_ptr = NULL;
v->__start = 0;
- unlock_kernel();
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH cleanup] Re: + mips-vpe-remove-leftover-unlock_kernel.patch added to -mm tree
[not found] ` <20091103085613.373634ab.akpm@linux-foundation.org>
@ 2009-11-03 21:52 ` John Kacur
2009-11-03 22:15 ` Ralf Baechle
0 siblings, 1 reply; 4+ messages in thread
From: John Kacur @ 2009-11-03 21:52 UTC (permalink / raw)
To: Andrew Morton; +Cc: Ralf Baechle, Thomas Gleixner, Ingo Molnar, linux-kernel
On Tue, 3 Nov 2009, Andrew Morton wrote:
> On Tue, 3 Nov 2009 15:09:53 +0100 Ralf Baechle <ralf@linux-mips.org> wrote:
>
> > On Mon, Nov 02, 2009 at 11:01:38PM -0800, akpm@linux-foundation.org wrote:
> >
> > I already have an equivalent patch in the MIPS kernel - Linus only needs
> > to finally pull ...
> >
>
> I didn't see such a patch in yesterday's linux-next..
>
Linus picked-up Ralf's version of the patch for 2.6.32-rc6
a22d621c80fb08f668ae3bfe62aeaae64b0b2b89
That has the important part that removes the stray unlock_kernel, but
unfortunately doesn't remove the smp_lock.h header.
So, the last patch is just a clean-up. (no urgency).
>From ae20d9169893112daaad1dc3af9f8559003a2bfb Mon Sep 17 00:00:00 2001
From: John Kacur <jkacur@redhat.com>
Date: Tue, 3 Nov 2009 22:15:55 +0100
Subject: [PATCH] Remove BKL header.
Now that the bkl has been removed, also remove the smp_lock.h include
Signed-off-by: John Kacur <jkacur@redhat.com>
---
arch/mips/kernel/vpe.c | 1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/arch/mips/kernel/vpe.c b/arch/mips/kernel/vpe.c
index 6047752..2bd2151 100644
--- a/arch/mips/kernel/vpe.c
+++ b/arch/mips/kernel/vpe.c
@@ -38,7 +38,6 @@
#include <linux/vmalloc.h>
#include <linux/elf.h>
#include <linux/seq_file.h>
-#include <linux/smp_lock.h>
#include <linux/syscalls.h>
#include <linux/moduleloader.h>
#include <linux/interrupt.h>
--
1.6.0.6
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH cleanup] Re: + mips-vpe-remove-leftover-unlock_kernel.patch added to -mm tree
2009-11-03 21:52 ` [PATCH cleanup] " John Kacur
@ 2009-11-03 22:15 ` Ralf Baechle
2009-11-03 22:24 ` John Kacur
0 siblings, 1 reply; 4+ messages in thread
From: Ralf Baechle @ 2009-11-03 22:15 UTC (permalink / raw)
To: John Kacur; +Cc: Andrew Morton, Thomas Gleixner, Ingo Molnar, linux-kernel
On Tue, Nov 03, 2009 at 10:52:41PM +0100, John Kacur wrote:
> > > I already have an equivalent patch in the MIPS kernel - Linus only needs
> > > to finally pull ...
> > >
> >
> > I didn't see such a patch in yesterday's linux-next..
I only have stuff to be pulled for 2.6.33 on my -next tree. Anything for
2.6.32 was in another tree.
> Linus picked-up Ralf's version of the patch for 2.6.32-rc6
> a22d621c80fb08f668ae3bfe62aeaae64b0b2b89
>
> That has the important part that removes the stray unlock_kernel, but
> unfortunately doesn't remove the smp_lock.h header.
>
> So, the last patch is just a clean-up. (no urgency).
>
> >From ae20d9169893112daaad1dc3af9f8559003a2bfb Mon Sep 17 00:00:00 2001
> From: John Kacur <jkacur@redhat.com>
> Date: Tue, 3 Nov 2009 22:15:55 +0100
> Subject: [PATCH] Remove BKL header.
>
> Now that the bkl has been removed, also remove the smp_lock.h include
In fact every single use of <linux/smp_lock.h> in arch/mips/ is unnecessary;
see patch for 2.6.33 in
http://www.linux-mips.org/git?p=upstream-sfr.git;a=commit;h=defde8dd822ddbab4ea1f8d2f5831e693bc75ad4
Ralf
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH cleanup] Re: + mips-vpe-remove-leftover-unlock_kernel.patch added to -mm tree
2009-11-03 22:15 ` Ralf Baechle
@ 2009-11-03 22:24 ` John Kacur
0 siblings, 0 replies; 4+ messages in thread
From: John Kacur @ 2009-11-03 22:24 UTC (permalink / raw)
To: Ralf Baechle; +Cc: Andrew Morton, Thomas Gleixner, Ingo Molnar, linux-kernel
On Tue, Nov 3, 2009 at 11:15 PM, Ralf Baechle <ralf@linux-mips.org> wrote:
> On Tue, Nov 03, 2009 at 10:52:41PM +0100, John Kacur wrote:
>
>> > > I already have an equivalent patch in the MIPS kernel - Linus only needs
>> > > to finally pull ...
>> > >
>> >
>> > I didn't see such a patch in yesterday's linux-next..
>
> I only have stuff to be pulled for 2.6.33 on my -next tree. Anything for
> 2.6.32 was in another tree.
>
>> Linus picked-up Ralf's version of the patch for 2.6.32-rc6
>> a22d621c80fb08f668ae3bfe62aeaae64b0b2b89
>>
>> That has the important part that removes the stray unlock_kernel, but
>> unfortunately doesn't remove the smp_lock.h header.
>>
>> So, the last patch is just a clean-up. (no urgency).
>>
>> >From ae20d9169893112daaad1dc3af9f8559003a2bfb Mon Sep 17 00:00:00 2001
>> From: John Kacur <jkacur@redhat.com>
>> Date: Tue, 3 Nov 2009 22:15:55 +0100
>> Subject: [PATCH] Remove BKL header.
>>
>> Now that the bkl has been removed, also remove the smp_lock.h include
>
> In fact every single use of <linux/smp_lock.h> in arch/mips/ is unnecessary;
> see patch for 2.6.33 in
>
> http://www.linux-mips.org/git?p=upstream-sfr.git;a=commit;h=defde8dd822ddbab4ea1f8d2f5831e693bc75ad4
>
> Ralf
Ah, great! Thanks Ralf.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2009-11-03 22:24 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-03 7:01 + mips-vpe-remove-leftover-unlock_kernel.patch added to -mm tree akpm
[not found] ` <20091103140953.GA23611@linux-mips.org>
[not found] ` <20091103085613.373634ab.akpm@linux-foundation.org>
2009-11-03 21:52 ` [PATCH cleanup] " John Kacur
2009-11-03 22:15 ` Ralf Baechle
2009-11-03 22:24 ` John Kacur
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.