linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Greg KH <gregkh@suse.de>
To: "Eric W. Biederman" <ebiederm@xmission.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Adrian Bunk <bunk@stusta.de>,
	Andrew Morton <akpm@linux-foundation.org>,
	David Miller <davem@davemloft.net>, Jeff Garzik <jeff@garzik.org>
Subject: Re: Linux v2.6.21-rc3
Date: Tue, 13 Mar 2007 12:40:24 -0700	[thread overview]
Message-ID: <20070313194024.GA14847@suse.de> (raw)
In-Reply-To: <m1zm6hq70n.fsf@ebiederm.dsl.xmission.com>

On Tue, Mar 13, 2007 at 01:26:48PM -0600, Eric W. Biederman wrote:
> - We have a network ABI regression caused by the latest sysfs
>   changes to net-sysfs.c   In particular we now cannot rename network
>   devices if our destination name happens to be the name of a sysfs file that
>   the network device appears in, and if we try the kernel gets very
>   confused and we loose access to the network device. 
> 
>   Do we just want to revert commit 43cb76d91ee85f579a69d42bc8efc08bac560278
>   Greg has been working on this off and on and has not found a
>   simple solution yet.

I do not think this should be reverted, as the odds that some one will
rename their network device to be "irq" or something else that is in the
pci device's directory is pretty slim.  It also only shows up if
CONFIG_SYSFS_DEPRECATED is disabled, not the common option.

But I am still working on it, I sent you and Kay a patch that, while it
locks up at boot time, should be close to what we need to address this
:)

> - pci_save_state, pci_restore_state are broken and have been for a
>   while if used on anything besides plain pci (pci-x, pci-e and msi)
>   and are not used in pairs. (gregkh and Andrew have the patches to 
>   correct this).

I think these are already in Linus's tree right now, right?

thanks,

greg k-h

  reply	other threads:[~2007-03-13 19:42 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-03-07  4:59 Linux v2.6.21-rc3 Linus Torvalds
2007-03-07 10:25 ` Benjamin Herrenschmidt
2007-03-07 13:26   ` Greg KH
2007-03-07 14:15     ` Mark Lord
2007-03-07 14:22       ` Greg KH
2007-03-07 15:39   ` Linus Torvalds
2007-03-07 20:52     ` Arnd Bergmann
2007-03-08  8:10       ` Benjamin Herrenschmidt
2007-03-08  8:08     ` Benjamin Herrenschmidt
2007-03-07 12:56 ` Michal Piotrowski
2007-03-07 16:34   ` Linus Torvalds
2007-03-07 17:12     ` [patch] CPU hotplug: call check_tsc_sync_source() with irqs off Ingo Molnar
2007-03-07 17:45       ` Michal Piotrowski
2007-03-07 19:12       ` Linux-2.6.21-rc3 : Dynticks and High resolution Timer hanging the system Stephane Casset
2007-03-07 19:52         ` Thomas Gleixner
2007-03-07 21:16           ` Stephane Casset
2007-03-07 22:09             ` Thomas Gleixner
2007-03-07 13:09 ` Linux v2.6.21-rc3 Michal Piotrowski
2007-03-07 16:25   ` Linus Torvalds
2007-03-07 17:14   ` [Linux-parport] " Stephen Mollett
2007-03-07 17:35     ` Russell King
2007-03-07 14:22 ` Thomas Gleixner
2007-03-07 17:14   ` Thomas Gleixner
2007-03-07 17:42   ` Soeren Sonnenburg
2007-03-08 17:28 ` Alistair John Strachan
2007-03-13 12:49 ` [1/6] 2.6.21-rc3: known regressions Adrian Bunk
2007-03-13 13:08   ` Pierre Ossman
2007-03-13 13:36     ` Oliver Neukum
2007-03-13 18:11       ` Pavel Machek
2007-03-13 19:07         ` Pierre Ossman
2007-03-13 19:12           ` Mws
2007-03-13 19:15           ` Adrian Bunk
2007-03-13 20:05           ` Pavel Machek
2007-03-13 20:31             ` Pierre Ossman
2007-03-13 13:40   ` Takashi Iwai
2007-03-13 12:50 ` [2/6] " Adrian Bunk
2007-03-13 13:30   ` Cornelia Huck
2007-03-13 13:35     ` Mark Lord
2007-03-13 18:13       ` Pavel Machek
2007-03-13 12:50 ` [3/6] " Adrian Bunk
2007-03-13 14:03   ` Alan Cox
2007-03-13 20:12     ` Fabio Comolli
2007-03-13 15:13   ` Andi Kleen
2007-03-13 12:50 ` [4/6] " Adrian Bunk
2007-03-13 12:50 ` [5/6] " Adrian Bunk
2007-03-13 13:29   ` Lukas Hejtmanek
2007-03-13 18:14   ` Pavel Machek
2007-03-13 21:46   ` Arkadiusz Miskiewicz
2007-03-13 12:50 ` [6/6] " Adrian Bunk
2007-03-13 20:05   ` Thomas Gleixner
2007-03-14 11:31     ` Adrian Bunk
2007-03-13 20:46   ` Thomas Gleixner
2007-03-14 11:44     ` Adrian Bunk
2007-03-14 12:16       ` Jiri Slaby
2007-03-14 17:31         ` Adrian Bunk
2007-03-14 18:02       ` Florian Lohoff
2007-03-14 18:28         ` Thomas Gleixner
2007-03-13 19:26 ` Linux v2.6.21-rc3 Eric W. Biederman
2007-03-13 19:40   ` Greg KH [this message]
2007-03-13 19:48     ` Linus Torvalds
2007-03-13 20:04     ` Eric W. Biederman
2007-03-14 18:11 ` 2.6.21-rc3: known regressions with patches Adrian Bunk

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=20070313194024.GA14847@suse.de \
    --to=gregkh@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=bunk@stusta.de \
    --cc=davem@davemloft.net \
    --cc=ebiederm@xmission.com \
    --cc=jeff@garzik.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    /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;
as well as URLs for NNTP newsgroup(s).