diff options
| author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-09-04 17:26:39 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-09-04 17:26:39 -0700 |
| commit | 1d0da63c6781741b6e9cc9558b13918c09445903 (patch) | |
| tree | 069426fd313f3a6758c1ece7c74e63530796d481 /f05.patch | |
| parent | f2a4447ad2255b39ebc3de81bf020ea667fdf983 (diff) | |
| download | patches-1d0da63c6781741b6e9cc9558b13918c09445903.tar.gz | |
more patches added to clean up dev_attrs
Diffstat (limited to 'f05.patch')
| -rw-r--r-- | f05.patch | 97 |
1 files changed, 97 insertions, 0 deletions
diff --git a/f05.patch b/f05.patch new file mode 100644 index 00000000000000..f77e0239e38971 --- /dev/null +++ b/f05.patch @@ -0,0 +1,97 @@ +--- + drivers/hv/channel.c | 2 -- + drivers/hv/vmbus_drv.c | 22 +++++++++++++++------- + include/linux/hyperv.h | 1 - + 3 files changed, 15 insertions(+), 10 deletions(-) + +--- a/drivers/hv/channel.c ++++ b/drivers/hv/channel.c +@@ -69,8 +69,6 @@ void vmbus_get_debug_info(struct vmbus_c + u8 monitor_group = (u8)channel->offermsg.monitorid / 32; + u8 monitor_offset = (u8)channel->offermsg.monitorid % 32; + +- memcpy(&debuginfo->interfacetype, +- &channel->offermsg.offer.if_type, sizeof(uuid_le)); + memcpy(&debuginfo->interface_instance, + &channel->offermsg.offer.if_instance, + sizeof(uuid_le)); +--- a/drivers/hv/vmbus_drv.c ++++ b/drivers/hv/vmbus_drv.c +@@ -47,7 +47,6 @@ static struct completion probe_event; + static int irq; + + struct hv_device_info { +- uuid_le chn_type; + uuid_le chn_instance; + + u32 server_monitor_pending; +@@ -80,8 +79,6 @@ static void get_channel_info(struct hv_d + + vmbus_get_debug_info(device->channel, &debug_info); + +- memcpy(&info->chn_type, &debug_info.interfacetype, +- sizeof(uuid_le)); + memcpy(&info->chn_instance, &debug_info.interface_instance, + sizeof(uuid_le)); + +@@ -139,9 +136,7 @@ static ssize_t vmbus_show_device_attr(st + + get_channel_info(hv_dev, device_info); + +- if (!strcmp(dev_attr->attr.name, "class_id")) { +- ret = sprintf(buf, "{%pUl}\n", device_info->chn_type.b); +- } else if (!strcmp(dev_attr->attr.name, "device_id")) { ++ if (!strcmp(dev_attr->attr.name, "device_id")) { + ret = sprintf(buf, "{%pUl}\n", device_info->chn_instance.b); + } else if (!strcmp(dev_attr->attr.name, "out_intr_mask")) { + ret = sprintf(buf, "%d\n", device_info->outbound.int_mask); +@@ -220,6 +215,18 @@ static ssize_t monitor_id_show(struct de + } + static DEVICE_ATTR_RO(monitor_id); + ++static ssize_t class_id_show(struct device *dev, ++ struct device_attribute *dev_attr, char *buf) ++{ ++ struct hv_device *hv_dev = device_to_hv_device(dev); ++ ++ if (!hv_dev->channel) ++ return -ENODEV; ++ return sprintf(buf, "{%pUl}\n", ++ hv_dev->channel->offermsg.offer.if_type.b); ++} ++static DEVICE_ATTR_RO(class_id); ++ + static ssize_t modalias_show(struct device *dev, + struct device_attribute *dev_attr, char *buf) + { +@@ -231,10 +238,12 @@ static ssize_t modalias_show(struct devi + } + static DEVICE_ATTR_RO(modalias); + ++ + static struct attribute *vmbus_attrs[] = { + &dev_attr_id.attr, + &dev_attr_state.attr, + &dev_attr_monitor_id.attr, ++ &dev_attr_class_id.attr, + &dev_attr_modalias.attr, + NULL, + }; +@@ -242,7 +251,6 @@ ATTRIBUTE_GROUPS(vmbus); + + /* Set up per device attributes in /sys/bus/vmbus/devices/<bus device> */ + static struct device_attribute vmbus_device_attrs[] = { +- __ATTR(class_id, S_IRUGO, vmbus_show_device_attr, NULL), + __ATTR(device_id, S_IRUGO, vmbus_show_device_attr, NULL), + + __ATTR(server_monitor_pending, S_IRUGO, vmbus_show_device_attr, NULL), +--- a/include/linux/hyperv.h ++++ b/include/linux/hyperv.h +@@ -900,7 +900,6 @@ enum vmbus_channel_state { + }; + + struct vmbus_channel_debug_info { +- uuid_le interfacetype; + uuid_le interface_instance; + u32 servermonitor_pending; + u32 servermonitor_latency; |