linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: u.kleine-koenig@pengutronix.de (Uwe Kleine-König)
To: linux-arm-kernel@lists.infradead.org
Subject: [patch 00/12] arm: raw_spinlock annotations
Date: Tue, 19 Oct 2010 16:37:17 +0200	[thread overview]
Message-ID: <20101019143717.GH11713@pengutronix.de> (raw)
In-Reply-To: <20100217125328.791176536@linutronix.de>

Hello,

On Wed, Feb 17, 2010 at 01:19:56PM -0000, Thomas Gleixner wrote:
> Preempt-RT converts spinlocks to sleeping spinlocks, but some
> spinlocks need to be kept as real spinlocks. In 2.6.33 we merged the
> spinlock namespace cleanup which allows us to annotate those
> locks. The lock type is change to raw_spinlock which is the same as a
> common spinlock in !RT kernels.
> 
> The following series is a collection of lock annotations from the
> preempt-rt tree. Please merge into 2.6.34.
Back then I imported these changes in my repo and found them now when
cleaning it up.  I once tried to get them in[1], and Russell answered:

	Thomas asked me about a pull request for his patches, and I said
	no because they're spread across many different mach-*
	directories.  So this isn't helpful.

I don't understand that critism, as apart from "ARM: footbridge: Convert
gpio and led locks to raw_spinlock" most patches (9) only touch a single
file, the others only touch a single lock and fix all callers.  The
footbridge patch admittedly is not that nice, but that's because the
touched lock is used all across the tree.

While cleaning up my repo I refound the patches and rebased them on top
of today's Linus' tree and only needed to fix up the l2x0_lock patch as
in the meantime a new usage hit mainline.

Best regards
Uwe

[1] http://thread.gmane.org/gmane.linux.ports.arm.kernel/74675

-- 
Pengutronix e.K.                           | Uwe Kleine-K?nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

  parent reply	other threads:[~2010-10-19 14:37 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-17 13:19 [patch 00/12] arm: raw_spinlock annotations Thomas Gleixner
2010-02-17 13:21 ` [patch 01/12] arm: Convert dma_spin_lock to raw_spinlock Thomas Gleixner
2010-02-17 13:21 ` [patch 02/12] arm: Convert stop_lock " Thomas Gleixner
2010-02-17 13:21 ` [patch 03/12] arm-convert-die_lock-undef_lock-to-raw-spinlocks.patch Thomas Gleixner
2010-02-17 13:21 ` [patch 04/12] arm: Convert l2x0_lock to raw_spinlock Thomas Gleixner
2010-02-17 13:22 ` [patch 05/12] arm: Convert cpu_asid_lock " Thomas Gleixner
2010-02-17 13:22 ` [patch 06/12] arm: Convert xscale minicache_lock " Thomas Gleixner
2010-02-17 13:22 ` [patch 07/12] arm: Convert v4 " Thomas Gleixner
2010-02-17 13:23 ` [patch 08/12] arm: Convert v6_lock " Thomas Gleixner
2010-02-17 13:23 ` [patch 09/12] arm: Convert footbridge gpio and led locks " Thomas Gleixner
2010-02-17 13:23 ` [patch 10/12] arm: Convert integrator cm_lock and v3_lock " Thomas Gleixner
2010-02-17 13:23 ` [patch 11/12] arm: Convert ixp4xx pci_lock " Thomas Gleixner
2010-02-17 13:23 ` [patch 12/12] arm: Convert shark leds_lock " Thomas Gleixner
2010-10-19 14:37 ` Uwe Kleine-König [this message]
2010-10-19 14:38   ` [PATCH 01/12] ARM: Convert dma_spin_lock " Uwe Kleine-König
2010-10-19 14:38   ` [PATCH 02/12] ARM: smp: Convert stop_lock " Uwe Kleine-König
2010-10-19 14:38   ` [PATCH 03/12] ARM: Convert die_lock and undef_lock to raw_spinlocks Uwe Kleine-König
2010-10-19 14:38   ` [PATCH 04/12] ARM: Convert l2x0_lock to raw_spinlock Uwe Kleine-König
2010-10-19 14:39   ` [PATCH 05/12] ARM: Convert cpu_asid_lock " Uwe Kleine-König
2010-10-19 14:39   ` [PATCH 06/12] ARM: Convert xscale minicache_lock " Uwe Kleine-König
2010-10-19 14:39   ` [PATCH 07/12] ARM: Convert v4 " Uwe Kleine-König
2010-10-19 14:39   ` [PATCH 08/12] ARM: Convert v6_lock " Uwe Kleine-König
2010-10-19 14:39   ` [PATCH 09/12] ARM: footbridge: Convert gpio and led locks " Uwe Kleine-König
2010-10-19 14:39   ` [PATCH 10/12] ARM: integrator: Convert cm_lock and v3_lock " Uwe Kleine-König
2010-10-19 14:39   ` [PATCH 11/12] ARM: ixp4xx: Convert pci_lock " Uwe Kleine-König
2010-10-19 14:39   ` [PATCH 12/12] ARM: shark: Convert leds_lock " Uwe Kleine-König
2010-10-19 15:26   ` [patch 00/12] arm: raw_spinlock annotations Arnd Bergmann
2010-10-19 18:44     ` Russell King - ARM Linux
2010-10-19 20:03       ` Thomas Gleixner
2010-10-19 20:20         ` Arnd Bergmann
2010-10-19 22:07           ` Thomas Gleixner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20101019143717.GH11713@pengutronix.de \
    --to=u.kleine-koenig@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).