[nvmewin] OFA driver patch release procedure

Kwok Kong Kwok.Kong at pmcs.com
Thu Dec 12 17:12:57 PST 2013


This is a reminder of the procedure to submit  patch:

Process:

- Submitter needs to base their changes on the latest (and re-base/re-test prior to sending their patch).
- They send the patch to the email list "nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>".
- Some review will happen over the reflector, the maintainer will send a message out that the db is locked
  when they're ready to apply the patch which will be once at least one member from each company
  on the review panel has approved (can be via email or con call if needed).  Once the patch is applied,
  the maintainer will send an email out.

Patch contents:

- Code changes, short summary for SVN log, more verbose write up for release notes, confirmation of what Windows platforms had been tested.
- Patch must follow the coding guidelines as attached.
- All source files should be zipped up into a .zip file with password enabled. The zip file name should properly describe the main changes of the patch

Reviews:
- Patches submitted by anyone, email to distribution list "nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>".
- Patch submission should include time sensitivity/expectations
- Patch submission should include justification for the patch (what value will it add, and are
  tradeoffs what are they and why would we want to take a hit).  If multiple implementation
  options were considered, what data/reasoning was behind the implementation choice.
- At a minimum reviews need to be completed by Intel, PMC and LSI representative
- Reviews include compliance with coding guidelines (in SVN) as well as logic

Testing:
- All patches and release candidates required, at a minimum, the following;
                - 1 hour of data integrity testing using sdstress (Microsoft Tool)
                - 1 hour of heavy stress testing using IOMETER covering, at least, 512B, 4KB and 128KB ranging
                  from 1 OIO to 64 OIO both sequential and random
                - Quick and slow format of both MBR and GPT partitioning
                - Microsoft SCSI Compliance, no failures except (warnings OK)
                - Additional testing with other tools is encouraged
                - Occurs in all supported OSs for the release
                                - 64-bit, Windows 7, 8.0, 8,1, server 2008R2 and 2012
                                - 32-bit, Windows 7, 8.0
                - Minimum test platform is latest QEMU.  Those with their HW should test on it as well.
                 QEMU is available at https://github.com/nvmeqemu/nvmeqemu



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20131213/8c8d7547/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Coding_guidelines.docx
Type: application/vnd.openxmlformats-officedocument.wordprocessingml.document
Size: 76721 bytes
Desc: Coding_guidelines.docx
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20131213/8c8d7547/attachment.docx>


More information about the nvmewin mailing list