Prefer the direct use of octal for permissions.
Done with checkpatch -f --types=SYMBOLIC_PERMS --fix-inplace
and some typing.
Miscellanea:
o Whitespace neatening around these conversions.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
        if (!vn->proc_vlan_dir)
                goto err;
 
-       vn->proc_vlan_conf = proc_create(name_conf, S_IFREG|S_IRUSR|S_IWUSR,
-                                    vn->proc_vlan_dir, &vlan_fops);
+       vn->proc_vlan_conf = proc_create(name_conf, S_IFREG | 0600,
+                                        vn->proc_vlan_dir, &vlan_fops);
        if (!vn->proc_vlan_conf)
                goto err;
        return 0;
        if (!strcmp(vlandev->name, name_conf))
                return -EINVAL;
        vlan->dent =
-               proc_create_data(vlandev->name, S_IFREG|S_IRUSR|S_IWUSR,
+               proc_create_data(vlandev->name, S_IFREG | 0600,
                                 vn->proc_vlan_dir, &vlandev_fops, vlandev);
        if (!vlan->dent)
                return -ENOBUFS;
 
        if (!atalk_proc_dir)
                goto out;
 
-       p = proc_create("interface", S_IRUGO, atalk_proc_dir,
+       p = proc_create("interface", 0444, atalk_proc_dir,
                        &atalk_seq_interface_fops);
        if (!p)
                goto out_interface;
 
-       p = proc_create("route", S_IRUGO, atalk_proc_dir,
+       p = proc_create("route", 0444, atalk_proc_dir,
                        &atalk_seq_route_fops);
        if (!p)
                goto out_route;
 
-       p = proc_create("socket", S_IRUGO, atalk_proc_dir,
+       p = proc_create("socket", 0444, atalk_proc_dir,
                        &atalk_seq_socket_fops);
        if (!p)
                goto out_socket;
 
-       p = proc_create("arp", S_IRUGO, atalk_proc_dir, &atalk_seq_arp_fops);
+       p = proc_create("arp", 0444, atalk_proc_dir, &atalk_seq_arp_fops);
        if (!p)
                goto out_arp;
 
 
        return scnprintf(buf, PAGE_SIZE, "%d\n", link_rate);
 }
 
-static DEVICE_ATTR(address, S_IRUGO, show_address, NULL);
-static DEVICE_ATTR(atmaddress, S_IRUGO, show_atmaddress, NULL);
-static DEVICE_ATTR(atmindex, S_IRUGO, show_atmindex, NULL);
-static DEVICE_ATTR(carrier, S_IRUGO, show_carrier, NULL);
-static DEVICE_ATTR(type, S_IRUGO, show_type, NULL);
-static DEVICE_ATTR(link_rate, S_IRUGO, show_link_rate, NULL);
+static DEVICE_ATTR(address, 0444, show_address, NULL);
+static DEVICE_ATTR(atmaddress, 0444, show_atmaddress, NULL);
+static DEVICE_ATTR(atmindex, 0444, show_atmindex, NULL);
+static DEVICE_ATTR(carrier, 0444, show_carrier, NULL);
+static DEVICE_ATTR(type, 0444, show_type, NULL);
+static DEVICE_ATTR(link_rate, 0444, show_link_rate, NULL);
 
 static struct device_attribute *atm_attrs[] = {
        &dev_attr_atmaddress,
 
        {
                struct proc_dir_entry *p;
 
-               p = proc_create("arp", S_IRUGO, atm_proc_root, &arp_seq_fops);
+               p = proc_create("arp", 0444, atm_proc_root, &arp_seq_fops);
                if (!p) {
                        pr_err("Unable to initialize /proc/net/atm/arp\n");
                        atm_clip_exit_noproc();
 
 #ifdef CONFIG_PROC_FS
        struct proc_dir_entry *p;
 
-       p = proc_create("lec", S_IRUGO, atm_proc_root, &lec_seq_fops);
+       p = proc_create("lec", 0444, atm_proc_root, &lec_seq_fops);
        if (!p) {
                pr_err("Unable to initialize /proc/net/atm/lec\n");
                return -ENOMEM;
 
        for (e = atm_proc_ents; e->name; e++) {
                struct proc_dir_entry *dirent;
 
-               dirent = proc_create(e->name, S_IRUGO,
+               dirent = proc_create(e->name, 0444,
                                     atm_proc_root, e->proc_fops);
                if (!dirent)
                        goto err_out_remove;
 
        dev_add_pack(&ax25_packet_type);
        register_netdevice_notifier(&ax25_dev_notifier);
 
-       proc_create("ax25_route", S_IRUGO, init_net.proc_net,
+       proc_create("ax25_route", 0444, init_net.proc_net,
                    &ax25_route_fops);
-       proc_create("ax25", S_IRUGO, init_net.proc_net, &ax25_info_fops);
-       proc_create("ax25_calls", S_IRUGO, init_net.proc_net, &ax25_uid_fops);
+       proc_create("ax25", 0444, init_net.proc_net, &ax25_info_fops);
+       proc_create("ax25_calls", 0444, init_net.proc_net, &ax25_uid_fops);
 out:
        return rc;
 }
 
        return sprintf(buf, "%d\n", dev->channel);
 }
 
-static DEVICE_ATTR(address, S_IRUGO, show_address, NULL);
-static DEVICE_ATTR(channel, S_IRUGO, show_channel, NULL);
+static DEVICE_ATTR(address, 0444, show_address, NULL);
+static DEVICE_ATTR(channel, 0444, show_channel, NULL);
 
 static struct rfcomm_dev *__rfcomm_dev_add(struct rfcomm_dev_req *req,
                                           struct rfcomm_dlc *dlc)
 
 
 static struct bin_attribute bridge_forward = {
        .attr = { .name = SYSFS_BRIDGE_FDB,
-                 .mode = S_IRUGO, },
+                 .mode = 0444, },
        .read = brforward_read,
 };
 
 
 {                                                              \
        return store_flag(p, v, _mask);                         \
 }                                                              \
-static BRPORT_ATTR(_name, S_IRUGO | S_IWUSR,                   \
+static BRPORT_ATTR(_name, 0644,                                        \
                   show_##_name, store_##_name)
 
 static int store_flag(struct net_bridge_port *p, unsigned long v,
        return sprintf(buf, "%d\n", p->path_cost);
 }
 
-static BRPORT_ATTR(path_cost, S_IRUGO | S_IWUSR,
+static BRPORT_ATTR(path_cost, 0644,
                   show_path_cost, br_stp_set_path_cost);
 
 static ssize_t show_priority(struct net_bridge_port *p, char *buf)
        return sprintf(buf, "%d\n", p->priority);
 }
 
-static BRPORT_ATTR(priority, S_IRUGO | S_IWUSR,
+static BRPORT_ATTR(priority, 0644,
                         show_priority, br_stp_set_port_priority);
 
 static ssize_t show_designated_root(struct net_bridge_port *p, char *buf)
 {
        return br_show_bridge_id(buf, &p->designated_root);
 }
-static BRPORT_ATTR(designated_root, S_IRUGO, show_designated_root, NULL);
+static BRPORT_ATTR(designated_root, 0444, show_designated_root, NULL);
 
 static ssize_t show_designated_bridge(struct net_bridge_port *p, char *buf)
 {
        return br_show_bridge_id(buf, &p->designated_bridge);
 }
-static BRPORT_ATTR(designated_bridge, S_IRUGO, show_designated_bridge, NULL);
+static BRPORT_ATTR(designated_bridge, 0444, show_designated_bridge, NULL);
 
 static ssize_t show_designated_port(struct net_bridge_port *p, char *buf)
 {
        return sprintf(buf, "%d\n", p->designated_port);
 }
-static BRPORT_ATTR(designated_port, S_IRUGO, show_designated_port, NULL);
+static BRPORT_ATTR(designated_port, 0444, show_designated_port, NULL);
 
 static ssize_t show_designated_cost(struct net_bridge_port *p, char *buf)
 {
        return sprintf(buf, "%d\n", p->designated_cost);
 }
-static BRPORT_ATTR(designated_cost, S_IRUGO, show_designated_cost, NULL);
+static BRPORT_ATTR(designated_cost, 0444, show_designated_cost, NULL);
 
 static ssize_t show_port_id(struct net_bridge_port *p, char *buf)
 {
        return sprintf(buf, "0x%x\n", p->port_id);
 }
-static BRPORT_ATTR(port_id, S_IRUGO, show_port_id, NULL);
+static BRPORT_ATTR(port_id, 0444, show_port_id, NULL);
 
 static ssize_t show_port_no(struct net_bridge_port *p, char *buf)
 {
        return sprintf(buf, "0x%x\n", p->port_no);
 }
 
-static BRPORT_ATTR(port_no, S_IRUGO, show_port_no, NULL);
+static BRPORT_ATTR(port_no, 0444, show_port_no, NULL);
 
 static ssize_t show_change_ack(struct net_bridge_port *p, char *buf)
 {
        return sprintf(buf, "%d\n", p->topology_change_ack);
 }
-static BRPORT_ATTR(change_ack, S_IRUGO, show_change_ack, NULL);
+static BRPORT_ATTR(change_ack, 0444, show_change_ack, NULL);
 
 static ssize_t show_config_pending(struct net_bridge_port *p, char *buf)
 {
        return sprintf(buf, "%d\n", p->config_pending);
 }
-static BRPORT_ATTR(config_pending, S_IRUGO, show_config_pending, NULL);
+static BRPORT_ATTR(config_pending, 0444, show_config_pending, NULL);
 
 static ssize_t show_port_state(struct net_bridge_port *p, char *buf)
 {
        return sprintf(buf, "%d\n", p->state);
 }
-static BRPORT_ATTR(state, S_IRUGO, show_port_state, NULL);
+static BRPORT_ATTR(state, 0444, show_port_state, NULL);
 
 static ssize_t show_message_age_timer(struct net_bridge_port *p,
                                            char *buf)
 {
        return sprintf(buf, "%ld\n", br_timer_value(&p->message_age_timer));
 }
-static BRPORT_ATTR(message_age_timer, S_IRUGO, show_message_age_timer, NULL);
+static BRPORT_ATTR(message_age_timer, 0444, show_message_age_timer, NULL);
 
 static ssize_t show_forward_delay_timer(struct net_bridge_port *p,
                                            char *buf)
 {
        return sprintf(buf, "%ld\n", br_timer_value(&p->forward_delay_timer));
 }
-static BRPORT_ATTR(forward_delay_timer, S_IRUGO, show_forward_delay_timer, NULL);
+static BRPORT_ATTR(forward_delay_timer, 0444, show_forward_delay_timer, NULL);
 
 static ssize_t show_hold_timer(struct net_bridge_port *p,
                                            char *buf)
 {
        return sprintf(buf, "%ld\n", br_timer_value(&p->hold_timer));
 }
-static BRPORT_ATTR(hold_timer, S_IRUGO, show_hold_timer, NULL);
+static BRPORT_ATTR(hold_timer, 0444, show_hold_timer, NULL);
 
 static int store_flush(struct net_bridge_port *p, unsigned long v)
 {
        br_fdb_delete_by_port(p->br, p, 0, 0); // Don't delete local entry
        return 0;
 }
-static BRPORT_ATTR(flush, S_IWUSR, NULL, store_flush);
+static BRPORT_ATTR(flush, 0200, NULL, store_flush);
 
 static ssize_t show_group_fwd_mask(struct net_bridge_port *p, char *buf)
 {
 
        return 0;
 }
-static BRPORT_ATTR(group_fwd_mask, S_IRUGO | S_IWUSR, show_group_fwd_mask,
+static BRPORT_ATTR(group_fwd_mask, 0644, show_group_fwd_mask,
                   store_group_fwd_mask);
 
 BRPORT_ATTR_FLAG(hairpin_mode, BR_HAIRPIN_MODE);
 {
        return br_multicast_set_port_router(p, v);
 }
-static BRPORT_ATTR(multicast_router, S_IRUGO | S_IWUSR, show_multicast_router,
+static BRPORT_ATTR(multicast_router, 0644, show_multicast_router,
                   store_multicast_router);
 
 BRPORT_ATTR_FLAG(multicast_fast_leave, BR_MULTICAST_FAST_LEAVE);
 
 MODULE_ALIAS_NETPROTO(PF_CAN);
 
 static int stats_timer __read_mostly = 1;
-module_param(stats_timer, int, S_IRUGO);
+module_param(stats_timer, int, 0444);
 MODULE_PARM_DESC(stats_timer, "enable timer for statistics (default:on)");
 
 static struct kmem_cache *rcv_cache __read_mostly;
 
 #define CGW_DEFAULT_HOPS 1
 
 static unsigned int max_hops __read_mostly = CGW_DEFAULT_HOPS;
-module_param(max_hops, uint, S_IRUGO);
+module_param(max_hops, uint, 0444);
 MODULE_PARM_DESC(max_hops,
                 "maximum " CAN_GW_NAME " routing hops for CAN frames "
                 "(valid values: " __stringify(CGW_MIN_HOPS) "-"
 
        .get = param_get_supported_features,
 };
 module_param_cb(supported_features, ¶m_ops_supported_features, NULL,
-               S_IRUGO);
+               0444);
 
 const char *ceph_msg_type_name(int type)
 {
 
 {
        int rc = -ENOMEM;
 
-       if (!proc_create("dev", S_IRUGO, net->proc_net, &dev_seq_fops))
+       if (!proc_create("dev", 0444, net->proc_net, &dev_seq_fops))
                goto out;
-       if (!proc_create("softnet_stat", S_IRUGO, net->proc_net,
+       if (!proc_create("softnet_stat", 0444, net->proc_net,
                         &softnet_seq_fops))
                goto out_dev;
-       if (!proc_create("ptype", S_IRUGO, net->proc_net, &ptype_seq_fops))
+       if (!proc_create("ptype", 0444, net->proc_net, &ptype_seq_fops))
                goto out_softnet;
 
        if (wext_proc_init(net))
 
        return netdev_store(dev, attr, buf, len, change_group);
 }
 NETDEVICE_SHOW(group, fmt_dec);
-static DEVICE_ATTR(netdev_group, S_IRUGO | S_IWUSR, group_show, group_store);
+static DEVICE_ATTR(netdev_group, 0644, group_show, group_store);
 
 static int change_proto_down(struct net_device *dev, unsigned long proto_down)
 {
 }
 
 static struct rx_queue_attribute rps_cpus_attribute __ro_after_init
-       = __ATTR(rps_cpus, S_IRUGO | S_IWUSR, show_rps_map, store_rps_map);
+       = __ATTR(rps_cpus, 0644, show_rps_map, store_rps_map);
 
 static struct rx_queue_attribute rps_dev_flow_table_cnt_attribute __ro_after_init
-       = __ATTR(rps_flow_cnt, S_IRUGO | S_IWUSR,
+       = __ATTR(rps_flow_cnt, 0644,
                 show_rps_dev_flow_table_cnt, store_rps_dev_flow_table_cnt);
 #endif /* CONFIG_RPS */
 
 }
 
 static struct netdev_queue_attribute bql_hold_time_attribute __ro_after_init
-       = __ATTR(hold_time, S_IRUGO | S_IWUSR,
+       = __ATTR(hold_time, 0644,
                 bql_show_hold_time, bql_set_hold_time);
 
 static ssize_t bql_show_inflight(struct netdev_queue *queue,
 }
 
 static struct netdev_queue_attribute bql_inflight_attribute __ro_after_init =
-       __ATTR(inflight, S_IRUGO, bql_show_inflight, NULL);
+       __ATTR(inflight, 0444, bql_show_inflight, NULL);
 
 #define BQL_ATTR(NAME, FIELD)                                          \
 static ssize_t bql_show_ ## NAME(struct netdev_queue *queue,           \
 }                                                                      \
                                                                        \
 static struct netdev_queue_attribute bql_ ## NAME ## _attribute __ro_after_init \
-       = __ATTR(NAME, S_IRUGO | S_IWUSR,                               \
+       = __ATTR(NAME, 0644,                            \
                 bql_show_ ## NAME, bql_set_ ## NAME)
 
 BQL_ATTR(limit, limit);
 
 
 static __net_init int proto_init_net(struct net *net)
 {
-       if (!proc_create("protocols", S_IRUGO, net->proc_net, &proto_seq_fops))
+       if (!proc_create("protocols", 0444, net->proc_net, &proto_seq_fops))
                return -ENOMEM;
 
        return 0;
 
        dev_add_pack(&dn_dix_packet_type);
        register_netdevice_notifier(&dn_dev_notifier);
 
-       proc_create("decnet", S_IRUGO, init_net.proc_net, &dn_socket_seq_fops);
+       proc_create("decnet", 0444, init_net.proc_net, &dn_socket_seq_fops);
        dn_register_sysctl();
 out:
        return rc;
 
        rtnl_register_module(THIS_MODULE, PF_DECnet, RTM_GETADDR,
                             NULL, dn_nl_dump_ifaddr, 0);
 
-       proc_create("decnet_dev", S_IRUGO, init_net.proc_net, &dn_dev_seq_fops);
+       proc_create("decnet_dev", 0444, init_net.proc_net, &dn_dev_seq_fops);
 
 #ifdef CONFIG_SYSCTL
        {
 
 void __init dn_neigh_init(void)
 {
        neigh_table_init(NEIGH_DN_TABLE, &dn_neigh_table);
-       proc_create("decnet_neigh", S_IRUGO, init_net.proc_net,
+       proc_create("decnet_neigh", 0444, init_net.proc_net,
                    &dn_neigh_seq_fops);
 }
 
 
 
        dn_dst_ops.gc_thresh = (dn_rt_hash_mask + 1);
 
-       proc_create("decnet_cache", S_IRUGO, init_net.proc_net,
+       proc_create("decnet_cache", 0444, init_net.proc_net,
                    &dn_rt_cache_seq_fops);
 
 #ifdef CONFIG_DECNET_ROUTER
 
 MODULE_LICENSE("GPL");
 
 unsigned int dns_resolver_debug;
-module_param_named(debug, dns_resolver_debug, uint, S_IWUSR | S_IRUGO);
+module_param_named(debug, dns_resolver_debug, uint, 0644);
 MODULE_PARM_DESC(debug, "DNS Resolver debugging mask");
 
 const struct cred *dns_resolver_cache;
 
 
 static int __net_init arp_net_init(struct net *net)
 {
-       if (!proc_create("arp", S_IRUGO, net->proc_net, &arp_seq_fops))
+       if (!proc_create("arp", 0444, net->proc_net, &arp_seq_fops))
                return -ENOMEM;
        return 0;
 }
 
 
 int __net_init fib_proc_init(struct net *net)
 {
-       if (!proc_create("fib_trie", S_IRUGO, net->proc_net, &fib_trie_fops))
+       if (!proc_create("fib_trie", 0444, net->proc_net, &fib_trie_fops))
                goto out1;
 
-       if (!proc_create("fib_triestat", S_IRUGO, net->proc_net,
+       if (!proc_create("fib_triestat", 0444, net->proc_net,
                         &fib_triestat_fops))
                goto out2;
 
-       if (!proc_create("route", S_IRUGO, net->proc_net, &fib_route_fops))
+       if (!proc_create("route", 0444, net->proc_net, &fib_route_fops))
                goto out3;
 
        return 0;
 
        struct proc_dir_entry *pde;
        int err;
 
-       pde = proc_create("igmp", S_IRUGO, net->proc_net, &igmp_mc_seq_fops);
+       pde = proc_create("igmp", 0444, net->proc_net, &igmp_mc_seq_fops);
        if (!pde)
                goto out_igmp;
-       pde = proc_create("mcfilter", S_IRUGO, net->proc_net,
+       pde = proc_create("mcfilter", 0444, net->proc_net,
                          &igmp_mcf_seq_fops);
        if (!pde)
                goto out_mcfilter;
 
        unsigned int i;
 
 #ifdef CONFIG_PROC_FS
-       proc_create("pnp", S_IRUGO, init_net.proc_net, &pnp_seq_fops);
+       proc_create("pnp", 0444, init_net.proc_net, &pnp_seq_fops);
 #endif /* CONFIG_PROC_FS */
 
        if (!ic_enable)
 
 
                /* create proc dir entry */
                sprintf(buffer, "%pI4", &ip);
-               c->pde = proc_create_data(buffer, S_IWUSR|S_IRUSR,
+               c->pde = proc_create_data(buffer, 0600,
                                          cn->procdir,
                                          &clusterip_proc_fops, c);
                if (!c->pde) {
 
 int ping_proc_register(struct net *net, struct ping_seq_afinfo *afinfo)
 {
        struct proc_dir_entry *p;
-       p = proc_create_data(afinfo->name, S_IRUGO, net->proc_net,
+       p = proc_create_data(afinfo->name, 0444, net->proc_net,
                             afinfo->seq_fops, afinfo);
        if (!p)
                return -ENOMEM;
 
 
 static __net_init int ip_proc_init_net(struct net *net)
 {
-       if (!proc_create("sockstat", S_IRUGO, net->proc_net,
+       if (!proc_create("sockstat", 0444, net->proc_net,
                         &sockstat_seq_fops))
                goto out_sockstat;
-       if (!proc_create("netstat", S_IRUGO, net->proc_net, &netstat_seq_fops))
+       if (!proc_create("netstat", 0444, net->proc_net, &netstat_seq_fops))
                goto out_netstat;
-       if (!proc_create("snmp", S_IRUGO, net->proc_net, &snmp_seq_fops))
+       if (!proc_create("snmp", 0444, net->proc_net, &snmp_seq_fops))
                goto out_snmp;
 
        return 0;
 
 
 static __net_init int raw_init_net(struct net *net)
 {
-       if (!proc_create("raw", S_IRUGO, net->proc_net, &raw_seq_fops))
+       if (!proc_create("raw", 0444, net->proc_net, &raw_seq_fops))
                return -ENOMEM;
 
        return 0;
 
 {
        struct proc_dir_entry *pde;
 
-       pde = proc_create("rt_cache", S_IRUGO, net->proc_net,
+       pde = proc_create("rt_cache", 0444, net->proc_net,
                          &rt_cache_seq_fops);
        if (!pde)
                goto err1;
 
-       pde = proc_create("rt_cache", S_IRUGO,
+       pde = proc_create("rt_cache", 0444,
                          net->proc_net_stat, &rt_cpu_seq_fops);
        if (!pde)
                goto err2;
 
        afinfo->seq_ops.next            = tcp_seq_next;
        afinfo->seq_ops.stop            = tcp_seq_stop;
 
-       p = proc_create_data(afinfo->name, S_IRUGO, net->proc_net,
+       p = proc_create_data(afinfo->name, 0444, net->proc_net,
                             afinfo->seq_fops, afinfo);
        if (!p)
                rc = -ENOMEM;
 
        afinfo->seq_ops.next            = udp_seq_next;
        afinfo->seq_ops.stop            = udp_seq_stop;
 
-       p = proc_create_data(afinfo->name, S_IRUGO, net->proc_net,
+       p = proc_create_data(afinfo->name, 0444, net->proc_net,
                             afinfo->seq_fops, afinfo);
        if (!p)
                rc = -ENOMEM;
 
 
 static int __net_init if6_proc_net_init(struct net *net)
 {
-       if (!proc_create("if_inet6", S_IRUGO, net->proc_net, &if6_fops))
+       if (!proc_create("if_inet6", 0444, net->proc_net, &if6_fops))
                return -ENOMEM;
        return 0;
 }
 
 
 int __net_init ac6_proc_init(struct net *net)
 {
-       if (!proc_create("anycast6", S_IRUGO, net->proc_net, &ac6_seq_fops))
+       if (!proc_create("anycast6", 0444, net->proc_net, &ac6_seq_fops))
                return -ENOMEM;
 
        return 0;
 
 
 static int __net_init ip6_flowlabel_proc_init(struct net *net)
 {
-       if (!proc_create("ip6_flowlabel", S_IRUGO, net->proc_net,
+       if (!proc_create("ip6_flowlabel", 0444, net->proc_net,
                         &ip6fl_seq_fops))
                return -ENOMEM;
        return 0;
 
        int err;
 
        err = -ENOMEM;
-       if (!proc_create("igmp6", S_IRUGO, net->proc_net, &igmp6_mc_seq_fops))
+       if (!proc_create("igmp6", 0444, net->proc_net, &igmp6_mc_seq_fops))
                goto out;
-       if (!proc_create("mcfilter6", S_IRUGO, net->proc_net,
+       if (!proc_create("mcfilter6", 0444, net->proc_net,
                         &igmp6_mcf_seq_fops))
                goto out_proc_net_igmp6;
 
 
        if (!net->mib.proc_net_devsnmp6)
                return -ENOENT;
 
-       p = proc_create_data(idev->dev->name, S_IRUGO,
+       p = proc_create_data(idev->dev->name, 0444,
                             net->mib.proc_net_devsnmp6,
                             &snmp6_dev_seq_fops, idev);
        if (!p)
 
 static int __net_init ipv6_proc_init_net(struct net *net)
 {
-       if (!proc_create("sockstat6", S_IRUGO, net->proc_net,
+       if (!proc_create("sockstat6", 0444, net->proc_net,
                         &sockstat6_seq_fops))
                return -ENOMEM;
 
-       if (!proc_create("snmp6", S_IRUGO, net->proc_net, &snmp6_seq_fops))
+       if (!proc_create("snmp6", 0444, net->proc_net, &snmp6_seq_fops))
                goto proc_snmp6_fail;
 
        net->mib.proc_net_devsnmp6 = proc_mkdir("dev_snmp6", net->proc_net);
 
 
 static int __net_init raw6_init_net(struct net *net)
 {
-       if (!proc_create("raw6", S_IRUGO, net->proc_net, &raw6_seq_fops))
+       if (!proc_create("raw6", 0444, net->proc_net, &raw6_seq_fops))
                return -ENOMEM;
 
        return 0;
 
 {
 #ifdef CONFIG_PROC_FS
        proc_create("ipv6_route", 0, net->proc_net, &ipv6_route_proc_fops);
-       proc_create("rt6_stats", S_IRUGO, net->proc_net, &rt6_stats_seq_fops);
+       proc_create("rt6_stats", 0444, net->proc_net, &rt6_stats_seq_fops);
 #endif
        return 0;
 }
 
        struct proc_dir_entry *p;
        int rc = 0;
 
-       p = proc_create_data(muxinfo->name, S_IRUGO, net->proc_net,
+       p = proc_create_data(muxinfo->name, 0444, net->proc_net,
                             muxinfo->seq_fops, muxinfo);
        if (!p)
                rc = -ENOMEM;
 {
        int err;
 
-       if (!proc_create("kcm_stats", S_IRUGO, net->proc_net,
+       if (!proc_create("kcm_stats", 0444, net->proc_net,
                         &kcm_stats_seq_fops)) {
                err = -ENOMEM;
                goto out_kcm_stats;
 
        struct proc_dir_entry *pde;
        int err = 0;
 
-       pde = proc_create("pppol2tp", S_IRUGO, net->proc_net,
+       pde = proc_create("pppol2tp", 0444, net->proc_net,
                          &pppol2tp_proc_fops);
        if (!pde) {
                err = -ENOMEM;
 
        if (!llc_proc_dir)
                goto out;
 
-       p = proc_create("socket", S_IRUGO, llc_proc_dir, &llc_seq_socket_fops);
+       p = proc_create("socket", 0444, llc_proc_dir, &llc_seq_socket_fops);
        if (!p)
                goto out_socket;
 
-       p = proc_create("core", S_IRUGO, llc_proc_dir, &llc_seq_core_fops);
+       p = proc_create("core", 0444, llc_proc_dir, &llc_seq_core_fops);
        if (!p)
                goto out_core;
 
 
 #ifdef CONFIG_MAC80211_DEBUGFS
        mp->fixed_rate_idx = (u32) -1;
        mp->dbg_fixed_rate = debugfs_create_u32("fixed_rate_idx",
-                       S_IRUGO | S_IWUGO, debugfsdir, &mp->fixed_rate_idx);
+                       0666, debugfsdir, &mp->fixed_rate_idx);
 #endif
 
        minstrel_init_cck_rates(mp);
 
 {
        struct minstrel_sta_info *mi = priv_sta;
 
-       mi->dbg_stats = debugfs_create_file("rc_stats", S_IRUGO, dir, mi,
-                       &minstrel_stat_fops);
+       mi->dbg_stats = debugfs_create_file("rc_stats", 0444, dir, mi,
+                                           &minstrel_stat_fops);
 
-       mi->dbg_stats_csv = debugfs_create_file("rc_stats_csv", S_IRUGO, dir,
-                       mi, &minstrel_stat_csv_fops);
+       mi->dbg_stats_csv = debugfs_create_file("rc_stats_csv", 0444, dir, mi,
+                                               &minstrel_stat_csv_fops);
 }
 
 void
 
 {
        struct minstrel_ht_sta_priv *msp = priv_sta;
 
-       msp->dbg_stats = debugfs_create_file("rc_stats", S_IRUGO, dir, msp,
-                       &minstrel_ht_stat_fops);
-       msp->dbg_stats_csv = debugfs_create_file("rc_stats_csv", S_IRUGO,
-                            dir, msp, &minstrel_ht_stat_csv_fops);
+       msp->dbg_stats = debugfs_create_file("rc_stats", 0444, dir, msp,
+                                            &minstrel_ht_stat_fops);
+       msp->dbg_stats_csv = debugfs_create_file("rc_stats_csv", 0444, dir, msp,
+                                                &minstrel_ht_stat_csv_fops);
 }
 
 void
 
 MODULE_ALIAS_NFCT_HELPER("netbios_ns");
 
 static unsigned int timeout __read_mostly = 3;
-module_param(timeout, uint, S_IRUSR);
+module_param(timeout, uint, 0400);
 MODULE_PARM_DESC(timeout, "timeout for master connection/replies in seconds");
 
 static struct nf_conntrack_expect_policy exp_policy = {
 
 MODULE_ALIAS_NFCT_HELPER("snmp");
 
 static unsigned int timeout __read_mostly = 30;
-module_param(timeout, uint, S_IRUSR);
+module_param(timeout, uint, 0400);
 MODULE_PARM_DESC(timeout, "timeout for master connection/replies in seconds");
 
 int (*nf_nat_snmp_hook)(struct sk_buff *skb,
 
        if (uid_valid(root_uid) && gid_valid(root_gid))
                proc_set_user(pde, root_uid, root_gid);
 
-       pde = proc_create("nf_conntrack", S_IRUGO, net->proc_net_stat,
+       pde = proc_create("nf_conntrack", 0444, net->proc_net_stat,
                          &ct_cpu_seq_fops);
        if (!pde)
                goto out_stat_nf_conntrack;
 
        int ret = -ENOMEM;
 
 #ifdef CONFIG_PROC_FS
-       if (!proc_create("nf_log", S_IRUGO,
+       if (!proc_create("nf_log", 0444,
                         net->nf.proc_netfilter, &nflog_file_ops))
                return ret;
 #endif
 
 
 static int __net_init synproxy_proc_init(struct net *net)
 {
-       if (!proc_create("synproxy", S_IRUGO, net->proc_net_stat,
+       if (!proc_create("synproxy", 0444, net->proc_net_stat,
                         &synproxy_cpu_seq_fops))
                return -ENOMEM;
        return 0;
 
                ret = -ENOMEM;
                goto out_free_timer;
        }
-       info->timer->attr.attr.mode = S_IRUGO;
+       info->timer->attr.attr.mode = 0444;
        info->timer->attr.show = idletimer_tg_show;
 
        ret = sysfs_create_file(idletimer_tg_kobj, &info->timer->attr.attr);
 
 module_param(ip_list_tot, uint, 0400);
 module_param(ip_list_hash_size, uint, 0400);
 module_param(ip_list_perms, uint, 0400);
-module_param(ip_list_uid, uint, S_IRUGO | S_IWUSR);
-module_param(ip_list_gid, uint, S_IRUGO | S_IWUSR);
+module_param(ip_list_uid, uint, 0644);
+module_param(ip_list_gid, uint, 0644);
 MODULE_PARM_DESC(ip_list_tot, "number of IPs to remember per list");
 MODULE_PARM_DESC(ip_list_hash_size, "size of hash table used to look up IPs");
 MODULE_PARM_DESC(ip_list_perms, "permissions on /proc/net/xt_recent/* files");
 
 
        nr_loopback_init();
 
-       proc_create("nr", S_IRUGO, init_net.proc_net, &nr_info_fops);
-       proc_create("nr_neigh", S_IRUGO, init_net.proc_net, &nr_neigh_fops);
-       proc_create("nr_nodes", S_IRUGO, init_net.proc_net, &nr_nodes_fops);
+       proc_create("nr", 0444, init_net.proc_net, &nr_info_fops);
+       proc_create("nr_neigh", 0444, init_net.proc_net, &nr_neigh_fops);
+       proc_create("nr_nodes", 0444, init_net.proc_net, &nr_nodes_fops);
 out:
        return rc;
 fail:
 
 
        rose_add_loopback_neigh();
 
-       proc_create("rose", S_IRUGO, init_net.proc_net, &rose_info_fops);
-       proc_create("rose_neigh", S_IRUGO, init_net.proc_net,
+       proc_create("rose", 0444, init_net.proc_net, &rose_info_fops);
+       proc_create("rose_neigh", 0444, init_net.proc_net,
                    &rose_neigh_fops);
-       proc_create("rose_nodes", S_IRUGO, init_net.proc_net,
+       proc_create("rose_nodes", 0444, init_net.proc_net,
                    &rose_nodes_fops);
-       proc_create("rose_routes", S_IRUGO, init_net.proc_net,
+       proc_create("rose_routes", 0444, init_net.proc_net,
                    &rose_routes_fops);
 out:
        return rc;
 
 MODULE_ALIAS_NETPROTO(PF_RXRPC);
 
 unsigned int rxrpc_debug; // = RXRPC_DEBUG_KPROTO;
-module_param_named(debug, rxrpc_debug, uint, S_IWUSR | S_IRUGO);
+module_param_named(debug, rxrpc_debug, uint, 0644);
 MODULE_PARM_DESC(debug, "RxRPC debugging mask");
 
 static struct proto rxrpc_proto;
 
        net->sctp.proc_net_sctp = proc_net_mkdir(net, "sctp", net->proc_net);
        if (!net->sctp.proc_net_sctp)
                return -ENOMEM;
-       if (!proc_create("snmp", S_IRUGO, net->sctp.proc_net_sctp,
-                       &sctp_snmp_seq_fops))
+       if (!proc_create("snmp", 0444, net->sctp.proc_net_sctp,
+                        &sctp_snmp_seq_fops))
                goto cleanup;
-       if (!proc_create("eps", S_IRUGO, net->sctp.proc_net_sctp,
-                       &sctp_eps_seq_fops))
+       if (!proc_create("eps", 0444, net->sctp.proc_net_sctp,
+                        &sctp_eps_seq_fops))
                goto cleanup;
-       if (!proc_create("assocs", S_IRUGO, net->sctp.proc_net_sctp,
-                       &sctp_assocs_seq_fops))
+       if (!proc_create("assocs", 0444, net->sctp.proc_net_sctp,
+                        &sctp_assocs_seq_fops))
                goto cleanup;
-       if (!proc_create("remaddr", S_IRUGO, net->sctp.proc_net_sctp,
-                       &sctp_remaddr_seq_fops))
+       if (!proc_create("remaddr", 0444, net->sctp.proc_net_sctp,
+                        &sctp_remaddr_seq_fops))
                goto cleanup;
        return 0;
 
 
        struct proc_dir_entry **p = &sn->use_gssp_proc;
 
        sn->use_gss_proxy = -1;
-       *p = proc_create_data("use-gss-proxy", S_IFREG|S_IRUSR|S_IWUSR,
+       *p = proc_create_data("use-gss-proxy", S_IFREG | 0600,
                              sn->proc_net_rpc,
                              &use_gss_proxy_ops, net);
        if (!*p)
 
        if (cd->procfs == NULL)
                goto out_nomem;
 
-       p = proc_create_data("flush", S_IFREG|S_IRUSR|S_IWUSR,
+       p = proc_create_data("flush", S_IFREG | 0600,
                             cd->procfs, &cache_flush_operations_procfs, cd);
        if (p == NULL)
                goto out_nomem;
 
        if (cd->cache_request || cd->cache_parse) {
-               p = proc_create_data("channel", S_IFREG|S_IRUSR|S_IWUSR,
-                               cd->procfs, &cache_file_operations_procfs, cd);
+               p = proc_create_data("channel", S_IFREG | 0600, cd->procfs,
+                                    &cache_file_operations_procfs, cd);
                if (p == NULL)
                        goto out_nomem;
        }
        if (cd->cache_show) {
-               p = proc_create_data("content", S_IFREG|S_IRUSR,
-                               cd->procfs, &content_file_operations_procfs, cd);
+               p = proc_create_data("content", S_IFREG | 0400, cd->procfs,
+                                    &content_file_operations_procfs, cd);
                if (p == NULL)
                        goto out_nomem;
        }
 
                return;
 
        /* make tasks file */
-       if (!debugfs_create_file("tasks", S_IFREG | S_IRUSR, clnt->cl_debugfs,
+       if (!debugfs_create_file("tasks", S_IFREG | 0400, clnt->cl_debugfs,
                                 clnt, &tasks_fops))
                goto out_err;
 
                return;
 
        /* make tasks file */
-       if (!debugfs_create_file("info", S_IFREG | S_IRUSR, xprt->debugfs,
+       if (!debugfs_create_file("info", S_IFREG | 0400, xprt->debugfs,
                                 xprt, &xprt_info_fops)) {
                debugfs_remove_recursive(xprt->debugfs);
                xprt->debugfs = NULL;
        if (!faultdir)
                return NULL;
 
-       if (!debugfs_create_file("disconnect", S_IFREG | S_IRUSR, faultdir,
+       if (!debugfs_create_file("disconnect", S_IFREG | 0400, faultdir,
                                 NULL, &fault_disconnect_fops))
                return NULL;
 
 
 {
        struct dentry *dentry;
        struct inode *dir = d_inode(parent);
-       umode_t umode = S_IFIFO | S_IRUSR | S_IWUSR;
+       umode_t umode = S_IFIFO | 0600;
        int err;
 
        if (pipe->ops->upcall == NULL)
-               umode &= ~S_IRUGO;
+               umode &= ~0444;
        if (pipe->ops->downcall == NULL)
-               umode &= ~S_IWUGO;
+               umode &= ~0222;
 
        inode_lock_nested(dir, I_MUTEX_PARENT);
        dentry = __rpc_lookup_create_exclusive(parent, name);
        [RPCAUTH_info] = {
                .name = "info",
                .i_fop = &rpc_info_operations,
-               .mode = S_IFREG | S_IRUSR,
+               .mode = S_IFREG | 0400,
        },
 };
 
 {
        struct dentry *ret;
 
-       ret = rpc_mkdir_populate(dentry, name, S_IRUGO | S_IXUGO, NULL,
-                       rpc_clntdir_populate, rpc_client);
+       ret = rpc_mkdir_populate(dentry, name, 0555, NULL,
+                                rpc_clntdir_populate, rpc_client);
        if (!IS_ERR(ret)) {
                rpc_client->cl_pipedir_objects.pdh_dentry = ret;
                rpc_create_pipe_dir_objects(&rpc_client->cl_pipedir_objects);
        [0] = {
                .name = "channel",
                .i_fop = &cache_file_operations_pipefs,
-               .mode = S_IFREG|S_IRUSR|S_IWUSR,
+               .mode = S_IFREG | 0600,
        },
        [1] = {
                .name = "content",
                .i_fop = &content_file_operations_pipefs,
-               .mode = S_IFREG|S_IRUSR,
+               .mode = S_IFREG | 0400,
        },
        [2] = {
                .name = "flush",
                .i_fop = &cache_flush_operations_pipefs,
-               .mode = S_IFREG|S_IRUSR|S_IWUSR,
+               .mode = S_IFREG | 0600,
        },
 };
 
 static const struct rpc_filelist files[] = {
        [RPCAUTH_lockd] = {
                .name = "lockd",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
        [RPCAUTH_mount] = {
                .name = "mount",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
        [RPCAUTH_nfs] = {
                .name = "nfs",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
        [RPCAUTH_portmap] = {
                .name = "portmap",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
        [RPCAUTH_statd] = {
                .name = "statd",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
        [RPCAUTH_nfsd4_cb] = {
                .name = "nfsd4_cb",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
        [RPCAUTH_cache] = {
                .name = "cache",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
        [RPCAUTH_nfsd] = {
                .name = "nfsd",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
        [RPCAUTH_gssd] = {
                .name = "gssd",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
 };
 
 static const struct rpc_filelist gssd_dummy_clnt_dir[] = {
        [0] = {
                .name = "clntXX",
-               .mode = S_IFDIR | S_IRUGO | S_IXUGO,
+               .mode = S_IFDIR | 0555,
        },
 };
 
        [0] = {
                .name = "info",
                .i_fop = &rpc_dummy_info_operations,
-               .mode = S_IFREG | S_IRUSR,
+               .mode = S_IFREG | 0400,
        },
 };
 
        sb->s_d_op = &simple_dentry_operations;
        sb->s_time_gran = 1;
 
-       inode = rpc_get_inode(sb, S_IFDIR | S_IRUGO | S_IXUGO);
+       inode = rpc_get_inode(sb, S_IFDIR | 0555);
        sb->s_root = root = d_make_root(inode);
        if (!root)
                return -ENOMEM;
 
 int __net_init wext_proc_init(struct net *net)
 {
        /* Create /proc/net/wireless entry */
-       if (!proc_create("wireless", S_IRUGO, net->proc_net,
+       if (!proc_create("wireless", 0444, net->proc_net,
                         &wireless_seq_fops))
                return -ENOMEM;
 
 
        if (!proc_mkdir("x25", init_net.proc_net))
                return -ENOMEM;
 
-       if (!proc_create("x25/route", S_IRUGO, init_net.proc_net,
-                       &x25_seq_route_fops))
+       if (!proc_create("x25/route", 0444, init_net.proc_net,
+                        &x25_seq_route_fops))
                goto out;
 
-       if (!proc_create("x25/socket", S_IRUGO, init_net.proc_net,
-                       &x25_seq_socket_fops))
+       if (!proc_create("x25/socket", 0444, init_net.proc_net,
+                        &x25_seq_socket_fops))
                goto out;
 
-       if (!proc_create("x25/forward", S_IRUGO, init_net.proc_net,
-                       &x25_seq_forward_fops))
+       if (!proc_create("x25/forward", 0444, init_net.proc_net,
+                        &x25_seq_forward_fops))
                goto out;
        return 0;
 
 
 
 int __net_init xfrm_proc_init(struct net *net)
 {
-       if (!proc_create("xfrm_stat", S_IRUGO, net->proc_net,
+       if (!proc_create("xfrm_stat", 0444, net->proc_net,
                         &xfrm_statistics_seq_fops))
                return -ENOMEM;
        return 0;