From: Boaz Harrosh <ooo@electrozaur.com>
To: Jeff Garzik <jgpobox@gmail.com>, Christoph Hellwig <hch@infradead.org>
Cc: Jeff Garzik <jgarzik@pobox.com>,
Kernel development list <linux-kernel@vger.kernel.org>,
osd-dev@open-osd.org
Subject: Re: state of the osdblk driver?
Date: Tue, 21 Oct 2014 15:47:36 +0300 [thread overview]
Message-ID: <544655E8.8070004@electrozaur.com> (raw)
In-Reply-To: <CAFtxgO5cPNTMqyY5Foqp_Y+pqCH=zTiatREn-4xRCTm8o5GLwA@mail.gmail.com>
On 10/21/2014 03:21 PM, Jeff Garzik wrote:
> On Tue, Oct 21, 2014 at 8:06 AM, Christoph Hellwig <hch@infradead.org> wrote:
>> The osdblk driver hsa seen literally no change except for global
>> API changes since it was merged, and I've never heard of a user.
>>
>> Is this something we want to keep around? I just noticed it because
>> it's the only non-SCSI user of some of the old block layer tagging
>> functions.
>
> Drop it, if there are no users.
>
> It was simulating an OSD, but there did not seem to be much beyond
> academic interest, and some interest in the use of aforementioned
> block functions as a useful exercise in block driver writing.
>
>
It is used in testing, its a good way to compare ext4 over objects
vs exofs over objects, kind of compare apples with apples.
(Also at the get go it was a good test that found some bugs in the
bring up of the osd target)
If there is an old tagging API need removing its fine to just drop
the support, and/or do the minimal change possible. What was the
tag used for anyway?
If it bothers you very much then please remove it, send a patch
and I'll ACK it.
[Just that I thought we don't do that. I can show you a dozen
ISA scsi LLD controllers which are a complete pain in the neck
with a all subsection core support. There is not a single system
on the planet that has this even possible, yet we do not remove
them nor the ISA bus crap. osdblk at least is a SW only exercise
that can be very much valid in any system.
]
Your call sir, I'm with you on this
Thanks
Boaz
next prev parent reply other threads:[~2014-10-21 12:47 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-21 12:06 state of the osdblk driver? Christoph Hellwig
2014-10-21 12:21 ` Jeff Garzik
2014-10-21 12:47 ` Boaz Harrosh [this message]
2014-10-21 12:56 ` Jeff Garzik
2014-10-21 13:01 ` Christoph Hellwig
2014-10-21 13:24 ` Boaz Harrosh
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=544655E8.8070004@electrozaur.com \
--to=ooo@electrozaur.com \
--cc=hch@infradead.org \
--cc=jgarzik@pobox.com \
--cc=jgpobox@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=osd-dev@open-osd.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