From: "Martin K. Petersen" <martin.petersen@oracle.com>
To: Christoph Hellwig <hch@lst.de>
Cc: Damien Le Moal <damien.lemoal@wdc.com>,
Jens Axboe <axboe@kernel.dk>,
linux-block@vger.kernel.org,
"Martin K . Petersen" <martin.petersen@oracle.com>,
linux-scsi@vger.kernel.org, MPT-FusionLinux.pdl@broadcom.com,
Hannes Reinecke <hare@suse.de>,
Bart Van Assche <bart.vanassche@sandisk.com>
Subject: Re: [PATCH 0/3] Separate zone requests from medium access requests
Date: Tue, 28 Feb 2017 21:52:31 -0500 [thread overview]
Message-ID: <yq160jtu2s0.fsf@oracle.com> (raw)
In-Reply-To: <20170228160259.GA22755@lst.de> (Christoph Hellwig's message of "Tue, 28 Feb 2017 17:02:59 +0100")
>>>>> "Christoph" == Christoph Hellwig <hch@lst.de> writes:
Christoph> I don't really like this too much - this is too many SCSI
Christoph> specifics for the block layer to care. Maybe using bios for
Christoph> the zone ops was a mistake after all, and we should just have
Christoph> operations in struct block_device instead..
Yeah, I'm afraid I don't like this either.
Short term I still think we should adjust in sd. And then maybe longer
term revisit whether these hybrid commands should be something other
than bios.
--
Martin K. Petersen Oracle Linux Engineering
next prev parent reply other threads:[~2017-03-01 2:52 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-28 10:25 [PATCH 0/3] Separate zone requests from medium access requests Damien Le Moal
2017-02-28 10:25 ` Damien Le Moal
2017-02-28 10:25 ` [PATCH 1/3] block: Introduce blk_rq_accesses_medium() Damien Le Moal
2017-02-28 10:25 ` Damien Le Moal
2017-02-28 10:25 ` [PATCH 2/3] block: Separate zone requests from medium access requests Damien Le Moal
2017-02-28 10:25 ` Damien Le Moal
2017-02-28 16:57 ` Bart Van Assche
2017-02-28 16:57 ` Bart Van Assche
2017-03-09 5:27 ` [lkp-robot] [block] 670fcc25d7: kernel_BUG_at_block/blk-core.c kernel test robot
2017-03-09 5:27 ` kernel test robot
2017-02-28 10:25 ` [PATCH 3/3] mpt3sas: Do not check resid for non medium access commands Damien Le Moal
2017-02-28 10:25 ` Damien Le Moal
2017-02-28 16:02 ` [PATCH 0/3] Separate zone requests from medium access requests Christoph Hellwig
2017-02-28 16:16 ` Bart Van Assche
2017-02-28 16:16 ` Bart Van Assche
2017-03-01 2:52 ` Martin K. Petersen [this message]
2017-03-01 3:02 ` Damien Le Moal
2017-03-01 3:02 ` Damien Le Moal
2017-03-01 3:21 ` Martin K. Petersen
2017-03-01 9:06 ` Damien Le Moal
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=yq160jtu2s0.fsf@oracle.com \
--to=martin.petersen@oracle.com \
--cc=MPT-FusionLinux.pdl@broadcom.com \
--cc=axboe@kernel.dk \
--cc=bart.vanassche@sandisk.com \
--cc=damien.lemoal@wdc.com \
--cc=hare@suse.de \
--cc=hch@lst.de \
--cc=linux-block@vger.kernel.org \
--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.