mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jean Tourrilhes <jt@bougret.hpl.hp.com>
To: Linus Torvalds <torvalds@transmeta.com>,
	Jeff Garzik <jgarzik@mandrakesoft.com>,
	Martin Diehl <lists@mdiehl.de>,
	Linux kernel mailing list <linux-kernel@vger.kernel.org>,
	irda-users@lists.sourceforge.net
Subject: IrDA locking fixes for 2.5.X on the way...
Date: Thu, 29 Aug 2002 15:46:58 -0700	[thread overview]
Message-ID: <20020829224658.GA14118@bougret.hpl.hp.com> (raw)

	Hi Linus,

	The next few e-mails will contain the "trivial" patches
necessary to get rid of "save_flags(flags);cli();" in the IrDA
stack. I guess you were right, I only needed to add a few spinlocks
left and right.
	I would like those patches to go quickly in 2.5.X, because
they cover lot's of code, so any other changes to the IrDA stack is
likely to make those patches not apply. That's also the reason why I
separated the ALI fixes, as some wanabe IrDA hackers are playing with
it (and creating compile errors).
	The patches go in TWO phases. The first 3 patches are
independant and fixes things needed by the locking fixes. The
remaining 4 patches are also independant but depend on the 3 previous
patches. Patches done and tested on 2.5.32.
	I've got still other patches in my patch queue, and I'll send
that to Jeff in a few weeks.

	Regards,

	Jean

--------------------------------------------------------
[FEATURE] : Add a new feature to the IrDA stack
[CORRECT] : Fix to have the correct/expected behaviour
[CRITICA] : Fix potential kernel crash

ir252_hashbin_fixes-4.diff :
--------------------------
	o [CRITICA] Remove correct IAS Attribute/Object even if name is dup'ed
	o [CORRECT] Make irqueue 64 bit compliant (__u32 -> long)
	o [FEATURE] Don't use random handle for IrLMP handle, use self
		Remove dependancy on random generator in stack init

ir252_lap_unique_saddr.diff :
---------------------------
	o [CORRECT] Make sure LAP address is sane, which mean not NULL,
		not BROADCAST and not already in use by another link.

ir255_nsc_speed-4.diff :
----------------------
	o [FEATURE] Cleanly change speed back to 9600bps
	o [CORRECT] Change speed under spinlock/irq disabled
	o [CORRECT] Make sure interrupt handlers don't mess irq mask
	o [CORRECT] Don't change speed if we haven't fully finished to Tx

ir252_hashbin_locking_fixes-4.diff :
----------------------------------
	o [FEATURE] New hashbin locking scheme for irqueue
	o [FEATURE] Get rid of old broken hashbin locking schemes
	o [FEATURE] Lock hasbins while enumerating it in various places
	o [CORRECT] Remove all remaining "save_flags(flags);cli();"
	o [CORRECT] Fix two "return with spinlock" found by Stanford checker

ir252_ircomm_locking_fixes-4.diff :
---------------------------------
	o [FEATURE] Do the hashbin locking fixes for IrCOMM and IrLAN
	o [CORRECT] Remove all "save_flags(flags);cli();" in IrCOMM/IrLAN
	o [CORRECT] Fix other locking issues in IrCOMM

ir252_drivers_locking_fixes-3.diff :
----------------------------------
	o [CORRECT] Remove all "save_flags(flags);cli();" in IrDA driver
	o [FEATURE] Rework broken locking in irport
	o [FEATURE] Finish locking cleanup in nsc-ircc
	o [FEATURE] Improve locking in smc-ircc & w83977af_ir

ir252_ali_locking_fixes-3.diff :
------------------------------
	o [CORRECT] Remove all "save_flags(flags);cli();" in IrDA ali driver
	o [CORRECT] Fix stupid compilation error

                 reply	other threads:[~2002-08-29 22:46 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

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=20020829224658.GA14118@bougret.hpl.hp.com \
    --to=jt@bougret.hpl.hp.com \
    --cc=irda-users@lists.sourceforge.net \
    --cc=jgarzik@mandrakesoft.com \
    --cc=jt@hpl.hp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lists@mdiehl.de \
    --cc=torvalds@transmeta.com \
    /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

all inboxes | Powered by JetHome®