public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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


  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