sttus of the Altix mmtimer driver

sttus of the Altix mmtimer driver

[Christoph Hellwig]

Hi folks,

what's the status of the SGI support for the IA64 SN2-specific
altic mmtimer (drivers/char/mmtimer.c)?  It's the only driver that
register a k_lock outside the core kernel, so if we could get rid of
it because the remaining IA64 user are all on somewhat old distros
anyway it would make our life a whole lot easier.

Re: status of the Altix mmtimer driver

[Thomas Gleixner]

On Tue, 23 May 2017, Christoph Hellwig wrote:

Cc: + Mike

> Hi folks,
> 
> what's the status of the SGI support for the IA64 SN2-specific
> altic mmtimer (drivers/char/mmtimer.c)?  It's the only driver that
> register a k_lock outside the core kernel, so if we could get rid of
> it because the remaining IA64 user are all on somewhat old distros
> anyway it would make our life a whole lot easier.
> 

Re: status of the Altix mmtimer driver

[Dimitri Sivanich]

Since this is SN2 specific, this can be removed.

On Tue, May 23, 2017 at 05:27:16PM +0200, Thomas Gleixner wrote:
> On Tue, 23 May 2017, Christoph Hellwig wrote:
> 
> Cc: + Mike
> 
> > Hi folks,
> > 
> > what's the status of the SGI support for the IA64 SN2-specific
> > altic mmtimer (drivers/char/mmtimer.c)?  It's the only driver that
> > register a k_lock outside the core kernel, so if we could get rid of
> > it because the remaining IA64 user are all on somewhat old distros
> > anyway it would make our life a whole lot easier.
> > 

Re: status of the Altix mmtimer driver

[Dimitri Sivanich]

Note that the mmtimer struct can probably be removed from the k_itimer struct
in include/linux/posix-timers.h, just for completeness.

--- a/include/linux/posix-timers.h
+++ b/include/linux/posix-timers.h
@@ -73,12 +73,6 @@ struct k_itimer {
                } real;
                struct cpu_timer_list cpu;
                struct {
-                       unsigned int clock;
-                       unsigned int node;
-                       unsigned long incr;
-                       unsigned long expires;
-               } mmtimer;
-               struct {
                        struct alarm alarmtimer;
                        ktime_t interval;
                } alarm;

On Tue, May 23, 2017 at 07:36:01PM -0500, Dimitri Sivanich wrote:
> Since this is SN2 specific, this can be removed.
> 
> On Tue, May 23, 2017 at 05:27:16PM +0200, Thomas Gleixner wrote:
> > On Tue, 23 May 2017, Christoph Hellwig wrote:
> > 
> > Cc: + Mike
> > 
> > > Hi folks,
> > > 
> > > what's the status of the SGI support for the IA64 SN2-specific
> > > altic mmtimer (drivers/char/mmtimer.c)?  It's the only driver that
> > > register a k_lock outside the core kernel, so if we could get rid of
> > > it because the remaining IA64 user are all on somewhat old distros
> > > anyway it would make our life a whole lot easier.
> > > 

[tip:timers/core] posix-timers: Remove mmtimer leftovers

[tip-bot for Dimitri Sivanich]

Commit-ID:  f822798e3ced63427d57d128ee8d118126455f84
Gitweb:     http://git.kernel.org/tip/f822798e3ced63427d57d128ee8d118126455f84
Author:     Dimitri Sivanich <sivanich@hpe.com>
AuthorDate: Fri, 26 May 2017 23:30:52 +0200
Committer:  Thomas Gleixner <tglx@linutronix.de>
CommitDate: Sat, 27 May 2017 12:58:46 +0200

posix-timers: Remove mmtimer leftovers

After removing mmtimer, the mmtimer struct can be removed from the k_itimer
struct.

Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: Russ Anderson <rja@sgi.com>
Cc: Dimitri Sivanich <sivanich@sgi.com>
Cc: Mike Travis <mike.travis@hpe.com>
Cc: Nate Zimmer <nzimmer@sgi.com>
Cc: Christoph Hellwig <hch@lst.de>
Link: http://lkml.kernel.org/r/20170526130534.GE30788@hpe.com

---
 include/linux/posix-timers.h | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/include/linux/posix-timers.h b/include/linux/posix-timers.h
index b313ef2..34e893a 100644
--- a/include/linux/posix-timers.h
+++ b/include/linux/posix-timers.h
@@ -73,12 +73,6 @@ struct k_itimer {
 		} real;
 		struct cpu_timer_list cpu;
 		struct {
-			unsigned int clock;
-			unsigned int node;
-			unsigned long incr;
-			unsigned long expires;
-		} mmtimer;
-		struct {
 			struct alarm alarmtimer;
 			ktime_t interval;
 		} alarm;