lkml.org 
[lkml]   [2018]   [Feb]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH-next] sched/headers: Clean up <linux/sched.h>
From
Date
El vie, 16-02-2018 a las 10:44 +0100, Juri Lelli escribió:
> On 15/02/18 17:52, Peter Zijlstra wrote:
> > On Thu, Feb 15, 2018 at 10:43:18AM -0500, Christopher Diaz Riveros
> > wrote:
>
> [...]
>
> > > @@ -437,20 +437,28 @@ struct sched_dl_entity {
> > > * during sched_setattr(), they will remain the same
> > > until
> > > * the next sched_setattr().
> > > */
> > > - u64 dl_runtime; /*
> > > Maximum runtime for each instance */
> > > - u64 dl_deadline; /
> > > * Relative deadline of each instance */
> > > - u64 dl_period; /*
> > > Separation of two instances (period) */
> > > - u64 dl_bw;
> > > /* dl_runtime / dl_period */
> > > - u64 dl_density; /*
> > > dl_runtime / dl_deadline */
> > > + /* Maximum runtime for each instance */
> > > + u64 dl_runtime;
> > > + /* Relative deadline of each instance */
> > > + u64 dl_deadline;
> > > + /* Separation of two instances (period) */
> > > + u64 dl_period;
> > > + /* dl_runtime / dl_period */
> > > + u64 dl_bw;
> > > + /* dl_runtime / dl_deadline */
> > > + u64 dl_density;
> >
> > That's a whole lot less readable :/
>
> Yep. :(

Thank you all for the feedback, I'll consider this patch as NACK. Sorry
for wasting time in a low quality patch. I'll prepare a better one
next time :)

Regards,
--
Christopher Díaz Riveros
Gentoo Linux Developer
GPG Fingerprint: E517 5ECB 8152 98E4 FEBC 2BAA 4DBB D10F 0FDD 2547

\
 
 \ /
  Last update: 2018-02-16 14:25    [W:0.108 / U:0.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site