public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@suse.de>
To: Luiz Fernando Capitulino <lcapitulino@mandriva.com.br>
Cc: linux-kernel@vger.kernel.org,
	linux-usb-devel@lists.sourceforge.net, ehabkost@mandriva.com
Subject: Re: [PATCH 00/10] usb-serial: Switches from spin lock to atomic_t.
Date: Wed, 7 Dec 2005 08:41:18 -0800	[thread overview]
Message-ID: <20051207164118.GA28032@suse.de> (raw)
In-Reply-To: <20051206095610.29def5e7.lcapitulino@mandriva.com.br>

On Tue, Dec 06, 2005 at 09:56:10AM -0200, Luiz Fernando Capitulino wrote:
>  Greg,
> 
>  Don't get scared. :-)

I'm not scared, just not liking this patch series at all.

In the end, it's just moving from one locking scheme to another.  No big
deal.

The problem is, none of this should be needed at all.  We need to move
the usb-serial drivers over to use the serial core code.  If that
happens, then none of this locking is needed.

That's the right thing to do, so I'm not going to take this patch series
right now because of that.  If you all want to work on moving to use the
serial core, I would love to see that happen.

thanks,

greg k-h

  parent reply	other threads:[~2005-12-07 16:42 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-12-06 11:56 [PATCH 00/10] usb-serial: Switches from spin lock to atomic_t Luiz Fernando Capitulino
2005-12-06 19:40 ` Greg KH
2005-12-06 20:13   ` Eduardo Pereira Habkost
2005-12-06 22:48     ` [linux-usb-devel] " Oliver Neukum
2005-12-07 12:24       ` Luiz Fernando Capitulino
2005-12-07 12:27         ` Arjan van de Ven
2005-12-07 12:30           ` Luiz Fernando Capitulino
2005-12-07 12:34             ` Arjan van de Ven
2005-12-07 12:41               ` Luiz Fernando Capitulino
2005-12-07 12:54                 ` Luiz Fernando Capitulino
2005-12-07 15:07       ` Alan Stern
2005-12-07 15:22         ` Arjan van de Ven
2005-12-07 15:37           ` Oliver Neukum
2005-12-07 15:40             ` Arjan van de Ven
2005-12-07 15:50               ` Oliver Neukum
2005-12-07 16:02                 ` Alan Cox
2005-12-07 16:00           ` Eduardo Pereira Habkost
2005-12-07 16:02             ` Arjan van de Ven
2005-12-07 16:23               ` Eduardo Pereira Habkost
2005-12-07 16:01           ` Alan Stern
2005-12-07 16:04             ` Arjan van de Ven
2005-12-07 15:32         ` linux-os (Dick Johnson)
2005-12-07 16:08           ` Alan Stern
2005-12-06 20:14   ` Luiz Fernando Capitulino
2005-12-06 21:02     ` Pete Zaitcev
2005-12-06 21:18       ` Luiz Fernando Capitulino
2005-12-06 22:36         ` [linux-usb-devel] " Oliver Neukum
2005-12-07 12:25           ` Luiz Fernando Capitulino
2005-12-07 13:01             ` Oliver Neukum
2005-12-07 13:17               ` Luiz Fernando Capitulino
2005-12-07 16:41 ` Greg KH [this message]
2005-12-07 16:51   ` Luiz Fernando Capitulino
2005-12-07 17:13     ` Eduardo Pereira Habkost
2005-12-07 17:56       ` Greg KH
2005-12-07 19:10         ` Eduardo Pereira Habkost
2005-12-07 16:55   ` Otavio Salvador
2005-12-07 16:59     ` Greg KH

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=20051207164118.GA28032@suse.de \
    --to=gregkh@suse.de \
    --cc=ehabkost@mandriva.com \
    --cc=lcapitulino@mandriva.com.br \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    /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