Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-19 Thread Tom Rini
On Wed, Jun 19, 2024 at 10:21:51AM +0200, Rasmus Villemoes wrote: > On 18/06/2024 23.03, Tim Harvey wrote: > > On Tue, Jun 18, 2024 at 7:32 AM Tom Rini wrote: > >> > > > Stefan and Tom, > > > > I'm seeing CI issues here even with 5000us [1]: > > host bind 0 /tmp/sandbox/persistent-cyclic functio

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-19 Thread Rasmus Villemoes
On 18/06/2024 23.03, Tim Harvey wrote: > On Tue, Jun 18, 2024 at 7:32 AM Tom Rini wrote: >> > Stefan and Tom, > > I'm seeing CI issues here even with 5000us [1]: > host bind 0 /tmp/sandbox/persistent-cyclic function wdt-gpio-level > took too long: 5368us vs 5000us max Yes, 5ms is way too little

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-18 Thread Tim Harvey
On Tue, Jun 18, 2024 at 7:32 AM Tom Rini wrote: > > On Tue, Jun 18, 2024 at 03:00:50PM +0100, Jiaxun Yang wrote: > > > > > > 在2024年6月18日六月 上午12:29,Tom Rini写道: > > [...] > > >> Acked-by: Stefan Roese > > > > > > Can we please get either this, or > > > https://patchwork.ozlabs.org/project/uboot/pat

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-18 Thread Tom Rini
On Tue, Jun 18, 2024 at 03:00:50PM +0100, Jiaxun Yang wrote: > > > 在2024年6月18日六月 上午12:29,Tom Rini写道: > [...] > >> Acked-by: Stefan Roese > > > > Can we please get either this, or > > https://patchwork.ozlabs.org/project/uboot/patch/20240524210817.1953298-1-rasmus.villem...@prevas.dk/ > > merged

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-18 Thread Stefan Roese
Hi Jiaxun, On 6/18/24 16:00, Jiaxun Yang wrote: 在2024年6月18日六月 上午12:29,Tom Rini写道: [...] Acked-by: Stefan Roese Can we please get either this, or https://patchwork.ozlabs.org/project/uboot/patch/20240524210817.1953298-1-rasmus.villem...@prevas.dk/ merged for master? The number of false nega

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-18 Thread Jiaxun Yang
在2024年6月18日六月 上午12:29,Tom Rini写道: [...] >> Acked-by: Stefan Roese > > Can we please get either this, or > https://patchwork.ozlabs.org/project/uboot/patch/20240524210817.1953298-1-rasmus.villem...@prevas.dk/ > merged for master? The number of false negatives in CI due to this is > big issue for

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-18 Thread Stefan Roese
On 6/18/24 01:29, Tom Rini wrote: On Fri, Jun 14, 2024 at 04:13:54PM +0200, Stefan Roese wrote: On 6/12/24 18:50, Tom Rini wrote: On Wed, Jun 12, 2024 at 05:13:37PM +0100, Jiaxun Yang wrote: 在2024年6月12日六月 下午5:00,Tom Rini写道: [...] configs/octeon_nic23_defconfig | 1 - 2 files changed, 1

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-17 Thread Tom Rini
On Fri, Jun 14, 2024 at 04:13:54PM +0200, Stefan Roese wrote: > On 6/12/24 18:50, Tom Rini wrote: > > On Wed, Jun 12, 2024 at 05:13:37PM +0100, Jiaxun Yang wrote: > > > > > > > > > 在2024年6月12日六月 下午5:00,Tom Rini写道: > > > [...] > > > > > configs/octeon_nic23_defconfig | 1 - > > > > > 2 files ch

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-14 Thread Stefan Roese
On 6/12/24 18:50, Tom Rini wrote: On Wed, Jun 12, 2024 at 05:13:37PM +0100, Jiaxun Yang wrote: 在2024年6月12日六月 下午5:00,Tom Rini写道: [...] configs/octeon_nic23_defconfig | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) This seems similar to: https://patchwork.ozlabs.org/project/uboot/pa

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-12 Thread Tom Rini
On Wed, Jun 12, 2024 at 05:13:37PM +0100, Jiaxun Yang wrote: > > > 在2024年6月12日六月 下午5:00,Tom Rini写道: > [...] > >> configs/octeon_nic23_defconfig | 1 - > >> 2 files changed, 1 insertion(+), 2 deletions(-) > > > > This seems similar to: > > https://patchwork.ozlabs.org/project/uboot/patch/20240524

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-12 Thread Jiaxun Yang
在2024年6月12日六月 下午5:00,Tom Rini写道: [...] >> configs/octeon_nic23_defconfig | 1 - >> 2 files changed, 1 insertion(+), 2 deletions(-) > > This seems similar to: > https://patchwork.ozlabs.org/project/uboot/patch/20240524210817.1953298-1-rasmus.villem...@prevas.dk/ > > at least for CI. And for outs

Re: [PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-12 Thread Tom Rini
On Tue, Jun 11, 2024 at 10:04:11PM +0100, Jiaxun Yang wrote: > The default value CYCLIC_MAX_CPU_TIME_US was 1000, which is > a little bit too low for slower hardware and sandbox. > > On my MIPS Boston FPGA board with interaptiv CPU, wdt_cyclic > can easily take 3200 us to run. > > On azure pipel

[PATCH 12/20] cyclic: Rise default CYCLIC_MAX_CPU_TIME_US to 5000

2024-06-11 Thread Jiaxun Yang
The default value CYCLIC_MAX_CPU_TIME_US was 1000, which is a little bit too low for slower hardware and sandbox. On my MIPS Boston FPGA board with interaptiv CPU, wdt_cyclic can easily take 3200 us to run. On azure pipeline sandbox_clang, wdt_cyclic some times goes beyond 1300 us. Raise default