
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>

</head>

<body dir="ltr">

<div style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt; color: rgb(0, 0, 0);">

Hi,</div>

<div style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt; color: rgb(0, 0, 0);">

<br>

</div>

<div style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt; color: rgb(0, 0, 0);">

Ok, I will send out a new patch.</div>

<div style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt; color: rgb(0, 0, 0);">

<br>

</div>

<div style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt; color: rgb(0, 0, 0);">

Do I need to update syscalls/swapon/swapon03.c accordingly to make it a pattern? <br>

</div>

<div style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt; color: rgb(0, 0, 0);">

<br>

</div>

<div style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt; color: rgb(0, 0, 0);">

Cheers,</div>

<div style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt; color: rgb(0, 0, 0);">

Bogdan<br>

</div>

<div id="appendonsend"></div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Cyril Hrubis <chrubis@suse.cz><br>

<b>Sent:</b> Friday, February 11, 2022 15:15<br>

<b>To:</b> Petr Vorel <pvorel@suse.cz><br>

<b>Cc:</b> xuyang2018.jy@fujitsu.com <xuyang2018.jy@fujitsu.com>; Bogdan Lezhepekov <bogdan.lezhepekov@suse.com>; Li Wang <liwang@redhat.com>; LTP List <ltp@lists.linux.it><br>

<b>Subject:</b> Re: [LTP] [PATCH v1] mem/swapping01: Make sure swap is supported by kernel</font>

<div> </div>

</div>

<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">

<div class="PlainText">Hi!<br>

> 1) keep check access("/proc/swaps", F_OK)<br>

> 2) use .needs_config for "CONFIG_SWAP=y"<br>

> 3) Add .needs_swap flag to struct tst_test<br>

> <br>

> To be honest, I'm not sure, what's the best, probably .needs_config.<br>

> I worried little bit about .needs_config forcing config to be available in the<br>

> past, but that should be solved KCONFIG_SKIP_CHECK added by Xu.<br>

<br>

I guess that the easiest solution would be .needs_kconfig, we ended up<br>

depending on it in quite a few testcases anyway so I would say that<br>

nobody is runing LTP without having kernel config exproted anyways.<br>

<br>

-- <br>

Cyril Hrubis<br>

chrubis@suse.cz<br>

<br>

</div>

</span></font></div>

</body>

</html>