From owner-freebsd-fs@FreeBSD.ORG Fri Mar 20 03:49:34 2015 Return-Path: Delivered-To: freebsd-fs@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 23339788 for ; Fri, 20 Mar 2015 03:49:34 +0000 (UTC) Received: from mail-ig0-x232.google.com (mail-ig0-x232.google.com [IPv6:2607:f8b0:4001:c05::232]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id DB62E8DC for ; Fri, 20 Mar 2015 03:49:33 +0000 (UTC) Received: by igbqf9 with SMTP id qf9so7807588igb.1 for ; Thu, 19 Mar 2015 20:49:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:date:message-id:subject:from:to:content-type; bh=pLzua5+WGKXiRFMVARROa3G3/0Wc+YZZUOkPg3AaIxo=; b=NCHJXkFs3MdweEKG2AjFWegPCNIVJGRevX/EFCbn6vFXD9AylmyBUtn9xZ3ZBkPmmw uCu/rWx0HU51bymP6oJBjZFr6Vk5vkaWipUesyIULcBDKOzVtrKcAH14kfMybdMR1pp3 s2HiB02OULJe8LxsF1t/pPfgpUSq1h1mIMU6hWs7rVIN8zxl3nfqum8RxaEr56CGNJ0E k6pTO7CvwARLCh5+QQBbMxOEk1Nv/CtbERffrY2f3hF1UqrDotRFXC7HxctmLjDXdZzL c5L1TXblf96YLfX5lHtRURltcjBcLawbKRPm1dr6+a9w9FdMAHJEqunfnD2rq9vKqs/d IjXA== MIME-Version: 1.0 X-Received: by 10.50.137.99 with SMTP id qh3mr1880413igb.7.1426823373263; Thu, 19 Mar 2015 20:49:33 -0700 (PDT) Received: by 10.107.48.145 with HTTP; Thu, 19 Mar 2015 20:49:33 -0700 (PDT) Date: Thu, 19 Mar 2015 20:49:33 -0700 Message-ID: Subject: leftovers in fs/msdosfs/msdosfsmount.h From: Chris Torek To: freebsd-fs@freebsd.org Content-Type: multipart/mixed; boundary=001a11c3bcbc526b3f0511b03436 X-Content-Filtered-By: Mailman/MimeDel 2.1.18-1 X-BeenThere: freebsd-fs@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Filesystems List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 20 Mar 2015 03:49:34 -0000 --001a11c3bcbc526b3f0511b03436 Content-Type: text/plain; charset=UTF-8 I was poking at the kernel-side msdosfs code for reasons too boring :-) to go into here and found some stuff that should be safe to remove, based on the fact that there are no references to these names anywhere in the rest of the entire source tree. I've built a kernel with this patch in it (as a double check) but have not tested it on anything else. I don't really care if it goes in or not, I'm just providing it since I got misled slightly (in terms of how the UTF-16 en- and de-coding was to be done) when I was looking at the header. (In case gmail eats the straight text patch below, it's here as an attachment too. I'd send from a better mail system but my regular home FreeBSD system is still not quite up yet...) (A more interesting question is: when can the old compat mount stuff go away?) Chris commit 591612326e9c7ae2f12ad7d215a15cbefae0ff9a Author: Chris Torek Date: Thu Mar 19 19:30:02 2015 -0700 msdosfs: mark unused compat-mount fields The magic number MSDOSFS_ARGSMAGIC, which used to distinguish "old" vs "new" msdosfs mount arguments, has not been used since 2005; it should just go away now. Likewise, the local-to-Unicode table that changed at the same time is unused. Leave the space reserved in the old style mount arguments, though, since we still support the old mount call (via the cmount entry point). diff --git a/sys/fs/msdosfs/msdosfsmount.h b/sys/fs/msdosfs/msdosfsmount.h index 10ed95b..9446a3e 100644 --- a/sys/fs/msdosfs/msdosfsmount.h +++ b/sys/fs/msdosfs/msdosfsmount.h @@ -239,8 +239,8 @@ struct msdosfs_args { gid_t gid; /* gid that owns msdosfs files */ mode_t mask; /* file mask to be applied for msdosfs perms */ int flags; /* see below */ - int magic; /* version number */ - u_int16_t u2w[128]; /* Local->Unicode table */ + int unused1; /* unused, was version number */ + u_int16_t unused2[128]; /* no longer used, was Local->Unicode table */ char *cs_win; /* Windows(Unicode) Charset */ char *cs_dos; /* DOS Charset */ char *cs_local; /* Local Charset */ @@ -264,6 +264,4 @@ struct msdosfs_args { #define MSDOSFS_LARGEFS 0x10000000 /* perform fileno mapping */ #define MSDOSFS_FSIMOD 0x01000000 -#define MSDOSFS_ARGSMAGIC 0xe4eff300 - #endif /* !_MSDOSFS_MSDOSFSMOUNT_H_ */ --001a11c3bcbc526b3f0511b03436 Content-Type: application/octet-stream; name="msdosfsmount.h.patch" Content-Disposition: attachment; filename="msdosfsmount.h.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_i7h1l5g40 Y29tbWl0IDU5MTYxMjMyNmU5YzdhZTJmMTJhZDdkMjE1YTE1Y2JlZmFlMGZmOWEKQXV0aG9yOiBD aHJpcyBUb3JlayA8Y2hyaXMudG9yZWtAZ21haWwuY29tPgpEYXRlOiAgIFRodSBNYXIgMTkgMTk6 MzA6MDIgMjAxNSAtMDcwMAoKICAgIG1zZG9zZnM6IG1hcmsgdW51c2VkIGNvbXBhdC1tb3VudCBm aWVsZHMKICAgIAogICAgVGhlIG1hZ2ljIG51bWJlciBNU0RPU0ZTX0FSR1NNQUdJQywgd2hpY2gg dXNlZCB0byBkaXN0aW5ndWlzaAogICAgIm9sZCIgdnMgIm5ldyIgbXNkb3NmcyBtb3VudCBhcmd1 bWVudHMsIGhhcyBub3QgYmVlbiB1c2VkIHNpbmNlCiAgICAyMDA1OyBpdCBzaG91bGQganVzdCBn byBhd2F5IG5vdy4KICAgIAogICAgTGlrZXdpc2UsIHRoZSBsb2NhbC10by1Vbmljb2RlIHRhYmxl IHRoYXQgY2hhbmdlZCBhdCB0aGUgc2FtZQogICAgdGltZSBpcyB1bnVzZWQuCiAgICAKICAgIExl YXZlIHRoZSBzcGFjZSByZXNlcnZlZCBpbiB0aGUgb2xkIHN0eWxlIG1vdW50IGFyZ3VtZW50cywg dGhvdWdoLAogICAgc2luY2Ugd2Ugc3RpbGwgc3VwcG9ydCB0aGUgb2xkIG1vdW50IGNhbGwgKHZp YSB0aGUgY21vdW50IGVudHJ5CiAgICBwb2ludCkuCgpkaWZmIC0tZ2l0IGEvc3lzL2ZzL21zZG9z ZnMvbXNkb3Nmc21vdW50LmggYi9zeXMvZnMvbXNkb3Nmcy9tc2Rvc2ZzbW91bnQuaAppbmRleCAx MGVkOTViLi45NDQ2YTNlIDEwMDY0NAotLS0gYS9zeXMvZnMvbXNkb3Nmcy9tc2Rvc2ZzbW91bnQu aAorKysgYi9zeXMvZnMvbXNkb3Nmcy9tc2Rvc2ZzbW91bnQuaApAQCAtMjM5LDggKzIzOSw4IEBA IHN0cnVjdCBtc2Rvc2ZzX2FyZ3MgewogCWdpZF90CWdpZDsJCS8qIGdpZCB0aGF0IG93bnMgbXNk b3NmcyBmaWxlcyAqLwogCW1vZGVfdAltYXNrOwkJLyogZmlsZSBtYXNrIHRvIGJlIGFwcGxpZWQg Zm9yIG1zZG9zZnMgcGVybXMgKi8KIAlpbnQJZmxhZ3M7CQkvKiBzZWUgYmVsb3cgKi8KLQlpbnQg bWFnaWM7CQkvKiB2ZXJzaW9uIG51bWJlciAqLwotCXVfaW50MTZfdCB1MndbMTI4XTsgICAgIC8q IExvY2FsLT5Vbmljb2RlIHRhYmxlICovCisJaW50CXVudXNlZDE7CS8qIHVudXNlZCwgd2FzIHZl cnNpb24gbnVtYmVyICovCisJdV9pbnQxNl90IHVudXNlZDJbMTI4XTsJLyogbm8gbG9uZ2VyIHVz ZWQsIHdhcyBMb2NhbC0+VW5pY29kZSB0YWJsZSAqLwogCWNoYXIJKmNzX3dpbjsJLyogV2luZG93 cyhVbmljb2RlKSBDaGFyc2V0ICovCiAJY2hhcgkqY3NfZG9zOwkvKiBET1MgQ2hhcnNldCAqLwog CWNoYXIJKmNzX2xvY2FsOwkvKiBMb2NhbCBDaGFyc2V0ICovCkBAIC0yNjQsNiArMjY0LDQgQEAg c3RydWN0IG1zZG9zZnNfYXJncyB7CiAjZGVmaW5lCU1TRE9TRlNfTEFSR0VGUwkJMHgxMDAwMDAw MAkvKiBwZXJmb3JtIGZpbGVubyBtYXBwaW5nICovCiAjZGVmaW5lCU1TRE9TRlNfRlNJTU9ECQkw eDAxMDAwMDAwCiAKLSNkZWZpbmUgTVNET1NGU19BUkdTTUFHSUMJMHhlNGVmZjMwMAotCiAjZW5k aWYgLyogIV9NU0RPU0ZTX01TRE9TRlNNT1VOVF9IXyAqLwo= --001a11c3bcbc526b3f0511b03436--