Re: [PATCH v3] sched/deadline: remove useless param from setup_new_dl_entity

2016-07-12 Thread Juri Lelli
On 11/07/16 16:16, Xunlei Pang wrote: > On 2016/07/11 at 16:01, luca abeni wrote: > > Hello, > > > > On Mon, 11 Jul 2016 13:03:56 +0800 > > Xunlei Pang wrote: > > > >> On 2016/07/08 at 19:28, Juri Lelli wrote: > > [...] > >>> @@ -363,6 +364,15 @@ static inline void setup_new_dl_entity(struct > >>>

Re: [PATCH v3] sched/deadline: remove useless param from setup_new_dl_entity

2016-07-11 Thread luca abeni
On Mon, 11 Jul 2016 16:16:20 +0800 Xunlei Pang wrote: > On 2016/07/11 at 16:01, luca abeni wrote: > > Hello, > > > > On Mon, 11 Jul 2016 13:03:56 +0800 > > Xunlei Pang wrote: > > > >> On 2016/07/08 at 19:28, Juri Lelli wrote: > > [...] > >>> @@ -363,6 +364,15 @@ static inline void setup_ne

Re: [PATCH v3] sched/deadline: remove useless param from setup_new_dl_entity

2016-07-11 Thread Xunlei Pang
On 2016/07/11 at 16:01, luca abeni wrote: > Hello, > > On Mon, 11 Jul 2016 13:03:56 +0800 > Xunlei Pang wrote: > >> On 2016/07/08 at 19:28, Juri Lelli wrote: > [...] >>> @@ -363,6 +364,15 @@ static inline void setup_new_dl_entity(struct >>> sched_dl_entity *dl_se, return; >>> >>> /* >>> +

Re: [PATCH v3] sched/deadline: remove useless param from setup_new_dl_entity

2016-07-11 Thread luca abeni
Hello, On Mon, 11 Jul 2016 13:03:56 +0800 Xunlei Pang wrote: > On 2016/07/08 at 19:28, Juri Lelli wrote: [...] > > @@ -363,6 +364,15 @@ static inline void setup_new_dl_entity(struct > > sched_dl_entity *dl_se, return; > > > > /* > > +* Use the scheduling parameters of the top pi-waiter

Re: [PATCH v3] sched/deadline: remove useless param from setup_new_dl_entity

2016-07-10 Thread Xunlei Pang
On 2016/07/08 at 19:28, Juri Lelli wrote: > setup_new_dl_entity() takes two parameters, but it only actually uses > one of them, under a different name, to setup a new dl_entity, after: > > 2f9f3fdc928 "sched/deadline: Remove dl_new from struct sched_dl_entity" > > as we currently do > > setup_ne

[PATCH v3] sched/deadline: remove useless param from setup_new_dl_entity

2016-07-08 Thread Juri Lelli
setup_new_dl_entity() takes two parameters, but it only actually uses one of them, under a different name, to setup a new dl_entity, after: 2f9f3fdc928 "sched/deadline: Remove dl_new from struct sched_dl_entity" as we currently do setup_new_dl_entity(&p->dl, &p->dl) However, before Luca's cha