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

[PATCH v3] Added "Preserve Boot Time Support"

2017-05-19 Thread Bogdan Mirea
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 the cycles elapsed since timer start. The sched_clock part is

[PATCH v3] Added "Preserve Boot Time Support"

2017-05-19 Thread Bogdan Mirea
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 the cycles elapsed since timer start. The sched_clock part is