linux-kernel - trace name copy/paste in native_write

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>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20160603103250.GC2265@work-vm>
Date:	Fri, 3 Jun 2016 11:32:51 +0100
From:	"Dr. David Alan Gilbert" <dgilbert@...hat.com>
To:	ak@...ux.intel.com
Cc:	linux-kernel@...r.kernel.org
Subject: trace name copy/paste in native_write_msr ?

Hi Andi,
  In arch/x86/include/asm/msr.h native_write_msr(_safe)
you added a trace test in 7f47d8cc, should the
tracepoint_active's be testing for __tracepoint_write_msr ?

/* Can be uninlined because referenced by paravirt */
notrace static inline void native_write_msr(unsigned int msr,
                                            unsigned low, unsigned high)
{
        asm volatile("1: wrmsr\n"
                     "2:\n"
                     _ASM_EXTABLE_HANDLE(1b, 2b, ex_handler_wrmsr_unsafe)
                     : : "c" (msr), "a"(low), "d" (high) : "memory");
        if (msr_tracepoint_active(__tracepoint_read_msr))   <---------------
                do_trace_write_msr(msr, ((u64)high << 32 | low), 0);
}

/* Can be uninlined because referenced by paravirt */
notrace static inline int native_write_msr_safe(unsigned int msr,
                                        unsigned low, unsigned high)
{
        int err; 
        asm volatile("2: wrmsr ; xor %[err],%[err]\n"
                     "1:\n\t"
                     ".section .fixup,\"ax\"\n\t"
                     "3:  mov %[fault],%[err] ; jmp 1b\n\t"
                     ".previous\n\t"
                     _ASM_EXTABLE(2b, 3b)
                     : [err] "=a" (err)
                     : "c" (msr), "0" (low), "d" (high),
                       [fault] "i" (-EIO)
                     : "memory");
        if (msr_tracepoint_active(__tracepoint_read_msr))   <---------------
                do_trace_write_msr(msr, ((u64)high << 32 | low), err);
        return err;
}

Dave

--
Dr. David Alan Gilbert / dgilbert@...hat.com / Manchester, UK