Skip to content

Devel#56

Merged
wesleywesley merged 6 commits intomasterfrom
devel
Jan 8, 2019
Merged

Devel#56
wesleywesley merged 6 commits intomasterfrom
devel

Conversation

@wesleywesley
Copy link
Copy Markdown
Contributor

No description provided.

wesleywesley and others added 6 commits December 14, 2018 17:49
The reinit of shared MW when receive FORCE_LINK_DOWN message is
required for non-crosslink case, because the LUT settings for shared
MW of peer is reset after reboot.
But in crosslink case, the LUT setttings in the virtual partition
will not be impact by the peer's reboot, therefore it is unnecessary
to reinit it.

Signed-off-by: Wesley Sheng <wesley.sheng@microchip.com>
Remove ternary operator which doesn't make sense

This commit was response of feedback from Bjorn Helgaas
in the procedure of upstream.
https://lore.kernel.org/patchwork/patch/1023094/

Signed-off-by: Wesley Sheng <wesley.sheng@microchip.com>
There exists a corner case: MRPC completion event happened
immediately after other non-MRPC event, moreover exactly right
after MRPC event occurrence check but before the mask event loop
reach to its header register in event ISR, the MRPC event would
be masked unintentionally. Since there's no chance to enable it
again except module remove and install, all the following MRPC
execution completion check will be defer to timeout.

Fix this bug by bypass the mask operation same as to LINK event
in event ISR.

This corner case was exposed by concurrently running fw-update
and event-wait commands of switchtec-user. When it happened,
firmware update speed reduce evidently.

Signed-off-by: Wesley Sheng <wesley.sheng@microchip.com>
switchtec: Fix unintended mask of MRPC event
ntb_hw_switchtec: remove unnecessary shared MW reinit in crosslink case
switchtec: Remove unnecessary ternary operator
@wesleywesley
Copy link
Copy Markdown
Contributor Author

merge to master for release

@wesleywesley wesleywesley merged commit ba65e6c into master Jan 8, 2019
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.

1 participant