Project

General

Profile

Actions

Feature #7907

closed

Add model for macro DECLARE_COMPLETION_ONSTACK (rule specification linux:kernel:sched:completion)

Added by Evgeny Novikov over 7 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Immediate
Assignee:
Category:
Requirement specifications
Target version:
-
Start date:
01/25/2017
Due date:
% Done:

0%

Estimated time:
Published in build:

Description

Likely there are many false alarms due to lack of this model on every version of the Linux kernel including 3.14. In addition corresponding test cases should be added (at least safe shouldn't be detected in case of absence of the model).


Related issues 1 (1 open0 closed)

Related to Klever - Feature #7911: Add test cases for linux:kernel:locking:* rule specifications when CONFIG_LOCKDEP is unsetNew01/25/2017

Actions
Actions #1

Updated by Vitaly Mordan over 7 years ago

including 3.14

Incorrect. In >3.12 macro DECLARE_COMPLETION_ONSTACK() is processed as INIT_COMPLETION(), whereas for more earlier kernels those macros are different.

Actions #2

Updated by Vitaly Mordan over 7 years ago

For kernels ~2.6.33 fixed in branch fix_148_2.6.33.

Actions #3

Updated by Evgeny Novikov over 7 years ago

I didn't noticed any changes in corresponding API. What differs and matters is a configuration. Before your fix there is just a model that works when CONFIG_LOCKDEP is true (which is usually the case for us). But when it is false, there isn't a model.

So first macro DECLARE_COMPLETION_ONSTACK should be modeled as you did. Second there should be a test case with an appropriate configuration that detects this issue.

Actions #4

Updated by Vitaly Mordan over 7 years ago

Vitaly Mordan wrote:

Incorrect. In >3.12 macro DECLARE_COMPLETION_ONSTACK() is processed as INIT_COMPLETION()

My mistake, it is not connected to kernel version, but to its configuration.
With allmodconfig this issue cannot be revealed even in 2.6.33.

Actions #5

Updated by Vitaly Mordan over 7 years ago

Confirmed, that in branch fix_148_2.6.33 all related false positive have gone.

Actions #6

Updated by Evgeny Novikov over 7 years ago

There isn't still an appropriate test case that can reveal this issue. So I am going to add it myself.

Actions #7

Updated by Vitaly Mordan over 7 years ago

There should be new job with another configuration for that test case.

Actions #8

Updated by Evgeny Novikov over 7 years ago

Vitaly Mordan wrote:

There should be new job with another configuration for that test case.

Obviously, but you didn't add a corresponding sub-job.

Actions #9

Updated by Evgeny Novikov over 7 years ago

  • Status changed from New to Closed

I renamed the branch to fix-linux-kernel-sched-completion, add a required test case and test changes.

The most interesting thing was done within this issue is testing with using specific configuration files. This should be used within #7911.

Actions #10

Updated by Vitaly Mordan over 7 years ago

Adding 2 sub-jobs for one module/rule leads to #7917. This should be fixed.

Actions

Also available in: Atom PDF