netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: William Allen Simpson <william.allen.simpson@gmail.com>
To: huananhu@huawei.com
Cc: David Miller <davem@davemloft.net>,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 17/11]Optimize the upload speed for PPP connection.
Date: Tue, 17 Nov 2009 05:20:09 -0500	[thread overview]
Message-ID: <4B0278D9.6030806@gmail.com> (raw)
In-Reply-To: <20091117.012248.86306050.davem@davemloft.net>

David Miller wrote:
> Your patch is broken, please don't wate my time like this.
> 
> + git apply --check --whitespace=error-all diff
> error: patch failed: drivers/net/ppp_async.c:36
> error: drivers/net/ppp_async.c: patch does not apply
> 
What David may have meant, had he followed Documentation/ManagementStyle
or had any project management skills what-so-ever, is that you need to
follow Documentation/SubmittingPatches more carefully.

Look at 15) The canonical patch format

[PATCH 17/11] makes no sense, you don't have 11 patches, and the 17th
patch of 11 can never exist.

Need a better "subsystem: summary phrase", perhaps
   "net: PPP buffer too small for higher speed connections"

A marker line containing simply "---".

And your trailer boilerplate badly breaks the system, because it's
missing the SMTP email standard "--" line in front of it.

You'll get nicer formatting with 'git format-patch -o ~ HEAD~1' after you
do a 'git commit -a' on your git tree.

It takes some time, even for a simple 1 line fix like this....

Hope that helps.

  reply	other threads:[~2009-11-17 10:20 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-11-17  9:19 [PATCH 17/11]Optimize the upload speed for PPP connection fangxiaozhi 00110321
2009-11-17  9:22 ` David Miller
2009-11-17 10:20   ` William Allen Simpson [this message]
2009-11-17 11:14     ` Jarek Poplawski
2009-11-17 11:49       ` William Allen Simpson
2009-11-17 11:54       ` Franko Fang
2009-11-17 11:49     ` David Miller
2009-11-18  5:04       ` William Allen Simpson
2009-11-18  7:51         ` Jarek Poplawski
2009-11-21  9:29       ` Pavel Machek
2009-11-21 12:21         ` Jarek Poplawski
2009-11-21 19:00         ` David Miller
2009-11-17 17:37     ` Alexander Clouter
2009-11-17 18:49     ` Valdis.Kletnieks
2009-11-18  4:00       ` William Allen Simpson

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=4B0278D9.6030806@gmail.com \
    --to=william.allen.simpson@gmail.com \
    --cc=davem@davemloft.net \
    --cc=huananhu@huawei.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@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 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).