On 01/22/12 12:17, NeilBrown wrote: > On Tue, 17 Jan 2012 11:48:48 +0100 Jes.Sorensen@xxxxxxxxxx wrote: > >> From: Jes Sorensen <Jes.Sorensen@xxxxxxxxxx> >> + /* >> + * --initrd sets first char of argv[0] to @. This is used >> + * by systemd to signal that the tast was launched from >> + * initrd/initramfs and should be preserved during shutdown >> + */ >> + case InitrdOpt: >> + c = argv[0]; >> + c[0] = '@'; >> + continue; > > Why not: > argv[0][0] = '@'; > ?? I always get confused by multi-indexed arrays, so I open-coded it. I will change it. > Also I'm wondering about the choice of "initrd" as the option name. > After all, we mostly use initramfs these days. > > --preboot ?? > --systemd-root-storage-daemon ?? > > OK, the second is a joke, and the first is introducing terminology that isn't > widely used... I was actually thinking --attacked-by-thunderbunnies-from-space but users might find it a bit too long..... > --for-root > ?? > > I'm open to suggestions, but I'm not seeing --initrd as the best choice just > yet. I was trying to come up with a better name, and I think --preboot is the best I have seen so far. Other ideas I had were: --systemd-root --no-shutdown however I like --preboot better, so I will go with that, unless someone starts throwing things at me. Cheers, Jes -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
