On 07/01/2013 09:55 PM, [email protected] wrote:
> Hi!
>> Yes, for inotify04 on RHEL5. Below commit is back-ported to RHEL5 updated
>> version, so then inotify will send IN_ATTRIB events when link count changes.
>> But in test case, we only judge the kernel version 2.6.25 to add IN_ATTRIB
>> event.
>
>> commit ece95912db94d98e202cbedb8f35206deb29d83d
>> Author: Jan Kara <[email protected]>
>> Date: Wed Feb 6 01:37:13 2008 -0800
>
> <snip>
>
>> RHEL5 changelog:
>> - [fs] inotify: send IN_ATTRIB event on link count changes (Eric Paris )
>> [471893]
>>
>>
>> The man page of inotify said that:
>> IN_ATTRIB Metadata changed, e.g., permissions, timestamps, extended
>> attributes, link count (since Linux 2.6.25), UID, GID, etc. (*).
>>
>> So, in the test case, we judge that if the kernel version is less than
>> 2.6.25, we
>> assume that the IN_ATTRIB will not be sent in the event. But after back-port
>> this patch, RHEL5 includes this event when the link count changes.
>>
>> There are many test cases are suffering such situation, because sometimes
>> the patch
>> back-port may break our kernel version check. So I considering introduce some
>> vendor specific kernel version check like RHEL and SLES.
>
> So the case here is that the test is disabled because the we think that
> it will not work. I'm fine with adding more information to the
> tst_kvercmp() but we should do it more extensible and data driven
> manner.
>
> What about we create an array with extra version strings per
> distribution that gets passed to tst_kvercmp2() function and is used
> when particular distribution is detected?
RHEL kernel with a suffix like ".el5" on RHEL5 and ".el6" on RHEL6,
I wonder how to detect SLES kernels?
Thanks,
Wanlong Gao
>
> Roughly something as:
>
> In the LTP lib:
>
> struct tst_kern_extraver {
> char *dist_name;
> char *extra_ver;
> };
>
> int tst_kvercmp2(int r1, int r2, int r3, struct tst_kern_extraver *vers);
>
>
> In the testcase:
>
> static struct tst_kern_extraver exver[] = {
> {"RHEL6", "234"},
> {"SLES10", "0.91"},
> {NULL, NULL}
> };
>
>
> ...
>
> if (tst_kvercmp2(2, 6, 25, exver) < 0)
> ...
> ...
>
>
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:
Build for Windows Store.
http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
