linux-kernel - Re: post 2.6.34 bug: new code enabled by default

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

Message-ID: <4BFC9FA5.3040201@redhat.com>
Date:	Wed, 26 May 2010 01:12:21 -0300
From:	Mauro Carvalho Chehab <mchehab@...hat.com>
To:	Stefan Richter <stefanr@...6.in-berlin.de>
CC:	linux-media@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: post 2.6.34 bug: new code enabled by default

Stefan Richter wrote:
> $ make oldconfig
> ...
>   * Multimedia drivers
>   *
>   Compile Remote Controller keymap modules (RC_MAP) [M/n/?] (NEW) n
>   Enable IR raw decoder for the NEC protocol (IR_NEC_DECODER) [M/n/?] (NEW) n
>   Enable IR raw decoder for the RC-5 protocol (IR_RC5_DECODER) [M/n/?] (NEW) n
>   Enable IR raw decoder for the RC6 protocol (IR_RC6_DECODER) [M/n/?] (NEW) n
>   Enable IR raw decoder for the JVC protocol (IR_JVC_DECODER) [M/n/?] (NEW) n
>   Enable IR raw decoder for the Sony protocol (IR_SONY_DECODER) [M/n/?] (NEW) n
> 
> Please leave the default of new options at N.
> 
> (Unless this were a special case of new options that replaced older
> options and need to be migrated to 'on' per default in make oldconfig.
> I think this is not the case here.)

This is the case here. Before the RC subsystem, the decoding for NEC and RC-5
were done inside ir-core (at ir-functions). Also, all the keymap entries (RC_MAP)
were compiled in-kernel.

The intention is to move the RC_MAP keytables to userspace, by default, but there
are still some work to be done at the userspace application before we can do it.

So, without RC_MAP, IR_NEC_DECODER and IR_RC5_DECODER, the input driver will be 
compiled, but the IR support may not actually work, if the board needs IR raw
support.

That's said, the support for RC-6, JVC and Sony protocols (IR_RC6_DECODER, 
IR_JVC_DECODER and IR_SONY_DECODER) are new. Yet, even on this case, it may make
sense to keep the default to Y, or to add a logic to auto-select the IR decoder 
protocols that are needed in order to support the bundled IR found at the compiled
drivers.

-- 

Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/