[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] How to test my patch before I post it to public?
On 30/04/12 15:25, wang zhihao wrote:
> Hi George:
> I don't have a patch now, but may have one in future. : P
> Ok, I will try to make a patch and then ask this question : )
Well, as I said, the general idea is, "try to break it". If you add a
new option, for instance, check to make sure that it works right:
* If you put in a reasonable value
* If you put in an unreasonably high value
* If you put in a negative number
* If you don't specify the option at all
* If you put a string instead of a number
> Wang zhihao
> 在 2012-4-30，下午10:19， George Dunlap 写道：
>> On Mon, Apr 30, 2012 at 3:12 PM, wang zhihao <accept.acm@xxxxxxxxx> wrote:
>>> Hi George:
>>> Thanks for your guides, But I don't know what "combination" means. Could
>>> you tell me more about it?
>> I can't really give you an example without an example to work with.
>> :-) What does your patch do?
>>> Best Regards
>>> Wang Zhihao
>>> 在 2012-4-30，下午5:16， George Dunlap 写道：
>>> On Sun, Apr 29, 2012 at 4:21 PM, wang zhihao <accept.acm@xxxxxxxxx> wrote:
>>> Hi , All
>>> I 'm a green hand and interested in open source development. I have a
>>> general question "How to test my patch before I post it to public?" Hope you
>>> guys give me some suggestions. : )
>>> Firstly, I can re-compile the code, to assure no syntax error. However,I
>>> don't know how to test my patch's function is right or not. Some software
>>> requires unit test for each function. Is there anything similar in xen
>>> There is no unit testing for Xen. What you need to test really
>>> depends on what your patch is doing. The main goal is to exercise the
>>> code you've just added or changed: try to put it in different
>>> combinations to make sure that it works as you expect. Try to break
>>> it, really. :-)
>>> If you're just tweaking a simple option in the xl config file, then
>>> you need to test a few different combinations to make sure that all
>>> the reasonable combinations work. If you're changing the locks in the
>>> memory management code in the hypervisor, then you need to run a bunch
>>> of benchmarks that exercise that code, probably for several hours.
>>> If you post your patch to the list (with an appropriate description)
>>> we may be able to give you some suggestions.
>>> Xen-devel mailing list
Xen-devel mailing list