All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: Vineeth Vijayan <vneethv@linux.ibm.com>, linux-s390@vger.kernel.org
Cc: oberpar@linux.ibm.com, Eric Farman <farman@linux.ibm.com>,
	Matthew Rosato <mjrosato@linux.ibm.com>
Subject: Re: [RFC v2 0/1]s390/cio: remove uevent suppress from cio driver
Date: Wed, 03 Nov 2021 16:41:00 +0100	[thread overview]
Message-ID: <87a6ilb62b.fsf@redhat.com> (raw)
In-Reply-To: <3ab690456a523951ad59c17ac71e6b294ff12d98.camel@linux.ibm.com>

On Wed, Nov 03 2021, Vineeth Vijayan <vneethv@linux.ibm.com> wrote:

>> > I think the potentially problematic case is "lots of I/O subchannels
>> with no valid device", and I think you can't get that under z/VM (which
>> will not give you those subchannels in the first place), but only on LPAR.
> Yes. But, this is in case of actual device. I just defined around 5k virtual
> dasd devices on zVM and did not enable them. i.e those subchannels are not
> blacklisted anymore, but they does not have an operational device. 
>
> other than zVM, other than testing this patch on different available LPARs,
> we tried to mimic the device availability with a custom test-kernel.
> Basically, modified the subchannel initialization code and inform the subchannel
> about the presence of a device, which then later fails during SNSID.
> It is a horrid way to test it, but i think it could simulate some LPARs with
> more than 1000 non-operational devices connected on it.

OK, that should be a way to figure out how userspace deals with the
extra uevents.

>
> ...snip...
>
>> - non-I/O subchannels (IIRC you can't have CHSC subchannels under z/VM,
>>   don't know about EADM, so that would need to be done on an LPAR as
>>   well)
> Thanks. Most of my tests were on IO-subchannel. I shall try few tests on CHSC
> and EADM.
>
>> - interaction with driverctl (and maybe mdevctl) for vfio-ccw
> I have done couple of tests with driverctl. Also, i was wondering if vfio-ccw
> can make use of 'dev_busid' sysfs-interface under each subchannels while writing
> the rules. This is totally different topic, which i do not want to mix up in
> this thread.

Oh, did not know about that interface.

<looks>
Doesn't the code need to check for 'w' for message subchannels, though?

'dev_busid' looks like a good fit for udev rules; maybe driverctl needs
special-casing? Or does it belong into mdevctl? cc:ing vfio-ccw
maintainers for awareness :)


  reply	other threads:[~2021-11-03 15:41 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-27  8:50 [RFC v2 0/1]s390/cio: remove uevent suppress from cio driver Vineeth Vijayan
2021-10-27  8:50 ` [PATCH] s390/cio: " Vineeth Vijayan
2021-11-02 15:42   ` Cornelia Huck
2021-11-02 15:31 ` [RFC v2 0/1]s390/cio: " Cornelia Huck
2021-11-03 13:17   ` Vineeth Vijayan
2021-11-03 15:41     ` Cornelia Huck [this message]
2021-11-05 14:11       ` Vineeth Vijayan

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=87a6ilb62b.fsf@redhat.com \
    --to=cohuck@redhat.com \
    --cc=farman@linux.ibm.com \
    --cc=linux-s390@vger.kernel.org \
    --cc=mjrosato@linux.ibm.com \
    --cc=oberpar@linux.ibm.com \
    --cc=vneethv@linux.ibm.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 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.