Re: CVS commit: src/sys/arch/xen/xen

2020-06-24 Thread Taylor R Campbell
> Module Name:src > Committed By: jdolecek > Date: Fri Apr 10 18:03:06 UTC 2020 > > Modified Files: > src/sys/arch/xen/xen: if_xennet_xenbus.c > > Log Message: > convert to bus_dma(9), remove now not necessary XENPVHVM redefines > [...] > +

Re: CVS commit: src/sys/arch/xen/xen

2020-04-13 Thread Manuel Bouyer
On Mon, Apr 13, 2020 at 08:09:13PM +, Jaromir Dolecek wrote: > Module Name: src > Committed By: jdolecek > Date: Mon Apr 13 20:09:13 UTC 2020 > > Modified Files: > src/sys/arch/xen/xen: xbd_xenbus.c > > Log Message: > KASSERT() that requested I/O size is <= XBD_MAX_XFER - this

Re: CVS commit: src/sys/arch/xen/xen

2020-04-09 Thread Frank Kardel
HI, I am not sur whether it is the commit below, but 2 out 4 times my xen-DOMU from today (20200409/9.99.55) panics with following locking botch: [ 29.9301379] panic: kernel diagnostic assertion "IFNET_LOCKED(ifp)" failed: file "/usr/src/sys/arch/xen/xen/if_xennet_xenbus.c", line 1120 [

Re: CVS commit: src/sys/arch/xen/xen

2020-04-05 Thread Manuel Bouyer
On Sun, Apr 05, 2020 at 05:26:47PM +, Jaromir Dolecek wrote: > Module Name: src > Committed By: jdolecek > Date: Sun Apr 5 17:26:47 UTC 2020 > > Modified Files: > src/sys/arch/xen/xen: if_xennet_xenbus.c xennetback_xenbus.c > > Log Message: > remove support for legacy rx-flip

Re: CVS commit: src/sys/arch/xen/xen

2014-10-17 Thread Christos Zoulas
On Oct 17, 6:02pm, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | Don't you think that you should also revert r1.46 [1]? Now that | privcmd_map_obj() frees everything correctly... Yes, I was going to and I forgot; done now. christos

Re: CVS commit: src/sys/arch/xen/xen

2014-10-17 Thread Maxime Villard
Le 10/10/2014 18:42, Maxime Villard a écrit : Le 03/10/2014 22:56, Christos Zoulas a écrit : Module Name:src Committed By:christos Date:Fri Oct 3 20:56:24 UTC 2014 Modified Files: src/sys/arch/xen/xen: privcmd.c Log Message: correct error paths; still need to verify that

Re: CVS commit: src/sys/arch/xen/xen

2014-10-10 Thread Maxime Villard
Le 03/10/2014 22:56, Christos Zoulas a écrit : Module Name:src Committed By: christos Date: Fri Oct 3 20:56:24 UTC 2014 Modified Files: src/sys/arch/xen/xen: privcmd.c Log Message: correct error paths; still need to verify that the didn't give us back... case is

Re: CVS commit: src/sys/arch/xen/xen

2014-10-03 Thread Maxime Villard
Le 27/09/2014 17:51, Christos Zoulas a écrit : On Sep 27, 8:36am, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | One however returns an error without freeing: | | if (newstart != start) { | printf(uvm_map didn't give us back our

Re: CVS commit: src/sys/arch/xen/xen

2014-09-27 Thread Maxime Villard
Le 21/09/2014 20:22, Christos Zoulas a écrit : On Sep 21, 7:57pm, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | Did you test this change? Verily I'm not sure it's a proper bug, but I | kept it in my list so that someone (bouyer@?) could investigate

Re: CVS commit: src/sys/arch/xen/xen

2014-09-27 Thread Christos Zoulas
On Sep 27, 8:36am, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | One however returns an error without freeing: | | if (newstart != start) { | printf(uvm_map didn't give us back our vm space\n); | return EINVAL

Re: CVS commit: src/sys/arch/xen/xen

2014-09-27 Thread Manuel Bouyer
On Sat, Sep 27, 2014 at 11:51:59AM -0400, Christos Zoulas wrote: On Sep 27, 8:36am, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | One however returns an error without freeing: | | if (newstart != start) { | printf(uvm_map

Re: CVS commit: src/sys/arch/xen/xen

2014-09-27 Thread Christos Zoulas
On Sep 27, 10:15pm, bou...@antioche.eu.org (Manuel Bouyer) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | if (newstart != start) { | printf(uvm_map didn't give us back our vm space\n); | + uvm_unmap1(map, newstart, newstart + size, 0

Re: CVS commit: src/sys/arch/xen/xen

2014-09-21 Thread Christos Zoulas
On Sep 21, 7:57pm, m...@m00nbsd.net (Maxime Villard) wrote: -- Subject: Re: CVS commit: src/sys/arch/xen/xen | Did you test this change? Verily I'm not sure it's a proper bug, but I | kept it in my list so that someone (bouyer@?) could investigate. This is why I committed the fix in two pieces

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread David Laight
On Sat, Feb 01, 2014 at 06:09:04PM -0800, John Nemeth wrote: } } XEN might need the bits of fpuinit() that don't play with cr0. } In particular a fninit and setting TS. } Although I'm not 100% sure the TS is needed on a single cpu system. Xen domU is SMP capable. NetBSD dom0

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread David Laight
On Sun, Feb 02, 2014 at 01:27:45PM +0100, Manuel Bouyer wrote: On Sat, Feb 01, 2014 at 10:49:44PM +, David Laight wrote: On Sat, Feb 01, 2014 at 08:07:07PM +, Manuel Bouyer wrote: Module Name: src Committed By: bouyer Date: Sat Feb 1 20:07:07 UTC 2014

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread Manuel Bouyer
On Sun, Feb 02, 2014 at 06:53:55PM +, David Laight wrote: It's been a while since I looked at this and I don't remmeber the details, but I don't think anything of fpuinit() is neeed for Xen. in netbsd-6 npxattach() doens't do anything either for Xen (only set i386_fpu_present to 1 and

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread David Laight
On Sun, Feb 02, 2014 at 08:41:29PM +0100, Manuel Bouyer wrote: On Sun, Feb 02, 2014 at 06:53:55PM +, David Laight wrote: Something needs to set the TS (task switched) flag when a new cpu is added. Both amd64 and i386 'bare metal' have direct calls to fpuinit() to do this. If TS

Re: CVS commit: src/sys/arch/xen/xen

2014-02-02 Thread Manuel Bouyer
On Sun, Feb 02, 2014 at 08:24:08PM +, David Laight wrote: On Sun, Feb 02, 2014 at 08:41:29PM +0100, Manuel Bouyer wrote: On Sun, Feb 02, 2014 at 06:53:55PM +, David Laight wrote: Something needs to set the TS (task switched) flag when a new cpu is added. Both amd64 and i386 'bare

Re: CVS commit: src/sys/arch/xen/xen

2014-02-01 Thread Manuel Bouyer
On Sat, Feb 01, 2014 at 05:48:04PM +, David Laight wrote: Module Name: src Committed By: dsl Date: Sat Feb 1 17:48:04 UTC 2014 Modified Files: src/sys/arch/xen/xen: hypervisor.c Log Message: Add a direct call to fpuinit(). I'm not sure this is architecturally the

Re: CVS commit: src/sys/arch/xen/xen

2014-02-01 Thread David Laight
On Sat, Feb 01, 2014 at 08:07:07PM +, Manuel Bouyer wrote: Module Name: src Committed By: bouyer Date: Sat Feb 1 20:07:07 UTC 2014 Modified Files: src/sys/arch/xen/xen: hypervisor.c Log Message: Revert previous: calling fpuinit() leads to a panic, as a domU is not

Re: CVS commit: src/sys/arch/xen/xen

2014-02-01 Thread Jeff Rizzo
On 2/1/14, 2:49 PM, David Laight wrote: On Sat, Feb 01, 2014 at 08:07:07PM +, Manuel Bouyer wrote: I can't say I've actuallly looked a xen. Presumably we somewhere compile the hypervisor as part of a kernel? The hypervisor is in pkgsrc - sysutils/xenkernel42 (and the accompanying

Re: CVS commit: src/sys/arch/xen/xen

2011-12-08 Thread Christoph Egger
On 12/08/11 04:34, Cherry G. Mathew wrote: Module Name:src Committed By: cherry Date: Thu Dec 8 03:34:48 UTC 2011 Modified Files: src/sys/arch/xen/xen: evtchn.c Log Message: kmem_free() the appropriate size. Thanks cegger@ This is still not enough: In

Re: CVS commit: src/sys/arch/xen/xen

2011-12-08 Thread Manuel Bouyer
On Thu, Dec 08, 2011 at 11:09:19AM +0100, Christoph Egger wrote: On 12/08/11 04:34, Cherry G. Mathew wrote: Module Name: src Committed By:cherry Date:Thu Dec 8 03:34:48 UTC 2011 Modified Files: src/sys/arch/xen/xen: evtchn.c Log Message: kmem_free() the

Re: CVS commit: src/sys/arch/xen/xen

2011-12-07 Thread Cherry G. Mathew
On 7 December 2011 19:19, Christoph Egger ceg...@netbsd.org wrote: Module Name:    src Committed By:   cegger Date:           Wed Dec  7 13:49:04 UTC 2011 Modified Files:        src/sys/arch/xen/xen: evtchn.c Log Message: build fix: add back sys/malloc.h. malloc(9) is still in use. Hi,

Re: CVS commit: src/sys/arch/xen/xen

2011-05-15 Thread Mindaugas Rasiukevicius
Hello, Jean-Yves Migeon j...@netbsd.org wrote: Module Name: src Committed By: jym Date: Sun May 15 07:24:15 UTC 2011 Modified Files: src/sys/arch/xen/xen: xbdback_xenbus.c Log Message: Use atomic_ops(3) for ref counting. + atomic_dec_uint((xbdip)-xbdi_refcnt);

Re: CVS commit: src/sys/arch/xen/xen

2011-05-15 Thread Jean-Yves Migeon
On 15.05.2011 21:11, Mindaugas Rasiukevicius wrote: This is not correct. Atomic op might decrease the reference count to 1 while other thread executes xbdi_put() before xbdi_refcnt is fetched, thus decreasing it to 0. In such case, both threads would fetch 0. The following is a correct

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Cherry G. Mathew
On 18 April 2011 04:21, Mindaugas Rasiukevicius rm...@netbsd.org wrote: Masao Uebayashi uebay...@tombi.co.jp wrote: On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: Module Name:        src Committed By:       rmind Date:               Mon Apr 18 03:04:31 UTC 2011

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Cherry G. Mathew
On 18 April 2011 08:00, Cherry G. Mathew cherry.g.mat...@gmail.com wrote: On 18 April 2011 04:21, Mindaugas Rasiukevicius rm...@netbsd.org wrote: Masao Uebayashi uebay...@tombi.co.jp wrote: On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: Module Name:        src

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
On 18.04.2011 09:23, Cherry G. Mathew wrote: On 18 April 2011 08:00, Cherry G. Mathew cherry.g.mat...@gmail.com wrote: On 18 April 2011 04:21, Mindaugas Rasiukevicius rm...@netbsd.org wrote: Masao Uebayashi uebay...@tombi.co.jp wrote: On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Cherry G. Mathew
On 18 April 2011 09:04, Jean-Yves Migeon jeanyves.mig...@free.fr wrote: On 18.04.2011 09:23, Cherry G. Mathew wrote: On 18 April 2011 08:00, Cherry G. Mathew cherry.g.mat...@gmail.com wrote: On 18 April 2011 04:21, Mindaugas Rasiukevicius rm...@netbsd.org wrote: Masao Uebayashi

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
On 18.04.2011 05:04, Mindaugas Rasiukevicius wrote: Module Name: src Committed By: rmind Date: Mon Apr 18 03:04:31 UTC 2011 Modified Files: src/sys/arch/xen/xen: balloon.c Log Message: balloon_xenbus_attach: use KM_SLEEP for allocation. Note: please do not use

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Mindaugas Rasiukevicius
Cherry G. Mathew cherry.g.mat...@gmail.com wrote: On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: Module Name:        src Committed By:       rmind Date:               Mon Apr 18 03:04:31 UTC 2011 Modified Files:     src/sys/arch/xen/xen: balloon.c

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Jean-Yves Migeon
On 18.04.2011 10:35, Mindaugas Rasiukevicius wrote: We used to check the return of big size allocations, when kmem(9) could fail even with KM_SLEEP due to KVA starvation. However, pretty much all kernel does not perform checks for smaller allocations and since the bug was fixed - we are no

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Adam Hamsik
On Apr,Monday 18 2011, at 10:26 AM, Mindaugas Rasiukevicius wrote: Cherry G. Mathew cherry.g.mat...@gmail.com wrote: On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: Module Name:src Committed By: rmind Date: Mon Apr 18 03:04:31 UTC 2011

Re: CVS commit: src/sys/arch/xen/xen

2011-04-18 Thread Mindaugas Rasiukevicius
Jean-Yves Migeon jeanyves.mig...@free.fr wrote: On 18.04.2011 10:35, Mindaugas Rasiukevicius wrote: We used to check the return of big size allocations, when kmem(9) could fail even with KM_SLEEP due to KVA starvation. However, pretty much all kernel does not perform checks for smaller

Re: CVS commit: src/sys/arch/xen/xen

2011-04-17 Thread Mindaugas Rasiukevicius
Masao Uebayashi uebay...@tombi.co.jp wrote: On Mon, Apr 18, 2011 at 03:04:32AM +, Mindaugas Rasiukevicius wrote: Module Name:src Committed By: rmind Date: Mon Apr 18 03:04:31 UTC 2011 Modified Files: src/sys/arch/xen/xen: balloon.c Log

Re: CVS commit: src/sys/arch/xen/xen

2011-04-06 Thread Taylor R Campbell
Date: Mon, 04 Apr 2011 23:46:19 +0200 From: Jean-Yves Migeon jeanyves.mig...@free.fr The newer scripts for Xen read the interface value from the vifname entry in Xenstore, so changing the name is not that critical. But this should be stabilized sooner than later. Personally, I find

Re: CVS commit: src/sys/arch/xen/xen

2011-04-06 Thread Jean-Yves Migeon
On 06.04.2011 20:01, Manuel Bouyer wrote: You could also use xvifxiy (e.g. xvif5i2, where i stands for 'index'). or any other letter ... Huh hmm indeed... I wonder why I did not think about this approach before... -- Jean-Yves Migeon jeanyves.mig...@free.fr

Re: CVS commit: src/sys/arch/xen/xen

2011-04-04 Thread Taylor R Campbell
Date: Sun, 3 Apr 2011 23:21:39 + From: Jean-Yves Migeon j...@netbsd.org Now that pkgsrc-2011Q1 has arrived, and before -6 chimes in, change ifxname for xvif(4) from xvif%d.%d to xvif%d-%d. This is needed to avoid sysctl(9) EINVAL errors when creating interface nodes. This

Re: CVS commit: src/sys/arch/xen/xen

2011-03-29 Thread David Young
On Wed, Mar 30, 2011 at 12:17:05AM +, Jean-Yves Migeon wrote: Module Name: src Committed By: jym Date: Wed Mar 30 00:17:04 UTC 2011 Modified Files: src/sys/arch/xen/xen: if_xennet_xenbus.c Log Message: printf(xennet: ...) = aprint_error_ifnet() It's silly, I know,