From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Andi Kleen <andi@firstfloor.org>
Cc: axboe@kernel.dk, linux-scsi@vger.kernel.org
Subject: Re: [PATCH] [2/21] Add blk_q_mask
Date: Sun, 16 Nov 2008 08:21:28 -0600 [thread overview]
Message-ID: <1226845288.3556.4.camel@localhost.localdomain> (raw)
In-Reply-To: <20081115231102.54DF53E661A@basil.firstfloor.org>
On Sun, 2008-11-16 at 00:11 +0100, Andi Kleen wrote:
> [Merge note: this patch will be in a SCSI patchkit and in the core
> mask allocator patchkit. They are equivalent. If one of them
> is merged first simply drop it in the other]
>
> Converts the queue bounce_pfn to a DMA mask suitable for the mask allocator
>
> Signed-off-by: Andi Kleen <ak@suse.de>
> Signed-off-by: Andi Kleen <andi@firstfloor.org>
> Signed-off-by: Andi Kleen <ak@linux.intel.com>
I'm confused by this signoff chain. You're supposed to give signoffs
from each of the entities whose hands the patch passed through ...
unless you've taken to emailing your own patches to yourself, this
doesn't look right ...
Could you just pick one? If it's about credit, then the reporting stats
go by Author. All three signoffs will do is inflate your rank in the
Signed-off-by stats (which I suppose we use for the kernel summit), but
I don't think it will give equal credit to the three entities.
James
next prev parent reply other threads:[~2008-11-16 14:21 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-15 23:11 [PATCH] [0/21] Remove isa_unchecked_dma and some more GFP_DMAs in the mid layer v5 Andi Kleen
2008-11-15 23:11 ` [PATCH] [1/21] Add the alloc/get_pages_mask calls Andi Kleen
2008-11-15 23:11 ` [PATCH] [2/21] Add blk_q_mask Andi Kleen
2008-11-16 14:21 ` James Bottomley [this message]
2008-11-16 17:40 ` Theodore Tso
2008-11-16 22:19 ` Andi Kleen
2008-11-15 23:11 ` [PATCH] [3/21] Pass gfp to scsi_allocate_command Andi Kleen
2008-11-16 15:29 ` James Bottomley
2008-11-16 22:21 ` Andi Kleen
2008-11-15 23:11 ` [PATCH] [4/21] Add sense_buffer_isa to host template Andi Kleen
2008-11-15 23:11 ` [PATCH] [5/21] Remove unchecked_isa in BusLogic Andi Kleen
2008-11-16 15:46 ` James Bottomley
2008-11-16 22:31 ` Andi Kleen
2008-11-17 11:03 ` James Bottomley
2008-11-15 23:11 ` [PATCH] [6/21] Remove unchecked_isa_dma in advansys.c Andi Kleen
2008-11-16 16:56 ` James Bottomley
2008-11-16 22:33 ` Andi Kleen
2008-11-17 11:32 ` James Bottomley
2008-11-15 23:11 ` [PATCH] [7/21] Remove unchecked_isa_dma in gdth Andi Kleen
2008-11-15 23:11 ` [PATCH] [8/21] Remove unchecked_isa_dma in eata.c Andi Kleen
2008-11-15 23:11 ` [PATCH] [9/21] Remove unchecked_isa_dma in aha1542 Andi Kleen
2008-11-15 23:11 ` [PATCH] [10/21] Remove unchecked_isa_dma in aha152x/wd7000/sym53c416/u14-34f/NCR53c406a Andi Kleen
2008-11-15 23:11 ` [PATCH] [11/21] Remove GFP_DMA uses in st/osst Andi Kleen
2008-11-15 23:11 ` [PATCH] [12/21] Remove unchecked_isa_dma support for hostdata Andi Kleen
2008-11-15 23:11 ` [PATCH] [13/21] Use blk_q_mask/get_pages_mask in sg driver Andi Kleen
2008-11-16 2:11 ` Douglas Gilbert
2008-11-16 22:17 ` Andi Kleen
2008-11-17 5:11 ` FUJITA Tomonori
2008-11-17 10:22 ` Andi Kleen
2008-11-17 11:07 ` FUJITA Tomonori
2008-11-17 11:23 ` Andi Kleen
2008-11-17 11:22 ` FUJITA Tomonori
2008-11-17 11:43 ` Andi Kleen
2008-11-17 11:47 ` FUJITA Tomonori
2008-11-15 23:11 ` [PATCH] [14/21] Rely on block layer bouncing for ISA DMA devices scanning Andi Kleen
2008-11-15 23:11 ` [PATCH] [15/21] Don't disable direct_io for unchecked_isa_dma in st.c Andi Kleen
2008-11-15 23:11 ` [PATCH] [16/21] Remove automatic block layer bouncing for unchecked_isa_dma Andi Kleen
2008-11-15 23:11 ` [PATCH] [17/21] Remove GFP_DMA use in sr.c Andi Kleen
2008-11-15 23:11 ` [PATCH] [18/21] Remove unchecked_isa_dma from sysfs Andi Kleen
2008-11-15 23:11 ` [PATCH] [19/21] Switch to a single SCSI command pool Andi Kleen
2008-11-15 23:11 ` [PATCH] [20/21] Finally kill unchecked_isa_dma Andi Kleen
2008-11-15 23:11 ` [PATCH] [21/21] Convert DMA buffers in ch.c to allocate via the block layer Andi Kleen
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=1226845288.3556.4.camel@localhost.localdomain \
--to=james.bottomley@hansenpartnership.com \
--cc=andi@firstfloor.org \
--cc=axboe@kernel.dk \
--cc=linux-scsi@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.