[LU-471] mkfs.lustre ext4 extended options passed via --mkfsoptions are ignored Created: 28/Jun/11  Updated: 29/Jul/11  Resolved: 29/Jul/11

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.1.0
Fix Version/s: Lustre 2.1.0, Lustre 1.8.7

Type: Bug Priority: Minor
Reporter: John Hammond Assignee: Zhenyu Xu
Resolution: Fixed Votes: 0
Labels: None
Environment:

RHEL 5.6, e2fsprogs-1.41.90.wc2-0redhat, mkfs.lustre v2.0.62

Severity: 3
Rank (Obsolete): 4958

 Description   

Using mkfs.ext4, I can set extended options using the -E flag:
mkfs.ext4 -b 4096 -E stride=64,stripe-width=384 -j -m 0 -J device=/dev/md25 /dev/md15

According to the manpage and mailing list http://comments.gmane.org/gmane.comp.file-systems.lustre.user/9766, I should be able to do the same for mkfs.lustre using the --mkfsoptions flag. However, mkfs.lustre calls mke2fs with its own set of extended options which causes mine to be ignored:

mke2fs -j -b 4096 -L punch-OST0000 -b 4096 -E stride=64,stripe-width=384 -j -m 0 -J device=/dev/md20 -I 256 -i 262144 -O extents,uninit_bg,dir_nlink,huge_file,flex_bg -G 256 -E resize=4290772992,lazy_journal_init, -F /dev/md10 732579456

The output of mkfs.lustre includes "Stride=0 blocks, Stripe width=0 blocks" suggesting that these options are ignored by mkfs.ext4.

 Comments   
Comment by Peter Jones [ 28/Jun/11 ]

Bobi

Could you please help out on this one?

Thanks

Peter

Comment by Andreas Dilger [ 28/Jun/11 ]

This is a defect in the underlying mke2fs that it doesn't accept multiple -E or -O options. I am working on a patch to mke2fs (and e2fsck and tune2fs) that will allow multiple options to be specified.

I wonder if we should also fix mkfs_lustre.c to avoid creating duplicate options, because I'm not sure whether the patch to mke2fs would be accepted upstream or not. That would involve changing the mkfs_lustre.c parsing of the mkfsoptions, so that it appends the command-line options for "-E" and "-O" to the internally-generated default values. This also would fix the problem that the options specified on the command line should come after the internally-generated ones, so that it is possible to override the internal default options.

Comment by Zhenyu Xu [ 29/Jun/11 ]

patch tracking at http://review.whamcloud.com/1035

Comment by Andreas Dilger [ 05/Jul/11 ]

Once this patch is landed to master and has had some time to soak, it should probably also be landed to b1_8.

Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » x86_64,client,el5,ofa #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » x86_64,client,el5,inkernel #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » x86_64,client,sles11,inkernel #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » x86_64,server,el5,inkernel #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » i686,client,el6,inkernel #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » x86_64,server,el5,ofa #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » x86_64,client,ubuntu1004,inkernel #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » i686,server,el5,ofa #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » i686,server,el5,inkernel #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » i686,server,el6,inkernel #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » i686,client,el5,ofa #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » i686,client,el5,inkernel #208
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » x86_64,server,el6,inkernel #209
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 14/Jul/11 ]

Integrated in lustre-master » x86_64,client,el6,inkernel #209
LU-471 fix mkfs.lustre to avoid creating duplicate options

Oleg Drokin : c0f12e41fbe766785ea0f6bb2db3a4801a4f3ff5
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Zhenyu Xu [ 14/Jul/11 ]

patch porting for b1_8 is tracking at http://review.whamcloud.com/1103

Comment by Zhenyu Xu [ 29/Jul/11 ]

landed on b1_8 for 1.8.7 and on master for 2.1.0

Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » x86_64,client,el5,inkernel #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » x86_64,client,el6,inkernel #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » x86_64,server,el5,inkernel #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » x86_64,client,el5,ofa #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » i686,client,el6,inkernel #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » i686,server,el5,ofa #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » i686,server,el5,inkernel #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » x86_64,client,ubuntu1004,inkernel #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » x86_64,server,el5,ofa #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/tests/conf-sanity.sh
  • lustre/utils/mkfs_lustre.c
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » i686,client,el5,inkernel #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Comment by Build Master (Inactive) [ 29/Jul/11 ]

Integrated in lustre-b1_8 » i686,client,el5,ofa #119
LU-471 fix mkfs.lustre to avoid creating duplicate options

Johann Lombardi : d294d3d067478cd22d7b21aa84b378399457a807
Files :

  • lustre/utils/mkfs_lustre.c
  • lustre/tests/conf-sanity.sh
Generated at Sat Feb 10 01:07:21 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.