linux-kernel - Fwd: [PATCH] x86/apic: fix two slight indenting

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Mon, 30 Jul 2018 11:35:06 +0300
From:   Andy Shevchenko <andy.shevchenko@...il.com>
To:     Ingo Molnar <mingo@...hat.com>, "H. Peter Anvin" <hpa@...or.com>,
        "maintainer:X86 ARCHITECTURE (32-BIT AND 64-BIT)" <x86@...nel.org>,
        Thomas Gleixner <tglx@...utronix.de>
Cc:     Steven Rostedt <rostedt@...dmis.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Fwd: [PATCH] x86/apic: fix two slight indenting

Restore Cc list (it seems I sent it privately)

---------- Forwarded message ----------
From: Andy Shevchenko <andy.shevchenko@...il.com>
Date: Fri, Jul 27, 2018 at 1:33 PM
Subject: Re: [PATCH] x86/apic: fix two slight indenting
To: Yi Wang <wang.yi59@....com.cn>


On Fri, Jul 27, 2018 at 9:15 AM, Yi Wang <wang.yi59@....com.cn> wrote:
> There are two inconsistent indenting in calibrate_APIC_clock()
> and activate_managed(). Remove the surplus TAB.

>         if (levt->features & CLOCK_EVT_FEAT_DUMMY) {
>                 pr_warning("APIC timer disabled due to verification failure\n");
> -                       return -1;
> +               return -1;
>         }

Slightly better fix is to replace with appropriate ERRNO constant at
the same time.

To avoid bigger change for now, just change this one and IIUC one more
above in the same function.

--
With Best Regards,
Andy Shevchenko


-- 
With Best Regards,
Andy Shevchenko