netdev
[Top] [All Lists]

Re: [PATCH][NET] make all protos partially use sk_prot

To: Marcel Holtmann <marcel@xxxxxxxxxxxx>
Subject: Re: [PATCH][NET] make all protos partially use sk_prot
From: Arnaldo Carvalho de Melo <arnaldo.melo@xxxxxxxxx>
Date: Sat, 26 Mar 2005 15:55:39 -0300
Cc: Arnaldo Carvalho de Melo <acme@xxxxxxxxxxxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, Network Development Mailing List <netdev@xxxxxxxxxxx>
Domainkey-signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:reply-to:to:subject:cc:in-reply-to:mime-version:content-type:content-transfer-encoding:references; b=Yph/wAmeC0LcYch9/LxzCzBI0J/JH09mtN0/SmSPqaIawaceOoXKnvFEhlFAFTbpmRF77Kr9qwCIhlpjeKjqFzCDsIyEq6pHLCS06pF6FqlXZF5TVNE5pY/JyJU6WgzNhUxKzv/YjlwiuNCTG+xkNS5hdW45T8JMzcflunKAU6w=
In-reply-to: <1111854713.9195.202.camel@pegasus>
References: <20050326144516.GA21949@conectiva.com.br> <1111854140.9195.199.camel@pegasus> <1111854713.9195.202.camel@pegasus>
Reply-to: acme@xxxxxxxxxxxxxxxx
Sender: netdev-bounce@xxxxxxxxxxx
On Sat, 26 Mar 2005 17:31:53 +0100, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote:
> Hi Arnaldo,
> 
> > >     Here is an updated patch, taking, I hope, all of Marcel
> > > considerations into account and also fixing a bug in proto_register,
> > > I was returning on error without dropping the lock.
> >
> > the error path of some init function in the Bluetooth subsystem is still
> > wrong. 

Which one? I'll check...

>> And I don't really like your way of using the labels, because
> > this twists my brain too much. 

Oh well, this is how things are done all over the place, but lets be fair
with your brain ;)

>> I fixed all of these and while we are at
> > it, I cleaned up the init functions. Please use the attached patch
> > instead of your changes to the Bluetooth subsystem.

Whatever, I'll cook up a new patch with your changes for bluetooth.

> oops, I forgot to include hci_sock.c into that patch. Here is a fixed
> one.

Regards,

- Arnaldo

<Prev in Thread] Current Thread [Next in Thread>