kernel/dev_ops: Make default dev_ops (noopen(), etc.) static.
authorSascha Wildner <saw@online.de>
Sun, 11 Jan 2015 11:05:17 +0000 (12:05 +0100)
committerSascha Wildner <saw@online.de>
Sun, 11 Jan 2015 11:05:17 +0000 (12:05 +0100)
They should only ever be needed by compile_dev_ops() (via default_dev_ops).

sys/kern/kern_device.c
sys/sys/device.h

index 56b0f3a..2a2fc39 100644 (file)
@@ -101,6 +101,20 @@ DEVOP_DESC_INIT(clone);
  */
 struct dev_ops dead_dev_ops;
 
+static d_open_t                noopen;
+static d_close_t       noclose;
+static d_read_t                noread;
+static d_write_t       nowrite;
+static d_ioctl_t       noioctl;
+static d_mmap_t                nommap;
+static d_mmap_single_t nommap_single;
+static d_strategy_t    nostrategy;
+static d_dump_t                nodump;
+static d_psize_t       nopsize;
+static d_kqfilter_t    nokqfilter;
+static d_clone_t       noclone;
+static d_revoke_t      norevoke;
+
 struct dev_ops default_dev_ops = {
        { "null" },
        .d_default = NULL,      /* must be NULL */
@@ -672,69 +686,69 @@ dev_ops_restore(cdev_t dev, struct dev_ops *oops)
  * Unsupported devswitch functions (e.g. for writing to read-only device).
  * XXX may belong elsewhere.
  */
-int
+static int
 norevoke(struct dev_revoke_args *ap)
 {
        /* take no action */
        return(0);
 }
 
-int
+static int
 noclone(struct dev_clone_args *ap)
 {
        /* take no action */
        return (0);     /* allow the clone */
 }
 
-int
+static int
 noopen(struct dev_open_args *ap)
 {
        return (ENODEV);
 }
 
-int
+static int
 noclose(struct dev_close_args *ap)
 {
        return (ENODEV);
 }
 
-int
+static int
 noread(struct dev_read_args *ap)
 {
        return (ENODEV);
 }
 
-int
+static int
 nowrite(struct dev_write_args *ap)
 {
        return (ENODEV);
 }
 
-int
+static int
 noioctl(struct dev_ioctl_args *ap)
 {
        return (ENODEV);
 }
 
-int
+static int
 nokqfilter(struct dev_kqfilter_args *ap)
 {
        return (ENODEV);
 }
 
-int
+static int
 nommap(struct dev_mmap_args *ap)
 {
        return (ENODEV);
 }
 
-int
+static int
 nommap_single(struct dev_mmap_single_args *ap)
 {
        return (ENODEV);
 }
 
-int
+static int
 nostrategy(struct dev_strategy_args *ap)
 {
        struct bio *bio = ap->a_bio;
@@ -745,14 +759,14 @@ nostrategy(struct dev_strategy_args *ap)
        return(0);
 }
 
-int
+static int
 nopsize(struct dev_psize_args *ap)
 {
        ap->a_result = 0;
        return(0);
 }
 
-int
+static int
 nodump(struct dev_dump_args *ap)
 {
        return (ENODEV);
index b9aca6c..2ce698a 100644 (file)
@@ -343,21 +343,6 @@ int dev_dflags(cdev_t dev);
 int dev_doperate(struct dev_generic_args *ap);
 int dev_doperate_ops(struct dev_ops *, struct dev_generic_args *ap);
 
-d_default_t    nodefault;
-d_open_t       noopen;
-d_close_t      noclose;
-d_read_t       noread;
-d_write_t      nowrite;
-d_ioctl_t      noioctl;
-d_mmap_t       nommap;
-d_mmap_single_t        nommap_single;
-d_strategy_t   nostrategy;
-d_dump_t       nodump;
-d_psize_t      nopsize;
-d_kqfilter_t   nokqfilter;
-d_clone_t      noclone;
-d_revoke_t     norevoke;
-
 d_open_t       nullopen;
 d_close_t      nullclose;