[PATCH]netfront: need release all resources after adding and removing NICs time and again

Liuyingdong liuyingdong at huawei.com
Thu Dec 15 02:01:20 UTC 2016


Hello Roger,
	Thank you for the time and patience you devoted to reading my messages and e-mails. I really appreciate that.
	I can't use git send-email so I attach the patches directly. In the 0001 patch I introduce suspend_cancel mechanism for frontend devices and in the 0002 patch I release all resources after hot unplug NICs.

    Note: These two patches is on the base of the origin/release/10.2.0 branch and the 0002 patch is made after the 0001 patch.

-----邮件原件-----
发件人: freebsd xen [mailto:roger.pau at citrix.com] 
发送时间: 2016年12月13日 22:29
收件人: Liuyingdong
抄送: freebsd-xen at freebsd.org; Zhaojun (Euler); Suoben; Ouyangzhaowei (Charles)
主题: Re: [PATCH]netfront: need release all resources after adding and removing NICs time and again

On Tue, Dec 13, 2016 at 02:03:08PM +0800, liuyingdong wrote:
> Hello Roger,
>     I want to know how about this patch,Please let me know if you have any questions.
>     Thanks.

Hello,

Thanks for the patches! This one is looking fine, it's just that I'm a little bit busy at the moment, and there are some issues in HEAD related to Xen that I would like to fix before pushing anything new.

In any case, I see that you are sending the patches using Thunderbird, which is not ideal (MUAs tend to mangle patches). The preferred way for sending patches is using "git send-email"[0] directly. There are also several tutorials online that will help you setup git send-email correctly. If there's some reason why you can't use git send-email I would recommend that you also attach the patches directly to emails, that way they won't probably get mangled.

Roger.

[0] https://git-scm.com/docs/git-send-email
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-introduce-suspend-cancel-mechanism-for-frontend-devices.patch
Type: application/octet-stream
Size: 9427 bytes
Desc: 0001-introduce-suspend-cancel-mechanism-for-frontend-devices.patch
URL: <http://lists.freebsd.org/pipermail/freebsd-xen/attachments/20161215/fd29ee77/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-netfront-need-release-all-resources-after-hot-plug.patch
Type: application/octet-stream
Size: 1807 bytes
Desc: 0002-netfront-need-release-all-resources-after-hot-plug.patch
URL: <http://lists.freebsd.org/pipermail/freebsd-xen/attachments/20161215/fd29ee77/attachment-0001.obj>


More information about the freebsd-xen mailing list