public inbox for systemtap@sourceware.org
 help / color / mirror / Atom feed
* 回复: Re: Change for aux_syscalls.stp
@ 2007-08-11 15:58 WENJI.HUANG
  2007-08-11 17:46 ` Frank Ch. Eigler
  0 siblings, 1 reply; 2+ messages in thread
From: WENJI.HUANG @ 2007-08-11 15:58 UTC (permalink / raw)
  To: Frank Ch. Eigler; +Cc: systemTAP

Frank,

   Sorry for my ambiguous expression.
   In fact, I found NFSCTL_UGIDUPDATE,NFSCTL_GETFH, NFSCTL_GETFD and NFSCTL_GETFS in kernel 2.4
   But NFSCTL_UGIDUPDATE and NFSCTL_GETFH are discarded in kernel >= 2.6.0

   Don't know why these changes happened(from Changelog). 
 
Regards,
Wenji
> 
> wenji.huang wrote:
> 
> >     Since 2.6.16, new SCHED_BATCH policy is added .
> >     Since 2.6
> > 
> >     NFSCTL_UGIDUPDATE and NFSCTL_GETFH are discarded.
> >     NFSCTL_GETFD and NFSCTL_GETFS are added.
> > [...]
> 
> I'm not sure how to read this.  Do UGIDUPDATE and GETFH never
> appear
> on any 2.6 kernel?  If so, it could be OK to remove them.  If
> those
> values are still used on some 2.6 kernels, I see no harm in
> keeping
> the number->name mappings around.
> 
> - FChE

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: 回复: Re: Change for aux_syscalls.stp
  2007-08-11 15:58 回复: Re: Change for aux_syscalls.stp WENJI.HUANG
@ 2007-08-11 17:46 ` Frank Ch. Eigler
  0 siblings, 0 replies; 2+ messages in thread
From: Frank Ch. Eigler @ 2007-08-11 17:46 UTC (permalink / raw)
  To: WENJI.HUANG; +Cc: systemtap


WENJI.HUANG wrote:

>  In fact, I found NFSCTL_UGIDUPDATE,NFSCTL_GETFH, NFSCTL_GETFD and
>  NFSCTL_GETFS in kernel 2.4 But NFSCTL_UGIDUPDATE and NFSCTL_GETFH
>  are discarded in kernel >= 2.6.0

OK.  Given the absence of conflict between between all these enums, I
would prefer to leave the entry in the tapset.  This is especially
appropriate for data values that may come from old userspace, and
would otherwise fall through to anonymous "default:" branches.

- FChE

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-08-11 15:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-08-11 15:58 回复: Re: Change for aux_syscalls.stp WENJI.HUANG
2007-08-11 17:46 ` Frank Ch. Eigler

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).