Hi Qu, Thanks for resolving the issue and update. I'll re-use your testcase with common.convert/generate_dataset() when it becomes available on master branch.
---- Cheers, Lakshmipathi.G On Mon, Oct 10, 2016 at 6:54 AM, Qu Wenruo <quwen...@cn.fujitsu.com> wrote: > Thanks for the report. > > I first thought it's a convert rewrite regression again, but I checked out > v4.5.3(the last version without convert rewrite) and just found it's still > the same problem. > > So the good news is, it's not a regression(at least for me), and the bad > news is, attr copy never works from the beginning. > > I'll fix it in recent days, and enrich the convert test framework to prevent > such problem happens again. > > Thanks, > Qu > > > At 10/10/2016 01:51 AM, Lakshmipathi.G wrote: >> >> Hi. >> >> Looks like btrfs-progs/btrfs-convert doesn't retain file attributes >> like immutable flags, is this expected behavior ? >> >> // test.img is ext2 >> [root@ mnt]# touch a b >> [root@ mnt]# chattr +a a >> [root@ mnt]# chattr +i b >> [root@ mnt]# lsattr a b >> -----a---------- a >> ----i----------- b >> >> >> // after btrfs-convert test.img >> >> [root@ mnt]# lsattr a b >> ---------------- a >> ---------------- b >> >> >> >> ---- >> Cheers, >> Lakshmipathi.G >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in >> the body of a message to majord...@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> > > -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
