public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* state of the osdblk driver?
@ 2014-10-21 12:06 Christoph Hellwig
  2014-10-21 12:21 ` Jeff Garzik
  0 siblings, 1 reply; 6+ messages in thread
From: Christoph Hellwig @ 2014-10-21 12:06 UTC (permalink / raw)
  To: Boaz Harrosh, Jeff Garzik; +Cc: linux-kernel, osd-dev

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.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: state of the osdblk driver?
  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
  0 siblings, 1 reply; 6+ messages in thread
From: Jeff Garzik @ 2014-10-21 12:21 UTC (permalink / raw)
  To: Christoph Hellwig
  Cc: Boaz Harrosh, Jeff Garzik, Kernel development list, osd-dev

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.


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.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: state of the osdblk driver?
  2014-10-21 12:21 ` Jeff Garzik
@ 2014-10-21 12:47   ` Boaz Harrosh
  2014-10-21 12:56     ` Jeff Garzik
  2014-10-21 13:01     ` Christoph Hellwig
  0 siblings, 2 replies; 6+ messages in thread
From: Boaz Harrosh @ 2014-10-21 12:47 UTC (permalink / raw)
  To: Jeff Garzik, Christoph Hellwig
  Cc: Jeff Garzik, Kernel development list, osd-dev

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: state of the osdblk driver?
  2014-10-21 12:47   ` Boaz Harrosh
@ 2014-10-21 12:56     ` Jeff Garzik
  2014-10-21 13:01     ` Christoph Hellwig
  1 sibling, 0 replies; 6+ messages in thread
From: Jeff Garzik @ 2014-10-21 12:56 UTC (permalink / raw)
  To: Boaz Harrosh
  Cc: Christoph Hellwig, Jeff Garzik, Kernel development list, osd-dev

On Tue, Oct 21, 2014 at 8:47 AM, Boaz Harrosh <ooo@electrozaur.com> wrote:
> [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.


+1 also a fair point.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: state of the osdblk driver?
  2014-10-21 12:47   ` Boaz Harrosh
  2014-10-21 12:56     ` Jeff Garzik
@ 2014-10-21 13:01     ` Christoph Hellwig
  2014-10-21 13:24       ` Boaz Harrosh
  1 sibling, 1 reply; 6+ messages in thread
From: Christoph Hellwig @ 2014-10-21 13:01 UTC (permalink / raw)
  To: Boaz Harrosh; +Cc: Jeff Garzik, Jeff Garzik, Kernel development list, osd-dev

On Tue, Oct 21, 2014 at 03:47:36PM +0300, Boaz Harrosh wrote:
> 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.

I'm mostly just trying to figure out if anyone cares.  Would you be
willing to sign up as a maintainer, and maybe even look into a blk-mq
conversion?  I'd be happy to help to answer questions and review 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.
> ]

Not sure what exactly you mean, but I'm open to any serious proposal to
get rid of not actually usable cruft in old SCSI HBA drivers.  Feel
free to send a proposal to linux-scsi - I was planning to come up with
one of my own, but never got to it.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: state of the osdblk driver?
  2014-10-21 13:01     ` Christoph Hellwig
@ 2014-10-21 13:24       ` Boaz Harrosh
  0 siblings, 0 replies; 6+ messages in thread
From: Boaz Harrosh @ 2014-10-21 13:24 UTC (permalink / raw)
  To: Christoph Hellwig
  Cc: Jeff Garzik, Jeff Garzik, Kernel development list, osd-dev

On 10/21/2014 04:01 PM, Christoph Hellwig wrote:
<>
> 
> I'm mostly just trying to figure out if anyone cares.  Would you be
> willing to sign up as a maintainer, and maybe even look into a blk-mq
> conversion?  I'd be happy to help to answer questions and review it.
> 

I thought I was kind of am ;-) .

Good point I will put it on my list of things to do. Meanwhile can
I help you with the immediate problem of the tagging? could you do
something most brutal that even removes the tag support, just to
get you going?

And speaking of osdblk. Should it be its own full blown driver
or should it be under dm ? I know that at some LSF people said
it is better placed there? Specially since for a long time I wanted
to base it over ORE and not directly libosd so to support
RAID0/1/5/6 under the one simple block device.

> 
> Not sure what exactly you mean, but I'm open to any serious proposal to
> get rid of not actually usable cruft in old SCSI HBA drivers.  Feel
> free to send a proposal to linux-scsi - I was planning to come up with
> one of my own, but never got to it.
> 

Yes exactly that!
I wish I could have the time, perhaps one day I will. There is so much
more cleaning I can see.

Thanks Christoph
Boaz


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2014-10-21 13:24 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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
2014-10-21 12:56     ` Jeff Garzik
2014-10-21 13:01     ` Christoph Hellwig
2014-10-21 13:24       ` Boaz Harrosh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox