All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Gross <mgross@linux.intel.com>
To: Jeff Garzik <jeff@garzik.org>
Cc: LKML <linux-kernel@vger.kernel.org>,
	Eric Biederman <ebiederm@xmission.com>
Subject: Re: [PATCH 0/9] Remove 'irq' argument from all irq handlers
Date: Fri, 19 Oct 2007 11:45:35 -0700	[thread overview]
Message-ID: <20071019184535.GB1847@linux.intel.com> (raw)
In-Reply-To: <20071019075443.GA6407@havoc.gtf.org>

On Fri, Oct 19, 2007 at 03:54:43AM -0400, Jeff Garzik wrote:
> 
> WARNING NOT FOR MERGE WARNING NOT FOR MERGE WARNING NOT FOR MERGE

then whats the point ?

> 
> This posting is just to demonstrate something that I have been keeping
> alive in the background.  I have no urge to push it upstream anytime
> soon.

why not? 

--mgross

> 
> The overwhelming majority of drivers do not ever bother with the 'irq'
> argument that is passed to each driver's irq handler.
> 
> Of the minority of drivers that do use the arg, the majority of those
> have the irq number stored in their private-info structure somewhere.
> 
> There are a tiny few -- a couple Mac drivers -- which do weird things
> with that argument, but that's it.
> 
> For the large sweeps through the tree, these patches are grouped into
> "trivial" changes -- simply removing the unused irq arg -- or all other
> changes.
> 
>    [IRQ ARG REMOVAL] core interrupt delivery infrastructure updates
>    [IRQ ARG REMOVAL] various non-trivial arch updates
>    [IRQ ARG REMOVAL] trivial arch updates
>    [IRQ ARG REMOVAL] non-trivial driver updates
>    [IRQ ARG REMOVAL] trivial net driver updates
>    [IRQ ARG REMOVAL] trivial sound driver updates
>    [IRQ ARG REMOVAL] trivial scsi driver updates
>    [IRQ ARG REMOVAL] trivial driver updates
>    [IRQ ARG REMOVAL] x86-64 build fixes, cleanups
> 
> WARNING NOT FOR MERGE WARNING NOT FOR MERGE WARNING NOT FOR MERGE
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

  parent reply	other threads:[~2007-10-19 18:49 UTC|newest]

Thread overview: 48+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-10-19  7:54 [PATCH 0/9] Remove 'irq' argument from all irq handlers Jeff Garzik
2007-10-19  7:55 ` [PATCH 1/9] irq-remove: core Jeff Garzik
2007-10-19 17:27   ` Eric W. Biederman
2007-10-19 17:48     ` Jeff Garzik
2007-10-19 18:04   ` Eric W. Biederman
2007-10-19 18:21     ` Jeff Garzik
2007-10-19 19:50       ` Eric W. Biederman
2007-10-19 19:58         ` Jeff Garzik
2007-10-19 23:13           ` Eric W. Biederman
2007-10-19 23:46             ` Jeff Garzik
2007-10-19 23:53             ` Jeff Garzik
2007-10-19  7:55 ` [PATCH 2/9] irq-remove: arch non-trivial Jeff Garzik
2007-10-19 16:54   ` Jeremy Fitzhardinge
2007-10-19 17:31     ` Jeff Garzik
2007-10-19 17:50     ` Jeff Garzik
2007-10-19 17:11   ` Eric W. Biederman
2007-10-19 17:16     ` Jeff Garzik
2007-10-19 19:38       ` Eric W. Biederman
2007-10-19  7:56 ` [PATCH 3/9] irq-remove: arch trivial Jeff Garzik
2007-10-19  7:56 ` [PATCH 4/9] irq-remove: driver non-trivial Jeff Garzik
2007-10-19 18:19   ` Eric W. Biederman
2007-10-19 18:36     ` Jeff Garzik
2007-10-19  7:57 ` [PATCH 5/9] irq-remove: net driver trivial Jeff Garzik
2007-10-19  7:57 ` [PATCH 6/9] irq-remove: sound " Jeff Garzik
2007-10-19  7:58 ` [PATCH 7/9] irq-remove: scsi " Jeff Garzik
2007-10-19 13:00   ` Salyzyn, Mark
2007-10-26 21:35     ` Andrew Morton
2007-10-26 21:47       ` Jeff Garzik
2007-10-26 23:50         ` Arjan van de Ven
2007-10-27  0:12           ` Jeff Garzik
2007-10-27  0:16             ` Arjan van de Ven
2007-10-27  0:37               ` Jeff Garzik
2007-10-27  5:31                 ` Arjan van de Ven
2007-10-27  7:06                   ` Jeff Garzik
2007-10-27  7:46                     ` Eric W. Biederman
2007-10-27 14:17                       ` Arjan van de Ven
2007-10-19  7:58 ` [PATCH 8/9] irq-remove: " Jeff Garzik
2007-10-19  7:59 ` [PATCH 9/9] irq-remove: misc fixes and cleanups Jeff Garzik
2007-10-19 14:53 ` [PATCH 0/9] Remove 'irq' argument from all irq handlers Thomas Gleixner
2007-10-19 18:38   ` Eric W. Biederman
2007-10-19 18:57     ` Jeff Garzik
2007-10-19 19:02       ` Jeff Garzik
2007-10-19 19:07     ` Ingo Molnar
2007-10-19 19:35       ` Eric W. Biederman
2007-10-19 19:41         ` Thomas Gleixner
2007-10-19 19:55           ` Jeff Garzik
2007-10-19 18:45 ` Mark Gross [this message]
2007-10-20  6:07 ` Greg KH

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=20071019184535.GB1847@linux.intel.com \
    --to=mgross@linux.intel.com \
    --cc=ebiederm@xmission.com \
    --cc=jeff@garzik.org \
    --cc=linux-kernel@vger.kernel.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 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.