diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2012-09-07 16:12:54 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-09-10 15:30:41 -0400 |
commit | 15e473046cb6e5d18a4d0057e61d76315230382b (patch) | |
tree | 893d2df5d46a6ce156933ac57a1398f0ad22b889 /net/ieee802154 | |
parent | 9f00d9776bc5beb92e8bfc884a7e96ddc5589e2e (diff) |
netlink: Rename pid to portid to avoid confusion
It is a frequent mistake to confuse the netlink port identifier with a
process identifier. Try to reduce this confusion by renaming fields
that hold port identifiers portid instead of pid.
I have carefully avoided changing the structures exported to
userspace to avoid changing the userspace API.
I have successfully built an allyesconfig kernel with this change.
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Acked-by: Stephen Hemminger <shemminger@vyatta.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ieee802154')
-rw-r--r-- | net/ieee802154/nl-mac.c | 6 | ||||
-rw-r--r-- | net/ieee802154/nl-phy.c | 6 |
2 files changed, 6 insertions, 6 deletions
diff --git a/net/ieee802154/nl-mac.c b/net/ieee802154/nl-mac.c index 1e9917124e75..96bb08abece2 100644 --- a/net/ieee802154/nl-mac.c +++ b/net/ieee802154/nl-mac.c | |||
@@ -246,7 +246,7 @@ nla_put_failure: | |||
246 | } | 246 | } |
247 | EXPORT_SYMBOL(ieee802154_nl_start_confirm); | 247 | EXPORT_SYMBOL(ieee802154_nl_start_confirm); |
248 | 248 | ||
249 | static int ieee802154_nl_fill_iface(struct sk_buff *msg, u32 pid, | 249 | static int ieee802154_nl_fill_iface(struct sk_buff *msg, u32 portid, |
250 | u32 seq, int flags, struct net_device *dev) | 250 | u32 seq, int flags, struct net_device *dev) |
251 | { | 251 | { |
252 | void *hdr; | 252 | void *hdr; |
@@ -534,7 +534,7 @@ static int ieee802154_list_iface(struct sk_buff *skb, | |||
534 | if (!msg) | 534 | if (!msg) |
535 | goto out_dev; | 535 | goto out_dev; |
536 | 536 | ||
537 | rc = ieee802154_nl_fill_iface(msg, info->snd_pid, info->snd_seq, | 537 | rc = ieee802154_nl_fill_iface(msg, info->snd_portid, info->snd_seq, |
538 | 0, dev); | 538 | 0, dev); |
539 | if (rc < 0) | 539 | if (rc < 0) |
540 | goto out_free; | 540 | goto out_free; |
@@ -565,7 +565,7 @@ static int ieee802154_dump_iface(struct sk_buff *skb, | |||
565 | if (idx < s_idx || (dev->type != ARPHRD_IEEE802154)) | 565 | if (idx < s_idx || (dev->type != ARPHRD_IEEE802154)) |
566 | goto cont; | 566 | goto cont; |
567 | 567 | ||
568 | if (ieee802154_nl_fill_iface(skb, NETLINK_CB(cb->skb).pid, | 568 | if (ieee802154_nl_fill_iface(skb, NETLINK_CB(cb->skb).portid, |
569 | cb->nlh->nlmsg_seq, NLM_F_MULTI, dev) < 0) | 569 | cb->nlh->nlmsg_seq, NLM_F_MULTI, dev) < 0) |
570 | break; | 570 | break; |
571 | cont: | 571 | cont: |
diff --git a/net/ieee802154/nl-phy.c b/net/ieee802154/nl-phy.c index d54be34cca94..22b1a7058fd3 100644 --- a/net/ieee802154/nl-phy.c +++ b/net/ieee802154/nl-phy.c | |||
@@ -35,7 +35,7 @@ | |||
35 | 35 | ||
36 | #include "ieee802154.h" | 36 | #include "ieee802154.h" |
37 | 37 | ||
38 | static int ieee802154_nl_fill_phy(struct sk_buff *msg, u32 pid, | 38 | static int ieee802154_nl_fill_phy(struct sk_buff *msg, u32 portid, |
39 | u32 seq, int flags, struct wpan_phy *phy) | 39 | u32 seq, int flags, struct wpan_phy *phy) |
40 | { | 40 | { |
41 | void *hdr; | 41 | void *hdr; |
@@ -105,7 +105,7 @@ static int ieee802154_list_phy(struct sk_buff *skb, | |||
105 | if (!msg) | 105 | if (!msg) |
106 | goto out_dev; | 106 | goto out_dev; |
107 | 107 | ||
108 | rc = ieee802154_nl_fill_phy(msg, info->snd_pid, info->snd_seq, | 108 | rc = ieee802154_nl_fill_phy(msg, info->snd_portid, info->snd_seq, |
109 | 0, phy); | 109 | 0, phy); |
110 | if (rc < 0) | 110 | if (rc < 0) |
111 | goto out_free; | 111 | goto out_free; |
@@ -138,7 +138,7 @@ static int ieee802154_dump_phy_iter(struct wpan_phy *phy, void *_data) | |||
138 | return 0; | 138 | return 0; |
139 | 139 | ||
140 | rc = ieee802154_nl_fill_phy(data->skb, | 140 | rc = ieee802154_nl_fill_phy(data->skb, |
141 | NETLINK_CB(data->cb->skb).pid, | 141 | NETLINK_CB(data->cb->skb).portid, |
142 | data->cb->nlh->nlmsg_seq, | 142 | data->cb->nlh->nlmsg_seq, |
143 | NLM_F_MULTI, | 143 | NLM_F_MULTI, |
144 | phy); | 144 | phy); |