The patch titled arch/powerpc/kernel/vio.c: add missing unlock has been added to the -mm tree. Its filename is arch-powerpc-kernel-vioc-add-missing-unlock.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: arch/powerpc/kernel/vio.c: add missing unlock From: Julia Lawall <julia@xxxxxxx> Add an unlock before exiting the function. A simplified version of the semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @r exists@ expression E1; identifier f; @@ f (...) { <+... * spin_lock_irq (E1,...); ... when != E1 * return ...; ...+> } // </smpl> Signed-off-by: Julia Lawall <julia@xxxxxxx> Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> Acked-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- arch/powerpc/kernel/vio.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff -puN arch/powerpc/kernel/vio.c~arch-powerpc-kernel-vioc-add-missing-unlock arch/powerpc/kernel/vio.c --- a/arch/powerpc/kernel/vio.c~arch-powerpc-kernel-vioc-add-missing-unlock +++ a/arch/powerpc/kernel/vio.c @@ -645,8 +645,10 @@ void vio_cmo_set_dev_desired(struct vio_ found = 1; break; } - if (!found) + if (!found) { + spin_unlock_irqrestore(&vio_cmo.lock, flags); return; + } /* Increase/decrease in desired device entitlement */ if (desired >= viodev->cmo.desired) { _ Patches currently in -mm which might be from julia@xxxxxxx are linux-next.patch arch-powerpc-kernel-vioc-add-missing-unlock.patch drivers-acpi-use-kasprintf.patch drivers-acpi-use-kasprintf-fix.patch arch-x86-pci-use-kasprintf.patch arch-arm-plat-pxa-dmac-correct-null-test.patch arch-powerpc-platforms-pseries-use-kasprintf.patch drivers-media-video-avoid-null-dereference.patch drivers-scsi-libsas-use-sam_good.patch drivers-scsi-remove-unnecessary-null-test.patch drivers-message-move-dereference-after-null-test.patch drivers-scsi-correct-the-size-argument-to-kmalloc.patch drivers-scsi-bfa-bfad_imc-eliminate-useless-code.patch drivers-char-ppdevc-use-kasprintf.patch drivers-edac-introduce-missing-kfree.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html