Page MenuHomeFreeBSD
Differential D4315

artifical frame should be 3 in arm platform.
ClosedPublic
Actions

Authored by howard0su_gmail.com on Nov 30 2015, 11:50 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 11, 1:44 PM
Unknown Object (File)
Mon, Nov 10, 4:31 PM
Unknown Object (File)
Sun, Nov 9, 2:00 AM
Unknown Object (File)
Wed, Nov 5, 7:40 AM
Unknown Object (File)
Mon, Nov 3, 2:19 AM
Unknown Object (File)
Sun, Nov 2, 10:29 AM
Unknown Object (File)
Sun, Nov 2, 1:49 AM
Unknown Object (File)
Sun, Nov 2, 1:49 AM
View All Files
Subscribers
imp

Details

Reviewers
stas
andrew
rwatson
Group Reviewers
Contributor Reviewers (ports)
Commits
rS291855: Allow the artificial profile frames to be adjusted as needed by the user.
Summary

Original value is 10 is incorrect.

Test Plan

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

check the output

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

howard0su_gmail.com updated this revision to Diff 10592.Nov 30 2015, 11:50 AM
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)
Herald added a reviewer: Contributor Reviewers (ports). · View Herald TranscriptNov 30 2015, 11:50 AM
Herald added a subscriber: imp. · View Herald Transcript
howard0su_gmail.com updated this revision to Diff 10655.Dec 1 2015, 11:43 PM
howard0su_gmail.com edited edge metadata.

Add sysctl to tune artifical frame

howard0su_gmail.com added reviewers: rwatson, andrew.Dec 1 2015, 11:46 PM

I think it make sense to make it tuneable.

howard0su_gmail.com updated this revision to Diff 10656.Dec 2 2015, 1:00 AM

Make logic of arframes more explict.

howard0su_gmail.com added a reviewer: stas.Dec 2 2015, 2:55 AM
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 ↗(On Diff #10656)

SYSCTL_INT?

This revision is now accepted and ready to land.Dec 2 2015, 3:11 AM
howard0su_gmail.com updated this revision to Diff 10662.Dec 2 2015, 3:30 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
Closed by commit rS291855: Allow the artificial profile frames to be adjusted as needed by the user. (authored by andrew). · Explain WhyDec 5 2015, 10:00 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
cddl/
dev/
profile/
15 lines

Diff 10771

View Options

head/sys/cddl/dev/profile/profile.c

Loading...