From: Artem Bityutskiy <dedekind@infradead.org>
To: Hamish Moffatt <hamish@cloud.net.au>
Cc: linux-mtd@lists.infradead.org
Subject: Re: [PATCH] mkfs.ubifs: support --squash-uids
Date: Tue, 06 May 2008 09:38:36 +0300 [thread overview]
Message-ID: <1210055916.3645.21.camel@sauron> (raw)
In-Reply-To: <20080505084617.GA11042@cloud.net.au>
On Mon, 2008-05-05 at 18:46 +1000, Hamish Moffatt wrote:
> > Thanks for the patch. Before applying it, we'd like to know what do you
> > expect to happen if -U is used simultaneously with the device table
> > (-D). Should we use what device table says for directories which are
> > created because they are specified in the device table, or we should use
> > 0 anyway? Or we should just prohibit simultaneous usage of -U and -D ?
>
> Good point. mkfs.jffs2 (from my reading of the source code) squashes
> ownership of files read from the specified root directory, but the
> ownerships specified in the device table are still used. I think that
> makes sense, because you can always specify 0:0 in the device table if
> that's what you need.
>
> However it's not what is implemented by my patch :-) Would you like me
> to fix it up and resubmit, or will you? I applied the squashing at the
> lowest level without realizing the effect on the device table.
I've committed your patch and will do the "prefer dev. table" change
later. Just a note - the patch was against some old mkfs.ubifs, so I had
to do conflict resolution. Also note, we changed -p (--orph-lebs) option
to -P at some point. You are unlikely using it - this is just for
information.
Thank you!
--
Best regards,
Artem Bityutskiy (Битюцкий Артём)
next prev parent reply other threads:[~2008-05-06 6:39 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-02 5:16 [PATCH] mkfs.ubifs: support --squash-uids Hamish Moffatt
2008-05-05 7:31 ` Artem Bityutskiy
2008-05-05 8:46 ` Hamish Moffatt
2008-05-05 8:52 ` Artem Bityutskiy
2008-05-06 11:00 ` Artem Bityutskiy
2008-05-06 6:38 ` Artem Bityutskiy [this message]
2008-05-06 9:46 ` Hamish Moffatt
2008-05-06 10:23 ` Artem Bityutskiy
2008-05-08 0:16 ` Hamish Moffatt
2008-05-08 6:34 ` Artem Bityutskiy
2008-05-07 12:55 ` Artem Bityutskiy
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=1210055916.3645.21.camel@sauron \
--to=dedekind@infradead.org \
--cc=hamish@cloud.net.au \
--cc=linux-mtd@lists.infradead.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.