Re: [PATCH] ext4: add cond_resched() to ext4_mb_init_backend()

[Khazhismel Kumykov]

On Thu, Apr 18, 2019 at 4:53 AM Jan Kara <jack@xxxxxxx> wrote:
>
> On Mon 15-04-19 19:59:34, Khazhismel Kumykov wrote:
> > on non-preempt kernels for filesystems with large number of groups we
> > may take a long time (>50 ticks) initializing all the groups.
> >
> > Signed-off-by: Khazhismel Kumykov <khazhy@xxxxxxxxxx>
>
> Thanks for the patch. I'm not opposed to this but I'm just wondering: Has
> this caused any real issues to you? Or how have you come across this
> particular loop? Because I'd think we have more similar loops in ext4
> codebase...
>
>                                                                 Honza
>

We have some instrumentation to warn for these longer periods without
scheduling, which admittedly does trigger relatively frequently from
various places. This particular loop just so happens to be during
system startup where it caught my eye.

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature