linux-kernel - Re:Re: [PATCH] hrtimer.c : Remove redundant thread state settings

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <257fc950.14ca.18cfc0e0a5a.Coremail.sensor1010@163.com>
Date: Fri, 12 Jan 2024 13:03:47 +0800 (CST)
From: lizhe  <sensor1010@....com>
To: "Thomas Gleixner" <tglx@...utronix.de>
Cc: ilpo.jarvinen@...ux.intel.com, linux-kernel@...r.kernel.org
Subject: Re:Re: [PATCH] hrtimer.c : Remove redundant thread state settings







Hi, 
    Please review this patch, It does not check the condition when executing the scheduler() function.
       
                                                                                                       Lizhe
                                                                                           thanks











At 2024-01-12 00:44:20, "Thomas Gleixner" <tglx@...utronix.de> wrote:
>On Wed, Jan 10 2024 at 06:13, Lizhe wrote:
>> After a thread is awakened, its state is already task_running
>
>That's correct, but please look at hrtimer_wakeup() and the conditional
>schedule() invocation in schedule_hrtimeout_range_clock(). You break the
>guarantee that this function returns with task state == TASK_RUNNING.
>
>Thanks,
>
>        tglx

Content of type "text/html" skipped

Download attachment "0001-drivers-dm-crypt.c-remove-redundant-state-settings-a.patch" of type "application/octet-stream" (775 bytes)