Skip to content

[Sensor] Modify the coretemp data format of 7260#4256

Merged
kevinskwang merged 2 commits intosonic-net:masterfrom
kevinskwang:platform_sensors
Sep 15, 2021
Merged

[Sensor] Modify the coretemp data format of 7260#4256
kevinskwang merged 2 commits intosonic-net:masterfrom
kevinskwang:platform_sensors

Conversation

@kevinskwang
Copy link
Contributor

@kevinskwang kevinskwang commented Sep 14, 2021

What is the motivation for this PR?
The coretemp data format of 7260 has been changed to "Package id" since 202012 image.

How did you do it?
Modify the coretemp data format of 7260 from "Physical id" to "Package id".

How did you verify/test it?
Run the sensor test.

Signed-off-by: Kevin(Shengkai) Wang [email protected]

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911

Approach

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

What is the motivation for this PR?
The coretemp data format of 7260 has been changed to "Package id" since 202012 image.

How did you do it?
Modify the coretemp data format of 7260 from "Physical id" to "Package id".

How did you verify/test it?
Run the sensor test.

Signed-off-by: Kevin(Shengkai) Wang <[email protected]>
@kevinskwang kevinskwang requested a review from wangxin September 14, 2021 06:28
@kevinskwang kevinskwang requested a review from a team as a code owner September 14, 2021 06:28
@kevinskwang kevinskwang self-assigned this Sep 14, 2021
@kevinskwang
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@kevinskwang kevinskwang merged commit 95663bb into sonic-net:master Sep 15, 2021
vmittal-msft pushed a commit to vmittal-msft/sonic-mgmt that referenced this pull request Sep 28, 2021
What is the motivation for this PR?
The coretemp data format of 7260 has been changed to "Package id" since 202012 image.

How did you do it?
Modify the coretemp data format of 7260 from "Physical id" to "Package id".

How did you verify/test it?
Run the sensor test.

Signed-off-by: Kevin(Shengkai) Wang <[email protected]>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…lly (#25637)

#### Why I did it
src/sonic-swss
```
* dbe0768e - (HEAD -> master, origin/master, origin/HEAD) Fix the "Invalid BUFFER QUEUE" error (sonic-net#4224) (6 hours ago) [saksarav-nokia]
* d29d7ea1 - This patch is to fix a major bug in the for loop of the function AclRange::remove. (sonic-net#4256) (9 hours ago) [ashishalok-arista]
* 35288a65 - Fix performance issue in fpmsyncd for non-ZMQ configuration (sonic-net#4219) (9 hours ago) [venkit-nexthop]
* 1c384fb6 - Merge pull request sonic-net#4022 from ksravani-hcl/swss_1 (12 hours ago) [StephenWangGoogle]
|\ 
| failure_prs.log skip_prs.log 36d98d61 - Enable response pipeline for P4Orch. (22 hours ago) [mint570]
|/ 
* 63860664 - [build] Add docker-based build environment (sonic-net#3715) (22 hours ago) [Lawrence Lee]
* 26ec7561 - [buffermgrd] Wait for lossless buffer profile to be applied to SAI (sonic-net#4154) (23 hours ago) [ganglv]
* 599eae77 - [tunnel] use warning message when setting a create-only attribute (sonic-net#4212) (27 hours ago) [Yakiv Huryk]
* ebf8c73f - [orchagent] Fix getopt string for -R option (ring_thread_enabled regression) (sonic-net#4207) (27 hours ago) [Chiranjeevi U - MapleLabs]
* aa081017 - Avoid using an uninitialized source MAC address (sonic-net#4201) (28 hours ago) [Andriy Yurkiv]
* 2a258c2 - Added mux neighbor_mode support with prefix-route (sonic-net#4152) (31 hours ago) [manamand2020]
* 9616cd52 - [ci] Install redis from Debian (sonic-net#4241) (34 hours ago) [Saikrishna Arcot]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants