linux-kernel - Re: Remoteproc adaptations for ST-Ericsson modems

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAJK669bMZ1FKJbG6nh_rpm6EELAPDXQgxzck0JqtJ16TFf-bmA@mail.gmail.com>
Date:	Mon, 21 May 2012 17:21:46 +0200
From:	Sjur Brændeland <sjurbren@...il.com>
To:	Ohad Ben-Cohen <ohad@...ery.com>
Cc:	Loic PALLARDY <loic.pallardy@...ricsson.com>,
	Ludovic BARRE <ludovic.barre@...ricsson.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Arnd Bergmann <arnd@...db.de>,
	Linus Walleij <linus.walleij@...aro.org>
Subject: Re: Remoteproc adaptations for ST-Ericsson modems

> Can you please check if the below helps ? this will also protects
> against premature invocation of rproc_get_by_name().

Sure this works for me. If you like you can add:
Tested-by: Sjur Brændeland <sjur.brandeland@...ricsson.com>

But, are you sure you don't want to do use wait_for_completion_timeout()
so you don't risk to block the clients if firmware loading fails?


For reference the crach without this patch looks like this:
------------[ cut here ]------------
WARNING: at fs/sysfs/dir.c:508 sysfs_add_one+0xb9/0xd4()
sysfs: cannot create duplicate filename '/devices/platform/foo.1/firmware/foo'
Modules linked in: virtio_rpmsg_bus remoteproc virtio_ring virtio
Call Trace:
6316f928:  [<6003536e>] warn_slowpath_common+0x86/0xb0
6316f958:  [<60189e2a>] strcat+0x0/0x28
6316f978:  [<600354d6>] warn_slowpath_fmt+0x94/0x96
6316f9a8:  [<6018505a>] ida_get_new_above+0x119/0x1dc
6316f9c0:  [<60035442>] warn_slowpath_fmt+0x0/0x96
6316f9d8:  [<60102c49>] sysfs_pathname+0x4f/0x57
6316f9f8:  [<60102c49>] sysfs_pathname+0x4f/0x57
6316fa18:  [<60102c49>] sysfs_pathname+0x4f/0x57
6316fa38:  [<60102c49>] sysfs_pathname+0x4f/0x57
6316fa58:  [<60103407>] sysfs_add_one+0xb9/0xd4
6316fa88:  [<6010366e>] create_dir+0x89/0xd9
6316fae8:  [<60103af6>] sysfs_create_dir+0x141/0x15f
6316faf8:  [<6018bc35>] vsnprintf+0x212/0x48b
6316fb08:  [<60185a01>] kobject_get+0x0/0x3a
6316fb28:  [<60185c87>] kobject_add_internal+0xe1/0x24b
6316fb68:  [<60186332>] kobject_add+0x110/0x124
6316fba8:  [<60186222>] kobject_add+0x0/0x124
6316fbc0:  [<6002c0f7>] unblock_signals+0x0/0x82
6316fbe8:  [<6002c2b7>] set_signals+0x29/0x3f
6316fbf0:  [<6002c0f7>] unblock_signals+0x0/0x82
6316fc18:  [<600a639b>] kmem_cache_alloc+0x9f/0xab
6316fc38:  [<6019ff09>] get_device_parent+0x1e0/0x210
6316fc68:  [<601a162d>] device_add+0x11e/0x6dd
6316fc90:  [<60053f82>] up_read+0x0/0x18
6316fc98:  [<600502f4>] prepare_to_wait+0x0/0x8d
6316fcd8:  [<601a879f>] _request_firmware_load+0x40/0x239
6316fd18:  [<601a93ae>] request_firmware+0xb5/0x107
6316fd48:  [<6006954a>] try_module_get+0x0/0x20
6316fd58:  [<6388ecab>] rproc_boot+0x141/0x3e2 [remoteproc]


Regards,
Sjur
--
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/