linux-kernel - Re: [PATCH] mux: include compiler.h from mux/consumer.h

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

Message-ID: <e5d7b2bc-39b0-638d-f4bf-6bf3d5038628@axentia.se>
Date:   Mon, 31 Jul 2017 18:28:57 +0200
From:   Peter Rosin <peda@...ntia.se>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc:     Ulrich Hecht <ulrich.hecht+renesas@...il.com>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] mux: include compiler.h from mux/consumer.h

On 2017-07-31 18:00, Greg Kroah-Hartman wrote:
> On Mon, Jul 31, 2017 at 12:04:35PM +0200, Peter Rosin wrote:
>> From: Ulrich Hecht <ulrich.hecht+renesas@...il.com>
>>
>> Required for __must_check.
>>
>> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@...il.com>
>> Signed-off-by: Peter Rosin <peda@...ntia.se>
>> ---
>>  include/linux/mux/consumer.h | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> Hi Greg,
>>
>> Please pick this up for 4.14.
> 
> What commit is this patch fixing?

I don't think it's needed for any existing user, I at least do not have any
report about that. So, I don't think it's needed for stable.

Ulrich, the way I understood this, it was needed for your serdev series [1],
and not for any existing usage. Right?

If I misunderstood, it would be

Fixes: a3b02a9c6591 ("mux: minimal mux subsystem")

Cheers,
Peter

[1] https://www.spinics.net/lists/linux-i2c/msg30475.html