All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: bharrosh@panasas.com, tj@kernel.org, petkovbb@gmail.com,
	bzolnier@gmail.com, linux-kernel@vger.kernel.org,
	axboe@kernel.dk, linux-ide@vger.kernel.org
Subject: Re: [PATCHSET pata-2.6] ide: rq->buffer, data, special and misc	cleanups
Date: Mon, 13 Apr 2009 04:31:08 -0400	[thread overview]
Message-ID: <49E2F84C.6080804@garzik.org> (raw)
In-Reply-To: <20090413172143B.fujita.tomonori@lab.ntt.co.jp>

FUJITA Tomonori wrote:
> On Mon, 13 Apr 2009 03:54:38 -0400
> Jeff Garzik <jeff@garzik.org> wrote:
> 
>> FUJITA Tomonori wrote:
>>> On Tue, 31 Mar 2009 16:04:39 +0300
>>> Boaz Harrosh <bharrosh@panasas.com> wrote:
>>>
>>>>> I
>>>>> just don't think bvec should be used outside of block/fs interface.
>>>>> As I wrote before, non-FS users have no reason to worry about bio.
>>>>> All it should think about is the requst it needs to issue and the
>>>>> memory area it's gonna use as in/out buffers.  bio just doesn't have a
>>>>> place there.
>>>>>
>>>> I don't understand what happens to all the people that start to work on the block
>>>> layer, they start getting defensive about bio being private to the request
>>>> level. But the Genie is out of the bag already (I know cats and bottles).
>>>> bio is already heavily used above the block layer from directly inside filesystems
>>>> to all kind of raid engines, DM MD managers, multi paths, integrity information ...
>>>>
>>>> Because bio is exactly that Ideal page carrier you are talking about.
>>> Wrong. multi path doesn't use bio. md accesses to the bio internal
>>> (it's not nice) and has the own way to carry pages. dm has the own
>>> mechanism on the top of bio. And bio doesn't work nicely for file
>>> systems such as btrfs, which handle multiple devices.
>>>
>>> Please stop your wrong argument 'bio is the ideal page carrier'.
>> What is the multi-device problem with bio?
> 
> Well, if it works nicely, I guess that we don't have something like
> drivers/dm/{dm-bio-record.h, dm-bio-list.h}, btrfs_multi_bio struct,
> or md's own page carrier?

It was an honest question.  I am seeking information, not denying your 
argument.

What, specifically, is this multi-device problem with bio, please?

	Jeff


  reply	other threads:[~2009-04-13  8:31 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-24 16:06 [PATCHSET pata-2.6] ide: rq->buffer, data, special and misc cleanups Tejun Heo
2009-03-24 16:06 ` [PATCH 01/14] block: clear req->errors on bio completion only for fs requests Tejun Heo
2009-03-24 16:06 ` [PATCH 02/14] block: reorganize [__]bio_map_kern() Tejun Heo
2009-03-24 16:06 ` [PATCH 03/14] block: implement blk_rq_map_kern_prealloc() Tejun Heo
2009-03-25 15:18   ` Boaz Harrosh
2009-03-26  2:10     ` Tejun Heo
2009-03-26  7:42       ` Tejun Heo
2009-03-26  8:05         ` Boaz Harrosh
2009-03-26  8:10           ` Boaz Harrosh
2009-03-26 10:19             ` Tejun Heo
2009-03-26 11:23               ` Boaz Harrosh
2009-03-26 12:07                 ` Tejun Heo
2009-03-26 14:44                   ` Boaz Harrosh
2009-03-27  2:26                     ` Tejun Heo
2009-04-13 10:07           ` FUJITA Tomonori
2009-03-24 16:06 ` [PATCH 04/14] ide: use blk_run_queue() instead of blk_start_queueing() Tejun Heo
2009-03-24 16:06 ` [PATCH 05/14] ide: don't set REQ_SOFTBARRIER Tejun Heo
2009-03-24 16:06 ` [PATCH 06/14] ide kill unused ide_cmd->special Tejun Heo
2009-03-24 16:06 ` [PATCH 07/14] ide-cd: clear sense buffer before issuing request sense Tejun Heo
2009-03-26  7:20   ` Borislav Petkov
2009-03-26  7:26     ` Tejun Heo
2009-03-24 16:06 ` [PATCH 08/14] ide-floppy: block pc always uses bio Tejun Heo
2009-03-24 16:06 ` [PATCH 09/14] ide-taskfile: don't abuse rq->buffer Tejun Heo
2009-03-24 16:06 ` [PATCH 10/14] ide-atapi: " Tejun Heo
2009-03-24 16:06 ` [PATCH 11/14] ide-cd: " Tejun Heo
2009-03-26  8:34   ` Borislav Petkov
2009-03-24 16:06 ` [PATCH 12/14] ide-pm: don't abuse rq->data Tejun Heo
2009-03-24 16:06 ` [PATCH 13/14] ide-atapi: use bio for request sense Tejun Heo
2009-03-28  8:43   ` Borislav Petkov
2009-03-24 16:06 ` [PATCH 14/14] ide-cd: " Tejun Heo
2009-04-13  8:52   ` Borislav Petkov
2009-03-28 13:51 ` [PATCHSET pata-2.6] ide: rq->buffer, data, special and misc cleanups Bartlomiej Zolnierkiewicz
2009-03-28 14:04   ` Borislav Petkov
2009-03-30  9:12     ` Tejun Heo
2009-03-30 11:14       ` Boaz Harrosh
2009-03-30 17:20         ` Tejun Heo
2009-03-31  8:43           ` Boaz Harrosh
2009-03-31  9:05             ` Tejun Heo
2009-03-31  9:10               ` Tejun Heo
2009-03-31 13:04               ` Boaz Harrosh
2009-03-31 13:43                 ` Tejun Heo
2009-04-01 11:50                   ` Boaz Harrosh
2009-04-13  7:41                 ` FUJITA Tomonori
2009-04-13  7:54                   ` Jeff Garzik
2009-04-13  8:22                     ` FUJITA Tomonori
2009-04-13  8:31                       ` Jeff Garzik [this message]
2009-04-13 10:07                         ` FUJITA Tomonori
2009-04-13 14:16                         ` James Bottomley
2009-03-30 14:50       ` Bartlomiej Zolnierkiewicz
2009-03-30 14:50         ` Bartlomiej Zolnierkiewicz
2009-03-30 17:21         ` Tejun Heo

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=49E2F84C.6080804@garzik.org \
    --to=jeff@garzik.org \
    --cc=axboe@kernel.dk \
    --cc=bharrosh@panasas.com \
    --cc=bzolnier@gmail.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=petkovbb@gmail.com \
    --cc=tj@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.