netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marcel Holtmann <holtmann@linux.intel.com>
To: David Miller <davem@davemloft.net>
Cc: tomasw@gmail.com, linville@tuxdriver.com,
	linux-wireless@vger.kernel.org, netdev@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: pull request: wireless-2.6 2008-08-26
Date: Wed, 27 Aug 2008 13:10:10 +0200	[thread overview]
Message-ID: <1219835410.5359.24.camel@californication> (raw)
In-Reply-To: <20080827.014041.230100807.davem@davemloft.net>

Hi Dave,

> > Afther massive ath9k merge I didn't see it was a conceptual problem.
> 
> Stop right there.
> 
> You don't see it as a problem because you're not the one who has to
> explain all of this shit to Linus and ends up getting flamed by him
> when inappropriate changes are in my tree outside of the merge window.

I did a quick review of the offending patch queue. And some of them
should go in since they are fixing real bugs (as far as I can tell from
a quick review). Others have to clearly wait for the next merge window.

Thomas, please go through the list of patches and pick the real bug
fixes. The cleanup patches can't go in since they are after the merge
window. And being one of the offenders why Dave got flamed by Linus,
there is no way to get these through. And my big offender was a
regression that broke userspace and came with a lengthy commit message.

Speaking of which, you better have a more convincing commit message if
you wanna get something in after the merge window. It should make clear
why this is a bug or a regression and why it can't wait until the next
merge window.

Please take this into account for any future patches since Linus now set
a really high bar for patches after the merge window.

Regards

Marcel



  parent reply	other threads:[~2008-08-27  9:10 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-27  1:30 pull request: wireless-2.6 2008-08-26 John W. Linville
     [not found] ` <20080827013009.GA15781-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
2008-08-27  7:38   ` Tomas Winkler
     [not found]     ` <1ba2fa240808270038r24d5eeb9l36fd70ef5ae904f5-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-27  8:40       ` David Miller
2008-08-27  9:13         ` Tomas Winkler
2008-08-27 11:10         ` Marcel Holtmann [this message]
2008-08-27 10:05           ` Tomas Winkler
2008-08-27 10:10             ` Johannes Berg
     [not found]               ` <1219831828.3891.3.camel-YfaajirXv214zXjbi5bjpg@public.gmane.org>
2008-08-27 10:33                 ` David Miller
2008-08-27 11:34                 ` Tomas Winkler
2008-08-27 11:45                   ` David Miller
     [not found]                     ` <20080827.044551.102233262.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>
2008-08-27 12:26                       ` Tomas Winkler
     [not found]                   ` <1ba2fa240808270434y1589761xd2ff0a48c2e99033-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-27 13:10                     ` Michael Buesch
2008-08-27 14:55                       ` Tomas Winkler
     [not found]                         ` <1ba2fa240808270755t2f3d0c28rcb874011f5540ef3-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-27 15:22                           ` Michael Buesch
     [not found]                             ` <200808271722.55627.mb-fseUSCV1ubazQB+pC5nmwQ@public.gmane.org>
2008-08-27 15:45                               ` Tomas Winkler
     [not found]             ` <1ba2fa240808270305m7f505efak9fd0d36d5db09a1e-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-27 10:32               ` David Miller
2008-08-27 11:42                 ` Tomas Winkler
2008-08-27 13:57               ` Arjan van de Ven
2008-08-27 11:39 ` David Miller
2008-08-27 19:26 ` Tomas Winkler
     [not found]   ` <1ba2fa240808271226i95a7789k481a68b09dc60164-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-27 20:25     ` Michael Buesch
     [not found]       ` <200808272225.10557.mb-fseUSCV1ubazQB+pC5nmwQ@public.gmane.org>
2008-08-27 23:11         ` Tomas Winkler
     [not found]           ` <1ba2fa240808271611v382631ecn2a24e2816562d434-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-27 23:31             ` Luis R. Rodriguez
2008-08-28  0:19               ` Tomas Winkler
     [not found]                 ` <1ba2fa240808271719o2a3617e7re0499670de785b2f-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-28  1:30                   ` Luis R. Rodriguez
     [not found]                     ` <43e72e890808271830k6e0a9e65j790ee1e0d6d52d36-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-28  7:59                       ` Tomas Winkler
2008-08-28 10:35               ` Bruno Randolf
2008-08-28 10:52                 ` Tomas Winkler
     [not found]                   ` <1ba2fa240808280352q4a913c1cv33a6bc0998cdbcb9-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2008-08-28 11:13                     ` Bruno Randolf
2008-08-28  8:31             ` Michael Buesch

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=1219835410.5359.24.camel@californication \
    --to=holtmann@linux.intel.com \
    --cc=davem@davemloft.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=netdev@vger.kernel.org \
    --cc=tomasw@gmail.com \
    /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).