All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cornelia Huck <cohuck@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>, Halil Pasic <pasic@linux.ibm.com>
Cc: Parav Pandit <parav@nvidia.com>,
	"virtio-dev@lists.oasis-open.org"
	<virtio-dev@lists.oasis-open.org>,
	"sgarzare@redhat.com" <sgarzare@redhat.com>,
	"virtio-comment@lists.oasis-open.org"
	<virtio-comment@lists.oasis-open.org>,
	Shahaf Shuler <shahafs@nvidia.com>
Subject: [virtio-comment] Participation (was: Re: [virtio-comment] RE: [virtio-dev] RE: [PATCH v12 03/10] content: Rename confusing queue_notify_data and vqn names)
Date: Mon, 17 Apr 2023 10:47:47 +0200	[thread overview]
Message-ID: <878reqlwss.fsf@redhat.com> (raw)
In-Reply-To: <20230417024402-mutt-send-email-mst@kernel.org>

On Mon, Apr 17 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:

> I am not saying don't give feedback but I'm saying please help us
> all be more organized, feedback time really should be within a
> day or two, in rare cases up to a week.

Given that there are things like weekends, public holidays, etc. one day
does not look reasonable; while it certainly makes sense to continue if
no feedback is forthcoming for a few days, not accounting for the fact
that this is not the exclusive job for most/any of us is just a fast
track to either burnout or people dropping out of virtio standardization
altogether.

> And I'd like to remind everyone if you are going away you are supposed
> to report a leave of absence.

Well... "Each request shall be made by sending an email to the TC
mailing list at least 7 days before the Leave is to start." is probably
not going to work for many cases. (Also, in any other community I'm
participating in it is expected that you just might not be there or have
time to work on it every week -- I've always seen that leave of absence
thingy as something for a really long vacation or for something like
parental leave, for which the max 45 days is really too short...) Not to
mention that it applies to voting, not to participation on the lists.

> TC's that have meetings just take away voting rights from someone who
> does not attend two meetings in a row.  We do it by ballot so this does
> not apply, but I think we should set some limits in group's bylaws,
> anyway. Ideas on formalizing this? If not we can just have informal
> guidelines.  There's of course a flip side to this. Some patches
> seemingly go through two versions a day. Keeping up becomes a full time
> job. We'd need a guideline for that, too.

What do we actually expect from TC members? "Reply to emails" is not
part of any formal requirement AFAICS (and not all TC members do
participate on the lists on a regular basis anyway). The requirement is
only to vote on the ballots, and there you're completely free to vote
"abstain", so you can always squeeze in voting even if you're not able
to participate otherwise. I think that's fine.

"If I don't get a reply after $NUMBER_OF_WORKING_DAYS, I'll assume I can
proceed as I think fit" is a reasonable assumption to make, e.g. to
request a vote. Not sure if/how to formalize that. Also, how is this
supposed to work if the original author doesn't reply to comments?
Should the proposal be considered abandoned?

I agree that patch reposting is happening much too fast in some
cases. Not sure how to formalize that, either. Can we please just be
more mindful of that? Reviewing time is not free. If I'm trying to do a
timely review of something and constantly see new versions while I'm not
finished yet, I do not feel like my feedback is actually valued.

> I also feel high latency is one of the reasons people are beginning to
> ask to split into subcommitees where they won't have to deal with this
> kind of thing. Let's try to keep the latency low, please.

I think there's multiple things to unpack here.

- The very common strain of limited reviewer time. This seems to be an
  issue nearly everywhere. Encouraging more review helps; but if review
  and ensuing discussing turns into a time sink, it just cannot be
  handled at a reasonable activity level anymore.
- Latency due to missing feedback. Can be solved by just requesting a
  vote if no feedback is forthcoming in a reasonable time frame.
- Latency due to missing reaction to feedback. This means the proposal
  just doesn't make any progress. The onus is on the submitter here.
- Conflicting approaches favoured by different people. This cannot be
  resolved in a formal way; either people need to be convinced that a
  certain approach will work, a middle ground found, or a way worked out
  that the different approaches can co-exist. In any case, this usually
  means long discussions which can be very frustrating, but unless we
  want to bulldoze over some people this is something we'll have to
  live with. [Personally, I think this is the worst contributor to
  frustration, and not something that can be solved by subcommittees.]
- [I'm also not happy with the tone of some emails I've been seeing. I
  won't point to them in order not to stir up things that have already
  calmed down again.]


This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.

In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.

Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/


WARNING: multiple messages have this Message-ID (diff)
From: Cornelia Huck <cohuck@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>, Halil Pasic <pasic@linux.ibm.com>
Cc: Parav Pandit <parav@nvidia.com>,
	"virtio-dev@lists.oasis-open.org"
	<virtio-dev@lists.oasis-open.org>,
	"sgarzare@redhat.com" <sgarzare@redhat.com>,
	"virtio-comment@lists.oasis-open.org"
	<virtio-comment@lists.oasis-open.org>,
	Shahaf Shuler <shahafs@nvidia.com>
Subject: [virtio-dev] Participation (was: Re: [virtio-comment] RE: [virtio-dev] RE: [PATCH v12 03/10] content: Rename confusing queue_notify_data and vqn names)
Date: Mon, 17 Apr 2023 10:47:47 +0200	[thread overview]
Message-ID: <878reqlwss.fsf@redhat.com> (raw)
In-Reply-To: <20230417024402-mutt-send-email-mst@kernel.org>

On Mon, Apr 17 2023, "Michael S. Tsirkin" <mst@redhat.com> wrote:

> I am not saying don't give feedback but I'm saying please help us
> all be more organized, feedback time really should be within a
> day or two, in rare cases up to a week.

Given that there are things like weekends, public holidays, etc. one day
does not look reasonable; while it certainly makes sense to continue if
no feedback is forthcoming for a few days, not accounting for the fact
that this is not the exclusive job for most/any of us is just a fast
track to either burnout or people dropping out of virtio standardization
altogether.

> And I'd like to remind everyone if you are going away you are supposed
> to report a leave of absence.

Well... "Each request shall be made by sending an email to the TC
mailing list at least 7 days before the Leave is to start." is probably
not going to work for many cases. (Also, in any other community I'm
participating in it is expected that you just might not be there or have
time to work on it every week -- I've always seen that leave of absence
thingy as something for a really long vacation or for something like
parental leave, for which the max 45 days is really too short...) Not to
mention that it applies to voting, not to participation on the lists.

> TC's that have meetings just take away voting rights from someone who
> does not attend two meetings in a row.  We do it by ballot so this does
> not apply, but I think we should set some limits in group's bylaws,
> anyway. Ideas on formalizing this? If not we can just have informal
> guidelines.  There's of course a flip side to this. Some patches
> seemingly go through two versions a day. Keeping up becomes a full time
> job. We'd need a guideline for that, too.

What do we actually expect from TC members? "Reply to emails" is not
part of any formal requirement AFAICS (and not all TC members do
participate on the lists on a regular basis anyway). The requirement is
only to vote on the ballots, and there you're completely free to vote
"abstain", so you can always squeeze in voting even if you're not able
to participate otherwise. I think that's fine.

"If I don't get a reply after $NUMBER_OF_WORKING_DAYS, I'll assume I can
proceed as I think fit" is a reasonable assumption to make, e.g. to
request a vote. Not sure if/how to formalize that. Also, how is this
supposed to work if the original author doesn't reply to comments?
Should the proposal be considered abandoned?

I agree that patch reposting is happening much too fast in some
cases. Not sure how to formalize that, either. Can we please just be
more mindful of that? Reviewing time is not free. If I'm trying to do a
timely review of something and constantly see new versions while I'm not
finished yet, I do not feel like my feedback is actually valued.

> I also feel high latency is one of the reasons people are beginning to
> ask to split into subcommitees where they won't have to deal with this
> kind of thing. Let's try to keep the latency low, please.

I think there's multiple things to unpack here.

- The very common strain of limited reviewer time. This seems to be an
  issue nearly everywhere. Encouraging more review helps; but if review
  and ensuing discussing turns into a time sink, it just cannot be
  handled at a reasonable activity level anymore.
- Latency due to missing feedback. Can be solved by just requesting a
  vote if no feedback is forthcoming in a reasonable time frame.
- Latency due to missing reaction to feedback. This means the proposal
  just doesn't make any progress. The onus is on the submitter here.
- Conflicting approaches favoured by different people. This cannot be
  resolved in a formal way; either people need to be convinced that a
  certain approach will work, a middle ground found, or a way worked out
  that the different approaches can co-exist. In any case, this usually
  means long discussions which can be very frustrating, but unless we
  want to bulldoze over some people this is something we'll have to
  live with. [Personally, I think this is the worst contributor to
  frustration, and not something that can be solved by subcommittees.]
- [I'm also not happy with the tone of some emails I've been seeing. I
  won't point to them in order not to stir up things that have already
  calmed down again.]


---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org


  reply	other threads:[~2023-04-17  8:47 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-05  1:06 [virtio-comment] [PATCH v12 00/10] Rename queue number to queue index Parav Pandit
2023-04-05  1:06 ` [virtio-dev] " Parav Pandit
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 01/10] content: Add vq index text Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  5:26   ` [virtio-comment] " Michael S. Tsirkin
2023-04-05  5:26     ` [virtio-dev] " Michael S. Tsirkin
2023-04-05 13:00     ` [virtio-comment] " Parav Pandit
2023-04-05 13:00       ` [virtio-dev] " Parav Pandit
2023-04-05  9:18   ` [virtio-comment] " Michael S. Tsirkin
2023-04-05  9:18     ` [virtio-dev] " Michael S. Tsirkin
2023-04-05 13:03     ` [virtio-comment] " Parav Pandit
2023-04-05 13:03       ` [virtio-dev] " Parav Pandit
2023-04-07 11:30       ` [virtio-comment] " Michael S. Tsirkin
2023-04-07 11:30         ` Michael S. Tsirkin
2023-04-07 15:34         ` [virtio-comment] " Parav Pandit
2023-04-07 15:34           ` Parav Pandit
2023-04-07 15:54           ` [virtio-comment] " Michael S. Tsirkin
2023-04-07 15:54             ` Michael S. Tsirkin
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 02/10] content.tex Replace virtqueue number with index Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  9:46   ` [virtio-comment] " Cornelia Huck
2023-04-05  9:46     ` [virtio-dev] " Cornelia Huck
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 03/10] content: Rename confusing queue_notify_data and vqn names Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  5:22   ` [virtio-comment] " Michael S. Tsirkin
2023-04-05  5:22     ` [virtio-dev] " Michael S. Tsirkin
2023-04-05 13:20     ` [virtio-comment] " Parav Pandit
2023-04-05 13:20       ` [virtio-dev] " Parav Pandit
2023-04-10 13:25       ` [virtio-comment] " Michael S. Tsirkin
2023-04-10 13:25         ` [virtio-dev] " Michael S. Tsirkin
2023-04-05  5:30   ` [virtio-comment] " Michael S. Tsirkin
2023-04-05  5:30     ` [virtio-dev] " Michael S. Tsirkin
2023-04-05  9:57     ` [virtio-comment] " Cornelia Huck
2023-04-05  9:57       ` [virtio-dev] " Cornelia Huck
2023-04-05 13:21     ` [virtio-comment] " Parav Pandit
2023-04-05 13:21       ` [virtio-dev] " Parav Pandit
2023-04-05 15:27       ` [virtio-comment] " Halil Pasic
2023-04-05 15:27         ` Halil Pasic
2023-04-05 15:58         ` [virtio-comment] " Parav Pandit
2023-04-05 15:58           ` Parav Pandit
2023-04-07 11:43           ` [virtio-comment] " Michael S. Tsirkin
2023-04-07 11:43             ` [virtio-dev] " Michael S. Tsirkin
2023-04-11  8:56             ` Cornelia Huck
2023-04-11  8:56               ` [virtio-dev] " Cornelia Huck
2023-04-11 13:35               ` Parav Pandit
2023-04-11 13:35                 ` [virtio-dev] " Parav Pandit
2023-04-17  3:18                 ` Halil Pasic
2023-04-17  3:18                   ` [virtio-dev] " Halil Pasic
2023-04-17  7:04                   ` Michael S. Tsirkin
2023-04-17  7:04                     ` [virtio-dev] " Michael S. Tsirkin
2023-04-17  8:47                     ` Cornelia Huck [this message]
2023-04-17  8:47                       ` [virtio-dev] Participation (was: Re: [virtio-comment] RE: [virtio-dev] RE: [PATCH v12 03/10] content: Rename confusing queue_notify_data and vqn names) Cornelia Huck
2023-04-17 11:45                       ` [virtio-comment] " Michael S. Tsirkin
2023-04-17 11:45                         ` [virtio-dev] " Michael S. Tsirkin
2023-04-17 16:13                     ` [virtio-comment] Re: [virtio-dev] Re: [virtio-comment] RE: [virtio-dev] RE: [PATCH v12 03/10] content: Rename confusing queue_notify_data and vqn names Halil Pasic
2023-04-17 16:13                       ` Halil Pasic
2023-04-07 11:44       ` [virtio-comment] " Michael S. Tsirkin
2023-04-07 11:44         ` [virtio-dev] " Michael S. Tsirkin
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 04/10] transport-pci: Avoid first vq index reference Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 05/10] transport-mmio: Rename QueueNum register Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 06/10] transport-mmio: Avoid referring to zero based index Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 07/10] transport-ccw: Rename queue depth/size to other transports Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 08/10] transport-ccw: Refer to the vq by its index Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 09/10] virtio-net: Avoid duplicate receive queue example Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  1:06 ` [virtio-comment] [PATCH v12 10/10] virtio-net: Describe RSS using rss rq id Parav Pandit
2023-04-05  1:06   ` [virtio-dev] " Parav Pandit
2023-04-05  9:17   ` [virtio-comment] " Michael S. Tsirkin
2023-04-05  9:17     ` [virtio-dev] " Michael S. Tsirkin
2023-04-05 13:02     ` [virtio-comment] " Parav Pandit
2023-04-05 13:02       ` [virtio-dev] " Parav Pandit
2023-04-07 11:32       ` [virtio-comment] " Michael S. Tsirkin
2023-04-07 11:32         ` [virtio-dev] " Michael S. Tsirkin
2023-04-05  5:32 ` [virtio-comment] Re: [PATCH v12 00/10] Rename queue number to queue index Michael S. Tsirkin
2023-04-05  5:32   ` [virtio-dev] " Michael S. Tsirkin
2023-04-05 13:11   ` [virtio-comment] " Parav Pandit
2023-04-05 13:11     ` [virtio-dev] " Parav Pandit
2023-04-05  9:20 ` [virtio-comment] " Michael S. Tsirkin
2023-04-05  9:20   ` [virtio-dev] " Michael S. Tsirkin
2023-04-05 13:05   ` [virtio-comment] " Parav Pandit
2023-04-05 13:05     ` [virtio-dev] " Parav Pandit

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=878reqlwss.fsf@redhat.com \
    --to=cohuck@redhat.com \
    --cc=mst@redhat.com \
    --cc=parav@nvidia.com \
    --cc=pasic@linux.ibm.com \
    --cc=sgarzare@redhat.com \
    --cc=shahafs@nvidia.com \
    --cc=virtio-comment@lists.oasis-open.org \
    --cc=virtio-dev@lists.oasis-open.org \
    /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.