Various little cleanups and commenting fixes. Fixed up the patchset so
each one, incrementally, should give a properly compiling and running
kernel.
I'd still like Hugh to ack the anon/swap changes when he can find the time.
It would be desirable to get at least one ack as to the overall problem
Ensure pages are uptodate after returning from read_cache_page, which allows
us to cut out most of the filesystem-internal PageUptodate calls.
I didn't have a great look down the call chains, but this appears to fixes 7
possible use-before uptodate in hfs, 2 in hfsplus, 1 in jfs, a few in
On Wednesday February 14, [EMAIL PROTECTED] wrote:
I am well aware of what Greg KHs position is, in fact he is the reason
I started the whole rant. This is only a plea to the higher
authorities. Linus, please save Linux!
Linus is not in any position to do anything. The die is cast.
You
__block_write_full_page is calling SetPageUptodate without the page locked.
This is unusual, but not incorrect, as PG_writeback is still set.
However the next patch will require that SetPageUptodate always be called
with the page locked. Simply don't bother setting the page uptodate in this
case
After running SetPageUptodate, preceeding stores to the page contents to
actually bring it uptodate may not be ordered with the store to set the page
uptodate.
Therefore, another CPU which checks PageUptodate is true, then reads the
page contents can get stale data.
Fix this by ensuring
please ack if O.K.
-Kame
--
bind_zonelist() can create zero-length zonelist if there is a
memory-less-node. This patch checks the length of zonelist.
If length is 0, returns -EINVAL.
Changelog: v3 - v4:
- changes a name of a temporal void* variable as error_code
Changelog: v2 - v3
- removed
On Wed, Feb 14, 2007 at 10:46:13PM -0800, v j wrote:
You don't get it do you. Our source code is meaningless to the Open
Source community at large.
Linux supports entire _architectures_ of which there are single figures
of people using it. What makes your hardware special ?
We are only
On Wed, 2007-02-14 at 21:16 -0800, v j wrote:
This is in reference to the following thread:
http://lkml.org/lkml/2006/12/14/63
I am not sure if this is ever addressed in LKML, but linux is _very_
popular in the embedded space. We (an embedded vendor) chose Linux 3
years back because of
Ben Nizette wrote:
v j wrote:
This is in reference to the following thread:
http://lkml.org/lkml/2006/12/14/63
I am not sure if this is ever addressed in LKML, but linux is _very_
popular in the embedded space. We (an embedded vendor) chose Linux 3
years back because of its lack of royalty
On Wednesday February 14, [EMAIL PROTECTED] wrote:
On 2/14/07, Randy Dunlap [EMAIL PROTECTED] wrote:
We seem to have different definitions of open and closed.
Open = 3rd party Linux drivers can be loaded. Closed = No third party
Linux drivers can be loaded.
Loading a driver is not at
Nadia Derbey [EMAIL PROTECTED] writes:
But, what do you do with Oracle that's asking maxfiles to be set to 0x1,
while the default value might be enough for a system that's not running
Oracle.
I'm afraid that giving boot time values to the max_* tunables we will loose
all
the benefits
On 2/15/07, Neil Brown [EMAIL PROTECTED] wrote:
[..] then it is less clear what people believe
Another area where it is less clear what people believe is if you are
distributing the module separately to the kernel, but, as I understand
it, vj says he is not.
But of course the person who's
On Wednesday February 14, [EMAIL PROTECTED] wrote:
You don't get it do you. Our source code is meaningless to the Open
Source community at large. It is only useful to our tiny set of
competitors that have nothing to do with Linux. The Embedded space is
very specific. We are only _using_ Linux.
Jakub Narebski [EMAIL PROTECTED] wrote:
Junio C Hamano wrote:
- git-blame learned a new option, --incremental, that tells it
to output the blames as they are assigned. A sample script
to use it is also included as contrib/blameview.
And there are example GUI blameview (Perk
On Wednesday 14 February 2007 20:13, Dave Jones wrote:
I've not investigated it, but I hear rumours that suse has something
similar.
Actually, no. We don't belive that module signing adds significant value, and
it also doesn't work well with external modules. (The external modules we
really
On Wed, Feb 14, 2007 at 09:35:40PM -0800, Andreas Gruenbacher wrote:
On Wednesday 14 February 2007 20:13, Dave Jones wrote:
I've not investigated it, but I hear rumours that suse has something
similar.
Actually, no. We don't belive that module signing adds significant value,
ok, then
Jeff,
Please accept the following patchset for the atl1 network device driver.
* Drop unnecessary NET_PCI config
* Fix incorrect hash table address
* Read MAC address from register
* Remove unused define
* Add Attansic L1 device id to pci_ids
* Bump version number
This patchset contains changes
From: Jay Cliburn [EMAIL PROTECTED]
The atl1 driver doesn't need NET_PCI. Remove it from Kconfig.
Noticed by Chad Sprouse.
Signed-off-by: Jay Cliburn [EMAIL PROTECTED]
Signed-off-by: Chris Snook [EMAIL PROTECTED]
---
drivers/net/Kconfig |2 +-
1 files changed, 1 insertions(+), 1
From: Al Viro [EMAIL PROTECTED]
An ioread32 statement reads the wrong address. Fix it.
Signed-off-by: Al Viro [EMAIL PROTECTED]
Signed-off-by: Jay Cliburn [EMAIL PROTECTED]
Signed-off-by: Chris Snook [EMAIL PROTECTED]
---
drivers/net/atl1/atl1_hw.c |2 +-
1 files changed, 1 insertions(+),
From: Jay Cliburn [EMAIL PROTECTED]
On some Asus motherboards containing the L1 NIC, the MAC address is
written by the BIOS directly to the MAC register during POST, and is
not stored in eeprom. If we don't succeed in fetching the MAC address
from eeprom or spi, try reading it directly from the
From: Chris Snook [EMAIL PROTECTED]
Remove unused define from atl1_main.c.
Signed-off-by: Chris Snook [EMAIL PROTECTED]
Signed-off-by: Jay Cliburn [EMAIL PROTECTED]
---
drivers/net/atl1/atl1_main.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git
From: Chris Snook [EMAIL PROTECTED]
Add device id for the Attansic L1 chip to pci_ids.h, then use it.
Signed-off-by: Chris Snook [EMAIL PROTECTED]
Signed-off-by: Jay Cliburn [EMAIL PROTECTED]
---
drivers/net/atl1/atl1_main.c |2 +-
include/linux/pci_ids.h |1 +
2 files changed, 2
From: Jay Cliburn [EMAIL PROTECTED]
Bump the version number.
Signed-off-by: Jay Cliburn [EMAIL PROTECTED]
---
drivers/net/atl1/atl1_main.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/net/atl1/atl1_main.c b/drivers/net/atl1/atl1_main.c
index
On Wednesday 14 February 2007 21:45, Dave Jones wrote:
well, the situation for external modules is no worse than usual.
They still work, they just aren't signed. Which from a distributor point
of view, is actually a nice thing, as they stick out like a sore thumb
in oops reports with (U)
On Wed, Feb 14, 2007 at 10:14:53PM -0800, Andreas Gruenbacher wrote:
On Wednesday 14 February 2007 21:45, Dave Jones wrote:
well, the situation for external modules is no worse than usual.
They still work, they just aren't signed. Which from a distributor point
of view, is actually a
Linus,
Please pull 'master' from:
git://git.kernel.org:/pub/scm/linux/kernel/git/mchehab/v4l-dvb.git
master
Basically, this series adds support for a bunch of newer cards and newer
drivers, do some relevant cleanups on cx88 (improving source code
readability and reducing binary code
On Wed, 07 Feb 2007 13:07:40 -0800 Sriram Chidambaram [EMAIL PROTECTED] wrote:
This patch provides the Fabric7 VIOC driver source code.
This git mbox patch is built against
git://git.kernel.org/pub/scm/linux/kernel/git/jgarzik/netdev-2.6.git
The patch can be pulled from
Jeremy Fitzhardinge [EMAIL PROTECTED] writes:
Eric W. Biederman wrote:
Ok. If that is all this may be a difference that makes no difference.
binutils has a bad habit of looking at sections (which are fully
optional) instead of segments on ET_EXEC and ET_DYN objects. Only
ET_REL objects (.o
Eric W. Biederman wrote:
Reasonable and it's probably worth letting the binutils developer know.
I do agree that it is weird. It might be that something in binutils
doesn't like us dropping some of the notes.
What do you mean by dropping some of the notes? I think the only
notes (at
Jeremy Fitzhardinge [EMAIL PROTECTED] writes:
Eric W. Biederman wrote:
Reasonable and it's probably worth letting the binutils developer know.
I do agree that it is weird. It might be that something in binutils
doesn't like us dropping some of the notes.
What do you mean by dropping
Eric W. Biederman wrote:
I'm pretty certain we explicitly drop the weird GNU note that
is automatically generated by gcc and specifies something informational.
But that's something else again, since it appears as a PT_GNU_STACK phdr.
I don't think anything we are doing is wrong but ld gets
Jeremy Fitzhardinge [EMAIL PROTECTED] writes:
Eric W. Biederman wrote:
I'm pretty certain we explicitly drop the weird GNU note that
is automatically generated by gcc and specifies something informational.
But that's something else again, since it appears as a PT_GNU_STACK phdr.
Not
Hi Andrew,
Jeff Moyer has identified a race between mount and expire.
What happens is that during an expire the situation can arise
that a directory is removed and another lookup is done before
the expire issues a completion status to the kernel module.
In this case, since the the lookup gets a
Hi Andrew,
This problem was identified and fixed some time ago by Jeff Moyer
but it fell through the cracks somehow.
It is possible that a user space application could remove and
re-create a directory during a request. To avoid returning a
failure from lookup incorrectly when our current dentry
On Thu, 15 Feb 2007 04:43:41 +0100 Blaisorblade [EMAIL PROTECTED] wrote:
I sent an equivalent patch in earlier today:
Doh! Interesting this timing...
Index: linux-2.6/arch/x86_64/ia32/ptrace32.c
===
---
While testing out some libata FUA changes I was working on, I was
inadvertently able to reproduce the kind of NCQ command timeouts in
sata_nv that a few people have reported. I since verified that the FUA
stuff had nothing to do with it as it still happens even with FUA
disabled. However I'm
Hello Roland!
Here is a patch set for ehca with the following changes resp. bug fixes:
* Reworked irq handler to avoid/reduce missed irq events
* Fix race condition bug in find_next_online_cpu() and other potential
locking issue of scaling code
* Allow scaling code to be configurable
Hi,
here is a patch for ehca with the reworked irq handler.
Thanks
Nam
Signed-off-by: Hoang-Nam Nguyen [EMAIL PROTECTED]
---
ehca_classes.h | 18 +++--
ehca_eq.c |1
ehca_irq.c | 200 -
ehca_irq.h |1
Hi,
this patch fixes a race condition in find_next_cpu_online() and some
other locking issues in scaling code.
Thanks
Nam
Signed-off-by: Hoang-Nam Nguyen [EMAIL PROTECTED]
---
ehca_irq.c | 68 +
1 files changed, 33 insertions(+),
Hi,
this patch removes yield() and uses wait_for_completion() in order
to wait for running completion handlers finished before destroying
associated completion queue.
Thanks
Nam
Signed-off-by: Hoang-Nam Nguyen [EMAIL PROTECTED]
---
ehca_classes.h |3 +++
ehca_cq.c |3 ++-
Hi,
here is a patch for ehca that allows users to en/disable scaling code
when loading ib_ehca module.
Thanks
Nam
Signed-off-by: Hoang-Nam Nguyen [EMAIL PROTECTED]
---
Kconfig|8
ehca_classes.h |1 +
ehca_irq.c | 47
Hi,
this patch sets port phys state as a result of ehca_query_port() to LINK_UP.
On pSeries ehca actually represents a logical HCA, whose phys/link state always
is LINK_UP.
Thanks
Nam
Signed-off-by: Hoang-Nam Nguyen [EMAIL PROTECTED]
---
ehca_hca.c |3 +++
1 files changed, 3
On Wed, Feb 14, 2007 at 05:40:47PM +0100, Hoang-Nam Nguyen wrote:
Hi,
here is a patch for ehca with the reworked irq handler.
Thanks
Nam
This looks okay to me (and sorry for new replying earlier to you private
mail)
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
Looks fine but this patch at least has serious whitespace
damage... please resend a fixed version.
- R.
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please
@@ -332,7 +333,7 @@ int ehca_destroy_cq(struct ib_cq *cq)
spin_lock_irqsave(ehca_cq_idr_lock, flags);
while (my_cq-nr_callbacks) {
spin_unlock_irqrestore(ehca_cq_idr_lock, flags);
- yield();
+
I agree with Christoph -- the use of wait_for_completion() in a loop
makes no sense. When you send a new copy of this patch without
whitespace damage, please fix that up too...
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
701 - 746 of 746 matches
Mail list logo