Page MenuHomeFreeBSD

artifical frame should be 3 in arm platform.
ClosedPublic

Authored by howard0su_gmail.com on Nov 30 2015, 11:50 AM.

Details

Summary

Original value is 10 is incorrect.

Test Plan

dtrace -n 'tick-1s { stack(); }'

check the output

Diff Detail

Repository
rS FreeBSD src repository
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 1376
Build 1381: arc lint + arc unit

Event Timeline

howard0su_gmail.com retitled this revision from to artifical frame should be 3 in arm platform..
howard0su_gmail.com updated this object.
howard0su_gmail.com edited the test plan for this revision. (Show Details)
howard0su_gmail.com edited edge metadata.

Add sysctl to tune artifical frame

I think it make sense to make it tuneable.

Make logic of arframes more explict.

stas accepted this revision.Dec 2 2015, 3:11 AM
stas edited edge metadata.

Looks good besides the SYSCTL_INT vs UINT issue.

sys/cddl/dev/profile/profile.c
238

SYSCTL_INT?

This revision is now accepted and ready to land.Dec 2 2015, 3:11 AM
howard0su_gmail.com edited edge metadata.

Address review comments. aframes is INT

This revision now requires review to proceed.Dec 2 2015, 3:30 AM
stas accepted this revision.Dec 2 2015, 3:33 AM
stas edited edge metadata.
This revision is now accepted and ready to land.Dec 2 2015, 3:33 AM
This revision was automatically updated to reflect the committed changes.