Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1195449

Re: [PATCH] net: switchdev: restrict vid range abstraction

From David Miller <davem@davemloft.net>
Newsgroups linux.kernel
Subject Re: [PATCH] net: switchdev: restrict vid range abstraction
Date 2015-07-29 21:30 +0200
Message-ID <pRFd8-AH-21@gated-at.bofh.it> (permalink)
References <pRu81-1ux-11@gated-at.bofh.it> <pREh4-7GV-9@gated-at.bofh.it> <pRF3s-pr-21@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH] net: switchdev: restrict vid range abstraction Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2015-07-27 01:50 +0200
  Re: [PATCH] net: switchdev: restrict vid range abstraction Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2015-07-29 21:20 +0200
    Re: [PATCH] net: switchdev: restrict vid range abstraction David Miller <davem@davemloft.net> - 2015-07-29 21:30 +0200
    Re: [PATCH] net: switchdev: restrict vid range abstraction Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2015-08-01 03:10 +0200

csiph-web