OPW 2014 - Advanced Scheduling Parameters Project

OPW 2014 - Advanced Scheduling Parameters Project

[Uma Sharma]

[-- Attachment #1.1: Type: text/plain, Size: 168 bytes --]

I would like to work on the project Advanced Scheduling Parameters. I
studied about the Credit Scheduler. Kindly help me to start some work on it.

Regards,
Uma Sharma

[-- Attachment #1.2: Type: text/html, Size: 234 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: OPW 2014 - Advanced Scheduling Parameters Project

[George Dunlap]

On Mon, Oct 13, 2014 at 6:31 PM, Uma Sharma <uma.sharma523@gmail.com> wrote:
> I would like to work on the project Advanced Scheduling Parameters. I
> studied about the Credit Scheduler. Kindly help me to start some work on it.

According to tools/libxl.h:

 * IDL-generated libxl types should be used as follows: the user must
 * always call the "init" function before using a type, even if the
 * variable is simply being passed by reference as an out parameter
 * to a libxl function.  The user must always calls "dispose" exactly
 * once afterwards, to clean up, regardless of whether operations on
 * this object succeeded or failed.  See the xl code for examples.


However, the code in xl (namely tools/xl_cmdimpl.c) fails to do this
several times when calling sched_domain_get().  Please write a patch
to address this.

Coding style conventions are very important, so try to copy the style
of the surrounding code.

Extra credit: Find other IDL-generated libxl types in xl_cmdimpl.c
which aren't initialized properly (if there are any) and send patches
for those.

You can find information about submitting patches here:

http://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches

I would personally recommend using stgit to create patch series
(that's what I use - there's a link near the top of the page above on
using stgit), but you're free to use whatever you want, as long as the
patches apply properly. :-)

Feel free to contact me on IRC or over e-mail if you run into any
problems or have any questions.

Also, there are a million conventions about how to do patches and
coding that would be overwhelming either for us to write down or for
you to read.  Don't worry about trying to get it 100% perfect; just
dive in and send your patch when you think it's ready, and expect to
get lots of feedback on how it needs to be changed.  That's part of
the learning process.

 -George

Re: OPW 2014 - Advanced Scheduling Parameters Project

[George Dunlap]

On Mon, Oct 13, 2014 at 10:00 PM, George Dunlap
<George.Dunlap@eu.citrix.com> wrote:
> On Mon, Oct 13, 2014 at 6:31 PM, Uma Sharma <uma.sharma523@gmail.com> wrote:
>> I would like to work on the project Advanced Scheduling Parameters. I
>> studied about the Credit Scheduler. Kindly help me to start some work on it.
>
> According to tools/libxl.h:
>
>  * IDL-generated libxl types should be used as follows: the user must
>  * always call the "init" function before using a type, even if the
>  * variable is simply being passed by reference as an out parameter
>  * to a libxl function.  The user must always calls "dispose" exactly
>  * once afterwards, to clean up, regardless of whether operations on
>  * this object succeeded or failed.  See the xl code for examples.
>
>
> However, the code in xl (namely tools/xl_cmdimpl.c) fails to do this
> several times when calling sched_domain_get().  Please write a patch
> to address this.
>
> Coding style conventions are very important, so try to copy the style
> of the surrounding code.
>
> Extra credit: Find other IDL-generated libxl types in xl_cmdimpl.c
> which aren't initialized properly (if there are any) and send patches
> for those.
>
> You can find information about submitting patches here:
>
> http://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches
>
> I would personally recommend using stgit to create patch series
> (that's what I use - there's a link near the top of the page above on
> using stgit), but you're free to use whatever you want, as long as the
> patches apply properly. :-)
>
> Feel free to contact me on IRC or over e-mail if you run into any
> problems or have any questions.
>
> Also, there are a million conventions about how to do patches and
> coding that would be overwhelming either for us to write down or for
> you to read.  Don't worry about trying to get it 100% perfect; just
> dive in and send your patch when you think it's ready, and expect to
> get lots of feedback on how it needs to be changed.  That's part of
> the learning process.
>
>  -George

Re: OPW 2014 - Advanced Scheduling Parameters Project

[George Dunlap]

Sorry, sent this too early... meant to say, Meng, you're off the hook
for this one. :-)

 -George

On Mon, Oct 13, 2014 at 10:01 PM, George Dunlap
<George.Dunlap@eu.citrix.com> wrote:
> On Mon, Oct 13, 2014 at 10:00 PM, George Dunlap
> <George.Dunlap@eu.citrix.com> wrote:
>> On Mon, Oct 13, 2014 at 6:31 PM, Uma Sharma <uma.sharma523@gmail.com> wrote:
>>> I would like to work on the project Advanced Scheduling Parameters. I
>>> studied about the Credit Scheduler. Kindly help me to start some work on it.
>>
>> According to tools/libxl.h:
>>
>>  * IDL-generated libxl types should be used as follows: the user must
>>  * always call the "init" function before using a type, even if the
>>  * variable is simply being passed by reference as an out parameter
>>  * to a libxl function.  The user must always calls "dispose" exactly
>>  * once afterwards, to clean up, regardless of whether operations on
>>  * this object succeeded or failed.  See the xl code for examples.
>>
>>
>> However, the code in xl (namely tools/xl_cmdimpl.c) fails to do this
>> several times when calling sched_domain_get().  Please write a patch
>> to address this.
>>
>> Coding style conventions are very important, so try to copy the style
>> of the surrounding code.
>>
>> Extra credit: Find other IDL-generated libxl types in xl_cmdimpl.c
>> which aren't initialized properly (if there are any) and send patches
>> for those.
>>
>> You can find information about submitting patches here:
>>
>> http://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches
>>
>> I would personally recommend using stgit to create patch series
>> (that's what I use - there's a link near the top of the page above on
>> using stgit), but you're free to use whatever you want, as long as the
>> patches apply properly. :-)
>>
>> Feel free to contact me on IRC or over e-mail if you run into any
>> problems or have any questions.
>>
>> Also, there are a million conventions about how to do patches and
>> coding that would be overwhelming either for us to write down or for
>> you to read.  Don't worry about trying to get it 100% perfect; just
>> dive in and send your patch when you think it's ready, and expect to
>> get lots of feedback on how it needs to be changed.  That's part of
>> the learning process.
>>
>>  -George

Re: OPW 2014 - Advanced Scheduling Parameters Project

[Meng Xu]

[-- Attachment #1.1: Type: text/plain, Size: 2996 bytes --]

2014-10-13 17:09 GMT-04:00 George Dunlap <George.Dunlap@eu.citrix.com>:

> Sorry, sent this too early... meant to say, Meng, you're off the hook
> for this one. :-)
>
>
Thank you very much for letting me know. :-) Sorry about not sending them
now because I'm rushing for a paper deadline this month. :-(
I will send the patches about synchronizing the vcpu data in rtds scheduler
after the paper deadline (which is Oct. 20th).

Thanks,

Meng



>  -George
>
> On Mon, Oct 13, 2014 at 10:01 PM, George Dunlap
> <George.Dunlap@eu.citrix.com> wrote:
> > On Mon, Oct 13, 2014 at 10:00 PM, George Dunlap
> > <George.Dunlap@eu.citrix.com> wrote:
> >> On Mon, Oct 13, 2014 at 6:31 PM, Uma Sharma <uma.sharma523@gmail.com>
> wrote:
> >>> I would like to work on the project Advanced Scheduling Parameters. I
> >>> studied about the Credit Scheduler. Kindly help me to start some work
> on it.
> >>
> >> According to tools/libxl.h:
> >>
> >>  * IDL-generated libxl types should be used as follows: the user must
> >>  * always call the "init" function before using a type, even if the
> >>  * variable is simply being passed by reference as an out parameter
> >>  * to a libxl function.  The user must always calls "dispose" exactly
> >>  * once afterwards, to clean up, regardless of whether operations on
> >>  * this object succeeded or failed.  See the xl code for examples.
> >>
> >>
> >> However, the code in xl (namely tools/xl_cmdimpl.c) fails to do this
> >> several times when calling sched_domain_get().  Please write a patch
> >> to address this.
> >>
> >> Coding style conventions are very important, so try to copy the style
> >> of the surrounding code.
> >>
> >> Extra credit: Find other IDL-generated libxl types in xl_cmdimpl.c
> >> which aren't initialized properly (if there are any) and send patches
> >> for those.
> >>
> >> You can find information about submitting patches here:
> >>
> >> http://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches
> >>
> >> I would personally recommend using stgit to create patch series
> >> (that's what I use - there's a link near the top of the page above on
> >> using stgit), but you're free to use whatever you want, as long as the
> >> patches apply properly. :-)
> >>
> >> Feel free to contact me on IRC or over e-mail if you run into any
> >> problems or have any questions.
> >>
> >> Also, there are a million conventions about how to do patches and
> >> coding that would be overwhelming either for us to write down or for
> >> you to read.  Don't worry about trying to get it 100% perfect; just
> >> dive in and send your patch when you think it's ready, and expect to
> >> get lots of feedback on how it needs to be changed.  That's part of
> >> the learning process.
> >>
> >>  -George
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
>



-- 


-----------
Meng Xu
PhD Student in Computer and Information Science
University of Pennsylvania

[-- Attachment #1.2: Type: text/html, Size: 4911 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

patches for proper serialization in RTDS scheduler [Was: Re: OPW 2014 - Advanced Scheduling Parameters Project]

[Dario Faggioli]

[-- Attachment #1.1: Type: text/plain, Size: 1238 bytes --]

On Mon, 2014-10-13 at 17:17 -0400, Meng Xu wrote:

> 2014-10-13 17:09 GMT-04:00 George Dunlap
> <George.Dunlap@eu.citrix.com>:
>         Sorry, sent this too early... meant to say, Meng, you're off
>         the hook
>         for this one. :-)
>         
Indeed! :-P

But no worries, it happens! :-D

However...

> Thank you very much for letting me know. :-) Sorry about not sending
> them now because I'm rushing for a paper deadline this month. :-( 
> I will send the patches about synchronizing the vcpu data in rtds
> scheduler after the paper deadline (which is Oct. 20th). 
> 
I was just about to send you a 'ping' about the patches for curing the
serialization issue in RTDS.

Ideally, we'd have wanted them before RC1. Of course, the fact that we
accepted the scheduler as experimental makes things a bit less critical
and urgent, but I really would like to have them in before the release.

So, Meng, let us know :-)

regards,
Dario

-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: patches for proper serialization in RTDS scheduler [Was: Re: OPW 2014 - Advanced Scheduling Parameters Project]

[Meng Xu]

[-- Attachment #1.1: Type: text/plain, Size: 1672 bytes --]

Hi Dario,

2014-10-24 12:54 GMT-04:00 Dario Faggioli <dario.faggioli@citrix.com>:

> On Mon, 2014-10-13 at 17:17 -0400, Meng Xu wrote:
>
> > 2014-10-13 17:09 GMT-04:00 George Dunlap
> > <George.Dunlap@eu.citrix.com>:
> >         Sorry, sent this too early... meant to say, Meng, you're off
> >         the hook
> >         for this one. :-)
> >
> Indeed! :-P
>
> But no worries, it happens! :-D
>
> However...
>
> > Thank you very much for letting me know. :-) Sorry about not sending
> > them now because I'm rushing for a paper deadline this month. :-(
> > I will send the patches about synchronizing the vcpu data in rtds
> > scheduler after the paper deadline (which is Oct. 20th).
> >
> I was just about to send you a 'ping' about the patches for curing the
> serialization issue in RTDS.
>
> Ideally, we'd have wanted them before RC1. Of course, the fact that we
> accepted the scheduler as experimental makes things a bit less critical
> and urgent, but I really would like to have them in before the release.
>
> So, Meng, let us know :-)
>


​Sure! I just sent out the patches to solve the sanity check and
serialization issue in RTDS scheduler.​
In the cover letter of the patch, I cited your summarization of the
solutions we discussed.

I'm sorry for the delay. :-( I was pushed by my bosses (well my advisor) to
try to submit a paper in RTAS 2015 whose deadline is Oct. 20th. So many
things were delayed. :-(
​​
​But anyway, the patches are out and hope these two are ok. :-)​

​Thanks,

Meng​

-----------
Meng Xu
PhD Student in Computer and Information Science
University of Pennsylvania

[-- Attachment #1.2: Type: text/html, Size: 2785 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: patches for proper serialization in RTDS scheduler [Was: Re: OPW 2014 - Advanced Scheduling Parameters Project]

[Dario Faggioli]

[-- Attachment #1.1: Type: text/plain, Size: 643 bytes --]

On Sat, 2014-10-25 at 10:22 -0400, Meng Xu wrote:
> Hi Dario,
> 
Hi,

> 2014-10-24 12:54 GMT-04:00 Dario Faggioli <dario.faggioli@citrix.com>:
>         So, Meng, let us know :-)
> 

> ​Sure! I just sent out the patches to solve the sanity check and
> serialization issue in RTDS scheduler.​
>
Perfect, thanks, I'll have a look at them today.

Regards,
Dario

-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel