From owner-freebsd-geom@FreeBSD.ORG Fri Feb 11 09:13:00 2011 Return-Path: Delivered-To: geom@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id E62FF106566B for ; Fri, 11 Feb 2011 09:13:00 +0000 (UTC) (envelope-from alexander@leidinger.net) Received: from mail.ebusiness-leidinger.de (mail.ebusiness-leidinger.de [217.11.53.44]) by mx1.freebsd.org (Postfix) with ESMTP id 3FB3A8FC18 for ; Fri, 11 Feb 2011 09:13:00 +0000 (UTC) Received: from outgoing.leidinger.net (p57B3B9CC.dip.t-dialin.net [87.179.185.204]) by mail.ebusiness-leidinger.de (Postfix) with ESMTPSA id 0CA85844012; Fri, 11 Feb 2011 10:03:02 +0100 (CET) Received: from webmail.leidinger.net (unknown [IPv6:fd73:10c7:2053:1::2:102]) by outgoing.leidinger.net (Postfix) with ESMTP id DF3622F45; Fri, 11 Feb 2011 10:02:58 +0100 (CET) Received: (from www@localhost) by webmail.leidinger.net (8.14.4/8.13.8/Submit) id p1B92weP065959; Fri, 11 Feb 2011 10:02:58 +0100 (CET) (envelope-from Alexander@Leidinger.net) Received: from pslux.ec.europa.eu (pslux.ec.europa.eu [158.169.9.14]) by webmail.leidinger.net (Horde Framework) with HTTP; Fri, 11 Feb 2011 10:02:58 +0100 Message-ID: <20110211100258.18243dlaaz0iulog@webmail.leidinger.net> Date: Fri, 11 Feb 2011 10:02:58 +0100 From: Alexander Leidinger To: geom@freebsd.org References: <20110211100047.14771t9tnmafolog@webmail.leidinger.net> In-Reply-To: <20110211100047.14771t9tnmafolog@webmail.leidinger.net> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=_30palfo2nku8" Content-Transfer-Encoding: 7bit User-Agent: Dynamic Internet Messaging Program (DIMP) H3 (1.1.4) X-EBL-MailScanner-Information: Please contact the ISP for more information X-EBL-MailScanner-ID: 0CA85844012.AA112 X-EBL-MailScanner: Found to be clean X-EBL-MailScanner-SpamCheck: not spam, spamhaus-ZEN, SpamAssassin (not cached, score=1.274, required 6, autolearn=disabled, RDNS_NONE 1.27) X-EBL-MailScanner-SpamScore: s X-EBL-MailScanner-From: alexander@leidinger.net X-EBL-MailScanner-Watermark: 1298019786.34765@ZRZISRx1ZgtTwfp+gE1t5g X-EBL-Spam-Status: No Cc: kibab@freebsd.org Subject: Re: CFR: FEATURE macros for GEOM classes (with patch) X-BeenThere: freebsd-geom@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: GEOM-specific discussions and implementations List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 11 Feb 2011 09:13:01 -0000 This message is in MIME format. --=_30palfo2nku8 Content-Type: text/plain; charset=UTF-8; DelSp="Yes"; format="flowed" Content-Disposition: inline Content-Transfer-Encoding: 7bit Quoting Alexander Leidinger (from Fri, 11 Feb 2011 10:00:47 +0100): > Can someone please review if the patch is OK (bikesheds about the > description part need to contain corresponding patches for NOTES, > else we will not take the arguments into account)? It helps to attach the patch... Bye, Alexander. -- The great nations have always acted like gangsters and the small nations like prostitutes. -- Stanley Kubrick http://www.Leidinger.net Alexander @ Leidinger.net: PGP ID = B0063FE7 http://www.FreeBSD.org netchild @ FreeBSD.org : PGP ID = 72077137 --=_30palfo2nku8 Content-Type: text/x-patch; charset=UTF-8; name="features_geom.diff" Content-Disposition: attachment; filename="features_geom.diff" Content-Transfer-Encoding: 7bit Index: geom/label/g_label.c =================================================================== --- geom/label/g_label.c (Revision 218482) +++ geom/label/g_label.c (Arbeitskopie) @@ -36,10 +36,12 @@ #include #include #include +#include #include #include #include +FEATURE(geom_label, "GEOM labeling support"); SYSCTL_DECL(_kern_geom); SYSCTL_NODE(_kern_geom, OID_AUTO, label, CTLFLAG_RW, 0, "GEOM_LABEL stuff"); Index: geom/gate/g_gate.c =================================================================== --- geom/gate/g_gate.c (Revision 218482) +++ geom/gate/g_gate.c (Arbeitskopie) @@ -53,6 +53,8 @@ #include #include +FEATURE(geom_gate, "GEOM Gate module"); + static MALLOC_DEFINE(M_GATE, "gg_data", "GEOM Gate Data"); SYSCTL_DECL(_kern_geom); Index: geom/uzip/g_uzip.c =================================================================== --- geom/uzip/g_uzip.c (Revision 218482) +++ geom/uzip/g_uzip.c (Arbeitskopie) @@ -36,10 +36,13 @@ #include #include #include +#include #include #include +FEATURE(geom_uzip, "GEOM uzip read-only compressed disks support"); + #undef GEOM_UZIP_DEBUG #ifdef GEOM_UZIP_DEBUG #define DPRINTF(a) printf a Index: geom/geom_sunlabel.c =================================================================== --- geom/geom_sunlabel.c (Revision 218482) +++ geom/geom_sunlabel.c (Arbeitskopie) @@ -39,6 +39,7 @@ #include #include #include +#include #include #include #include @@ -51,6 +52,8 @@ #include #include +FEATURE(geom_sunlabel, "GEOM Sun/Solaris partitioning support"); + #define SUNLABEL_CLASS_NAME "SUN" struct g_sunlabel_softc { Index: geom/geom_bsd.c =================================================================== --- geom/geom_bsd.c (Revision 218482) +++ geom/geom_bsd.c (Arbeitskopie) @@ -45,6 +45,7 @@ #include #include #include +#include #include #include #include @@ -60,6 +61,8 @@ #include #include +FEATURE(geom_bsd, "GEOM BSD disklabels support"); + #define BSD_CLASS_NAME "BSD" #define ALPHA_LABEL_OFFSET 64 Index: geom/cache/g_cache.c =================================================================== --- geom/cache/g_cache.c (Revision 218482) +++ geom/cache/g_cache.c (Arbeitskopie) @@ -42,6 +42,8 @@ #include #include +FEATURE(geom_cache, "GEOM cache module"); + static MALLOC_DEFINE(M_GCACHE, "gcache_data", "GEOM_CACHE Data"); SYSCTL_DECL(_kern_geom); Index: geom/mirror/g_mirror.c =================================================================== --- geom/mirror/g_mirror.c (Revision 218482) +++ geom/mirror/g_mirror.c (Arbeitskopie) @@ -45,6 +45,7 @@ #include #include +FEATURE(geom_mirror, "GEOM mirroring support"); static MALLOC_DEFINE(M_MIRROR, "mirror_data", "GEOM_MIRROR Data"); Index: geom/geom_fox.c =================================================================== --- geom/geom_fox.c (Revision 218482) +++ geom/geom_fox.c (Arbeitskopie) @@ -37,6 +37,7 @@ #include #include +#include #include #include #include @@ -52,6 +53,8 @@ #define FOX_CLASS_NAME "FOX" #define FOX_MAGIC "GEOM::FOX" +FEATURE(geom_fox, "GEOM FOX redundant path mitigation support"); + struct g_fox_softc { off_t mediasize; u_int sectorsize; Index: geom/geom_vol_ffs.c =================================================================== --- geom/geom_vol_ffs.c (Revision 218482) +++ geom/geom_vol_ffs.c (Arbeitskopie) @@ -30,6 +30,7 @@ #include #include #include +#include #include #include #include @@ -42,6 +43,8 @@ #include #include +FEATURE(geom_vol, "GEOM support for volume names from UFS superblock"); + #define VOL_FFS_CLASS_NAME "VOL_FFS" static int superblocks[] = SBLOCKSEARCH; Index: geom/bde/g_bde.c =================================================================== --- geom/bde/g_bde.c (Revision 218482) +++ geom/bde/g_bde.c (Arbeitskopie) @@ -41,6 +41,7 @@ #include #include #include +#include #include #include @@ -48,6 +49,8 @@ #include #define BDE_CLASS_NAME "BDE" +FEATURE(geom_bde, "GEOM-based Disk Encryption"); + static void g_bde_start(struct bio *bp) { Index: geom/raid3/g_raid3.c =================================================================== --- geom/raid3/g_raid3.c (Revision 218482) +++ geom/raid3/g_raid3.c (Arbeitskopie) @@ -45,6 +45,7 @@ #include #include +FEATURE(geom_raid3, "GEOM RAID-3 functionality"); static MALLOC_DEFINE(M_RAID3, "raid3_data", "GEOM_RAID3 Data"); Index: geom/shsec/g_shsec.c =================================================================== --- geom/shsec/g_shsec.c (Revision 218482) +++ geom/shsec/g_shsec.c (Arbeitskopie) @@ -40,6 +40,7 @@ #include #include +FEATURE(geom_shsec, "GEOM shared secret device support"); static MALLOC_DEFINE(M_SHSEC, "shsec_data", "GEOM_SHSEC Data"); Index: geom/part/g_part_pc98.c =================================================================== --- geom/part/g_part_pc98.c (Revision 218482) +++ geom/part/g_part_pc98.c (Arbeitskopie) @@ -40,11 +40,14 @@ #include #include #include +#include #include #include #include "g_part_if.h" +FEATURE(geom_part_pc98, "GEOM partitioning class for PC-9800 disk partitions"); + #define SECSIZE 512 struct g_part_pc98_table { Index: geom/part/g_part_vtoc8.c =================================================================== --- geom/part/g_part_vtoc8.c (Revision 218482) +++ geom/part/g_part_vtoc8.c (Arbeitskopie) @@ -39,12 +39,15 @@ #include #include #include +#include #include #include #include #include "g_part_if.h" +FEATURE(geom_part_vtoc8, "GEOM partitioning class for SMI VTOC8 disk labels"); + struct g_part_vtoc8_table { struct g_part_table base; struct vtoc8 vtoc; Index: geom/part/g_part_bsd.c =================================================================== --- geom/part/g_part_bsd.c (Revision 218482) +++ geom/part/g_part_bsd.c (Arbeitskopie) @@ -40,11 +40,14 @@ #include #include #include +#include #include #include #include "g_part_if.h" +FEATURE(geom_part_bsd, "GEOM partitioning class for BSD disklabels"); + struct g_part_bsd_table { struct g_part_table base; u_char *bbarea; Index: geom/part/g_part_ebr.c =================================================================== --- geom/part/g_part_ebr.c (Revision 218482) +++ geom/part/g_part_ebr.c (Arbeitskopie) @@ -42,11 +42,19 @@ #include #include #include +#include #include #include #include "g_part_if.h" +FEATURE(geom_part_ebr, + "GEOM partitioning class for extended boot records support"); +#if defined(GEOM_PART_EBR_COMPAT) +FEATURE(geom_part_ebr_compat, + "GEOM EBR partitioning class: backward-compatible partition names"); +#endif + #define EBRSIZE 512 struct g_part_ebr_table { Index: geom/part/g_part_gpt.c =================================================================== --- geom/part/g_part_gpt.c (Revision 218482) +++ geom/part/g_part_gpt.c (Arbeitskopie) @@ -41,12 +41,15 @@ #include #include #include +#include #include #include #include #include "g_part_if.h" +FEATURE(geom_part_gpt, "GEOM partitioning class for GPT partitions support"); + CTASSERT(offsetof(struct gpt_hdr, padding) == 92); CTASSERT(sizeof(struct gpt_ent) == 128); Index: geom/part/g_part_apm.c =================================================================== --- geom/part/g_part_apm.c (Revision 218482) +++ geom/part/g_part_apm.c (Arbeitskopie) @@ -41,11 +41,14 @@ #include #include #include +#include #include #include #include "g_part_if.h" +FEATURE(geom_part_apm, "GEOM partitioning class for Apple-style partitions"); + struct g_part_apm_table { struct g_part_table base; struct apm_ddr ddr; Index: geom/part/g_part_mbr.c =================================================================== --- geom/part/g_part_mbr.c (Revision 218482) +++ geom/part/g_part_mbr.c (Arbeitskopie) @@ -40,11 +40,14 @@ #include #include #include +#include #include #include #include "g_part_if.h" +FEATURE(geom_part_mbr, "GEOM partitioning class for MBR support"); + #define MBRSIZE 512 struct g_part_mbr_table { Index: geom/stripe/g_stripe.c =================================================================== --- geom/stripe/g_stripe.c (Revision 218482) +++ geom/stripe/g_stripe.c (Arbeitskopie) @@ -40,6 +40,7 @@ #include #include +FEATURE(geom_stripe, "GEOM striping support"); static MALLOC_DEFINE(M_STRIPE, "stripe_data", "GEOM_STRIPE Data"); Index: geom/concat/g_concat.c =================================================================== --- geom/concat/g_concat.c (Revision 218482) +++ geom/concat/g_concat.c (Arbeitskopie) @@ -39,6 +39,7 @@ #include #include +FEATURE(geom_concat, "GEOM concatenation support"); static MALLOC_DEFINE(M_CONCAT, "concat_data", "GEOM_CONCAT Data"); Index: geom/multipath/g_multipath.c =================================================================== --- geom/multipath/g_multipath.c (Revision 218482) +++ geom/multipath/g_multipath.c (Arbeitskopie) @@ -44,6 +44,7 @@ #include #include +FEATURE(geom_multipath, "GEOM multipath support"); SYSCTL_DECL(_kern_geom); SYSCTL_NODE(_kern_geom, OID_AUTO, multipath, CTLFLAG_RW, 0, Index: geom/virstor/g_virstor.c =================================================================== --- geom/virstor/g_virstor.c (Revision 218482) +++ geom/virstor/g_virstor.c (Arbeitskopie) @@ -52,6 +52,8 @@ #include #include +FEATURE(g_virstor, "GEOM virtual storage support"); + /* Declare malloc(9) label */ static MALLOC_DEFINE(M_GVIRSTOR, "gvirstor", "GEOM_VIRSTOR Data"); Index: geom/geom_mbr.c =================================================================== --- geom/geom_mbr.c (Revision 218482) +++ geom/geom_mbr.c (Arbeitskopie) @@ -37,6 +37,7 @@ #include #include #include +#include #include #include #include @@ -50,6 +51,8 @@ #include #include +FEATURE(geom_mbr, "GEOM DOS/MBR partitioning support"); + #define MBR_CLASS_NAME "MBR" #define MBREXT_CLASS_NAME "MBREXT" Index: geom/eli/g_eli.c =================================================================== --- geom/eli/g_eli.c (Revision 218482) +++ geom/eli/g_eli.c (Arbeitskopie) @@ -51,6 +51,7 @@ #include #include +FEATURE(geom_eli, "GEOM crypto module"); MALLOC_DEFINE(M_ELI, "eli data", "GEOM_ELI Data"); Index: geom/journal/g_journal.c =================================================================== --- geom/journal/g_journal.c (Revision 218482) +++ geom/journal/g_journal.c (Arbeitskopie) @@ -55,6 +55,7 @@ #include +FEATURE(geom_journal, "GEOM journaling support"); /* * On-disk journal format: Index: geom/geom_pc98.c =================================================================== --- geom/geom_pc98.c (Revision 218482) +++ geom/geom_pc98.c (Arbeitskopie) @@ -36,6 +36,7 @@ #include #include #include +#include #include #include #include @@ -47,6 +48,8 @@ #include #include +FEATURE(geom_pc98, "GEOM NEC PC9800 partitioning support"); + #define PC98_CLASS_NAME "PC98" struct g_pc98_softc { Index: geom/linux_lvm/g_linux_lvm.c =================================================================== --- geom/linux_lvm/g_linux_lvm.c (Revision 218482) +++ geom/linux_lvm/g_linux_lvm.c (Arbeitskopie) @@ -42,6 +42,8 @@ #include +FEATURE(geom_linux_lvm, "GEOM Linux LVM partitioning support"); + /* Declare malloc(9) label */ static MALLOC_DEFINE(M_GLLVM, "gllvm", "GEOM_LINUX_LVM Data"); --=_30palfo2nku8--