* Re: [LTP] [PATCH V3] lib: add tst_kvercmp2 to compare specific vendor extra kernel versions [not found] <1373356491-15338-1-git-send-email-gaowanlong@cn.fujitsu.com> @ 2013-07-10 13:38 ` chrubis [not found] ` <51DD9B64.7050802@casparzhang.com> 1 sibling, 0 replies; 4+ messages in thread From: chrubis @ 2013-07-10 13:38 UTC (permalink / raw) To: Wanlong Gao; +Cc: Garrett Cooper, LTP, Mike Frysinger Hi! > We can now test specific verdor kernel versions in > test cases like: > > static struct tst_kern_exv exv[] = { > {"RHEL6", "234"}, > {"SLES10", "0.91"}, > {NULL, NULL} > }; > > ... > > if (tst_kvercmp2(2, 6, 25, exv) < 0) > ... > ... This one looks fine, acked. -- Cyril Hrubis chrubis@suse.cz ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <51DD9B64.7050802@casparzhang.com>]
* Re: [LTP] [PATCH V3] lib: add tst_kvercmp2 to compare specific vendor extra kernel versions [not found] ` <51DD9B64.7050802@casparzhang.com> @ 2013-07-10 18:33 ` Jan Stancek 2013-07-11 1:14 ` Wanlong Gao 2013-07-11 9:37 ` chrubis 0 siblings, 2 replies; 4+ messages in thread From: Jan Stancek @ 2013-07-10 18:33 UTC (permalink / raw) To: Caspar Zhang, Wanlong Gao; +Cc: LTP ----- Original Message ----- > From: "Caspar Zhang" <caspar@casparzhang.com> > To: "Wanlong Gao" <gaowanlong@cn.fujitsu.com> > Cc: "LTP" <ltp-list@lists.sourceforge.net>, "Cyril Hrubis" <chrubis@suse.cz>, "Garrett Cooper" <yanegomi@gmail.com>, > "Mike Frysinger" <vapier@gentoo.org>, jstancek@redhat.com > Sent: Wednesday, 10 July, 2013 7:35:32 PM > Subject: Re: [PATCH V3] lib: add tst_kvercmp2 to compare specific vendor extra kernel versions > > On 07/09/2013 03:54 PM, Wanlong Gao wrote: > > We can now test specific verdor kernel versions in > > test cases like: > > > > static struct tst_kern_exv exv[] = { > > {"RHEL6", "234"}, > > {"SLES10", "0.91"}, > > {NULL, NULL} > > }; > > > > ... > > > > if (tst_kvercmp2(2, 6, 25, exv) < 0) > > ... > > ... > > > > Thank you for Cyril's suggestion. > > > > > > Signed-off-by: Wanlong Gao <gaowanlong@cn.fujitsu.com> > > --- > > include/test.h | 7 +++++++ > > lib/tst_kvercmp.c | 46 +++++++++++++++++++++++++++++++++++++++++++--- > > 2 files changed, 50 insertions(+), 3 deletions(-) > > > > diff --git a/include/test.h b/include/test.h > > index ee3efc3..a76fafc 100644 > > --- a/include/test.h > > +++ b/include/test.h > > @@ -173,6 +173,13 @@ char *get_high_address(void); > > void tst_getkver(int *k1, int *k2, int *k3); > > int tst_kvercmp(int r1, int r2, int r3); > > > > +struct tst_kern_exv { > > + char *dist_name; > > + char *extra_ver; > > +}; > > + > > +int tst_kvercmp2(int r1, int r2, int r3, struct tst_kern_exv *vers); > > + > > /* lib/tst_is_cwd.c */ > > int tst_is_cwd_nfs(void); > > int tst_is_cwd_v9fs(void); > > diff --git a/lib/tst_kvercmp.c b/lib/tst_kvercmp.c > > index cb4fd67..15fb571 100644 > > --- a/lib/tst_kvercmp.c > > +++ b/lib/tst_kvercmp.c > > @@ -34,15 +34,13 @@ > > #include <unistd.h> > > #include <string.h> > > #include <sys/utsname.h> > > +#include "test.h" > > > > void get_kver(int *k1, int *k2, int *k3) > > { > > struct utsname uval; > > char *kver; > > char *r1, *r2, *r3; > > -#if !defined(linux) > > - extern char *strsep(); /* shut up some compilers */ > > -#endif > > > > uname(&uval); > > kver = uval.release; > > @@ -66,3 +64,45 @@ int tst_kvercmp(int r1, int r2, int r3) > > > > return currver - testver; > > } > > + > > +static int tst_kexvcmp(char *tst_exv, char *cur_ver) > > +{ > > + int c1 = 0, c2 = 0, t1 = 0, t2 = 0; > > + int ret; > > + > > + sscanf(cur_ver, "%*d.%*d.%*d-%d.%d", &c1, &c2); > > + sscanf(tst_exv, "%d.%d", &t1, &t2); > > + if ((ret = c1 - t1)) > > not sure if the brackets are duplicated... What caught my (and checkpatch's) eye is that these are assignments in if condition, but I'm not sure how strictly we take checkpatch output. > > > + return ret; > > + else > > + return c2 - t2; > > +} > > + > > +int tst_kvercmp2(int r1, int r2, int r3, struct tst_kern_exv *vers) > > +{ > > + int ret; > > + int i; > > + struct utsname uval; > > + char *kver; > > + const char *cur_dist_name = NULL; > > + > > + if ((ret = tst_kvercmp(r1, r2, r3))) > > here as well. > > > + return ret; > > + > > + uname(&uval); > > + kver = uval.release; > > + if (strstr(kver, ".el5")) { > > + cur_dist_name = "RHEL5"; > > + } else if (strstr(kver, ".el6")) { > > + cur_dist_name = "RHEL6"; > > + } else { > > + return ret; > > + } > > + > > + for (i = 0; vers[i].dist_name; i++) { > > + if (!strcmp(vers[i].dist_name, cur_dist_name)) > > + return tst_kexvcmp(vers[i].extra_ver, kver); > > + } > > + > > + return ret; > > +} > > > > Others look good. Looks good to me as well. Regards, Jan > > Reviewed-by: Caspar Zhang <caspar@casparzhang.com> > ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [LTP] [PATCH V3] lib: add tst_kvercmp2 to compare specific vendor extra kernel versions 2013-07-10 18:33 ` Jan Stancek @ 2013-07-11 1:14 ` Wanlong Gao 2013-07-11 9:37 ` chrubis 1 sibling, 0 replies; 4+ messages in thread From: Wanlong Gao @ 2013-07-11 1:14 UTC (permalink / raw) To: Jan Stancek, Caspar Zhang; +Cc: LTP On 07/11/2013 02:33 AM, Jan Stancek wrote: > > > ----- Original Message ----- >> From: "Caspar Zhang" <caspar@casparzhang.com> >> To: "Wanlong Gao" <gaowanlong@cn.fujitsu.com> >> Cc: "LTP" <ltp-list@lists.sourceforge.net>, "Cyril Hrubis" <chrubis@suse.cz>, "Garrett Cooper" <yanegomi@gmail.com>, >> "Mike Frysinger" <vapier@gentoo.org>, jstancek@redhat.com >> Sent: Wednesday, 10 July, 2013 7:35:32 PM >> Subject: Re: [PATCH V3] lib: add tst_kvercmp2 to compare specific vendor extra kernel versions >> >> On 07/09/2013 03:54 PM, Wanlong Gao wrote: >>> We can now test specific verdor kernel versions in >>> test cases like: >>> >>> static struct tst_kern_exv exv[] = { >>> {"RHEL6", "234"}, >>> {"SLES10", "0.91"}, >>> {NULL, NULL} >>> }; >>> >>> ... >>> >>> if (tst_kvercmp2(2, 6, 25, exv) < 0) >>> ... >>> ... >>> >>> Thank you for Cyril's suggestion. >>> >>> >>> Signed-off-by: Wanlong Gao <gaowanlong@cn.fujitsu.com> >>> --- >>> include/test.h | 7 +++++++ >>> lib/tst_kvercmp.c | 46 +++++++++++++++++++++++++++++++++++++++++++--- >>> 2 files changed, 50 insertions(+), 3 deletions(-) >>> >>> diff --git a/include/test.h b/include/test.h >>> index ee3efc3..a76fafc 100644 >>> --- a/include/test.h >>> +++ b/include/test.h >>> @@ -173,6 +173,13 @@ char *get_high_address(void); >>> void tst_getkver(int *k1, int *k2, int *k3); >>> int tst_kvercmp(int r1, int r2, int r3); >>> >>> +struct tst_kern_exv { >>> + char *dist_name; >>> + char *extra_ver; >>> +}; >>> + >>> +int tst_kvercmp2(int r1, int r2, int r3, struct tst_kern_exv *vers); >>> + >>> /* lib/tst_is_cwd.c */ >>> int tst_is_cwd_nfs(void); >>> int tst_is_cwd_v9fs(void); >>> diff --git a/lib/tst_kvercmp.c b/lib/tst_kvercmp.c >>> index cb4fd67..15fb571 100644 >>> --- a/lib/tst_kvercmp.c >>> +++ b/lib/tst_kvercmp.c >>> @@ -34,15 +34,13 @@ >>> #include <unistd.h> >>> #include <string.h> >>> #include <sys/utsname.h> >>> +#include "test.h" >>> >>> void get_kver(int *k1, int *k2, int *k3) >>> { >>> struct utsname uval; >>> char *kver; >>> char *r1, *r2, *r3; >>> -#if !defined(linux) >>> - extern char *strsep(); /* shut up some compilers */ >>> -#endif >>> >>> uname(&uval); >>> kver = uval.release; >>> @@ -66,3 +64,45 @@ int tst_kvercmp(int r1, int r2, int r3) >>> >>> return currver - testver; >>> } >>> + >>> +static int tst_kexvcmp(char *tst_exv, char *cur_ver) >>> +{ >>> + int c1 = 0, c2 = 0, t1 = 0, t2 = 0; >>> + int ret; >>> + >>> + sscanf(cur_ver, "%*d.%*d.%*d-%d.%d", &c1, &c2); >>> + sscanf(tst_exv, "%d.%d", &t1, &t2); >>> + if ((ret = c1 - t1)) >> >> not sure if the brackets are duplicated... > > What caught my (and checkpatch's) eye is that these are assignments in if condition, > but I'm not sure how strictly we take checkpatch output. Adding one more parenthesis just shut down the gcc warning here. Thanks, Wanlong Gao ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [LTP] [PATCH V3] lib: add tst_kvercmp2 to compare specific vendor extra kernel versions 2013-07-10 18:33 ` Jan Stancek 2013-07-11 1:14 ` Wanlong Gao @ 2013-07-11 9:37 ` chrubis 1 sibling, 0 replies; 4+ messages in thread From: chrubis @ 2013-07-11 9:37 UTC (permalink / raw) To: Jan Stancek; +Cc: LTP Hi! > > > +static int tst_kexvcmp(char *tst_exv, char *cur_ver) > > > +{ > > > + int c1 = 0, c2 = 0, t1 = 0, t2 = 0; > > > + int ret; > > > + > > > + sscanf(cur_ver, "%*d.%*d.%*d-%d.%d", &c1, &c2); > > > + sscanf(tst_exv, "%d.%d", &t1, &t2); > > > + if ((ret = c1 - t1)) > > > > not sure if the brackets are duplicated... > > What caught my (and checkpatch's) eye is that these are assignments in if condition, > but I'm not sure how strictly we take checkpatch output. Assignment inside of condition is generally considered as a tricky statement which is why checkpatch gives a warning. I'm personally not against it as long as it's simple statement like this but if other people here disagree I'm fine with treating it as a wrong thing to do. -- Cyril Hrubis chrubis@suse.cz ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ Ltp-list mailing list Ltp-list@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ltp-list ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-07-11 9:35 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1373356491-15338-1-git-send-email-gaowanlong@cn.fujitsu.com>
2013-07-10 13:38 ` [LTP] [PATCH V3] lib: add tst_kvercmp2 to compare specific vendor extra kernel versions chrubis
[not found] ` <51DD9B64.7050802@casparzhang.com>
2013-07-10 18:33 ` Jan Stancek
2013-07-11 1:14 ` Wanlong Gao
2013-07-11 9:37 ` chrubis
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox