All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Kent Overstreet <kent.overstreet@gmail.com>
Cc: linux-bcache@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-fsdevel@vger.kernel.org
Subject: Re: Bcache version 9
Date: Sun, 21 Nov 2010 17:09:14 -0800	[thread overview]
Message-ID: <20101122010914.GB7688@kroah.com> (raw)
In-Reply-To: <20101121140808.GA6429@moria>

On Sun, Nov 21, 2010 at 06:09:34AM -0800, Kent Overstreet wrote:
> +++ b/Documentation/bcache.txt

For new sysfs files, please create Documentation/ABI files.

> +All configuration is done via sysfs. To use sde to cache md1, assuming the
> +SSD's erase block size is 128k:
> +
> +  make-bcache -b128k /dev/sde
> +  echo "/dev/sde" > /sys/kernel/bcache/register_cache
> +  echo "<UUID> /dev/md1" > /sys/kernel/bcache/register_dev

/sys/kernel/bcache/?  Really?

Come on, shouldn't this be somewhere else?  You only have 1 file here,
right?

Shouldn't it be a configfs file instead as that is what you are doing?

thanks,

greg k-h

  reply	other threads:[~2010-11-22  1:12 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-21 14:09 Bcache version 9 Kent Overstreet
2010-11-22  1:09 ` Greg KH [this message]
2010-11-23  8:07   ` Kent Overstreet
     [not found]     ` <4CEB7648.8030000-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2010-12-01  4:16       ` Greg KH
2010-12-01  4:16         ` Greg KH
     [not found]         ` <20101201041610.GA5891-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2010-12-04  3:24           ` Kent Overstreet
2010-12-04  3:24         ` Kent Overstreet
2010-12-04  3:24         ` Kent Overstreet
2010-12-16 11:21         ` Kent Overstreet
2010-12-16 11:21           ` Kent Overstreet
2010-11-23 23:35 ` Cédric Villemain
2010-11-23 23:35   ` Cédric Villemain
2010-11-24  6:25   ` Kent Overstreet
     [not found]     ` <4CECAFD1.10506-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2010-12-04  5:41       ` John Drescher
2010-12-04  5:41         ` John Drescher

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=20101122010914.GB7688@kroah.com \
    --to=greg@kroah.com \
    --cc=kent.overstreet@gmail.com \
    --cc=linux-bcache@vger.kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@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.