Signed-off-by: Wei Liu
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 0c55b0fedbe2..e212c6a42ddf 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -17295,7 +17295,7 @@ F: Documentation/ABI/stable/sysfs-hypervisor-xen
F
On Sat, Aug 10, 2019 at 12:31:08PM +0200, Greg Kroah-Hartman wrote:
> When calling debugfs functions, there is no need to ever check the
> return value. The function can work or not, but the code logic should
> never do something different based on this.
>
> Cc: Wei Liu
> Cc:
On Sat, Jan 16, 2021 at 02:02:01PM +0100, Andrea Parri wrote:
> On Fri, Jan 15, 2021 at 08:30:22PM -0800, Jakub Kicinski wrote:
> > On Thu, 14 Jan 2021 21:26:28 +0100 Andrea Parri (Microsoft) wrote:
> > > For additional robustness in the face of Hyper-V errors or malicious
> > > behavior, validate
On Tue, Nov 17, 2020 at 09:23:08PM +0100, Enrico Weigelt, metux IT consult
wrote:
> Make it possible to opt-out from hyperv support.
>
"Hyper-V support".
Have you tested this patch? If so, how?
> Signed-off-by: Enrico Weigelt, metux IT consult
> ---
> arch/x86/Kconfig | 7 +++
On Wed, Nov 18, 2020 at 05:37:15PM -0800, Jakub Kicinski wrote:
> On Wed, 18 Nov 2020 16:33:10 +0100 Andrea Parri (Microsoft) wrote:
> > Lack of validation could lead to out-of-bound reads and information
> > leaks (cf. usage of nvdev->chan_table[]). Check that the number of
> > allocated sub-chan
rea Parri (Microsoft)
> Reviewed-by: Michael Kelley
> Acked-by: Jakub Kicinski
> Cc: "David S. Miller"
> Cc: Jakub Kicinski
> Cc: netdev@vger.kernel.org
Reviewed-by: Wei Liu
On Mon, Nov 09, 2020 at 11:03:59AM +0100, Andrea Parri (Microsoft) wrote:
> Currently, VMbus drivers use pointers into guest memory as request IDs
> for interactions with Hyper-V. To be more robust in the face of errors
> or malicious behavior from a compromised Hyper-V, avoid exposing
> guest memo
On Tue, Dec 22, 2020 at 01:59:34PM +, Michael Kelley wrote:
> From: Andrea Parri (Microsoft) Sent: Monday,
> December 7, 2020 8:53 PM
> >
> > From: Andres Beltran
> >
> > Pointers to ring-buffer packets sent by Hyper-V are used within the
> > guest VM. Hyper-V can send packets with erroneo
n-netback/xenbus.c:1001: warning: Function parameter or member
> 'id' not described in 'netback_probe'
>
> Cc: Wei Liu
If this is ever needed:
Acked-by: Wei Liu
e a event being spurious if no rx OR no tx requests are
> pending.
>
> Fix that plus using local variables for rx/tx pending indicators in
> order to split function calls and if condition.
>
> Fixes: 23025393dbeb3b ("xen/netback: use lateeoi irq binding")
> Signed-off-by: Juergen Gross
Reviewed-by: Wei Liu
On Fri, Feb 12, 2021 at 05:50:50PM +0100, Andrea Parri wrote:
> Hi all,
[...]
> 2) IIUC a8c3209998afb5 could be dropped (after rebase) without further modi-
>fications to hyperv-next.
I've reverted the said patch from hyperv-next.
Wei.
On Wed, Apr 07, 2021 at 08:08:59AM +, Dexuan Cui wrote:
> > From: kernel test robot
> > Sent: Tuesday, April 6, 2021 6:31 PM
> > ...
> > Hi Dexuan,
> > I love your patch! Perhaps something to improve:
> >
> > All warnings (new ones prefixed by >>):
> >
> >drivers/pci/controller/pci-hype
On Tue, Apr 06, 2021 at 04:23:21PM -0700, Dexuan Cui wrote:
[...]
> +config MICROSOFT_MANA
> + tristate "Microsoft Azure Network Adapter (MANA) support"
> + default m
> + depends on PCI_MSI
> + select PCI_HYPERV
OOI which part of the code requires PCI_HYPERV?
Asking because I can'
On Wed, Apr 07, 2021 at 02:34:01PM +, Haiyang Zhang wrote:
>
>
> > -Original Message-
> > From: Wei Liu
> > Sent: Wednesday, April 7, 2021 9:17 AM
> > To: Dexuan Cui
> > Cc: da...@davemloft.net; k...@kernel.org; KY Srinivasan
> > ; H
On Mon, Feb 01, 2021 at 03:48:10PM +0100, Andrea Parri (Microsoft) wrote:
> Andrea Parri (Microsoft) (4):
> x86/hyperv: Load/save the Isolation Configuration leaf
> Drivers: hv: vmbus: Restrict vmbus_devices on isolated guests
> Drivers: hv: vmbus: Enforce 'VMBus version >= 5.2' on isolated g
> associated queue held.
>
> Reported-by: Igor Druzhinin
> Fixes: 23025393dbeb3b8b3 ("xen/netback: use lateeoi irq binding")
> Cc: sta...@vger.kernel.org
> Signed-off-by: Juergen Gross
Acked-by: Wei Liu
> Signed-off-by: Thomas Gleixner
Acked-by: Wei Liu
aces to take the pointer to the xenbus device as a
> parameter instead of the domain id of the other side.
>
> While at it remove the stale prototype of bind_evtchn_to_irq_lateeoi().
>
> Signed-off-by: Juergen Gross
Reviewed-by: Wei Liu
On Mon, Apr 30, 2018 at 11:01:48PM +0200, Marek Marczykowski-Górecki wrote:
> Tx response ID is fetched from shared page, so make sure it is sane
> before using it as an array index.
>
> CC: sta...@vger.kernel.org
> Signed-off-by: Marek Marczykowski-Górecki
> ---
> drivers/net/xen-netfront.c | 1
On Fri, Apr 20, 2018 at 10:47:31AM +, Stanislav Kinsburskii wrote:
>
> #include
> #include
> @@ -1649,6 +1650,7 @@ static int __init netback_init(void)
> PTR_ERR(xen_netback_dbg_root));
> #endif /* CONFIG_DEBUG_FS */
>
> + (void) xen_netbk_fi_init();
If you c
ng 'netdev_tx_t' in this driver too.
>
> Signed-off-by: Luc Van Oostenryck
Acked-by: Wei Liu
ng 'netdev_tx_t' in this driver too.
>
> Signed-off-by: Luc Van Oostenryck
Reviewed-by: Wei Liu
>
> Signed-off-by: Joe Perches
> ---
> drivers/net/xen-netback/xenbus.c | 4 +-
> drivers/net/xen-netfront.c | 6 +--
Reviewed-by: Wei Liu
initializers to match their
> nested position in the struct. After this, compilation succeeds again.
>
> Fixes: 4ab6c99d99bb ("sock: MSG_ZEROCOPY notification coalescing")
> Reported-by: kbuild bot
> Signed-off-by: Willem de Bruijn
Acked-by: Wei Liu
On Thu, Feb 28, 2019 at 09:46:57AM +, Paul Durrant wrote:
> > -Original Message-
> > From: Igor Druzhinin [mailto:igor.druzhi...@citrix.com]
> > Sent: 28 February 2019 02:03
> > To: xen-de...@lists.xenproject.org; netdev@vger.kernel.org;
> > linux-ker.
On Thu, Feb 28, 2019 at 12:07:07PM +, Paul Durrant wrote:
> Yes, I meant kfree_skb(nskb).
>
In that case I think your patch looks fine.
Wei.
of page cache writes coexist with active atomic skb allocations from
> network traffic. Additionally, rework the logic to deal with frag_list
> deallocation in a single place.
>
> Signed-off-by: Paul Durrant
> Signed-off-by: Igor Druzhinin
Acked-by: Wei Liu
> ---
> drivers/n
he guest can no longer be pinged.
>
> xen_net{front|back} should not allow the user to set this value which causes
> network problems.
>
> Reported-by: Chen Shi
> Signed-off-by: Mohammed Gamal
Acked-by: Wei Liu
CC netfront maintainers
22:6: warning: symbol 'xenvif_rx_skb'
> was not declared. Should it be static?
>
> Signed-off-by: Colin Ian King
Acked-by: Wei Liu
Thanks
On Mon, Apr 27, 2020 at 08:30:43PM -0700, Nathan Chancellor wrote:
> netvsc_start_xmit is used as a callback function for the ndo_start_xmit
> function pointer. ndo_start_xmit's return type is netdev_tx_t but
> netvsc_start_xmit's return type is int.
>
> This causes a failure with Control Flow Int
On Mon, Sep 14, 2020 at 04:46:00PM +0800, Boqun Feng wrote:
> On Thu, Sep 10, 2020 at 10:34:52PM +0800, Boqun Feng wrote:
> > When PAGE_SIZE > HV_HYP_PAGE_SIZE, we need the ringbuffer size to be at
> > least 2 * PAGE_SIZE: one page for the header and at least one page of
> > the data part (because
On Tue, Jul 21, 2020 at 09:41:25AM +0800, Boqun Feng wrote:
> Since the hypervisor always uses 4K as its page size, the size of PFNs
> used for gpadl should be HV_HYP_PAGE_SIZE rather than PAGE_SIZE, so
> adjust this accordingly as the preparation for supporting 16K/64K page
> size guests.
It may
two patches to make it easy to review.
>
> Signed-off-by: Boqun Feng
Reviewed-by: Wei Liu
Hi Chi
Thanks for your patch. A few things need to be fixed before it can be
accepted upstream.
On Fri, Jul 17, 2020 at 06:04:31AM +, Chi Song wrote:
> The network is observed with low performance, if TX indirection table
> is imbalance. But the table is in memory and set in runtime, it's
>
On Wed, Sep 16, 2020 at 11:48:06AM +0800, Boqun Feng wrote:
> This patchset add the necessary changes to support guests whose page
> size is larger than 4K. And the main architecture which we develop this
> for is ARM64 (also it's the architecture that I use to test this
> feature).
>
> Previous v
On Mon, Dec 10, 2018 at 10:53:29AM -0500, Yangtao Li wrote:
> Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.
>
> Signed-off-by: Yangtao Li
Acked-by: Wei Liu
Thanks for the patch.
CC netfront maintainers.
On Mon, Nov 20, 2017 at 11:41:09AM +0100, Eduardo Otubo wrote:
> When unloading module xen_netfront from guest, dmesg would output
> warning messages like below:
>
> [ 105.236836] xen:grant_table: WARNING: g.e. 0x903 still in use!
> [ 105.236839] deferring g.e. 0x90
On Thu, Jun 25, 2020 at 11:37:20AM -0400, Andres Beltran wrote:
> From: Andres Beltran (Microsoft)
>
> Currently, VMbus drivers use pointers into guest memory as request IDs
> for interactions with Hyper-V. To be more robust in the face of errors
> or malicious behavior from a compromised Hyper-V
On Fri, Jun 26, 2020 at 04:48:17PM +0200, Andrea Parri wrote:
> On Fri, Jun 26, 2020 at 01:42:27PM +0000, Wei Liu wrote:
> > On Thu, Jun 25, 2020 at 11:37:20AM -0400, Andres Beltran wrote:
> > > From: Andres Beltran (Microsoft)
> > >
> > > Currently, VMbus dr
On Mon, Jun 29, 2020 at 09:33:15PM +, Haiyang Zhang wrote:
>
>
> > -Original Message-
> > From: Andres Beltran
> > Sent: Monday, June 29, 2020 4:02 PM
> > To: KY Srinivasan ; Haiyang Zhang
> > ; Stephen Hemminger ;
> > wei@kernel.org
> > Cc: linux-hyp...@vger.kernel.org; linux-ke
David
Do you want this to go through net tree? I can submit it via hyperv tree
if that's preferred.
Wei.
On Tue, Apr 28, 2020 at 10:54:56AM -0700, Nathan Chancellor wrote:
> netvsc_start_xmit is used as a callback function for the ndo_start_xmit
> function pointer. ndo_start_xmit's return type i
On Fri, 13 Sep 2019 at 13:47, Paul Durrant wrote:
>
> My Citrix email address will expire shortly.
>
> Signed-off-by: Paul Durrant
Acked-by: Wei Liu
On Fri, 13 Sep 2019 at 16:28, Wei Liu wrote:
>
> On Fri, 13 Sep 2019 at 13:47, Paul Durrant wrote:
> >
> > My Citrix email address will expire shortly.
> >
> > Signed-off-by: Paul Durrant
>
> Acked-by: Wei Liu
Or rather:
Acked-by: Wei Liu
Add a flag to indicate if a queue is rate-limited. Test the flag in
NAPI poll handler and avoid rescheduling the queue if true, otherwise
we risk locking up the host. The rescheduling will be done in the
timer callback function.
Reported-by: Jean-Louis Dupond
Signed-off-by: Wei Liu
Tested-by
On Mon, Oct 16, 2017 at 05:29:38PM -0700, Kees Cook wrote:
> In preparation for unconditionally passing the struct timer_list pointer to
> all timer callbacks, switch to using the new timer_setup() and from_timer()
> to pass the timer pointer explicitly.
>
> Cc: Wei Liu
> Cc:
On Tue, May 12, 2015 at 07:18:26PM +0200, Joao Martins wrote:
> Checks for "feature-persistent" that indicates persistent grants
> support. Adds max_persistent_grants module param that specifies the max
> number of persistent grants, which if set to zero disables persistent
> grants.
>
> Signed-of
On Tue, May 12, 2015 at 07:18:30PM +0200, Joao Martins wrote:
> By introducing persistent grants we speed up the RX thread with the
> decreased copy cost, that leads to a throughput decrease of 20%.
> It is observed that the rx_queue stays mostly at 10% of its capacity,
> as opposed to full capacit
On Tue, May 19, 2015 at 11:56:39PM +0100, Julien Grall wrote:
>
> >>diff --git a/drivers/net/xen-netback/common.h
> >>b/drivers/net/xen-netback/common.h
> >>index 0eda6e9..c2a5402 100644
> >>--- a/drivers/net/xen-netback/common.h
> >>+++ b/drivers/net/xen-netback/common.h
> >>@@ -204,7 +204,7 @@
t; prevented bandwidth limiting from working. Initialize the field as it
> was done before multiqueue support was added.
>
> Signed-off-by: Ross Lagerwall
Acked-by: Wei Liu
> ---
>
> Changed in v2: Fixed multiple-assignment.
>
> drivers/net/xen-netback/xenbus.c | 1
that for you.
> >
> > Thanks, Konrad.
> >
> > I am copying Wei too who had fixed the below problem earlier.
> > It fixed the incorrect ip_hdr(). tcp_hdr() still needs to fixed.
> >
> > commit d554f73df6bc35ac8f6a65e5560bf1d31dfebed9
> > Author:
t; [-Wformat=]
> (txreq.offset&~PAGE_MASK) + txreq.size);
> ^
>
> txreq.offset and .size are uint16_t fields.
>
> Signed-off-by: Ian Campbell
Acked-by: Wei Liu
> ---
> drivers/net/xen-netback/netback.c |2 +-
> 1 file changed, 1 insertion(+), 1 de
On Fri, May 29, 2015 at 05:24:53PM +0100, Ian Campbell wrote:
[...]
> if (be->vif != NULL)
> return 0;
> @@ -417,12 +409,23 @@ static int backend_create_xenvif(struct backend_info
> *be)
> return (err < 0) ? err : -EINVAL;
> }
>
> + script = xenbus_rea
On Mon, Jun 01, 2015 at 09:52:45AM +0100, Ian Campbell wrote:
> On Fri, 2015-05-29 at 18:38 +0100, Wei Liu wrote:
> > On Fri, May 29, 2015 at 05:24:53PM +0100, Ian Campbell wrote:
> > [...]
> > > if (be->vif != NULL)
> > > return 0;
&g
t; [-Wformat=]
> (txreq.offset&~PAGE_MASK) + txreq.size);
> ^
>
> PAGE_MASK's type can vary by arch, so a cast is needed.
>
> Signed-off-by: Ian Campbell
Acked-by: Wei Liu
>
> v2: Cast to unsigned long, since PAGE_MASK can vary by arch.
>
t this state machine is
> fragile and prone to anger...
>
> A modern Xen system only relies on the hotplug uevent for driver
> domains, when the backend is in the same domain as the toolstack it
> will run the necessary setup/teardown directly in the correct sequence
> wrt xenstore
Hi Eric
Sorry for coming late to the discussion.
On Thu, Apr 16, 2015 at 05:42:16AM -0700, Eric Dumazet wrote:
> On Thu, 2015-04-16 at 11:01 +0100, George Dunlap wrote:
>
> > He suggested that after he'd been prodded by 4 more e-mails in which two
> > of us guessed what he was trying to get at.
On Fri, May 22, 2015 at 10:24:39AM +, Joao Martins wrote:
>
> On 19 May 2015, at 17:23, Wei Liu wrote:
> > On Tue, May 12, 2015 at 07:18:27PM +0200, Joao Martins wrote:
> >> Introduces persistent grants for TX path which follows similar code path
> >> as th
On Fri, May 22, 2015 at 10:25:10AM +, Joao Martins wrote:
>
> On 19 May 2015, at 17:32, Wei Liu wrote:
>
> > On Tue, May 12, 2015 at 07:18:28PM +0200, Joao Martins wrote:
> >> It starts by doing a lookup in the tree for a gref. If no persistent
> >> grant
On Fri, May 22, 2015 at 10:26:48AM +, Joao Martins wrote:
[...]
> >>return IRQ_HANDLED;
> >> }
> >> @@ -168,8 +169,12 @@ static int xenvif_start_xmit(struct sk_buff *skb,
> >> struct net_device *dev)
> >>cb = XENVIF_RX_CB(skb);
> >>cb->expires = jiffies + vif->drain_timeout;
> >>
On Thu, Oct 15, 2015 at 02:02:47PM -0400, Insu Yun wrote:
> I changed patch with valid format.
>
> On Thu, Oct 15, 2015 at 2:02 PM, Insu Yun wrote:
>
> > Since vzalloc can be failed in memory pressure,
> > writes -ENOMEM to xenstore to indicate error.
> >
> > Signed-off-by: Insu Yun
> > ---
> >
On Fri, Oct 16, 2015 at 10:05:21AM +0100, Wei Liu wrote:
> On Thu, Oct 15, 2015 at 02:02:47PM -0400, Insu Yun wrote:
> > I changed patch with valid format.
> >
> > On Thu, Oct 15, 2015 at 2:02 PM, Insu Yun wrote:
> >
> > > Since vzalloc can be failed in memo
ned-off-by: Joe Jin
> Cc: Wei Liu
> Cc: Ian Campbell
> Cc: David S. Miller
> ---
> drivers/net/xen-netfront.c |9 ++---
> 1 files changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/xen-netfront.c b/drivers/net/xen-netfront.c
> index f821a
On Fri, Oct 23, 2015 at 05:44:44PM +0800, Joe Jin wrote:
> Should not allocate xen vif queues number more than online cpus.
I think it's absolutely fine for administrators to override the value
should they choose to.
Wei.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the
et a hash on the skb.
>
> Signed-off-by: Paul Durrant
> Cc: Ian Campbell
> Cc: Wei Liu
Reviewed-by: Wei Liu
[...]
>
> + /* We support hash values. */
> + err = xenbus_printf(xbt, dev->nodename,
> + "feat
On Wed, Oct 21, 2015 at 11:36:24AM +0100, Paul Durrant wrote:
> Advertise the capability to handle a hash mapping specified by the
> frontend (see netif.h for details).
>
> Add an ndo_select() entry point so that, of the frontend does specify a
"if the frontend ..."
> hash mapping, the skb hash
> Cc: Ian Campbell
> Cc: Wei Liu
Reviewed-by: Wei Liu
> static int xenvif_rx_ring_slots_needed(struct xenvif *vif)
> {
> - if (vif->gso_mask)
> - return DIV_ROUND_UP(vif->dev->gso_max_size, PAGE_SIZE) + 1;
> + int needed;
> +
> + i
ithm itself was based on pseudo-code provided by
> Microsoft at:
>
> https://msdn.microsoft.com/en-us/library/windows/hardware/ff570725.aspx
>
> Signed-off-by: Paul Durrant
> Cc: Ian Campbell
> Cc: Wei Liu
[...]
>
> diff --git a/drivers/net/xen-netback/interface.
_netif_rx/g
> ,s/netif_extra_info/xen_netif_extra_info/g
> w
> EOF
>
> indent --linux-style netif.h -o include/xen/interface/io/netif.h
>
> Signed-off-by: Paul Durrant
> Cc: Konrad Rzeszutek Wilk
> Cc: Boris Ostrovsky
> Cc: David Vrabel
> Cc: Wei Liu
> ---
&g
n the array and
> removes many lines of code.
>
> Signed-off-by: Paul Durrant
> Cc: Ian Campbell
> Cc: Wei Liu
Reviewed-by: Wei Liu
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo i
modification, for better bisectability.
>
> Signed-off-by: Paul Durrant
> Cc: Ian Campbell
> Cc: Wei Liu
Reviewed-by: Wei Liu
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at
On Thu, May 05, 2016 at 12:19:27PM +0100, Paul Durrant wrote:
[...]
>
> +static int connect_ctrl_ring(struct backend_info *be)
> +{
Please use goto style error handling in this function.
Other than this the code looks good.
Wei.
in a hash extra
> info fragment passed from the guest frontend in a transmit-side
> (i.e. netback receive side) packet to set the skb hash accordingly.
>
> Signed-off-by: Paul Durrant
> Cc: Wei Liu
Acked-by: Wei Liu
calculated for
> guest receive-side packets (i.e. netback transmit side) to the frontend.
>
> Signed-off-by: Paul Durrant
> Cc: Wei Liu
Acked-by: Wei Liu
patch added the necessary boilerplate for mapping the control
> ring from the frontend, should it be created. This patch adds
> implementations for each of the defined protocol messages.
>
> Signed-off-by: Paul Durrant
> Cc: Wei Liu
> ---
> drivers/net/xen-netback/Makefile
patch added the necessary boilerplate for mapping the control
> ring from the frontend, should it be created. This patch adds
> implementations for each of the defined protocol messages.
>
> Signed-off-by: Paul Durrant
Acked-by: Wei Liu
necessary code to xen-netback to map this new shared
> ring, should it be created by a frontend, but does not add implementations
> for any of the defined protocol messages. These are added in a subsequent
> patch for clarity.
>
> Signed-off-by: Paul Durrant
Acked-by: Wei Liu
ntered when processing packets containing extra info fragments.
> This patch fixes the problem.
>
> Signed-off-by: Paul Durrant
> Reported-by: Jan Beulich
> Cc: Wei Liu
Acked-by: Wei Liu
> ---
> drivers/net/xen-netback/netback.c | 1 +
> 1 file changed, 1 insertio
patch added the necessary boilerplate for mapping the control
> ring from the frontend, should it be created. This patch adds
> implementations for each of the defined protocol messages.
>
> Signed-off-by: Paul Durrant
> Cc: Wei Liu
Acked-by: Wei Liu
static analysis checks.
>
> This patch replaces checks of 'len' with 'copy_op.len' in both cases
> to correct the incorrect check, keep the two checks consistent, and to
> make it clear what the checks are for.
>
> Signed-off-by: Paul Durrant
>
ient to ensure
> it is only called if xenvif_init_hash() was called previously.
>
> Signed-off-by: Paul Durrant
> Reported-by: Boris Ostrovsky
> Tested-by: Boris Ostrovsky
> Cc: Wei Liu
Acked-by: Wei Liu
> ---
> drivers/net/xen-netback/interface.c | 3 +--
> 1 file cha
On Mon, Feb 01, 2016 at 02:40:53PM +, Paul Durrant wrote:
[...]
> +static int xen_register_mcast_ctrl_watch(struct xenbus_device *dev,
> + struct xenvif *vif)
> +{
> + int err = 0;
> + char *node;
> + unsigned maxlen = strlen(dev->otherend) +
> +
On Tue, Feb 02, 2016 at 11:31:08AM +, Paul Durrant wrote:
[...]
> +static int xen_register_mcast_ctrl_watch(struct xenbus_device *dev,
> + struct xenvif *vif)
> +{
> + int err = 0;
> + char *node;
> + unsigned maxlen = strlen(dev->otherend) +
> +
ture off.
> Instead the backend watches the "request-multicast-control" key in xenstore
> and turns the feature off if the key value is written to zero.
>
> This patch adds support for dynamic multicast control in xen-netback.
>
> Signed-off-by: Paul Durrant
> Cc: Ian Campbell
> Cc: Wei Liu
Acked-by: Wei Liu
AB}/g
> w
> $
> w
> EOF
>
> indent --line-length 80 --linux-style netif.h \
> -o include/xen/interface/io/netif.h
>
> Signed-off-by: Paul Durrant
> Cc: Konrad Rzeszutek Wilk
> Cc: Boris Ostrovsky
> Cc: David Vrabel
> Cc: Wei Liu
Acked-by: Wei Liu
On Thu, Mar 10, 2016 at 12:30:28PM +, Paul Durrant wrote:
> Remove the "prepare for reconnect" pr_info in xenbus.c. It's largely
> uninteresting and the states of the frontend and backend can easily be
> observed by watching the (o)xenstored log.
>
> Signed-off-b
here
> is only ever a single extra info fragment.
>
> This patch modifies xenvif_get_extras() to pass back a count of extra
> info fragments, which is then passed to make_tx_response() (after
> possibly being stashed in pending_tx_info for deferred responses).
>
> Signed-off-b
On Fri, Feb 19, 2016 at 11:44:51AM +, Ian Campbell wrote:
> Wei has been picking this up for quite a while now.
>
> Signed-off-by: Ian Campbell
> Cc: Wei Liu
Acked-by: Wei Liu
> ---
> MAINTAINERS | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/MAINTA
I think the original code is fine.
Wei.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
The comment at the beginning of the file is the canonical source of
licenses for this module. Currently it contains GPL and MIT license. Fix
the code to reflect the reality.
Signed-off-by: Wei Liu
---
drivers/net/xen-netback/netback.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff
On Fri, Jan 22, 2016 at 01:14:24PM +, David Vrabel wrote:
> On 22/01/16 12:34, Wei Liu wrote:
> > The comment at the beginning of the file is the canonical source of
> > licenses for this module. Currently it contains GPL and MIT license. Fix
> > the code to reflect the r
nt, simply make the code to use "GPL" only.
Signed-off-by: Wei Liu
---
drivers/net/xen-netback/netback.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/xen-netback/netback.c
b/drivers/net/xen-netback/netback.c
index 61b97c3..e9601d1 100644
--- a/drivers/
On Fri, Jan 22, 2016 at 08:25:21PM +, One Thousand Gnomes wrote:
> > The fact what include/linux/license.h:license_is_gpl_compatible includes
> > "Dual MIT/GPL" as an option seems to suggest that it is enough of a thing
> > to be validly used as the contents of a MODULE_LICENSE() thing.
>
> Ye
On Thu, Mar 02, 2017 at 10:50:20AM +, Anoob Soman wrote:
> Allocation of new_hash, inside xenvif_new_hash(), always happen
> in softirq context, so use GFP_ATOMIC instead of GFP_KERNEL for new
> hash allocation.
>
> Signed-off-by: Anoob Soman
Acked-by: Wei Liu
On Thu, Mar 02, 2017 at 12:56:20PM +0100, Juergen Gross wrote:
> With commits f16f1df65 and 9a6cdf52b we get in our Xen testing:
>
> [ 174.512861] switch: port 2(vif3.0) entered disabled state
> [ 174.522735] BUG: sleeping function called from invalid context at
> /home/build/linux-linus/mm/vmal
On Thu, Mar 02, 2017 at 12:54:25PM +, Paul Durrant wrote:
> This patch replaces use of 'be->vif' with 'vif' and hence generally
> makes the function look tidier. No semantic change.
>
> Signed-off-by: Paul Durrant
Acked-by: Wei Liu
0x40/0x40
> [ 174.541898] ret_from_fork+0x21/0x2c
> [ 174.548635] switch: port 2(vif3.0) entered disabled state
>
> This patch defers the vfree() until after the spinlock is released.
>
> Reported-by: Juergen Gross
> Signed-off-by: Paul Durrant
Acked-by: Wei Liu
ximum on both sides to 8. Both
> frontend and backend maximum can be individually tuned via module
> parameters.
>
> Juergen Gross (2):
> xen/netfront: set default upper limit of tx/rx queues to 8
> xen/netback: set default upper limit of tx/rx queues to 8
>
Acked-by: Wei Liu
On Thu, Jan 12, 2017 at 05:51:56PM +, Igor Druzhinin wrote:
> Eliminate memory leaks introduced several years ago by cleaning the queue
> resources which are allocated on XenBus connection event. Namely, queue
> structure array and pages used for IO rings.
> vif->lock is used to protect statist
On Tue, Jan 17, 2017 at 08:49:38PM +, Igor Druzhinin wrote:
> vif->lock is used to protect statistics gathering agents from using the
> queue structure during cleaning.
>
> Signed-off-by: Igor Druzhinin
Acked-by: Wei Liu
Igor Druzhinin
Acked-by: Wei Liu
1 - 100 of 228 matches
Mail list logo
