Re: [PATCH] mmc: block: init force_ro sysfs attribute

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Apr 23, 2011 at 10:22 AM, Rabin Vincent <rabin@xxxxxx> wrote:
> To avoid lockdep warnings:
>
> BUG: key dc90a520 not in .data!
> ------------[ cut here ]------------
> WARNING: at /home/rabin/kernel/arm/kernel/lockdep.c:2701 sysfs_add_file_mode+0x4c/0xb0()
> Modules linked in:
> [<c004b5d8>] (unwind_backtrace+0x0/0xe4) from [<c0074f20>] (warn_slowpath_common+0x4c/0x64)
> [<c0074f20>] (warn_slowpath_common+0x4c/0x64) from [<c0074f50>] (warn_slowpath_null+0x18/0x1c)
> [<c0074f50>] (warn_slowpath_null+0x18/0x1c) from [<c0157fec>] (sysfs_add_file_mode+0x4c/0xb0)
> [<c0157fec>] (sysfs_add_file_mode+0x4c/0xb0) from [<c02d61e4>] (mmc_add_disk+0x40/0x64)
> [<c02d61e4>] (mmc_add_disk+0x40/0x64) from [<c02d64cc>] (mmc_blk_probe+0x188/0x1fc)
> [<c02d64cc>] (mmc_blk_probe+0x188/0x1fc) from [<c02ce820>] (mmc_bus_probe+0x14/0x18)
> ...
>
> Signed-off-by: Rabin Vincent <rabin@xxxxxx>
> ---
>  drivers/mmc/card/block.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/mmc/card/block.c b/drivers/mmc/card/block.c
> index 9e30cf6..179a607 100644
> --- a/drivers/mmc/card/block.c
> +++ b/drivers/mmc/card/block.c
> @@ -967,6 +967,7 @@ static int mmc_add_disk(struct mmc_blk_data *md)
>        add_disk(md->disk);
>        md->force_ro.show = force_ro_show;
>        md->force_ro.store = force_ro_store;
> +       sysfs_attr_init(&md->force_ro.attr);
>        md->force_ro.attr.name = "force_ro";
>        md->force_ro.attr.mode = S_IRUGO | S_IWUSR;
>        ret = device_create_file(disk_to_dev(md->disk), &md->force_ro);
> --
> 1.7.4.1
>
>

Thanks for catching this. I should really fix the lockdep issues with
our platform (some deadlock/hang with a watchdog reboot).

A
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux