Re: [PATCH] i2c: dev: Fix the race between the release of i2c_dev and cdev

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

 



On Fri, Mar 20, 2020 at 07:01:50PM +0100, Wolfram Sang wrote:
> Hi Kevin,
> 
> I am very sorry for not making it for 5.6. I work on it now to make it
> go into v5.7.

No worries.

> 
> > entirely released. We can easily get the following call trace with
> > CONFIG_DEBUG_KOBJECT_RELEASE and CONFIG_DEBUG_OBJECTS_TIMERS enabled.
> 
> I didn't get this trace on my ARM32 board (Renesas Lager). Anything more
> I need to do besides activating those options and using i2c-dev?

CONFIG_DEBUG_OBJECTS_FREE is also needed. I got this bug warning when running
the OF unittest, so you may need to enable the following kernel options if you
also want to reproduce this by using that.
    CONFIG_OF_UNITTEST
    CONFIG_OF_OVERLAY

I have attached the config I used.

> 
> > +#include <linux/cdev.h>
> 
> This line is not needed. We include this already.

Yes, there is a conflict with the latest kernel. Do I need to resend a new
version?

Thanks,
Kevin

> 
> Other than that, the patch looks good. Although I trust you, I like to
> verify the test results.
> 
> Kind regards,
> 
>    Wolfram
> 


Attachment: config.gz
Description: application/gzip

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux