xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: "Mathieu Gagné" <mgagne@iweb.com>
To: xen-devel@lists.xensource.com
Cc: ian.jackson@eu.citrix.com, stefano.stabellini@eu.citrix.com
Subject: [PATCH 0 of 4 v3 RESEND] xl: add support for vif rate limiting
Date: Tue, 24 Apr 2012 11:07:09 -0400	[thread overview]
Message-ID: <patchbomb.1335280029@mgagne.users.dev.iweb.com> (raw)

Hi,

This patch series implements the required plumbering for vif rate limiting in libxl/xl.

The first patch (already applied) fixes trivial indentation issues and introduces no functional changes.

The second patch (already applied) implements dry-run for `xl network-attach` for debugging and testing purposes.

The third patch adds the required plumbering in libxl/xl to add vif rate limiting support. It uses the `rate` option syntax from xm/xend, ensuring full backward compatiblity.

The final patch adds the "check-xl-vif-parse" test script which tests various `rate` syntax.

Changes in v3:
- Move xlu_cfg_init from parse_vif_rate to main_networkattach
- Add xlu_cfg_destroy to main_networkattach
- Use %"PRIuXX" instead of %llu and %lu

Changes in v2:
- Don't default to unlimited in case of overflow/underflow or invalid syntax in rate
- Add error handling
- Remove use of matches in regex which weren't used anyway
- Add docs about the syntax
- Add note in docs explaining that limitations are netback implementation specific
- Modify test cases according to new behavior

--
Mathieu

             reply	other threads:[~2012-04-24 15:07 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-24 15:07 Mathieu Gagné [this message]
2012-04-24 15:07 ` [PATCH 1 of 4 v3 RESEND] xl: cleanup indentation Mathieu Gagné
2012-04-24 15:07 ` [PATCH 2 of 4 v3 RESEND] xl: xl network-attach -N (dry run) option Mathieu Gagné
2012-04-24 15:07 ` [PATCH 3 of 4 v3 RESEND] xl: add support for vif rate limiting Mathieu Gagné
2012-04-24 15:07 ` [PATCH 4 of 4 v3 RESEND] xl: add "check-xl-vif-parse" test script Mathieu Gagné

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=patchbomb.1335280029@mgagne.users.dev.iweb.com \
    --to=mgagne@iweb.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xensource.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).