From: "Bob Pearson" <rpearson@systemfabricworks.com>
To: "'Andrew Morton'" <akpm@linux-foundation.org>,
"'Stephen Rothwell'" <sfr@canb.auug.org.au>
Cc: "'Joakim Tjernlund'" <joakim.tjernlund@transmode.se>,
<fzago@systemfabricworks.com>,
"'George Spelvin'" <linux@horizon.com>,
<linux-kernel@vger.kernel.org>
Subject: RE: [PATCH v6 05/10] crc32-misc-cleanup.diff
Date: Wed, 7 Sep 2011 11:30:31 -0500 [thread overview]
Message-ID: <035901cc6d7b$77065350$6512f9f0$@systemfabricworks.com> (raw)
In-Reply-To:
>
> The list of needed changes based on recent comments I am aware of are:
> - put back in a couple of header files per Andrew
> - fix the summary phrases to conform to coding standards per Andrew
> - add signed off by for Joakim to patch 04/10 per Joakim
> - fix bug in patch 06/10 noted in my email by passing bits as a parameter
to
> crc32_body
Adding three additional changes:
- update macro per Joakim
- shorter generator code per George
- remove unneeded local_irq_disable per Andrew
Any more?
Can we finish crc32 before we start boiling the ocean with crc32c, crc16
etc.? I am happy to pitch in on those but I would like to see something
actually get done. I actually need this one.
next prev parent reply other threads:[~2011-09-07 16:30 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20110831213729.395283830@systemfabricworks.com>
2011-08-31 22:29 ` [PATCH v6 01/10] crc32-remove-trailing-whitespace.diff Bob Pearson
2011-08-31 22:29 ` [PATCH v6 02/10] crc32-move-to-documentation.diff Bob Pearson
2011-08-31 22:29 ` [PATCH v6 03/10] crc32-replace-self-test.diff Bob Pearson
2011-09-02 23:51 ` Andrew Morton
2011-09-06 16:14 ` Bob Pearson
2011-08-31 22:30 ` [PATCH v6 04/10] crc32-add-pointer-to-tab.diff Bob Pearson
2011-09-01 8:16 ` Joakim Tjernlund
2011-08-31 22:30 ` [PATCH v6 05/10] crc32-misc-cleanup.diff Bob Pearson
2011-09-02 23:50 ` Andrew Morton
2011-09-03 1:44 ` Stephen Rothwell
2011-09-06 13:40 ` Joakim Tjernlund
2011-09-06 14:50 ` Stephen Rothwell
2011-09-06 19:38 ` Andrew Morton
2011-09-06 20:18 ` Bob Pearson
2011-09-07 7:39 ` Joakim Tjernlund
2011-09-07 16:30 ` Bob Pearson [this message]
2011-09-07 17:51 ` Joakim Tjernlund
2011-09-06 16:05 ` Bob Pearson
2011-08-31 22:30 ` [PATCH v6 06/10] crc32-fix-check-endian-warnings.diff Bob Pearson
2011-08-31 22:30 ` [PATCH v6 07/10] crc32-add-real-8-bit.diff Bob Pearson
2011-08-31 22:30 ` [PATCH v6 08/10] crc32-add-slicing-by-8.diff Bob Pearson
2011-09-07 7:31 ` Joakim Tjernlund
2011-09-07 19:44 ` Bob Pearson
[not found] ` <OF3D37A60B.7A33B855-ONC1257904.00276B5B-C1257904.002951AF@LocalDomain>
2011-09-07 8:30 ` Joakim Tjernlund
2011-08-31 22:30 ` [PATCH v6 09/10] crc32-optimize-loops-for-x86.diff Bob Pearson
2011-08-31 22:30 ` [PATCH v6 10/10] crc32-final.diff Bob Pearson
2011-09-01 3:03 ` [PATCH v6 08/10] crc32-add-slicing-by-8.diff Bob Pearson
2011-09-07 7:32 ` Joakim Tjernlund
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='035901cc6d7b$77065350$6512f9f0$@systemfabricworks.com' \
--to=rpearson@systemfabricworks.com \
--cc=akpm@linux-foundation.org \
--cc=fzago@systemfabricworks.com \
--cc=joakim.tjernlund@transmode.se \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@horizon.com \
--cc=sfr@canb.auug.org.au \
/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