[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#717689: linux: please review and merge m68k patch

Bastian Blank dixit:

>On Tue, Jul 23, 2013 at 08:42:04PM +0000, Thorsten Glaser wrote:
>Should be configured in the top config.

They’re =y there. I put them into modules to save space.

>> +# CONFIG_EXT2_FS_SECURITY is not set
>> +# CONFIG_EXT3_FS_SECURITY is not set

Saving space since CONFIG_SECURITY=n.

>You could still stick to this option.

The debian/config/config doesn’t use it and builds
separate ext2/ext3/ext4 modules instead, so I did
that too.

>Why here?

Because debian/config/config didn’t contain them.
But they could be moved there, yes – but I didn’t
want to touch !m68k space.

>Why override?

Saving some more space… I will retract this line
if you say so.

>> +## choice: Preemption Model
>> +# CONFIG_PREEMPT is not set
>> +## end choice
>Isn't this already set?


>> +##
>> +## file: unknown
>> +##
>You want to take a look at this options.

Right… I did have that planned but forgot.

I looked at them all, and the entire block can go,
they don’t appear in this century’s kernel source ;-)

17:08⎜«Vutral» früher gabs keine packenden smartphones und so
17:08⎜«Vutral» heute gibts frauen die sind facebooksüchtig
17:10⎜«Vutral» aber auch traurig; früher warst du als nerd voll am arsch
17:10⎜«Vutral» heute bist du als nerd der einzige der wirklich damit klarkommt

Reply to: