mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: vivien.didelot@savoirfairelinux.com
Cc: sfeldma@gmail.com, netdev@vger.kernel.org, jiri@resnulli.us,
	f.fainelli@gmail.com, linux-kernel@vger.kernel.org,
	kernel@savoirfairelinux.com
Subject: Re: [PATCH] net: switchdev: restrict vid range abstraction
Date: Wed, 29 Jul 2015 12:28:50 -0700 (PDT)	[thread overview]
Message-ID: <20150729.122850.41892076269756846.davem@davemloft.net> (raw)
In-Reply-To: <117624450.145106.1438197245281.JavaMail.zimbra@savoirfairelinux.com>

From: Vivien Didelot <vivien.didelot@savoirfairelinux.com>
Date: Wed, 29 Jul 2015 15:14:05 -0400 (EDT)

> While a range might be convenient to a user, exposing it to drivers is
> likely to end up writing the same vid_begin to vid_end for loop.

It is impossible not to expose this to the driver.

We have to have a prepare/commit sequence where the prepare part
can see the whole of the request.

It must know the full set of changes it must be able to support
in one go in order to signal a failure properly.

So in fact this is the most convenient way for the drivers to be
exposed to this issue.

  reply	other threads:[~2015-07-29 19:28 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-26 23:45 Vivien Didelot
2015-07-29  7:31 ` Scott Feldman
2015-07-29 18:28   ` David Miller
2015-07-29 19:14     ` Vivien Didelot
2015-07-29 19:28       ` David Miller [this message]
2015-07-29 21:17       ` Scott Feldman
2015-08-01  1:02         ` Vivien Didelot

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=20150729.122850.41892076269756846.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=f.fainelli@gmail.com \
    --cc=jiri@resnulli.us \
    --cc=kernel@savoirfairelinux.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=sfeldma@gmail.com \
    --cc=vivien.didelot@savoirfairelinux.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®