linux-btrfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* btrfs fi usage crash when multiple device volume contains seed device
@ 2017-05-14 17:38 Luis de Bethencourt
  0 siblings, 0 replies; 4+ messages in thread
From: Luis de Bethencourt @ 2017-05-14 17:38 UTC (permalink / raw)
  To: linux-btrfs

Hi,

Chris Murphy suggested we move the discussion in this bugzilla thread:
https://bugzilla.kernel.org/show_bug.cgi?id=115851

To here, the mailing list.

Going to quote him to give context:
"This might be better discussed on list to ensure there's congruence in
dev and user expectations; and in particular I think this needs a design
that accounts for the realistic long term goal, so that a short term
scope doesn't interfere or make the long term more difficult.

The matrix of possibilities, most of which are not yet implemented in
btrfs-progs:

1. 1 dev seed -> 1 dev sprout
2. 2+ dev seed -> 1 dev sprout
3. 1 dev seed -> 2+ dev sprout
4. 2+ dev seed -> 2+ dev sprout

Near as I can tell 2, 3, 4 are not implemented. It's an immediate
problem whether and how the profile (single, raid0, raid1) is to be
inherited from seed to sprout. If I have a 4 disk raid1 volume, to
create a sprout must I add a minimum of two devices? Or is it valid to
have raid1 profile seed chunks, where writes to go single profile sprout
chunks? Anyway point is, it needs a design to answer these things.

Next, and even more importantly as it applies to the simple case of
single to single, the way we do this right now is beyond confusing
because the remount ro to rw changes the volume UUID being mounted. The
ro mount is the seed, the rw mount is the sprout. This is not really a
remount, it's a umount of the seed, and a mount of the sprout. But what
if there's more than one sprout? This is asking for trouble so I think
the remount rw should be disallowed making it clear the ro seed cannot
be mounted rw. Instead it's necessary to umount it and explicitly mount
the rw sprout, and which sprout.

Also part of the ambiguity is that 'btrfs dev add' is more like
mkfs.btrfs in the context of seed-sprout. The new device isn't really
added to the seed, because the seed is read only. What's really
happening is a mkfs.btrfs with a "backing device" which is the seed; in
some sense it has more in common with the mkfs.btrfs --rootdir option. 

So I even wonder if 'btrfs dev add' is appropriate for creating sprouts,
and if instead it should be in mkfs.btfs with a --seed option to specify
the backing seed, and thereby what we are making is a sprout, which has
a new UUID, and possibly different chunk profiles than the seed."

Thanks,
Luis

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

* btrfs fi usage crash when multiple device volume contains seed device
@ 2017-06-18 19:12 Cengiz Can
  0 siblings, 0 replies; 4+ messages in thread
From: Cengiz Can @ 2017-06-18 19:12 UTC (permalink / raw)
  To: linux-btrfs

Hello,

How should we proceed on this?

I want to deep dive and contribute to btrfs and its toolings with this 
issue but I don't have enough experience with them.

As far as I understand, fixing this would require some design decisions 
which I hope to hear in this thread soon.

Thanks
Cengiz Can



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

* Re: btrfs fi usage crash when multiple device volume contains seed device
@ 2017-06-23  9:46 Cengiz C.
  2017-06-23 17:20 ` Chris Murphy
  0 siblings, 1 reply; 4+ messages in thread
From: Cengiz C. @ 2017-06-23  9:46 UTC (permalink / raw)
  To: linux-btrfs; +Cc: lists

Hello Chris,

Sorry, I'm a newbie on LKML flow, I didn't copy you while replying.

If you have any spare time to talk about this issue, I'd be extremely happy.

Thanks again

On Sun, Jun 18, 2017 at 10:12 PM, Cengiz Can <cengizc@gmail.com> wrote:
> Hello,
>
> How should we proceed on this?
>
> I want to deep dive and contribute to btrfs and its toolings with this issue
> but I don't have enough experience with them.
>
> As far as I understand, fixing this would require some design decisions
> which I hope to hear in this thread soon.
>
> Thanks
> Cengiz Can
>
>



-- 
Cengiz Can

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

* Re: btrfs fi usage crash when multiple device volume contains seed device
  2017-06-23  9:46 Cengiz C.
@ 2017-06-23 17:20 ` Chris Murphy
  0 siblings, 0 replies; 4+ messages in thread
From: Chris Murphy @ 2017-06-23 17:20 UTC (permalink / raw)
  To: Cengiz C.; +Cc: Btrfs BTRFS, Chris Murphy

On Fri, Jun 23, 2017 at 3:46 AM, Cengiz C. <cengizc@gmail.com> wrote:
> Hello Chris,
>
> Sorry, I'm a newbie on LKML flow, I didn't copy you while replying.
>
> If you have any spare time to talk about this issue, I'd be extremely happy.
>
> Thanks again
>
> On Sun, Jun 18, 2017 at 10:12 PM, Cengiz Can <cengizc@gmail.com> wrote:
>> Hello,
>>
>> How should we proceed on this?
>>
>> I want to deep dive and contribute to btrfs and its toolings with this issue
>> but I don't have enough experience with them.
>>
>> As far as I understand, fixing this would require some design decisions
>> which I hope to hear in this thread soon.

No idea how to proceed. Anand Jain understands the issues related to
seed devices much better than I do. The multiple device stuff is
always complicated.



-- 
Chris Murphy

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

end of thread, other threads:[~2017-06-23 17:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-06-18 19:12 btrfs fi usage crash when multiple device volume contains seed device Cengiz Can
  -- strict thread matches above, loose matches on Subject: below --
2017-06-23  9:46 Cengiz C.
2017-06-23 17:20 ` Chris Murphy
2017-05-14 17:38 Luis de Bethencourt

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).