* Re: [parisc-linux] review of Documentation/parisc/unwritten
2001-10-06 9:19 [parisc-linux] review of Documentation/parisc/unwritten Grant Grundler
2001-10-06 10:37 ` Helge Deller
@ 2001-10-09 19:44 ` Grant Grundler
1 sibling, 0 replies; 4+ messages in thread
From: Grant Grundler @ 2001-10-09 19:44 UTC (permalink / raw)
To: parisc-linux
Grant Grundler wrote:
> Hi all,
> In summary, it looks like unwritten file can be deleted.
> Notes from reviewing Documentation/parisc/unwritten are appended.
Update of remaining items appended. I'll file wishlist
severity bugs for the those items unless told otherwise.
And then delete unwritten and close kernel bug #35.
I've committed the "easy" changes (diff appended).
Since I'm only removing unused stuff, no -paXX change.
thanks,
grant
csum_partial_copy
csum_partial_copy_from_user
arch/parisc/lib/checksum.c
We want optimized asm for both of those.
do_check_pgt_cache arch/parisc/mm/init.c
NOP. this seems wrong.
all the other arches have (about) the same thing here.
sparc64 has some CONFIG_SMP in it.
sigaddset
sigdelset include/linux/signal.h
Only i386 and m68k define __HAVE_ARCH_SIG_BITOPS.
parisc ends up using "generic" code like most other arches.
Is this an issue?
Index: Documentation/parisc/unwritten
===================================================================
RCS file: /home/cvs/parisc/linux/Documentation/parisc/unwritten,v
retrieving revision 1.7
diff -u -p -r1.7 unwritten
--- unwritten 2001/10/09 05:23:19 1.7
+++ unwritten 2001/10/09 19:22:25
@@ -1,59 +1,16 @@
-Hi all,
-Notes from reviewing Documentation/parisc/unwritten.
-In summary, it looks like unwritten file can be deleted.
-
-I'll take care of things marked delete unless someone objects.
-
-Please give feedback on items marked "review?".
-It means I didn't know what to do but status quo didn't seem
-right to me either.
-
-grant
-
-
-sys_idle ./include/asm-parisc/unistd.h
- obsolete - delete usage.
-
-reboot_setup() arch/parisc/kernel/process.c
- not used. delete function.
-
-machine_halt BUG. NOP. Should update LEDs/ChassisCode.
-machine_power_on obsolete - delete it
- arch/parisc/kernel/process.c
-
-exit_thread NOP is ok.
-flush_thread review?
- arch/parisc/kernel/process.c.
- Only needs to handle fpu stuff or perf monitors.
- (several arches implement a "lazy fpu state"...interesting)
-
-strlen_user include/asm-parisc/uaccess.h
- #define strlen_user(str) lstrnlen_user(str, 0x7fffffffL)
- review?
+csum_partial_copy
+csum_partial_copy_from_user
+ arch/parisc/lib/checksum.c
+ We want optimized asm for both of those.
+
do_check_pgt_cache arch/parisc/mm/init.c
NOP. this seems wrong.
all the other arches have (about) the same thing here.
sparc64 has some CONFIG_SMP in it.
-set_pte_phys arch/parisc/mm/init.c
- orphaned. delete it.
-
-check_bugs include/asm-parisc/bugs.h
- NOP. Check for CPU bugs or misconfigured kernel.
- review?
-
-csum_partial_copy
-csum_partial_copy_from_user
- arch/parisc/lib/checksum.c
- We want optimized asm for both of those.
-
sigaddset
sigdelset include/linux/signal.h
Only i386 and m68k define __HAVE_ARCH_SIG_BITOPS.
parisc ends up using "generic" code like most other arches.
Is this an issue?
-
-get_cycles include/asm-parisc/timex.h
- Defined as "mfctl(16)". Is this ok?
- Only used in drivers/drm/ and kernel/sched.c.
Index: arch/parisc/kernel/process.c
===================================================================
RCS file: /home/cvs/parisc/linux/arch/parisc/kernel/process.c,v
retrieving revision 1.37
diff -u -p -r1.37 process.c
--- process.c 2001/06/30 23:47:53 1.37
+++ process.c 2001/10/09 19:22:25
@@ -74,9 +74,6 @@ void cpu_idle(void)
}
}
-void __init reboot_setup(char *str, int *ints)
-{
-}
#ifdef __LP64__
#define COMMAND_GLOBAL 0xfffffffffffe0030UL
@@ -130,11 +127,12 @@ void machine_restart(char *cmd)
void machine_halt(void)
{
+ /*
+ ** The LED/ChassisCodes are updated by the led_halt()
+ ** function, called by the reboot notifier chain.
+ */
}
-void machine_power_on(void)
-{
-}
/* This routine is called from sys_reboot to actually turn off the
* machine */
@@ -171,6 +169,9 @@ void exit_thread(void)
void flush_thread(void)
{
+ /* Only needs to handle fpu stuff or perf monitors.
+ ** REVISIT: several arches implement a "lazy fpu state".
+ */
set_fs(USER_DS);
}
Index: arch/parisc/mm/init.c
===================================================================
RCS file: /home/cvs/parisc/linux/arch/parisc/mm/init.c,v
retrieving revision 1.40
diff -u -p -r1.40 init.c
--- init.c 2001/10/07 01:15:02 1.40
+++ init.c 2001/10/09 19:22:25
@@ -467,9 +467,6 @@ void show_mem(void)
show_buffers();
}
-void set_pte_phys (unsigned long vaddr, unsigned long phys)
-{
-}
static void __init map_pages(unsigned long start_vaddr, unsigned long start_paddr, unsigned long size, pgprot_t pgprot)
{
Index: include/asm-parisc/unistd.h
===================================================================
RCS file: /home/cvs/parisc/linux/include/asm-parisc/unistd.h,v
retrieving revision 1.22
diff -u -p -r1.22 unistd.h
--- unistd.h 2001/07/05 23:09:06 1.22
+++ unistd.h 2001/10/09 19:22:26
@@ -809,13 +809,8 @@ type name(type1 arg1, type2 arg2, type3
return K_INLINE_SYSCALL(name, 6, arg1, arg2, arg3, arg4, arg5, arg6); \
}
-#ifdef __KERNEL_SYSCALLS__
-static inline int idle(void)
-{
- extern int sys_idle(void);
- return sys_idle();
-}
+#ifdef __KERNEL_SYSCALLS__
static inline int pause(void)
{
@@ -859,6 +854,13 @@ static inline int dup(int fd)
return sys_dup(fd);
}
+static inline int execve(char *filename, char * argv [],
+ char * envp[])
+{
+ extern int __execve(char *, char **, char **, struct task_struct *);
+ return __execve(filename, argv, envp, current);
+}
+
static inline int open(const char *file, int flag, int mode)
{
extern long sys_open(const char *, int, int);
@@ -891,15 +893,8 @@ static inline pid_t wait(int * wait_stat
{
return sys_wait4(-1, wait_stat, 0, NULL);
}
-
-static inline int execve(char *filename, char * argv [],
- char * envp[])
-{
- extern int __execve(char *, char **, char **, struct task_struct *);
- return __execve(filename, argv, envp, current);
-}
-#endif
+#endif /* __KERNEL_SYSCALLS__ */
#endif /* __ASSEMBLY__ */
^ permalink raw reply [flat|nested] 4+ messages in thread