From: Andi Kleen <andi@firstfloor.org>
To: Jens Axboe <jens.axboe@oracle.com>
Cc: Andi Kleen <andi@firstfloor.org>,
James Bottomley <James.Bottomley@HansenPartnership.com>,
linux-scsi@vger.kernel.org
Subject: Re: [PATCH] [9/20] Add blk_kmalloc/blk_alloc_pages
Date: Mon, 17 Mar 2008 22:34:22 +0100 [thread overview]
Message-ID: <20080317213422.GA11966@one.firstfloor.org> (raw)
In-Reply-To: <20080317204634.GQ17940@kernel.dk>
> Look further down in the email, queue_bounce_to_mask() or whatever you
> would want to call it. As also written there, the PRINCIPLE is the same.
> And that is that exporting a to_allocator_mask() helper is a lot saner
> than exporting an allocator api tied to the queue.
>
> Can we get over this, please?
Ok it would have helped if you had explained why it is saner, but I
bow to your superior experience on block layer issues.
The only open issue is right now if it isn't better to go back
for automatic bouncing for SCSI scan and the other users. Do you
have an opinion on that too? If you have one can you please convince
James of it too.
-Andi
next prev parent reply other threads:[~2008-03-17 21:31 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-07 17:53 [PATCH] [0/20] Remove isa_unchecked_dma and some more GFP_DMAs in the mid layer v3 Andi Kleen
2008-03-07 17:54 ` [PATCH] [1/20] Add sense_buffer_isa to host template Andi Kleen
2008-03-07 17:54 ` [PATCH] [2/20] Remove unchecked_isa in BusLogic Andi Kleen
2008-03-07 17:54 ` [PATCH] [3/20] Remove unchecked_isa_dma in advansys.c Andi Kleen
2008-03-07 17:54 ` [PATCH] [4/20] Remove unchecked_isa_dma in gdth Andi Kleen
2008-03-07 17:54 ` [PATCH] [5/20] Remove unchecked_isa_dma in eata.c Andi Kleen
2008-03-07 17:54 ` [PATCH] [6/20] Remove unchecked_isa_dma in aha1542 Andi Kleen
2008-03-07 17:54 ` [PATCH] [7/20] Remove unchecked_isa_dma in aha152x/wd7000/sym53c416/u14-34f/NCR53c406a Andi Kleen
2008-03-07 17:54 ` [PATCH] [8/20] Remove random noop unchecked_isa_dma users Andi Kleen
2008-03-07 17:54 ` [PATCH] [9/20] Add blk_kmalloc/blk_alloc_pages Andi Kleen
2008-03-13 22:06 ` James Bottomley
2008-03-14 13:48 ` Jens Axboe
2008-03-14 13:59 ` Andi Kleen
2008-03-17 8:27 ` Jens Axboe
2008-03-17 8:36 ` Andi Kleen
2008-03-17 8:38 ` Jens Axboe
2008-03-17 8:53 ` Andi Kleen
2008-03-17 9:18 ` Boaz Harrosh
2008-03-17 10:03 ` Andi Kleen
2008-03-17 20:29 ` Jens Axboe
2008-03-17 20:45 ` Andi Kleen
2008-03-17 20:46 ` Jens Axboe
2008-03-17 21:34 ` Andi Kleen [this message]
2008-03-18 7:26 ` Jens Axboe
2008-04-02 3:37 ` FUJITA Tomonori
2008-04-02 8:43 ` Boaz Harrosh
2008-04-02 11:08 ` FUJITA Tomonori
2008-04-02 11:32 ` Boaz Harrosh
2008-03-17 13:59 ` James Bottomley
2008-03-07 17:54 ` [PATCH] [11/20] Remove unchecked_isa_dma support for hostdata Andi Kleen
2008-03-07 17:54 ` [PATCH] [12/20] Remove unchecked_isa_dma checks in sg.c Andi Kleen
2008-03-07 17:54 ` [PATCH] [13/20] Use blk_kmalloc in scsi_scan Andi Kleen
2008-03-07 17:54 ` [PATCH] [14/20] Don't disable direct_io for unchecked_isa_dma in st.c Andi Kleen
2008-03-14 13:51 ` Jens Axboe
2008-03-14 14:24 ` Christoph Hellwig
2008-03-16 12:39 ` Boaz Harrosh
2008-03-16 12:44 ` Andi Kleen
2008-03-17 8:28 ` Jens Axboe
2008-03-27 17:26 ` Mike Christie
2008-03-17 8:27 ` Jens Axboe
2008-03-17 10:55 ` FUJITA Tomonori
2008-03-17 12:21 ` Boaz Harrosh
2008-03-07 17:54 ` [PATCH] [15/20] Remove automatic block layer bouncing for unchecked_isa_dma Andi Kleen
2008-03-07 17:54 ` [PATCH] [16/20] Convert sr driver over the blk_kmalloc Andi Kleen
2008-03-07 17:54 ` [PATCH] [17/20] Remove unchecked_isa_dma from sysfs Andi Kleen
2008-03-07 17:54 ` [PATCH] [18/20] Switch to a single SCSI command pool Andi Kleen
2008-03-07 17:54 ` [PATCH] [19/20] Finally kill unchecked_isa_dma Andi Kleen
2008-03-07 17:54 ` [PATCH] [20/20] Convert DMA buffers in ch.c to allocate via the block layer Andi Kleen
2008-03-11 17:55 ` [PATCH] [0/20] Remove isa_unchecked_dma and some more GFP_DMAs in the mid layer v3 Boaz Harrosh
2008-03-12 0:56 ` 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=20080317213422.GA11966@one.firstfloor.org \
--to=andi@firstfloor.org \
--cc=James.Bottomley@HansenPartnership.com \
--cc=jens.axboe@oracle.com \
--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 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).