Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-28 Thread Marc Kleine-Budde 
On 10/27/2016 06:28 PM, Cong Wang wrote:
>>> Hmm, bo->bound should guarantee it, so never mind, your patch
>>> looks fine.
>>
>> Can I add your Acked-by?
> 
> Of course.
> 
> Acked-by: Cong Wang 

Thanks,
Marc

-- 
Pengutronix e.K.  | Marc Kleine-Budde   |
Industrial Linux Solutions| Phone: +49-231-2826-924 |
Vertretung West/Dortmund  | Fax:   +49-5121-206917- |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |



signature.asc
Description: OpenPGP digital signature

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-28 Thread Marc Kleine-Budde 
On 10/27/2016 06:28 PM, Cong Wang wrote:
>>> Hmm, bo->bound should guarantee it, so never mind, your patch
>>> looks fine.
>>
>> Can I add your Acked-by?
> 
> Of course.
> 
> Acked-by: Cong Wang 

Thanks,
Marc

-- 
Pengutronix e.K.  | Marc Kleine-Budde   |
Industrial Linux Solutions| Phone: +49-231-2826-924 |
Vertretung West/Dortmund  | Fax:   +49-5121-206917- |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |



signature.asc
Description: OpenPGP digital signature

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-27 Thread Cong Wang 
On Thu, Oct 27, 2016 at 1:45 AM, Marc Kleine-Budde  wrote:
> On 10/24/2016 10:17 PM, Cong Wang wrote:
>> On Mon, Oct 24, 2016 at 1:10 PM, Cong Wang  wrote:
>>> On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
>>>  wrote:
 if (proc_dir) {
 /* unique socket address as filename */
 sprintf(bo->procname, "%lu", sock_i_ino(sk));
 bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
  proc_dir,
  _proc_fops, sk);
 +   if (!bo->bcm_proc_read) {
 +   ret = -ENOMEM;
 +   goto fail;
 +   }
>>>
>>> Well, I meant we need to call proc_create_data() once per socket,
>>> so we need a check before proc_create_data() too.
>>
>> Hmm, bo->bound should guarantee it, so never mind, your patch
>> looks fine.
>
> Can I add your Acked-by?

Of course.

Acked-by: Cong Wang 

Thanks.

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-27 Thread Cong Wang 
On Thu, Oct 27, 2016 at 1:45 AM, Marc Kleine-Budde  wrote:
> On 10/24/2016 10:17 PM, Cong Wang wrote:
>> On Mon, Oct 24, 2016 at 1:10 PM, Cong Wang  wrote:
>>> On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
>>>  wrote:
 if (proc_dir) {
 /* unique socket address as filename */
 sprintf(bo->procname, "%lu", sock_i_ino(sk));
 bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
  proc_dir,
  _proc_fops, sk);
 +   if (!bo->bcm_proc_read) {
 +   ret = -ENOMEM;
 +   goto fail;
 +   }
>>>
>>> Well, I meant we need to call proc_create_data() once per socket,
>>> so we need a check before proc_create_data() too.
>>
>> Hmm, bo->bound should guarantee it, so never mind, your patch
>> looks fine.
>
> Can I add your Acked-by?

Of course.

Acked-by: Cong Wang 

Thanks.

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-27 Thread Marc Kleine-Budde 
On 10/24/2016 10:17 PM, Cong Wang wrote:
> On Mon, Oct 24, 2016 at 1:10 PM, Cong Wang  wrote:
>> On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
>>  wrote:
>>> if (proc_dir) {
>>> /* unique socket address as filename */
>>> sprintf(bo->procname, "%lu", sock_i_ino(sk));
>>> bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
>>>  proc_dir,
>>>  _proc_fops, sk);
>>> +   if (!bo->bcm_proc_read) {
>>> +   ret = -ENOMEM;
>>> +   goto fail;
>>> +   }
>>
>> Well, I meant we need to call proc_create_data() once per socket,
>> so we need a check before proc_create_data() too.
> 
> Hmm, bo->bound should guarantee it, so never mind, your patch
> looks fine.

Can I add your Acked-by?

Marc

-- 
Pengutronix e.K.  | Marc Kleine-Budde   |
Industrial Linux Solutions| Phone: +49-231-2826-924 |
Vertretung West/Dortmund  | Fax:   +49-5121-206917- |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |



signature.asc
Description: OpenPGP digital signature

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-27 Thread Marc Kleine-Budde 
On 10/24/2016 10:17 PM, Cong Wang wrote:
> On Mon, Oct 24, 2016 at 1:10 PM, Cong Wang  wrote:
>> On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
>>  wrote:
>>> if (proc_dir) {
>>> /* unique socket address as filename */
>>> sprintf(bo->procname, "%lu", sock_i_ino(sk));
>>> bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
>>>  proc_dir,
>>>  _proc_fops, sk);
>>> +   if (!bo->bcm_proc_read) {
>>> +   ret = -ENOMEM;
>>> +   goto fail;
>>> +   }
>>
>> Well, I meant we need to call proc_create_data() once per socket,
>> so we need a check before proc_create_data() too.
> 
> Hmm, bo->bound should guarantee it, so never mind, your patch
> looks fine.

Can I add your Acked-by?

Marc

-- 
Pengutronix e.K.  | Marc Kleine-Budde   |
Industrial Linux Solutions| Phone: +49-231-2826-924 |
Vertretung West/Dortmund  | Fax:   +49-5121-206917- |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |



signature.asc
Description: OpenPGP digital signature

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-25 Thread Andrey Konovalov 
Hi Oliver,

I can confirm that your patch fixes the warnings for me.

Tested-by: Andrey Konovalov 

On Mon, Oct 24, 2016 at 10:17 PM, Cong Wang  wrote:
> On Mon, Oct 24, 2016 at 1:10 PM, Cong Wang  wrote:
>> On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
>>  wrote:
>>> if (proc_dir) {
>>> /* unique socket address as filename */
>>> sprintf(bo->procname, "%lu", sock_i_ino(sk));
>>> bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
>>>  proc_dir,
>>>  _proc_fops, sk);
>>> +   if (!bo->bcm_proc_read) {
>>> +   ret = -ENOMEM;
>>> +   goto fail;
>>> +   }
>>
>> Well, I meant we need to call proc_create_data() once per socket,
>> so we need a check before proc_create_data() too.
>
> Hmm, bo->bound should guarantee it, so never mind, your patch
> looks fine.

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-25 Thread Andrey Konovalov 
Hi Oliver,

I can confirm that your patch fixes the warnings for me.

Tested-by: Andrey Konovalov 

On Mon, Oct 24, 2016 at 10:17 PM, Cong Wang  wrote:
> On Mon, Oct 24, 2016 at 1:10 PM, Cong Wang  wrote:
>> On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
>>  wrote:
>>> if (proc_dir) {
>>> /* unique socket address as filename */
>>> sprintf(bo->procname, "%lu", sock_i_ino(sk));
>>> bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
>>>  proc_dir,
>>>  _proc_fops, sk);
>>> +   if (!bo->bcm_proc_read) {
>>> +   ret = -ENOMEM;
>>> +   goto fail;
>>> +   }
>>
>> Well, I meant we need to call proc_create_data() once per socket,
>> so we need a check before proc_create_data() too.
>
> Hmm, bo->bound should guarantee it, so never mind, your patch
> looks fine.

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-24 Thread Cong Wang 
On Mon, Oct 24, 2016 at 1:10 PM, Cong Wang  wrote:
> On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
>  wrote:
>> if (proc_dir) {
>> /* unique socket address as filename */
>> sprintf(bo->procname, "%lu", sock_i_ino(sk));
>> bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
>>  proc_dir,
>>  _proc_fops, sk);
>> +   if (!bo->bcm_proc_read) {
>> +   ret = -ENOMEM;
>> +   goto fail;
>> +   }
>
> Well, I meant we need to call proc_create_data() once per socket,
> so we need a check before proc_create_data() too.

Hmm, bo->bound should guarantee it, so never mind, your patch
looks fine.

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-24 Thread Cong Wang 
On Mon, Oct 24, 2016 at 1:10 PM, Cong Wang  wrote:
> On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
>  wrote:
>> if (proc_dir) {
>> /* unique socket address as filename */
>> sprintf(bo->procname, "%lu", sock_i_ino(sk));
>> bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
>>  proc_dir,
>>  _proc_fops, sk);
>> +   if (!bo->bcm_proc_read) {
>> +   ret = -ENOMEM;
>> +   goto fail;
>> +   }
>
> Well, I meant we need to call proc_create_data() once per socket,
> so we need a check before proc_create_data() too.

Hmm, bo->bound should guarantee it, so never mind, your patch
looks fine.

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-24 Thread Cong Wang 
On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
 wrote:
> if (proc_dir) {
> /* unique socket address as filename */
> sprintf(bo->procname, "%lu", sock_i_ino(sk));
> bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
>  proc_dir,
>  _proc_fops, sk);
> +   if (!bo->bcm_proc_read) {
> +   ret = -ENOMEM;
> +   goto fail;
> +   }

Well, I meant we need to call proc_create_data() once per socket,
so we need a check before proc_create_data() too.

Thanks.

Re: [PATCH] can: fix warning in bcm_connect/proc_register

2016-10-24 Thread Cong Wang 
On Mon, Oct 24, 2016 at 12:11 PM, Oliver Hartkopp
 wrote:
> if (proc_dir) {
> /* unique socket address as filename */
> sprintf(bo->procname, "%lu", sock_i_ino(sk));
> bo->bcm_proc_read = proc_create_data(bo->procname, 0644,
>  proc_dir,
>  _proc_fops, sk);
> +   if (!bo->bcm_proc_read) {
> +   ret = -ENOMEM;
> +   goto fail;
> +   }

Well, I meant we need to call proc_create_data() once per socket,
so we need a check before proc_create_data() too.

Thanks.