linux-kernel - Re: [PATCH] Input: serio

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <52A38A6C.6040005@vmware.com>
Date:	Sat, 07 Dec 2013 21:51:56 +0100
From:	Thomas Hellstrom <thellstrom@...are.com>
To:	Dmitry Torokhov <dmitry.torokhov@...il.com>
CC:	linux-input@...r.kernel.org,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Input: serio - fix sysfs layout

On 12/07/2013 04:41 PM, Dmitry Torokhov wrote:
> Restore previous layout of sysfs attributes that was broken by commit
> 3778a2129bcce84f684cc0017ed20d2524afd289 (input: serio: remove bus usage of
> dev_attrs) which moved all serio device attributes into 'id' group, when
> only 'type', 'proto', 'id', and 'extra' should be in 'id' group and the
> rest of attributes should be attached directly to the device.

Thanks for doing this. Makes vmmouse work again.

>
> Reported-by: Thomas Hellstrom <thellstrom@...are.com>
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@...il.com>

Tested-by: Thomas Hellstrom <thellstrom@...are.com>


> ---
>   drivers/input/serio/serio.c | 24 +++++++++++++++++-------
>   1 file changed, 17 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/input/serio/serio.c b/drivers/input/serio/serio.c
> index 98707fb..8f4c4ab 100644
> --- a/drivers/input/serio/serio.c
> +++ b/drivers/input/serio/serio.c
> @@ -455,16 +455,26 @@ static DEVICE_ATTR_RO(type);
>   static DEVICE_ATTR_RO(proto);
>   static DEVICE_ATTR_RO(id);
>   static DEVICE_ATTR_RO(extra);
> -static DEVICE_ATTR_RO(modalias);
> -static DEVICE_ATTR_WO(drvctl);
> -static DEVICE_ATTR(description, S_IRUGO, serio_show_description, NULL);
> -static DEVICE_ATTR(bind_mode, S_IWUSR | S_IRUGO, serio_show_bind_mode, serio_set_bind_mode);
>   
>   static struct attribute *serio_device_id_attrs[] = {
>   	&dev_attr_type.attr,
>   	&dev_attr_proto.attr,
>   	&dev_attr_id.attr,
>   	&dev_attr_extra.attr,
> +	NULL
> +};
> +
> +static struct attribute_group serio_id_attr_group = {
> +	.name	= "id",
> +	.attrs	= serio_device_id_attrs,
> +};
> +
> +static DEVICE_ATTR_RO(modalias);
> +static DEVICE_ATTR_WO(drvctl);
> +static DEVICE_ATTR(description, S_IRUGO, serio_show_description, NULL);
> +static DEVICE_ATTR(bind_mode, S_IWUSR | S_IRUGO, serio_show_bind_mode, serio_set_bind_mode);
> +
> +static struct attribute *serio_device_attrs[] = {
>   	&dev_attr_modalias.attr,
>   	&dev_attr_description.attr,
>   	&dev_attr_drvctl.attr,
> @@ -472,13 +482,13 @@ static struct attribute *serio_device_id_attrs[] = {
>   	NULL
>   };
>   
> -static struct attribute_group serio_id_attr_group = {
> -	.name	= "id",
> -	.attrs	= serio_device_id_attrs,
> +static struct attribute_group serio_device_attr_group = {
> +	.attrs	= serio_device_attrs,
>   };
>   
>   static const struct attribute_group *serio_device_attr_groups[] = {
>   	&serio_id_attr_group,
> +	&serio_device_attr_group,
>   	NULL
>   };
>   
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/