public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: "Serge E. Hallyn" <serue@us.ibm.com>
To: Garrett Cooper <yanegomi@gmail.com>
Cc: Linus Test Project <ltp-list@lists.sourceforge.net>,
	Michael Kerrisk <mtk.manpages@gmail.com>
Subject: Re: [LTP] [04/12 FAILURE] LTP? sysctl03 test fails
Date: Thu, 6 May 2010 12:55:12 -0500	[thread overview]
Message-ID: <20100506175512.GA25514@us.ibm.com> (raw)
In-Reply-To: <j2v364299f41005061038ucdbdaf19hcdad19491344e76d@mail.gmail.com>

Quoting Garrett Cooper (yanegomi@gmail.com):
> On Thu, May 6, 2010 at 6:53 AM, Serge E. Hallyn <serue@us.ibm.com> wrote:
> > Quoting Garrett Cooper (yanegomi@gmail.com):
> >> On May 5, 2010, at 11:56 PM, Subrata Modak wrote:
> >>
> >> > Subject: LTPś sysctl03 test fails
> >> >
> >> > Issues Description Below:
> >> > =====================================
> >> > # ./runltp -s sysctl03
> >> > <<<test_output>>>
> >> > sysctl03    1  TFAIL  :  Expected EPERM (1), got 13: Permission denied
> >> > sysctl03    2  TFAIL  :  Expected EPERM, got 13
> >> > sysctl03    1  TFAIL  :  Expected EPERM (1), got 13: Permission denied
> >> > <<<execution_status>>>
> >> > initiation_status="ok"
> >> > duration=0 termination_type=exited termination_id=1 corefile=no
> >> > cutime=0 cstime=0
> >> > <<<test_end>>>
> >>
> >>       Already known and recently discussed.
> >
> > Not only can things move glacially in kernel-land, but decisions not
> > yet implemented can be changed.
> >
> > In the meantime, the sysctl's sit there as a potential subject for
> > exploitation.
> >
> > So not meaning to be argumentative for its own sake, I nevertheless
> > think it's better to fix the test than either to ignore or remove
> > it.  Two untested patches below - the one just replaces EPERM with
> > EACCESS.  The other removes the (imo misuided) notion that we can
> > guess at the failing errno.
> 
> Except that the documentation (manpages) should explicitly state what
> the failing conditions are for any given libcall and syscall. If not,
> the Linux kernel devs and documentation team have failed to do their
> job.

So since we're all member of the doc team, send a patch for sysctl(2)
manpage ERRORS section :)

(mtk cc:d as this is probably news to him)

> > An LSM could choose to return -EPERM
> > after all, or perhaps even something different.  The thing that
> > should scare us is if the call succeeds.  If we give any false
> > positives, then true positives will seem less scary.
> 
> This will fail on older kernels as sysctl(2) always returned EPERM due

Sorry - what will fail?

I think you're saying the first patch will, and I agree, which is why
I advocate the second one i pasted in.

> to the way it was improperly designed. Please see the previous thread
> for more info: http://lkml.org/lkml/2010/3/4/354
> 
> Thanks,
> -Garrett

------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2010-05-06 17:55 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-06  6:55 [LTP] [00/12 FAILURE] Reporting LTP run issues on Fedora 13 ppc6 Subrata Modak
2010-05-06  6:56 ` [01/12 FAILURE] LTPX Memory CGROUP test creates Call Trace and Badness at mm/oom_kill.c:393 Subrata Modak
2010-05-06  6:56 ` [02/12 FAILURE] LTPX File Capabilities tests becomes defunct and does not complete beyond 12 hours Subrata Modak
2010-05-06  6:56 ` [03/12 FAILURE] LTPX Memory Conroller test fails and hangs system for more than " Subrata Modak
2010-05-06  6:56 ` [04/12 FAILURE] LTPX sysctl03 test fails Subrata Modak
2010-05-06  8:05   ` [LTP] [04/12 FAILURE] LTP? " Garrett Cooper
2010-05-06 13:53     ` Serge E. Hallyn
2010-05-06 17:38       ` Garrett Cooper
2010-05-06 17:55         ` Serge E. Hallyn [this message]
2010-05-06 17:59           ` Garrett Cooper
2010-05-06 18:39             ` Serge E. Hallyn
2010-05-07  9:05               ` Subrata Modak
2010-05-07  9:22               ` Garrett Cooper
2010-05-06  6:57 ` [05/12 FAILURE] LTPX proc01 " Subrata Modak
2010-05-06  8:04   ` [LTP] [05/12 FAILURE] LTP? " Garrett Cooper
2010-07-27 17:37     ` Subrata Modak
2010-07-27 17:38     ` Subrata Modak
2010-05-06  6:57 ` [LTP] [06/12 FAILURE] Many CONTAINER tests of LTP fail Subrata Modak
2010-05-06 17:53   ` Garrett Cooper
2010-05-06 17:59     ` Dan Smith
2010-07-27 17:40     ` Subrata Modak
2010-05-06  6:57 ` [07/12 FAILURE] LTPX MEMORY CGROUP FUNCTION tests fails Subrata Modak
2010-07-28 17:34   ` [LTP] [07/12 FAILURE] LTPE " Subrata Modak
2010-05-06  6:58 ` [08/12 FAILURE] LTPX clock gettime03 fails Subrata Modak
2010-05-06  7:04   ` [LTP] [08/12 FAILURE] LTPE " Suzuki Poulose
2010-05-06  7:55     ` Garrett Cooper
2010-05-06  9:03       ` Suzuki Poulose
2010-05-06 17:42         ` Garrett Cooper
2010-05-07  8:53           ` Subrata Modak
2010-06-01 15:40         ` Suzuki Poulose
2010-06-02 12:57           ` Subrata Modak
2010-07-27 18:03             ` Subrata Modak
2010-05-06  6:58 ` [09/12 FAILURE] LTPX timer create04 fails Subrata Modak
2010-05-07  2:12   ` [LTP] [09/12 FAILURE] LTP " Henry xu
2010-07-28 12:46   ` [LTP] [09/12 FAILURE] LTPE " Subrata Modak
2010-05-06  6:58 ` [10/12 FAILURE] LTPX ar01 fails Subrata Modak
2010-05-06  6:58 ` [11/12 FAILURE] LTPX file test fails Subrata Modak
2010-05-06  8:01   ` [LTP] [11/12 FAILURE] LTP? " Garrett Cooper
2010-05-06  6:59 ` [12/12 FAILURE] LTPX cron tests fails Subrata Modak

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100506175512.GA25514@us.ibm.com \
    --to=serue@us.ibm.com \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=mtk.manpages@gmail.com \
    --cc=yanegomi@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox