RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-30 Thread Mirea, Bogdan-Stefan
Hello Thomas, On Friday, May 26, 2017 1:05 PM, Thomas Gleixner wrote: > On Wed, 24 May 2017, Mirea, Bogdan-Stefan wrote: > > I am thinking about using timekeeping_inject_sleeptime64(delta) hook to > > That's not a hook. It's a regular function. Please use proper technical > terms. > > > add a

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-30 Thread Mirea, Bogdan-Stefan
Hello Thomas, On Friday, May 26, 2017 1:05 PM, Thomas Gleixner wrote: > On Wed, 24 May 2017, Mirea, Bogdan-Stefan wrote: > > I am thinking about using timekeeping_inject_sleeptime64(delta) hook to > > That's not a hook. It's a regular function. Please use proper technical > terms. > > > add a

Re: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-26 Thread Mark Rutland
On Fri, May 19, 2017 at 01:16:23PM +0300, Bogdan Mirea wrote: > This option enables Boot Time Preservation between Bootloader and > Linux Kernel. It is based on the idea that the Bootloader (or any > other early firmware) will start the HW Timer and Linux Kernel will > count the time starting with

Re: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-26 Thread Mark Rutland
On Fri, May 19, 2017 at 01:16:23PM +0300, Bogdan Mirea wrote: > This option enables Boot Time Preservation between Bootloader and > Linux Kernel. It is based on the idea that the Bootloader (or any > other early firmware) will start the HW Timer and Linux Kernel will > count the time starting with

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-26 Thread Thomas Gleixner
On Wed, 24 May 2017, Mirea, Bogdan-Stefan wrote: > I am thinking about using timekeeping_inject_sleeptime64(delta) hook to That's not a hook. It's a regular function. Please use proper technical terms. > add a delta time at boot to the CLOCK_BOOTTIME, but the problem that > arise here is that

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-26 Thread Thomas Gleixner
On Wed, 24 May 2017, Mirea, Bogdan-Stefan wrote: > I am thinking about using timekeeping_inject_sleeptime64(delta) hook to That's not a hook. It's a regular function. Please use proper technical terms. > add a delta time at boot to the CLOCK_BOOTTIME, but the problem that > arise here is that

Re: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-25 Thread Pavel Machek
Hi! > +#ifndef BOOT_TIME_PRESERVE_CMDLINE > + #define BOOT_TIME_PRESERVE_CMDLINE "preserve_boot_time" > +#endif Needs documentation in Doc*/, should not have separate define, that just makes it complex. -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures)

Re: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-25 Thread Pavel Machek
Hi! > +#ifndef BOOT_TIME_PRESERVE_CMDLINE > + #define BOOT_TIME_PRESERVE_CMDLINE "preserve_boot_time" > +#endif Needs documentation in Doc*/, should not have separate define, that just makes it complex. -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures)

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-24 Thread Mirea, Bogdan-Stefan
On Tuesday, May 23, 2017 7:39 PM, Thomas Gleixner wrote: > On Tue, 23 May 2017, Mirea, Bogdan-Stefan wrote: > > On Monday, May 22, 2017 12:36 AM, Thomas Gleixner wrote: > > > On Fri, 19 May 2017, Bogdan Mirea wrote: > > > This adds a arch_timer specific command line option. Why is this > > >

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-24 Thread Mirea, Bogdan-Stefan
On Tuesday, May 23, 2017 7:39 PM, Thomas Gleixner wrote: > On Tue, 23 May 2017, Mirea, Bogdan-Stefan wrote: > > On Monday, May 22, 2017 12:36 AM, Thomas Gleixner wrote: > > > On Fri, 19 May 2017, Bogdan Mirea wrote: > > > This adds a arch_timer specific command line option. Why is this > > >

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-23 Thread Thomas Gleixner
On Tue, 23 May 2017, Mirea, Bogdan-Stefan wrote: > On Monday, May 22, 2017 12:36 AM, Thomas Gleixner wrote: > > On Fri, 19 May 2017, Bogdan Mirea wrote: > > This adds a arch_timer specific command line option. Why is this > > arch_timer > > specific? So if any other platform wants to gain this

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-23 Thread Thomas Gleixner
On Tue, 23 May 2017, Mirea, Bogdan-Stefan wrote: > On Monday, May 22, 2017 12:36 AM, Thomas Gleixner wrote: > > On Fri, 19 May 2017, Bogdan Mirea wrote: > > This adds a arch_timer specific command line option. Why is this > > arch_timer > > specific? So if any other platform wants to gain this

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-23 Thread Mirea, Bogdan-Stefan
On Monday, May 22, 2017 12:36 AM, Thomas Gleixner wrote: > On Fri, 19 May 2017, Bogdan Mirea wrote: > This adds a arch_timer specific command line option. Why is this > arch_timer > specific? So if any other platform wants to gain this feature then we > end > up copying that mess to every single

RE: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-23 Thread Mirea, Bogdan-Stefan
On Monday, May 22, 2017 12:36 AM, Thomas Gleixner wrote: > On Fri, 19 May 2017, Bogdan Mirea wrote: > This adds a arch_timer specific command line option. Why is this > arch_timer > specific? So if any other platform wants to gain this feature then we > end > up copying that mess to every single

Re: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-21 Thread Thomas Gleixner
On Fri, 19 May 2017, Bogdan Mirea wrote: > +#ifdef CONFIG_BOOT_TIME_PRESERVE > +/* > + * Set the real system time(including the time spent in bootloader) > + * based on the timer counter. > + */ > + > +#ifndef BOOT_TIME_PRESERVE_CMDLINE > + #define BOOT_TIME_PRESERVE_CMDLINE

Re: [PATCH v3] Added "Preserve Boot Time Support"

2017-05-21 Thread Thomas Gleixner
On Fri, 19 May 2017, Bogdan Mirea wrote: > +#ifdef CONFIG_BOOT_TIME_PRESERVE > +/* > + * Set the real system time(including the time spent in bootloader) > + * based on the timer counter. > + */ > + > +#ifndef BOOT_TIME_PRESERVE_CMDLINE > + #define BOOT_TIME_PRESERVE_CMDLINE