linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Dongsu Park <dongsu.park@profitbricks.com>
To: Kent Overstreet <kmo@daterainc.com>
Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
	Jens Axboe <axboe@kernel.dk>,
	Christoph Hellwig <hch@infradead.org>
Subject: Re: Block layer projects that I haven't had time for
Date: Mon, 8 Dec 2014 12:48:13 +0100	[thread overview]
Message-ID: <20141208114813.GA2724@gmail.com> (raw)
In-Reply-To: <20141206030205.GA22669@kmo-pixel>

Thanks for the reply.

On 05.12.2014 19:02, Kent Overstreet wrote:
> On Thu, Dec 04, 2014 at 12:00:27PM +0100, Dongsu Park wrote:
> > Playing a little with your block_stuff tree based on 3.15, however,
> > I think there still seems to be a couple of issues.
> > First of all, it doesn't work with virtio-blk. A testing Qemu VM panics
> > at the very early stage of booting. This issue should be addressed as
> > the first step, so that other parts can be tested.
> 
> Really? I was testing with virtio-blk, that's odd..

The culprit seems to be the plugging commit.
Before that change, it works well also with virtio-blk.
Though that's not the only issue...

> > Moreover, I've already tried to rebase these patches on top of current
> > mainline, 3.18-rc7. It's now compilable, but it seems to introduce
> > more bugs about direct-IO. I didn't manage to find out the reason.
> > I'd need to also look at the previous review comments in [1], [2].
> > 
> > Don't you have other trees based on top of 3.17 or higher?
> > If not, can I create my own tree based on 3.18-rc7 to publish?
> 
> Yeah, I'd post what you have now and I'll try and take a look.

I've created a git tree to include what I have right now.
Please see <https://github.com/dongsupark/linux>.

To be able to handle different issues one by one,
I got the entire tree separated out into 4 branches based on 3.18.

* block-generic-req-for-next : the most stable branch you can test with.
  With this branch, you can test most of block drivers as well as file
  systems with less critical bugs. Though it's not 100% perfect yet,
  e.g. btrfs doesn't seem to work quite well. Thus more tests are needed.

* block-mpage-bvecs-for-next : block-generic-req-for-next + multipage bvecs.
  This branch shows a critical issue that writing blocks to ext4 rootfs
  causes the whole system to crash. Need-to-investigate.
  
* block-plug-for-next: block-mpage-bvecs-for-next + plugging.
  This branch has an additional bug with virtio-blk, that the kernel
  panics at the very early stage of booting. Need-to-investigate.

* block-dio-rewrite-for-next: block-plug-for-next + dio-rewriting.
  This branch has more issues w.r.t. direct-io. For example, dio_init()
  causes the kernel to panic at the early stage of booting.

All branches are compilable. But it's still somehow half-way complete.
Commit messages should be properly written too, so that they can be posted
to mailing lists.

Regards,
Dongsu

  reply	other threads:[~2014-12-08 11:48 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-24  4:16 Block layer projects that I haven't had time for Kent Overstreet
2014-12-04 11:00 ` Dongsu Park
2014-12-06  3:02   ` Kent Overstreet
2014-12-08 11:48     ` Dongsu Park [this message]
2014-12-10 22:42       ` Ming Lin
2014-12-10 22:57         ` Kent Overstreet
2014-12-10 23:11           ` Ming Lin
2014-12-11 10:07             ` Dongsu Park
2014-12-11 10:14               ` Kent Overstreet
2014-12-11 19:16               ` Ming Lin
2014-12-12  6:32               ` Ming Lin
2014-12-12 12:40                 ` Dongsu Park
2014-12-10 22:49       ` Kent Overstreet
2014-12-11 10:21         ` Dongsu Park

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=20141208114813.GA2724@gmail.com \
    --to=dongsu.park@profitbricks.com \
    --cc=axboe@kernel.dk \
    --cc=hch@infradead.org \
    --cc=kmo@daterainc.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).