lkml.org 
[lkml]   [2012]   [May]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: Remoteproc adaptations for ST-Ericsson modems
    From
    > 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@stericsson.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@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2012-05-21 18:01    [W:3.109 / U:0.004 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site