All of lore.kernel.org
 help / color / mirror / Atom feed
From: "xuyang2018.jy@fujitsu.com" <xuyang2018.jy@fujitsu.com>
To: Martin Doucha <mdoucha@suse.cz>
Cc: "ltp@lists.linux.it" <ltp@lists.linux.it>
Subject: Re: [LTP] [PATCH 2/3] quotactl08: Use do_mount()
Date: Fri, 4 Mar 2022 02:57:05 +0000	[thread overview]
Message-ID: <62218039.2070903@fujitsu.com> (raw)
In-Reply-To: <6d0a36c3-d317-b942-71b1-0c6fecb5c3e5@suse.cz>

Hi Martin
> On 24. 02. 22 2:47, xuyang2018.jy@fujitsu.com wrote:
>> Hi Petr
>>> to change
>>> quotactl08.c:160: TBROK: mount(/dev/loop0, mntpoint, ext4, 0, (nil)) failed: ESRCH (3)
>> I doubt why fs quota is not supported on this case since we have check
>> kernel config?
>
> In this case, kernel config says that quota is enabled as a module (=M).
> But the module is packaged in a separate RPM and the test is running on
> a minimalistic system where that RPM is not available. Therefore just
> checking the kernel config is not sufficient. We need to query actual
> module presence.
>
> (Sorry for duplicate mail, I forgot to CC the mailing list.)
Oh, I replied the personal mail.

Thanks for your explaination. I don't know this complex situation before.

I think the quota module is belong to kernel-default package but not 
belong to kernel-default-base package. So jeos use kernel-default-base 
will pass kernel config check but indeed it miss this module.

This problem also occurs on centos if we use module from 
kernel-module-extra.

So a better solution is that if the module is belong to 
kernel-default-base ,then we can use kconfig check, but if the module is 
not belong to kernel-default-base, then use need_driver maybe more correct.

Best Regards
Yang Xu


>

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

  reply	other threads:[~2022-03-04  2:57 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-23 15:32 [LTP] [PATCH 0/3] quotactl ESRCH fixes Petr Vorel
2022-02-23 15:32 ` [LTP] [PATCH 1/3] quotactl: Move do_mount() to header Petr Vorel
2022-02-23 15:32 ` [LTP] [PATCH 2/3] quotactl08: Use do_mount() Petr Vorel
2022-02-24  1:47   ` xuyang2018.jy
2022-02-28 13:49     ` Petr Vorel
2022-02-28 16:19     ` Petr Vorel
2022-03-01  1:11       ` xuyang2018.jy
2022-03-01  7:58         ` Petr Vorel
2022-03-01  8:10           ` xuyang2018.jy
2022-03-01 10:17             ` Petr Vorel
2022-03-03 11:41               ` Martin Doucha
2022-03-03 12:55                 ` Petr Vorel
2022-03-03 11:38     ` Martin Doucha
2022-03-04  2:57       ` xuyang2018.jy [this message]
2022-02-23 15:32 ` [LTP] [PATCH 3/3] quotactl09: " Petr Vorel

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=62218039.2070903@fujitsu.com \
    --to=xuyang2018.jy@fujitsu.com \
    --cc=ltp@lists.linux.it \
    --cc=mdoucha@suse.cz \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.