Skip to content

[ARISTA] adding 7060_cs32s to eMMC exclusions#2982

Merged
lguohan merged 5 commits intosonic-net:masterfrom
criteo-forks:7060-emmc
Jul 2, 2019
Merged

[ARISTA] adding 7060_cs32s to eMMC exclusions#2982
lguohan merged 5 commits intosonic-net:masterfrom
criteo-forks:7060-emmc

Conversation

@MichelMoriniaux
Copy link
Copy Markdown
Contributor

Following PR 2774 we added the 7060-cx32s according to the guidelines of
PR 2780

This adds the 7060-cx32s to the list f devices that mount /var/log as a
tmpfs to mitigate eMMC wearout

Signed-off-by: Michel Moriniaux m.moriniaux@criteo.com

yoda2-small

Following PR 2774 we added the 7060-cx32s according to the guidelines of
PR 2780

This adds the 7060-cx32s to the list f devices that mount /var/log as a
tmpfs to mitigate eMMC wearout

Signed-off-by: Michel Moriniaux <m.moriniaux@criteo.com>
@lguohan lguohan requested a review from yxieca June 8, 2019 05:46
@yxieca
Copy link
Copy Markdown
Contributor

yxieca commented Jun 11, 2019

@MichelMoriniaux I remember we had a chat about this change on my PR. You mentioned that you could keep it internal. Did things change since then?

We don't really see 7060 having hard drive issue and we have concerns for losing logs across reboot. If this is not necessary, we would like to keep this change out.

Can you let us know:

  1. How often do you see this issue?
  2. What percentage of devices you have ran into this issue?

Thanks,
Ying

@MichelMoriniaux
Copy link
Copy Markdown
Contributor Author

MichelMoriniaux commented Jun 11, 2019

@Staphylo can you please comment, as you suggested to us to implement this
@yxieca we have not observed any problems at this time but mostly because we do not want to take the risk, so we implemented it as soon as it was brought up to our attention.

As usual we prefer not to keep code internal that can be of utility to the community, especially code that can prevent bricking devices. We had agreed to remove our PR in favor of yours and keep it internal until we amended your code to include this platform. As we are moving our patches to 201904, this came up.

As you remember our original implementation was not vendor specific but depended on detecting the presence of an eMMC kernel driver, we believe that eMMC will fail at one point ( even if it's after 10 years ) and that it is best practice to have logs be streamed out of the device as soon as possible.
If reboot logs need to be preserved then I am sure we can come up with specific syslog rules to write those logs to a persistent store.

@Staphylo
Copy link
Copy Markdown
Collaborator

We indeed recommended to add the 7060CX-32S in the list of products that log in a tmpfs rather than the actual flash device. Log writing can be IO intensive depending on the verbosity.
These flash devices were not intended to support that amount of write during the design phase.
eMMC are known to have a lower number of write cycles than SSD or eUSB devices. We use SLC/pSLC which gives more endurance but still. It is the reason why we suggested to avoid adding this unnecessary load on the flash as to make sure it doesn't die prematurely.

Following PR 2774 we added the 7060-cx32s according to the guidelines of
PR 2780

This adds the 7060-cx32s to the list f devices that mount /var/log as a
tmpfs to mitigate eMMC wearout

Signed-off-by: Michel Moriniaux <m.moriniaux@criteo.com>
@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Jun 29, 2019

retest vs please

@lguohan lguohan merged commit dc74724 into sonic-net:master Jul 2, 2019
mssonicbld added a commit that referenced this pull request Dec 21, 2023
…lly (#17593)

#### Why I did it
src/sonic-swss
```
* 5643db9a - (HEAD -> 202305, origin/202305) [muxorch] Fixing cache bug in updateRoute logic (#2982) (6 hours ago) [Nikola Dancejic]
```
#### 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants