Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add systemd for binding dpdk port after reboot #212

Merged
merged 3 commits into from
Aug 6, 2018

Conversation

sufuf3
Copy link
Contributor

@sufuf3 sufuf3 commented Aug 1, 2018

No description provided.

@codecov-io
Copy link

codecov-io commented Aug 1, 2018

Codecov Report

Merging #212 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #212   +/-   ##
========================================
  Coverage    78.16%   78.16%           
========================================
  Files           49       49           
  Lines         2469     2469           
========================================
  Hits          1930     1930           
  Misses         416      416           
  Partials       123      123

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d65e8c8...96a70be. Read the comment docs.

@John-Lin
Copy link
Contributor

John-Lin commented Aug 1, 2018

加一下註解

@sufuf3 sufuf3 force-pushed the sufuf3/update-vagrant branch from 30d4cff to e88e4cc Compare August 1, 2018 09:41
@sufuf3
Copy link
Contributor Author

sufuf3 commented Aug 1, 2018

會再繼續研究用 systemd
好了的話,會再送另外的 PR 。

sudo ${DPDK_DIR}/usertools/dpdk-devbind.py --bind=igb_uio ${NET_IF_NAME}
# Note that this NIC setup will binding dpdk port after reboot
echo 'export IFDRV=igb_uio' | sudo tee -a /root/.bashrc
echo 'export IFnum=enp0s8' | sudo tee -a /root/.bashrc
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IFDRV IFnum 其實沒用到?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

喔 對的
這就拿掉他們

@sufuf3 sufuf3 force-pushed the sufuf3/update-vagrant branch from f3ca6d6 to 3702032 Compare August 1, 2018 17:09
@sufuf3 sufuf3 changed the title Add rc.local config for binding dpdk port after reboot Add systemd for binding dpdk port after reboot Aug 1, 2018
@@ -0,0 +1,11 @@
[Unit]
Description=Binding dpdk port after reboot
After=network-pre.target multi-user.target
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

multi-user.target 為什麼要在 dpdkbind.service 之後呀?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

恩~這邊沒有特別研究。
不過我把 network-pre.target 拿掉了
因為原本的寫法, reboot 後,不能 vagrant ssh 。拿掉就可以了~
然後重新進到 VM 中,也有看到 dpdk 的 port 有綁定了。

@sufuf3 sufuf3 force-pushed the sufuf3/update-vagrant branch from 3702032 to e5015bc Compare August 2, 2018 00:10
sufuf3 added 2 commits August 6, 2018 14:02
Remove rc.local config and use dpdkbind systemd to bind dpdk port after reboot
@sufuf3 sufuf3 force-pushed the sufuf3/update-vagrant branch 3 times, most recently from 02d5dc8 to 9c63b2a Compare August 6, 2018 08:38
@sufuf3
Copy link
Contributor Author

sufuf3 commented Aug 6, 2018

這個 PR 測過是 OK 的。
重開機後 (vagrant halt) 也綁好了。

@sufuf3 sufuf3 force-pushed the sufuf3/update-vagrant branch 2 times, most recently from 021f646 to f7b5ab3 Compare August 6, 2018 09:04
sudo ifconfig ${NET_IF_NAME} down
sudo ${DPDK_DIR}/usertools/dpdk-devbind.py --bind=igb_uio ${NET_IF_NAME}
# Note that this NIC setup will binding dpdk port after reboot
echo 'NET_IF_NAME=enp0s18' | sudo tee -a /etc/default/dpdkbind
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

echo 'NET_IF_NAME='${NET_IF_NAME} | sudo tee -a /etc/default/dpdkbind

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@sufuf3 sufuf3 force-pushed the sufuf3/update-vagrant branch from f7b5ab3 to 96a70be Compare August 6, 2018 09:11
sufuf3 added a commit to linkernetworks/vortex-installer that referenced this pull request Aug 6, 2018
- Add dpdkbind.service for systemd
- Update kernel module from uio to igb_uio
cc linkernetworks/vortex#212
sufuf3 added a commit to linkernetworks/vortex-installer that referenced this pull request Aug 6, 2018
- Add dpdkbind.service for systemd
- Update kernel module from uio to igb_uio
cc linkernetworks/vortex#212
sufuf3 added a commit to linkernetworks/vortex-installer that referenced this pull request Aug 6, 2018
- Add dpdkbind.service for systemd
- Update kernel module from uio to igb_uio
cc linkernetworks/vortex#212
@John-Lin John-Lin merged commit 35860c4 into develop Aug 6, 2018
@John-Lin John-Lin deleted the sufuf3/update-vagrant branch August 6, 2018 17:27
sufuf3 added a commit to linkernetworks/vortex-installer that referenced this pull request Aug 7, 2018
- Add dpdkbind.service for systemd
- Update kernel module from uio to igb_uio
cc linkernetworks/vortex#212
sufuf3 added a commit to linkernetworks/vortex-installer that referenced this pull request Aug 7, 2018
- Add dpdkbind.service for systemd
- Update kernel module from uio to igb_uio
cc linkernetworks/vortex#212
sufuf3 added a commit to linkernetworks/vortex-installer that referenced this pull request Aug 7, 2018
- Add dpdkbind.service for systemd
- Update kernel module from uio to igb_uio
cc linkernetworks/vortex#212
sufuf3 added a commit to linkernetworks/vortex-installer that referenced this pull request Aug 7, 2018
- Add dpdkbind.service for systemd
- Update kernel module from uio to igb_uio
cc linkernetworks/vortex#212
sufuf3 added a commit to linkernetworks/vortex-installer that referenced this pull request Aug 7, 2018
- Add dpdkbind.service for systemd
- Update kernel module from uio to igb_uio
cc linkernetworks/vortex#212
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.

4 participants