2023-08-11 09:13:04

by Thorsten Leemhuis

[permalink] [raw]
Subject: Re: RE: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to support more devices

[CCing the regression list, as it should be in the loop for regressions:
https://docs.kernel.org/admin-guide/reporting-regressions.html]

On 04.08.23 14:02, Gaurav Jain wrote:
> From: Herbert Xu <[email protected]>
>> On Mon, Jul 24, 2023 at 05:13:23AM +0000, Meenakshi Aggarwal wrote:
>>> Please share the required information.
>> Any progress on this?
>>
>> Should we revert the offending patch?
>
> Debugging is in progress. There should be some mismatch in TRNG configuration in customer board.
> Will be sharing a patch to dump the same.

Any progress on this? Afaics would be good to have either the fix or the
revert in by -rc7 to ensure things get at least one week of proper
testing before the final release.

Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat)
--
Everything you wanna know about Linux kernel regression tracking:
https://linux-regtracking.leemhuis.info/about/#tldr
If I did something stupid, please tell me, as explained on that page.

#regzbot poke


2023-08-11 12:16:35

by Meenakshi Aggarwal

[permalink] [raw]
Subject: RE: RE: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to support more devices

Hi,

we have tested this patch on multiple variants of imx board and it is working fine.

We are actively debugging the issue, it is taking time as we are not able to reproduce it at our end.

Thanks,
Meenakshi

> -----Original Message-----
> From: Linux regression tracking (Thorsten Leemhuis)
> <[email protected]>
> Sent: Friday, August 11, 2023 2:18 PM
> To: Gaurav Jain <[email protected]>; Herbert Xu
> <[email protected]>; Meenakshi Aggarwal
> <[email protected]>
> Cc: Bastian Krause <[email protected]>; [email protected]; linux-
> [email protected]; [email protected]; Dan Douglass
> <[email protected]>; [email protected]; Horia Geanta
> <[email protected]>; Varun Sethi <[email protected]>; Pankaj Gupta
> <[email protected]>; Linux kernel regressions list
> <[email protected]>
> Subject: Re: RE: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to support
> more devices
>
> [CCing the regression list, as it should be in the loop for regressions:
> https://docs.ker/
> nel.org%2Fadmin-guide%2Freporting-
> regressions.html&data=05%7C01%7Cmeenakshi.aggarwal%40nxp.com%7Cfde9
> ceeb9e0d40d4f77d08db9a47a5e1%7C686ea1d3bc2b4c6fa92cd99c5c301635%7
> C0%7C0%7C638273404742560069%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC
> 4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C
> %7C%7C&sdata=lt7nphOXYXc3GRfOquvM5jmqNszQkDd%2BuSRdAbJd8Ec%3D&r
> eserved=0]
>
> On 04.08.23 14:02, Gaurav Jain wrote:
> > From: Herbert Xu <[email protected]>
> >> On Mon, Jul 24, 2023 at 05:13:23AM +0000, Meenakshi Aggarwal wrote:
> >>> Please share the required information.
> >> Any progress on this?
> >>
> >> Should we revert the offending patch?
> >
> > Debugging is in progress. There should be some mismatch in TRNG
> configuration in customer board.
> > Will be sharing a patch to dump the same.
>
> Any progress on this? Afaics would be good to have either the fix or the revert in
> by -rc7 to ensure things get at least one week of proper testing before the final
> release.
>
> Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat)
> --
> Everything you wanna know about Linux kernel regression tracking:
> https://linux-/
> regtracking.leemhuis.info%2Fabout%2F%23tldr&data=05%7C01%7Cmeenakshi.
> aggarwal%40nxp.com%7Cfde9ceeb9e0d40d4f77d08db9a47a5e1%7C686ea1d3b
> c2b4c6fa92cd99c5c301635%7C0%7C0%7C638273404742560069%7CUnknown
> %7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwi
> LCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Xg3ZCWObptmF6xLKO7dpaIYB
> A17eKK5wNhlPZ6FR2XA%3D&reserved=0
> If I did something stupid, please tell me, as explained on that page.
>
> #regzbot poke

2023-08-11 13:34:52

by Meenakshi Aggarwal

[permalink] [raw]
Subject: RE: RE: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to support more devices

Hi,

We discussed about it and lets revert the patch for now, as the tag is to be applied this weekend.

Thanks,
Meenakshi

> -----Original Message-----
> From: Meenakshi Aggarwal <[email protected]>
> Sent: Friday, August 11, 2023 5:25 PM
> To: Linux regressions mailing list <[email protected]>; Gaurav Jain
> <[email protected]>; Herbert Xu <[email protected]>
> Cc: Bastian Krause <[email protected]>; [email protected]; linux-
> [email protected]; [email protected]; Dan Douglass
> <[email protected]>; [email protected]; Horia Geanta
> <[email protected]>; Varun Sethi <[email protected]>; Pankaj Gupta
> <[email protected]>
> Subject: RE: RE: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to support
> more devices
>
> Hi,
>
> we have tested this patch on multiple variants of imx board and it is working fine.
>
> We are actively debugging the issue, it is taking time as we are not able to
> reproduce it at our end.
>
> Thanks,
> Meenakshi
>
> > -----Original Message-----
> > From: Linux regression tracking (Thorsten Leemhuis)
> > <[email protected]>
> > Sent: Friday, August 11, 2023 2:18 PM
> > To: Gaurav Jain <[email protected]>; Herbert Xu
> > <[email protected]>; Meenakshi Aggarwal
> > <[email protected]>
> > Cc: Bastian Krause <[email protected]>; [email protected]; linux-
> > [email protected]; [email protected]; Dan Douglass
> > <[email protected]>; [email protected]; Horia Geanta
> > <[email protected]>; Varun Sethi <[email protected]>; Pankaj Gupta
> > <[email protected]>; Linux kernel regressions list
> > <[email protected]>
> > Subject: Re: RE: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to
> > support more devices
> >
> > [CCing the regression list, as it should be in the loop for regressions:
> > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs
> > .ker%2F&data=05%7C01%7Cmeenakshi.aggarwal%40nxp.com%7C84b2ed6da
> ac44625
> >
> 779408db9a61d801%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63
> 827351
> >
> 7248142302%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
> V2luMzI
> >
> iLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=OWH18Cwu
> h8zyAFqn
> > I6iiL3Kewpj5NiWaQorKcmiYrr0%3D&reserved=0
> > nel.org%2Fadmin-guide%2Freporting-
> >
> regressions.html&data=05%7C01%7Cmeenakshi.aggarwal%40nxp.com%7Cfde9
> >
> ceeb9e0d40d4f77d08db9a47a5e1%7C686ea1d3bc2b4c6fa92cd99c5c301635%7
> >
> C0%7C0%7C638273404742560069%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC
> >
> 4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C
> > %7C%7C&sdata=lt7nphOXYXc3GRfOquvM5jmqNszQkDd%2BuSRdAbJd8Ec%3D
> &r
> > eserved=0]
> >
> > On 04.08.23 14:02, Gaurav Jain wrote:
> > > From: Herbert Xu <[email protected]>
> > >> On Mon, Jul 24, 2023 at 05:13:23AM +0000, Meenakshi Aggarwal wrote:
> > >>> Please share the required information.
> > >> Any progress on this?
> > >>
> > >> Should we revert the offending patch?
> > >
> > > Debugging is in progress. There should be some mismatch in TRNG
> > configuration in customer board.
> > > Will be sharing a patch to dump the same.
> >
> > Any progress on this? Afaics would be good to have either the fix or
> > the revert in by -rc7 to ensure things get at least one week of proper
> > testing before the final release.
> >
> > Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker'
> > hat)
> > --
> > Everything you wanna know about Linux kernel regression tracking:
> > https://linux-/
> >
> regtracking.leemhuis.info%2Fabout%2F%23tldr&data=05%7C01%7Cmeenakshi.
> >
> aggarwal%40nxp.com%7Cfde9ceeb9e0d40d4f77d08db9a47a5e1%7C686ea1d3b
> >
> c2b4c6fa92cd99c5c301635%7C0%7C0%7C638273404742560069%7CUnknown
> > %7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haW
> wi
> >
> LCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Xg3ZCWObptmF6xLKO7dpaIYB
> > A17eKK5wNhlPZ6FR2XA%3D&reserved=0
> > If I did something stupid, please tell me, as explained on that page.
> >
> > #regzbot poke

2023-08-13 14:52:52

by Thorsten Leemhuis

[permalink] [raw]
Subject: Re: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to support more devices

On 11.08.23 14:58, Meenakshi Aggarwal wrote:
>
> We discussed about it and lets revert the patch for now,

Okay. So who will submit the revert?

> as the tag is to be applied this weekend.

/me can't follow, but whatever, likely not that imporant

Ciao, Thorsten

>> -----Original Message-----
>> From: Meenakshi Aggarwal <[email protected]>
>> Sent: Friday, August 11, 2023 5:25 PM
>> To: Linux regressions mailing list <[email protected]>; Gaurav Jain
>> <[email protected]>; Herbert Xu <[email protected]>
>> Cc: Bastian Krause <[email protected]>; [email protected]; linux-
>> [email protected]; [email protected]; Dan Douglass
>> <[email protected]>; [email protected]; Horia Geanta
>> <[email protected]>; Varun Sethi <[email protected]>; Pankaj Gupta
>> <[email protected]>
>> Subject: RE: RE: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to support
>> more devices
>>
>> Hi,
>>
>> we have tested this patch on multiple variants of imx board and it is working fine.
>>
>> We are actively debugging the issue, it is taking time as we are not able to
>> reproduce it at our end.
>>
>> Thanks,
>> Meenakshi
>>
>>> -----Original Message-----
>>> From: Linux regression tracking (Thorsten Leemhuis)
>>> <[email protected]>
>>> Sent: Friday, August 11, 2023 2:18 PM
>>> To: Gaurav Jain <[email protected]>; Herbert Xu
>>> <[email protected]>; Meenakshi Aggarwal
>>> <[email protected]>
>>> Cc: Bastian Krause <[email protected]>; [email protected]; linux-
>>> [email protected]; [email protected]; Dan Douglass
>>> <[email protected]>; [email protected]; Horia Geanta
>>> <[email protected]>; Varun Sethi <[email protected]>; Pankaj Gupta
>>> <[email protected]>; Linux kernel regressions list
>>> <[email protected]>
>>> Subject: Re: RE: [EXT] Re: [PATCH] crypto: caam - adjust RNG timing to
>>> support more devices
>>>
>>> [CCing the regression list, as it should be in the loop for regressions:
>>> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs
>>> .ker%2F&data=05%7C01%7Cmeenakshi.aggarwal%40nxp.com%7C84b2ed6da
>> ac44625
>>>
>> 779408db9a61d801%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C63
>> 827351
>>>
>> 7248142302%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoi
>> V2luMzI
>>>
>> iLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=OWH18Cwu
>> h8zyAFqn
>>> I6iiL3Kewpj5NiWaQorKcmiYrr0%3D&reserved=0
>>> nel.org%2Fadmin-guide%2Freporting-
>>>
>> regressions.html&data=05%7C01%7Cmeenakshi.aggarwal%40nxp.com%7Cfde9
>>>
>> ceeb9e0d40d4f77d08db9a47a5e1%7C686ea1d3bc2b4c6fa92cd99c5c301635%7
>>>
>> C0%7C0%7C638273404742560069%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC
>>>
>> 4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C
>>> %7C%7C&sdata=lt7nphOXYXc3GRfOquvM5jmqNszQkDd%2BuSRdAbJd8Ec%3D
>> &r
>>> eserved=0]
>>>
>>> On 04.08.23 14:02, Gaurav Jain wrote:
>>>> From: Herbert Xu <[email protected]>
>>>>> On Mon, Jul 24, 2023 at 05:13:23AM +0000, Meenakshi Aggarwal wrote:
>>>>>> Please share the required information.
>>>>> Any progress on this?
>>>>>
>>>>> Should we revert the offending patch?
>>>>
>>>> Debugging is in progress. There should be some mismatch in TRNG
>>> configuration in customer board.
>>>> Will be sharing a patch to dump the same.
>>>
>>> Any progress on this? Afaics would be good to have either the fix or
>>> the revert in by -rc7 to ensure things get at least one week of proper
>>> testing before the final release.
>>>
>>> Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker'
>>> hat)
>>> --
>>> Everything you wanna know about Linux kernel regression tracking:
>>> https://linux-/
>>>
>> regtracking.leemhuis.info%2Fabout%2F%23tldr&data=05%7C01%7Cmeenakshi.
>>>
>> aggarwal%40nxp.com%7Cfde9ceeb9e0d40d4f77d08db9a47a5e1%7C686ea1d3b
>>>
>> c2b4c6fa92cd99c5c301635%7C0%7C0%7C638273404742560069%7CUnknown
>>> %7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haW
>> wi
>>>
>> LCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=Xg3ZCWObptmF6xLKO7dpaIYB
>>> A17eKK5wNhlPZ6FR2XA%3D&reserved=0
>>> If I did something stupid, please tell me, as explained on that page.
>>>
>>> #regzbot poke
>
>