super_operations in -pre7

2001-01-16 Thread Matthew Wilcox
several new operations have been added to super_operations, presumably as part of the reiserfs merge. write_super_lockfs and unlockfs are never called. can we remove them? -- Revolutions do not require corporate support. - To unsubscribe from this list: send the line "unsubscribe

Re: CML2 design philosophy heads-up

2001-05-13 Thread Matthew Wilcox
Eric S. Raymond wrote: Reasoned objections can change my behavior. Grunting territorial challenges at me will not. You have two options: (1) persuade Linus that the whole CML2 thing is a bad idea and should be dropped, or (2) work with me to correct any errors I have made and improve the

Re: [RFD w/info-PATCH] device arguments from lookup, partion code

2001-05-19 Thread Matthew Wilcox
On Sat, May 19, 2001 at 10:22:55PM -0400, Richard Gooch wrote: The transaction(2) syscall can be just as easily abused as ioctl(2) in this respect. But read() and write() cannot. -- Revolutions do not require corporate support. - To unsubscribe from this list: send the line unsubscribe

Re: [RFD w/info-PATCH] device arguments from lookup, partion code

2001-05-22 Thread Matthew Wilcox
On Tue, May 22, 2001 at 08:49:04AM +0100, Alan Cox wrote: For _devices_, though? I don't expect my mouse to work if gpm and xfree both try to consume device events from the same filp. Heck, it doesn't even work when they try to consume events from the same inode :-) I think this is a

Re: [RFD w/info-PATCH] device arguments from lookup, partion code

2001-05-22 Thread Matthew Wilcox
On Tue, May 22, 2001 at 04:31:37PM +0100, Alan Cox wrote: `the class of devices in question' was cryptographic devices, and possibly other transactional DSPs. I don't consider audio to be transactional. in any case, you can do transactional things with two threads, as long as they each

Re: [RFD w/info-PATCH] device arguments from lookup, partion code

2001-05-21 Thread Matthew Wilcox
On Mon, May 21, 2001 at 06:13:18PM -0700, Linus Torvalds wrote: Nope. You can (and people do, quite often) share filps. So you can't associate state with it. For _devices_, though? I don't expect my mouse to work if gpm and xfree both try to consume device events from the same filp. Heck, it

Re: [RFD w/info-PATCH] device arguments from lookup, partion code

2001-05-21 Thread Matthew Wilcox
On Tue, May 22, 2001 at 02:22:34AM +0200, Ingo Oeser wrote: ioctl has actually 4 semantics: command only command + read command + write command + rw-transaction Separating these would be a first step. And yes, I consider each of them useful. command only: reset drive echo 'reset'

Re: [RFD w/info-PATCH] device arguments from lookup, partion code

2001-05-19 Thread Matthew Wilcox
On Sat, May 19, 2001 at 05:25:22PM +0100, Alan Cox wrote: Only to an English speaker. I suspect Quebec City canadians would prefer a different command set. Should we support `pas387' as well as `no387' as a kernel boot parameter then? Face it, a sysadmin has to know the limited subset of

Re: [RFD w/info-PATCH] device arguments from lookup, partion code

2001-05-20 Thread Matthew Wilcox
On Sun, May 20, 2001 at 03:11:53PM -0400, Alexander Viro wrote: Pheeew... Could you spell about megabyte of stuff in ioctl.c? No. $ ls -l arch/*/kernel/ioctl32*.c -rw-r--r--1 willywilly 22479 Jan 24 16:59 arch/mips64/kernel/ioctl32.c -rw-r--r--1 willywilly 109475 May

Re: Why side-effects on open(2) are evil. (was Re: [RFD w/info-PATCH] device arguments from lookup)

2001-05-19 Thread Matthew Wilcox
On Sat, May 19, 2001 at 12:51:07PM -0400, Alexander Viro wrote: clone(), walk(), clunk(), stat() and open() ;-) Basically, we can add unopened descriptors. I.e. no IO until you open it (turning the thing into opened one), but we can do lookups (move to child), we can clone and kill them and

Re: Semantics of remount

2000-09-24 Thread Matthew Wilcox
On Sat, Sep 09, 2000 at 03:18:57PM +0200, Andreas Gruenbacher wrote: Hi, What are the intended semantics for a remount: (a) equivalent to a mount, resetting all mount options that might be set (b) change mount options relative to the current mount Aoptions For ext2, the 2.2.17

Re: [PATCH] af_rose.c: s/suser/capable/ + micro cleanups

2000-08-31 Thread Matthew Wilcox
On Thu, Aug 31, 2000 at 12:26:29AM +0200, Rogier Wolff wrote: int mr (unsigned int rate, int r) { int e = 16+9; static int round[4]={0, 0, 0x, 0x8000}; if (!rate) return 0; for (; rate 0xfc00 ;rate = 1, e++); for (;!(rate 0xfe00);rate = 1,

dontdiff updates

2000-10-04 Thread Matthew Wilcox
hi tigran. can we add some more entries to dontdiff? *.ver sm_tbl_*.h -- Revolutions do not require corporate support. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] Please read the FAQ at http://www.tux.org/lkml/

[RFC] Imminent death of /proc/locks predicted; film at 11

2000-09-30 Thread Matthew Wilcox
Does anyone actually want /proc/locks to stay? The data structure which allows it to be generated is now only used by the code to print out /proc/locks. If it could be deleted, a lot of code and data pointers could go away. I don't think any program depends on its existance and it's a pretty

Re: [RFC] Imminent death of /proc/locks predicted; film at 11

2000-09-30 Thread Matthew Wilcox
On Sun, Oct 01, 2000 at 12:29:27AM +0100, Alan Cox wrote: Does anyone actually want /proc/locks to stay? The data structure I'd like a way to view the locks that exist - its useful for debugging weird app stuff out /proc/locks. If it could be deleted, a lot of code and data pointers

Re: [RFC] Imminent death of /proc/locks predicted; film at 11

2000-09-30 Thread Matthew Wilcox
On Sun, Oct 01, 2000 at 12:55:30AM +0100, Alan Cox wrote: When debugging this kind of problem, you're not interested in the non-conflicting locks, only the ones which are blocked waiting for another lock, right? All I care about is what locks are currently in force in the system. So for

Re: [RFC] Imminent death of /proc/locks predicted; film at 11

2000-09-30 Thread Matthew Wilcox
On Sat, Sep 30, 2000 at 08:23:37PM -0500, Peter Samuelson wrote: [Matthew Wilcox] if fcntl took a 4th argument specifying the length of the buffer, i'd recommend a F_GETLKS fcntl. a horrid work around for this would be that the first 4 bytes of the buffer pointed to by the third

[PATCH] Add documentation for string functions

2001-03-03 Thread Matthew Wilcox
diff -urNX dontdiff linux-2.4.2/Documentation/DocBook/Makefile linux-willy/Documentation/DocBook/Makefile --- linux-2.4.2/Documentation/DocBook/Makefile Sat Feb 3 13:43:55 2001 +++ linux-willy/Documentation/DocBook/Makefile Sat Mar 3 18:01:07 2001 @@ -82,6 +82,8 @@

[PATCH] Documentation for bitops

2001-03-04 Thread Matthew Wilcox
This is just the kernel-doc parts; the .tmpl file entry is missing until i get the chance to sync up with Alan again. --- linux-2.4.2/include/asm-i386/bitops.h Wed Feb 21 17:09:56 2001 +++ linux-willy/include/asm-i386/bitops.h Mon Mar 5 00:39:28 2001 @@ -23,6 +23,16 @@ #define

Re: (struct dentry *)-vfsmnt;

2001-03-14 Thread Matthew Wilcox
On Wed, Mar 14, 2001 at 10:26:50AM -0700, Andreas Dilger wrote: Let me put it that way: I don't understand why (if it is useful at all) it is done in the fs. Looks like a wrong level... For the same reason that the UUID and LABEL are stored in the superblock: you want this infomation kept

[PATCH] make config w/ Pentium-II

2000-12-12 Thread Matthew Wilcox
I built 2.4.0-test10 for my laptop and got caught out by `make config'. When I typed `Pentium-II' for my CPU type, it selected Pentium-III and my kernel wouldn't boot. To prevent errors of this type, please apply this patch. As a bonus, `Celeron' will now work as an answer too. Please apply.

[PATCH] find_vma_prev rewrite

2000-12-23 Thread Matthew Wilcox
find_vma_prev doesn't return a pointer to the `prev' vma if the address is greater than the last existing vma. This doesn't matter unless you're on a PA-RISC machine :-) This rewrite should speed up make find_vma_prev simpler, as well as fixing the previous behaviour. --- linux-t10/mm/mmap.c

[PATCH] kmalloc(HIGHUSER) crashes

2000-12-28 Thread Matthew Wilcox
doh, i'm a moron. -- This is a copy of the message -- Date: Thu, 28 Dec 2000 14:51:25 + From: Matthew Wilcox [EMAIL PROTECTED] To: [EMAIL PROTECTED] Cc: [EMAIL PROTECTED] Subject: [PATCH] kmalloc(HIGHUSER) crashes The slab cache accepts the __GFP_HIGHMEM flag, but it will then die

Re: [PATCH] move xchg/cmpxchg to atomic.h

2001-01-02 Thread Matthew Wilcox
On Tue, Jan 02, 2001 at 01:03:48AM -0800, David S. Miller wrote: If you require an external agent (f.e. your spinlock) because you cannot implement xchg with a real atomic sequence, this breaks the above assumptions. We really can't. We _only_ have load-and-zero. And it has to be 16-byte

Re: OK, let's try cleaning up another nit. Is anyone paying attention?

2001-04-19 Thread Matthew Wilcox
On Thu, Apr 19, 2001 at 18:50:34 EDT, Eric S. Raymond wrote: Remove dead CONFIG_BINFMT_JAVA symbol. Please don't do this, it just makes merging our patches with Linus harder. This has already been done in our tree since Feb 1. In fact, please don't touch anything in the tree which is PA

Re: OK, let's try cleaning up another nit. Is anyone paying attention?

2001-04-19 Thread Matthew Wilcox
On Thu, Apr 19, 2001 at 11:00:09PM -0400, Eric S. Raymond wrote: What is the right procedure for doing changes like this? Is "don't touch that tree" a permanent condition, or am I going to get a chance to clean up the global CONFIG_ namespace after your next merge-down? Our current status is

Re: OK, let's try cleaning up another nit. Is anyone paying attention?

2001-04-19 Thread Matthew Wilcox
On Thu, Apr 19, 2001 at 10:07:22PM -0600, james rich wrote: Doesn't this seem a little like the problems occurring with lvm right now? A separate tree maintained with the maintainers not wanting others submitting patches that conflict with their particular tree? It seems that any project

Re: [parisc-linux] Re: OK, let's try cleaning up another nit. Is anyone paying attention?

2001-04-20 Thread Matthew Wilcox
On Fri, Apr 20, 2001 at 11:15:12AM -0500, Bob McElrath wrote: This may be a dumb question, but is there some place where the arch maintainers are listed? Where the arch-specific trees are kept? Where would I go to get the latest set of relevant patches for alpha? http://www.kernel.org/ has

Re: [parisc-linux] Re: OK, let's try cleaning up another nit. Is anyone paying attention?

2001-04-20 Thread Matthew Wilcox
On Fri, Apr 20, 2001 at 02:00:00PM -0500, Jeff Dike wrote: [EMAIL PROTECTED] said: http://www.kernel.org/ has a list of architecture websites. Also the CREDITS / MAINTAINERS files tend to list the people who are involved. Except it's restricted to processor ports, which would leave you

Re: OK, let's try cleaning up another nit. Is anyone paying attention?

2001-04-20 Thread Matthew Wilcox
On Fri, Apr 20, 2001 at 03:47:43PM -0400, Eric S. Raymond wrote: CONFIG_BINFMT_SOM: arch/parisc/config.in arch/parisc/defconfig Not used in code anywhere. Can you get rid of this one? Code not merged yet. CONFIG_DMB_TRAP: arch/parisc/kernel/sba_iommu.c CONFIG_FUNC_SIZE:

Architecture-specific include files

2001-04-22 Thread Matthew Wilcox
Something which came up in one of the hallway discussions at the kernelsummit was that a lot of the architecture maintainers would find it more convenient if the arch-specific header files were moved from include/asm-$ARCH to arch/$ARCH/include. Since we use a symlink _anyway_, no global

Re: question on atomic_t

2001-04-22 Thread Matthew Wilcox
can I assume that a member of a structure of type atomic_t is 0 after using memset to zero on the structure ? You must not do this. use atomic_init instead. on parisc, the locked value of a spinlock is 0, so no more updates to that atomic value would ever work. -- Revolutions do not

Re: [Final call for testers][PATCH] superblock handling changes (2.4.6-pre3)

2001-06-15 Thread Matthew Wilcox
On Fri, Jun 15, 2001 at 01:10:00AM -0400, Alexander Viro wrote: +static inline void write_super(struct super_block *sb) +{ + lock_super(sb); + if (sb-s_root sb-s_dirt) When I first looked at this, I thought it was a typo. I don't think we should

Re: [RFC] sane access to per-fs metadata (was Re: [PATCH] Documentation/ioctl-number.txt)

2001-03-23 Thread Matthew Wilcox
On Fri, Mar 23, 2001 at 09:56:47AM -0700, Bryan Henderson wrote: There's a lot of cool simplicity in this, both in implementation and application, but it leaves something to be desired in functionality. This is partly because the price you pay for being able to use existing, well-worn

Re: 64-bit block sizes on 32-bit systems

2001-03-26 Thread Matthew Wilcox
On Mon, Mar 26, 2001 at 08:39:21AM -0800, LA Walsh wrote: I vaguely remember a discussion about this a few months back. If I remember, the reasoning was it would unnecessarily slow down smaller systems that would never have block devices in the 4-28T range attached. 4k page size * 2GB =

Re: 64-bit block sizes on 32-bit systems

2001-03-26 Thread Matthew Wilcox
On Mon, Mar 26, 2001 at 08:01:21PM +0200, Manfred Spraul wrote: drivers/block/ll_rw_blk.c, in submit_bh() bh-b_rsector = bh-b_blocknr * (bh-b_size 9); But it shouldn't cause data corruptions: It was discussed a few months ago, and iirc LVM refuses to create too large volumes. Ah yes,

Re: 64-bit block sizes on 32-bit systems

2001-03-26 Thread Matthew Wilcox
On Mon, Mar 26, 2001 at 10:47:13AM -0700, Andreas Dilger wrote: What do you mean by problems 5 years down the road? The real issue is that this 32-bit block count limit affects composite devices like MD RAID and LVM today, not just individual disks. There have been several postings I have

Re: 64-bit block sizes on 32-bit systems

2001-03-27 Thread Matthew Wilcox
On Mon, Mar 26, 2001 at 08:39:21AM -0800, LA Walsh wrote: So...is it the plan, or has it been though about -- 'abstracting' block numbes as a typedef 'block_nr', then at compile time having it be selectable as to whether or not this was to be a 32-bit or 64 bit quantity -- that way older

Re: [nameidata 1/2] Don't pass NULL nameidata to vfs_create

2007-04-16 Thread Matthew Wilcox
On Mon, Apr 16, 2007 at 06:11:30PM +0200, Andreas Gruenbacher wrote: +static inline int +nfsd_do_create(struct inode *dir, struct dentry *child, struct vfsmount *mnt, +int mode) +{ + struct nameidata nd = { + .dentry = child, + .mnt = mnt, + };

Re: [PATCH] add two SCSI command opcodes

2007-04-19 Thread Matthew Wilcox
On Thu, Apr 19, 2007 at 07:39:59PM +0300, Dan Aloni wrote: On Thu, Apr 19, 2007 at 05:47:43PM +0200, Jan-Benedict Glaw wrote: Where's the user? A privately maintained kernel driver. Do we _must_ have in-tree users? I'd consider the change for completion's sake. I agree with Dan -- if

Re: [RFC][PATCH] sys_fallocate() system call

2007-03-17 Thread Matthew Wilcox
On Sat, Mar 17, 2007 at 08:59:05PM +1100, Paul Mackerras wrote: ... but wouldn't work on 32-bit powerpc. :( We would end up with a pad argument between fd and offset, giving 7 arguments in all (counting the loff_t's as 2), but we only support 6. Ditto mips and parisc. - To unsubscribe from

Re: [RFC][PATCH] sys_fallocate() system call

2007-03-17 Thread Matthew Wilcox
On Fri, Mar 16, 2007 at 05:17:04PM +0100, Heiko Carstens wrote: +asmlinkage long sys_fallocate(int fd, int mode, loff_t offset, loff_t len) e.g. asmlinkage long sys_fallocate(int fd, loff_t offset, loff_t len, int mode) would work even on s390 ;) How about: asmlinkage long

Re: forced umount?

2007-03-18 Thread Matthew Wilcox
On Sun, Mar 18, 2007 at 08:16:19PM +0100, Arjan van de Ven wrote: the problem with the people who say they want forced umount is.. that most of the time they either want 1) get rid of the namespace entry or 2) want to stop any and all IO to a certain device/partition There is a third

Re: REISER4 FOR INCLUSION IN THE LINUX KERNEL.

2007-04-09 Thread Matthew Wilcox
On Sun, Apr 08, 2007 at 09:16:59PM -0700, [EMAIL PROTECTED] wrote: REISER4 FOR INCLUSION IN THE LINUX KERNEL. Fuck off. Cheerleading like this only hurts your cause. - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More

Re: [KJ] remove SPIN_LOCK_UNLOCKED

2007-04-10 Thread Matthew Wilcox
On Tue, Apr 10, 2007 at 05:45:07PM -0400, Robert P. J. Day wrote: that works fine if you're defining a single spinlock, but what do you do in cases like this: arch/sparc/lib/atomic32.c: [0 ... (ATOMIC_HASH_SIZE-1)] = SPIN_LOCK_UNLOCKED that is, when you're assigning an array of

Re: [patch] scsi: revert [SCSI] Get rid of scsi_cmnd-done

2008-01-07 Thread Matthew Wilcox
Personally, I've lost count of the number of times I've *not* reported a bug/oops just because of the NVidia users this means you statement, only to have the exact same issue reported several weeks/months later by somebody who's able to replicate it with an untainted kernel. If you can

Re: [patch] scsi: revert [SCSI] Get rid of scsi_cmnd-done

2008-01-07 Thread Matthew Wilcox
On Mon, Jan 07, 2008 at 06:04:25PM -0500, [EMAIL PROTECTED] wrote: Theoretically, at least. Sometimes, in the real world, other constraints enter into it... So you're saying that you can't find reliable ways to reproduce problems on demand? Those are some of the lower quality bug reports, so

Re: [JANITOR PROPOSAL] Switch ioctl functions to -unlocked_ioctl

2008-01-08 Thread Matthew Wilcox
On Tue, Jan 08, 2008 at 01:16:06PM -0700, Matthew Wilcox wrote: On Tue, Jan 08, 2008 at 09:03:13PM +0100, Paolo Ciarrocchi wrote: Yes of course, I've been silly in didn't verify whether the file compile but I would appreciate to know whether I'm on the right track or not. Well ... you're

Re: [JANITOR PROPOSAL] Switch ioctl functions to -unlocked_ioctl

2008-01-08 Thread Matthew Wilcox
On Tue, Jan 08, 2008 at 08:58:04PM +0100, Paolo Ciarrocchi wrote: -static const struct file_operations rtc_fops = { +static long rtc_fioctl(struct file_operations rtc_fops) +{ + lock_kernel(); .owner = THIS_MODULE, You should probably restrict yourself to files that you

Re: [JANITOR PROPOSAL] Switch ioctl functions to -unlocked_ioctl

2008-01-08 Thread Matthew Wilcox
On Tue, Jan 08, 2008 at 09:03:13PM +0100, Paolo Ciarrocchi wrote: On Jan 8, 2008 9:00 PM, Matthew Wilcox [EMAIL PROTECTED] wrote: On Tue, Jan 08, 2008 at 08:58:04PM +0100, Paolo Ciarrocchi wrote: -static const struct file_operations rtc_fops = { +static long rtc_fioctl(struct

Re: [PATCH] Change paride driver to use unlocked_ioctl instead of ioctl

2008-01-09 Thread Matthew Wilcox
On Thu, Jan 10, 2008 at 12:53:04PM +0530, Nikanth Karthikesan wrote: default: printk(%s: Unimplemented ioctl 0x%x\n, tape-name, cmd); + unlock_kernel(); return -EINVAL; Surely a bug ... shouldn't this return -ENOTTY? -- Intel are signing my

Re: sata_nv does not function in kernel 2.6.20.21

2008-01-09 Thread Matthew Wilcox
On Thu, Jan 10, 2008 at 02:47:33AM +, Matthew Hall wrote: I am using the Supermicro H8DCE motherboard. Some (not all) of the SATA channels quit working due to some kind of resource conflict when I upgrade to any kernel above 2.6.20.xx series, in my case I am running 2.6.20.21 SMP x86_64

Re: [PATCH] Change paride driver to use unlocked_ioctl instead of ioctl

2008-01-10 Thread Matthew Wilcox
On Thu, Jan 10, 2008 at 09:01:41PM +0530, Nikanth Karthikesan wrote: On Thu, 2008-01-10 at 16:01 +0100, Jiri Kosina wrote: On Wed, 9 Jan 2008, Alan Cox wrote: default: printk(%s: Unimplemented ioctl 0x%x\n, tape-name, cmd); +

Re: [patch 1/1] Switch ioctl functions of drivers/scsi/sg.c to unlocked_ioctl

2008-01-10 Thread Matthew Wilcox
On Thu, Jan 10, 2008 at 07:59:44PM +0100, Andi Kleen wrote: Really, all this is doing is open coding what the ioctl handler is doing anyway, isn't it? in which case, why bother to change it at all? Because once it's open coded it is visible and can then be eliminated. Does SCSI need the

Re: [patch 1/1] Switch ioctl functions of drivers/scsi/sg.c to unlocked_ioctl

2008-01-10 Thread Matthew Wilcox
On Thu, Jan 10, 2008 at 08:32:27PM +0100, Andi Kleen wrote: Not many really in the core kernel. Hardly any. Grep for lock_kernel to be sure, but there is not much. It's mostly drivers that still need it. How about the low level SCSI drivers that might called from the high level SCSI

Re: [PATCH] Random number driver: make random_ioctl as an unlocked_ioctl function

2008-01-11 Thread Matthew Wilcox
On Fri, Jan 11, 2008 at 05:09:52PM +0530, Nikanth Karthikesan wrote: The random_ioctl is registered as an ioctl function but it does not require BKL to be held when called. Changing it as an unlocked_ioctl function. Signed-off-by: Nikanth Karthikesan [EMAIL PROTECTED] Acked-by: Matthew

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-11 Thread Matthew Wilcox
On Fri, Jan 11, 2008 at 11:02:29AM -0800, Greg KH wrote: Can you send me a follow-on patch that documents this in Documentation/ABI please. Greg, if you integrate Ivan's patch, you don't need Arjan's patch. -- Intel are signing my paycheques ... these opinions are still mine Bill, look, we

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-11 Thread Matthew Wilcox
On Fri, Jan 11, 2008 at 01:28:30PM -0800, Linus Torvalds wrote: On Fri, 11 Jan 2008, Matthew Wilcox wrote: Did I miss a bug report? The only problems I'm currently aware of are the ones where using MMCONFIG during BAR probing causes a hard lockup on some Intel machines, and the ones

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-11 Thread Matthew Wilcox
On Fri, Jan 11, 2008 at 12:27:06PM -0800, Linus Torvalds wrote: On Fri, 11 Jan 2008, Matthew Wilcox wrote: Ivan's patch doesn't start enabling MMCONFIG in more places than we currently do. It makes us use conf1 accesses for all accesses below 256 bytes. That fixes all known problems

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-11 Thread Matthew Wilcox
On Fri, Jan 11, 2008 at 11:45:24AM -0800, Greg KH wrote: On Fri, Jan 11, 2008 at 11:40:02AM -0800, Arjan van de Ven wrote: Personally I absolutely don't agree with that. Ivan's patch is another attempt to make MMCONFIG work somewhat better, but does not provide the explicit opt-in that I

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-11 Thread Matthew Wilcox
On Fri, Jan 11, 2008 at 11:58:23AM -0800, Linus Torvalds wrote: On Fri, 11 Jan 2008, Matthew Wilcox wrote: So your argument is that MMCONFIG sucks, therefore Linux has to have a horrible interface to extended PCI config space? What's *your* point? MMCONFIG is known broken. If we ever

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-11 Thread Matthew Wilcox
On Fri, Jan 11, 2008 at 01:12:12PM -0800, Linus Torvalds wrote: On Fri, 11 Jan 2008, Matthew Wilcox wrote: But they can't. We limit the size they can access to 256 bytes, unless the kernel probed address 256 and it worked. Umm. Probing address 256 (or *any* address) using MMCONFIG

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-12 Thread Matthew Wilcox
On Sat, Jan 12, 2008 at 09:45:57AM -0800, Arjan van de Ven wrote: btw this is my main objection to your patch; it intertwines the conf1 and mmconfig code even more. When (and I'm saying when not if) systems arrive that only have MMCONFIG for some of the devices, we'll have to detangle this

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-12 Thread Matthew Wilcox
On Sat, Jan 12, 2008 at 08:42:48PM -0800, Arjan van de Ven wrote: Wanne bet there'll be devices that screw this up? THere's devices that even screwed up the 64-256 region after all. I don't know if they 'screwed it up'. There are devices that misbehave when registers are read from pci config

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-12 Thread Matthew Wilcox
On Sat, Dec 29, 2007 at 12:12:19AM +0300, Ivan Kokshaysky wrote: On Fri, Dec 28, 2007 at 12:40:53PM -0500, Loic Prylli wrote: One thing that could be changed in pci_cfg_space_size() is to avoid making a special case for PCI-X 266MHz/533Mhz (assume cfg_size == 256 for such devices too,

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-12 Thread Matthew Wilcox
On Sun, Jan 13, 2008 at 06:08:05PM +1100, Benjamin Herrenschmidt wrote: On Sat, 2008-01-12 at 17:40 +0300, Ivan Kokshaysky wrote: Actually I'm strongly against Arjan's patch. First, it's based on assumption that the MMCONFIG thing is sort of fundamentally broken on some systems, but none of

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-12 Thread Matthew Wilcox
On Sun, Jan 13, 2008 at 12:24:15AM -0700, Matthew Wilcox wrote: Here's a patch (on top of Ivan's) to improve things further. Oops. I forgot to check the ordering of mmconfig vs direct probing, so that patch would end up just using mmconfig for everything. Not what we want. Also, there's three

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-13 Thread Matthew Wilcox
On Sun, Jan 13, 2008 at 10:41:24AM -0800, Arjan van de Ven wrote: Note: There is not a 100% overlap between need and will not be used in the patches that use legacy for 256. In the other patches posted, extended config space will be used in cases where it won't be with my patch. (Most

[PATCH] Fix kprobes on ia64

2008-01-13 Thread Matthew Wilcox
If CONFIG_KPROBES is set, we get the error during build: kernel/kprobes.c:1057: error: __ksymtab_jprobe_return causes a section type conflict This is because ia64 defines a static inline jprobe_return which kprobes attempts to EXPORT_SYMBOL. Signed-off-by: Matthew Wilcox [EMAIL PROTECTED

Re: [PATCH] Convert drivers/scsi/ch.c to use unlocked_ioctl

2008-01-14 Thread Matthew Wilcox
On Mon, Jan 14, 2008 at 02:32:13PM +0100, Mathieu Segaud wrote: +#include linux/smp_lock.h You don't add any uses of lock_kernel() and there are none in the driver currently. - .owner= THIS_MODULE, - .open = ch_open, - .release = ch_release, - .ioctl

Re: [PATCH] Convert drivers/scsi/ch.c to use unlocked_ioctl

2008-01-14 Thread Matthew Wilcox
one tab too many for the first three lines. Please correct that one, and then add my: Reviewed-by: Matthew Wilcox [EMAIL PROTECTED] Thanks! -- Intel are signing my paycheques ... these opinions are still mine Bill, look, we understand that you're interested in selling us this operating system

Re: [patch] rewrite rd

2008-01-14 Thread Matthew Wilcox
On Tue, Dec 04, 2007 at 05:26:28AM +0100, Nick Piggin wrote: +static void copy_to_brd(struct brd_device *brd, const void *src, + sector_t sector, size_t n) +{ + struct page *page; + void *dst; + unsigned int offset = (sector (PAGE_SECTORS-1)) SECTOR_SHIFT;

Re: [Patch v2] Make PCI extended config space (MMCONFIG) a driver opt-in

2008-01-14 Thread Matthew Wilcox
() and raw_pci_write(). This is a better interface for ACPI, ia64 and now x86. Make pci_raw_ops private to the x86 arch, and use it to implement raw_pci_read/write. Add a raw_pci_ext_ops for extended config space. Signed-off-by: Matthew Wilcox [EMAIL PROTECTED] diff --git a/arch/ia64/pci/pci.c b/arch

Re: [RFC] A SCSI fault injection framework using SystemTap.

2008-01-14 Thread Matthew Wilcox
On Tue, Jan 15, 2008 at 12:04:09PM +0900, K.Tanaka wrote: I would like to introduce a SCSI fault injection framework using SystemTap. Currently, kernel has Fault-injection framework and Faulty mode for md, which can also be used for testing the error handling. But, they could only produce

Re: [PATCH]PCI:disable resource decode in PCI BAR detection

2007-09-13 Thread Matthew Wilcox
On Thu, Sep 13, 2007 at 02:21:07PM +0800, Shaohua Li wrote: http://bugzilla.kernel.org/show_bug.cgi?id=8833 We write 0x to BARs to detect BAR size, this will change BAR base to 0xfxx depends on BAR size. In the bug, PCI MCFG base address is 0xf400. One PCI device (gfx) has a

Re: [PATCH]PCI:disable resource decode in PCI BAR detection

2007-09-13 Thread Matthew Wilcox
On Thu, Sep 13, 2007 at 03:24:46PM +0800, Shaohua Li wrote: On Thu, 2007-09-13 at 01:31 -0600, Matthew Wilcox wrote: You missed the part where we have to avoid doing this for host bridges ... http://marc.info/?l=linux-kernelm=118809338631160w=2 (I believe it's now queued in Greg's

Re: [PATCH] Blackfin arch: add some missing syscall

2007-09-13 Thread Matthew Wilcox
On Thu, Sep 13, 2007 at 03:56:43PM +0800, Bryan Wu wrote: +/* Not relevant on no-mmu */ I thought this list seemed a little long, so I investigated a couple of them. mbind makes sense (it's only implemented for NUMA ... a NUMA embedded platform? not the kind which runs applications that use

Re: [PATCH]PCI:disable resource decode in PCI BAR detection

2007-09-16 Thread Matthew Wilcox
On Sun, Sep 16, 2007 at 03:13:55PM +0400, Ivan Kokshaysky wrote: Another possible solution is not to use mmconfig for bar sizing at all, if it's *that* broken. It would be more complicated though, since it probably requires changes to all architectures. I don't think it would be that

Re: [PATCH 0/5][RFC] Physical PCI slot objects

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 09:01:29AM -0800, Greg KH wrote: I'm still not sold on this idea at all. I'm really betting that there is a lot of incorrect acpi slot information floating around in machines and odd things will show up in these slot entries. Is that the end of the world? Instead of

Re: [BUG] New Kernel Bugs

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 12:50:08PM -0500, Mark Lord wrote: It's a 540MByte download over a slow link for everyone else. Where do you get this number from? $ du -sh .git/objects/pack/ 249M.git/objects/pack/ $ du -sh .git/objects/ 253M.git/objects/ ie about half what you claim. -- Intel

Re: [BUG] New Kernel Bugs

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 01:43:53PM -0500, Mark Lord wrote: Matthew Wilcox wrote: ie about half what you claim. .. No, it's from earlier in this very thread: Adrian Bunk wrote: git clone \ git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux-2.6.git .. mkdir t cd t git

Re: [PATCH 2/5] Construct one fakephp slot per pci slot

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 01:48:15PM -0600, Linas Vepstas wrote: On Mon, Nov 12, 2007 at 05:13:36PM -0700, Alex Chiang wrote: + slot-name = kmalloc(8, GFP_KERNEL); + sprintf(slot-name, fake%d, count++); Please use snprintf to avoid buffer overruns! Or, since kmalloc can return a 32-byte

Re: [PATCH 3/5, RFC] Introduce pci_slot

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 01:56:46PM -0600, Linas Vepstas wrote: On Mon, Nov 12, 2007 at 05:14:47PM -0700, Alex Chiang wrote: +/* pci_slot represents a physical slot */ +struct pci_slot { + struct pci_bus *bus;/* The bus this slot is on */ + struct pci_slot *next;

Re: [PATCH 0/5][RFC] Physical PCI slot objects

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 10:51:22AM -0800, Greg KH wrote: Ok, again, I want to see the IBM people sign off on this, after testing on all of their machines, before I'll consider this, as I know the IBM acpi tables are odd. That seems a little higher standard than patches are normally held to.

Re: [PATCH 0/5][RFC] Physical PCI slot objects

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 03:41:21PM -0600, Linas Vepstas wrote: /sys/bus/pci/slots /sys/bus/pci/slots/control /sys/bus/pci/slots/control/remove_slot /sys/bus/pci/slots/control/add_slot /sys/bus/pci/slots/0001:00:02.0 /sys/bus/pci/slots/0001:00:02.0/phy_location Ugh. Almost two years ago,

Re: [PATCH 0/5][RFC] Physical PCI slot objects

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 02:56:05PM -0800, Greg KH wrote: Why not just use the code in the linux firmware kit that does this already today from userspace (thanks to Kristen for pointing this out to me on irc.)? So then we have something that works on ACPI-based machines. Who will add support

Re: [PATCH 0/5][RFC] Physical PCI slot objects

2007-11-13 Thread Matthew Wilcox
On Tue, Nov 13, 2007 at 03:33:14PM -0800, Kristen Carlson Accardi wrote: As far as being able to retrieve the slot number (which it seemed from the HP manageablity application perspective is the goal here), that information is available from userspace as well for at least standard PCI and pcie

Re: [BUG] New Kernel Bugs

2007-11-14 Thread Matthew Wilcox
On Wed, Nov 14, 2007 at 12:46:20AM -0700, Denys Vlasenko wrote: Finally they replied and asked to rediff it against their git tree. I did that and sent patches back. No reply since then. And mind you, the patch is not trying to do anything complex, it mostly moves code around, removes

Re: [PATCH 0/5][RFC] Physical PCI slot objects

2007-11-14 Thread Matthew Wilcox
On Wed, Nov 14, 2007 at 02:07:56AM +0100, Andi Kleen wrote: It's not only complexity. Each new sysfs entry costs memory. Memory is not free. There should be always a good reason for those. It's not a lot of memory; it's one directory and a couple of files for each PCI slot in the system. Even

Re: v2.6.24-rc2-409-g9418d5d: attempt to access beyond end of device

2007-11-14 Thread Matthew Wilcox
On Wed, Nov 14, 2007 at 03:53:15PM +0100, Ingo Molnar wrote: By the way: Reverting commit 6f5391c283d7fdcf24bf40786ea79061919d1e1d makes the same cd medium readable again on v2.6.24-rc2-409-g9418d5d. nice - that commit should then be reverted. We're investigating; see bugzilla 9370.

Re: [PATCH 0/5][RFC] Physical PCI slot objects

2007-11-14 Thread Matthew Wilcox
On Wed, Nov 14, 2007 at 03:35:33PM +0100, Andi Kleen wrote: It becomes much more when someone does a find /sys. dentries are expensive. They eventually can get pruned again, but it's still costly to do that. Again, if this is a big concern for you, there are better places to look at for

Re: [PATCH 0/5][RFC] Physical PCI slot objects

2007-11-14 Thread Matthew Wilcox
On Wed, Nov 14, 2007 at 04:08:01PM +0100, Andi Kleen wrote: Whoever is proposing a feature has the burden to justify that its usefulness is larger than the overhead/cost it adds. Doesn't seem to be the case with this one so far. Huh? There are half a dozen people who think it does, and half

Re: [PATCH 2/5] Construct one fakephp slot per pci slot

2007-11-14 Thread Matthew Wilcox
On Wed, Nov 14, 2007 at 12:37:29PM -0700, Alex Chiang wrote: Register one slot per slot, rather than one slot per function. Change the name of the slot to fake%d instead of the pci address. +#define SLOT_NAME_SIZE KOBJ_NAME_LEN Defined, then never used ... how about s/KOBJ_NAME_LEN/8/, then

Re: SCSI breakage on non-cache coherent architectures

2007-11-19 Thread Matthew Wilcox
On Mon, Nov 19, 2007 at 04:35:23PM +1100, Benjamin Herrenschmidt wrote: The other one I'm hitting now is that the SCSI layer nowadays embeds the 'nowadays'? It has always been so. sense_buffer inside the scsi_cmnd structure without any kind of alignment whatsoever. I've been hitting

Re: 2.6.23.1-2.6.23.8 breaks root Makefile

2007-11-19 Thread Matthew Wilcox
On Mon, Nov 19, 2007 at 11:19:00AM -0500, David Ronis wrote: The current main Makefile incorrectly identified my ld as one that supports --build-id (it didn't). The problem is fixed by upgrading to the latest binutils, version 2.18, which does, but somehow I suspect that this wasn't what was

Re: [patch 16/26] hptiop: avoid buffer overflow when returning sense data

2007-11-19 Thread Matthew Wilcox
On Mon, Nov 19, 2007 at 10:19:12AM -0800, Greg Kroah-Hartman wrote: 2.6.22-stable review patch. If anyone has any objections, please let us know. Makes sense to backport this. Acked-by: Matthew Wilcox [EMAIL PROTECTED] -- Intel are signing my paycheques ... these opinions are still mine

Re: [PATCH] MAINTAINERS: remove Adam Fritzler, update his email address in other sources

2007-12-18 Thread Matthew Wilcox
On Mon, Dec 17, 2007 at 08:48:03PM -0800, Joe Perches wrote: diff --git a/MAINTAINERS b/MAINTAINERS index 9507b42..690f172 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3758,13 +3758,6 @@ W: http://www.kernel.org/pub/linux/kernel/people/bunk/trivial/ T: git

Re: INITIO scsi driver fails to work properly

2007-12-19 Thread Matthew Wilcox
On Wed, Dec 19, 2007 at 10:48:27AM +0200, Filippos Papadopoulos wrote: Would it be better to open a bug report at bugzilla? No, it wouldn't. Bugzilla is a place where bug reports go to be ignored. Witness 9370 where despite my best efforts to move discussion to the mailing list, it's been

Re: INITIO scsi driver fails to work properly

2007-12-19 Thread Matthew Wilcox
On Wed, Dec 19, 2007 at 10:50:40AM -0600, James Bottomley wrote: So, to get the best of both worlds, file a bugzilla and note the bugid. Then email a complete report to the relevant list, but add [BUG bugid] to the subject line and cc [EMAIL PROTECTED] If you do this, bugzilla will keep track

Re: [Fwd: Re: [PATCH 0/5]PCI: x86 MMCONFIG]

2007-12-20 Thread Matthew Wilcox
On Thu, Dec 20, 2007 at 09:22:05AM -0800, Greg KH wrote: The one device we know about that throws exceptions is the 830M/MG graphics chip. This chip passes the read-compare test, so the code merrily advances to bus sizing. When the bus sizing code writes the BAR at offset 0x18 in this

  1   2   3   4   5   6   7   8   9   10   >