- 19 Jun, 2002 7 commits
-
-
James Simmons authored
-
James Simmons authored
-
James Simmons authored
-
James Simmons authored
into heisenberg.transvirtual.com:/tmp/fbdev-2.5
-
http://linus.bkbits.net/linux-2.5James Simmons authored
into heisenberg.transvirtual.com:/tmp/linus-2.5
-
James Simmons authored
into heisenberg.transvirtual.com:/tmp/fbdev-2.5
-
http://linux.bkbits.net/linux-2.5James Simmons authored
into heisenberg.transvirtual.com:/tmp/linus-2.5
-
- 18 Jun, 2002 33 commits
-
-
Linus Torvalds authored
-
Linus Torvalds authored
-
Benjamin LaHaise authored
akpm pointed out that an EXPORT_SYMBOL is missing for default_wake_function.
-
http://gkernel.bkbits.net/net-drivers-2.5Linus Torvalds authored
into home.transmeta.com:/home/torvalds/v2.5/linux
-
Jeff Garzik authored
into mandrakesoft.com:/home/jgarzik/repo/net-drivers-2.5
-
Linus Torvalds authored
-
Linus Torvalds authored
Cset exclude: torvalds@penguin.transmeta.com|ChangeSet|20020619003306|07760 Cset exclude: ak@muc.de|ChangeSet|20020618172743|19150
-
Linus Torvalds authored
into penguin.transmeta.com:/home/penguin/torvalds/repositories/kernel/linux
-
Neil Brown authored
Previously each raid personality (Well, 1 and 5) started their own thread to do resync, but md.c had a single common thread to do reconstruct. Apart from being untidy, this means that you cannot have two arrays reconstructing at the same time, though you can have to array resyncing at the same time.. This patch changes the personalities so they don't start the resync, but just leave a flag to say that it is needed. The common thread (mdrecoveryd) now just monitors things and starts a separate per-array thread whenever resync or recovery (or both) is needed. When the recovery finishes, mdrecoveryd will be woken up to re-lock the device and activate the spares or whatever. raid1 needs to know when resync/recovery starts and ends so it can allocate and release resources. It allocated when a resync request for stripe 0 is received. Previously it deallocated for resync in it's own thread, and deallocated for recovery when the spare is made active or inactive (depending on success). As raid1 doesn't own a thread anymore this needed to change. So to match the "alloc on 0", the md_do_resync now calls sync_request one last time asking to sync one block past the end. This is a signal to release any resources.
-
Neil Brown authored
1/ don't free the rdev->sb on an error -- it might be accessed again later. Just wait for the device to be exported. 2/ Change md_update_sb to __md_update_sb and have it clear the sb_dirty flag. New md_update_sb locks the device and calls __md_update_sb if sb_dirty. This avoids any possbile races around updating the superblock
-
Neil Brown authored
Provide SMP safe locking for all_mddevs list. the all_mddevs_lock is added to protect all_mddevs and mddev_map. ITERATE_MDDEV is moved to md.c (it isn't needed elsewhere) and enhanced to take the lock appropriately and always have a refcount on the object that is given to the body of the loop. mddev_find is changed so that the structure is allocated outside a lock, but test-and-set is done inside the lock.
-
Neil Brown authored
If two md arrays which share real devices (i.e they each own a partition on some device) need to sync/reconstruct at the same time, it is much more efficient to have one wait while the other completes. The current code uses interruptible_sleep_on which isn't SMP safe (without the BKL). This patch re-does this code to make it more secure. Even it two start simultaneously, one will reliably get priority, and the other wont wait for ever.
-
Neil Brown authored
More mddev tidyup - remove recovery_sem and resync_sem recovery_sem and resync_sem get replaced by careful use of recovery_running protected by reconfig_sem. As part of this, the creative: down(&mddev->recovery_sem); up(&mddev->recovery_sem); when stopping an array gets replaced by a more obvious wait_event(resync_wait, mddev->recovery_running <= 0);
-
Neil Brown authored
Strengthen the locking of mddev. mddev is only ever locked in md.c, so we move {,un}lock_mddev out of the header and into md.c, and rename to mddev_{,un}lock for consistancy with mddev_{get,put,find}. When building arrays (typically at boot time) we now lock, and unlock as it is the "right" thing to do. The lock should never fail. When generating /proc/mdstat, we lock each array before inspecting it. In md_ioctl, we lock the mddev early and unlock at the end, rather than locking in two different places. In md_open we make sure we can get a lock before completing the open. This ensures that we sync with do_md_stop properly. In md_do_recovery, we lock each mddev before checking it's status. md_do_recovery must unlock while recovery happens, and a do_md_stop at this point will deadlock when md_do_recovery tries to regain the lock. This will be fixed in a later patch.
-
Neil Brown authored
md_ioctl doesn't need to mddev_find, as the mddev must be in the bd_inode->u.generic_ip. This means we don't need to mddev_put either.
-
Neil Brown authored
Only two users of kdev_to_mddev remain, md_release and md_queue_proc. For md_release we can store the mddev in the md_inode at md_open time so we can find it easily. For md_queue_proc, we use mddev_find because we only have the device number to work with. Hopefully the ->queue function will get more arguements one day...
-
Neil Brown authored
This patch gets md_open to use mddev_find instead of kdev_to_mddev, thus creating the mddev if necessary. This guarantees that md_release will be able to find an mddev to mddev_put. Now that we are certain of getting the refcount right at open/close time, we don't need the "countdev" stuff. If START_ARRAY happens to start and array other than that the one that is currently opened, it won't confuse things at all.
-
Neil Brown authored
First step to tidying mddev recounting and locking. This patches introduces mddev_get which incs the refcount on an mddev mddev_put which decs it and, if it becomes unused, frees it mddev_find which finds or allocated an mddev for a given minor This is mostly the old alloc_mddev free_mddev no longer actually frees it. It just disconnects all drives so that mddev_put will do the free. Now the test for "does an mddev exist" is not "mddev != NULL" but involves checking if the mddev has disks or a superblock attached. This makes the semantics of do_md_stop a bit cleaner. Previously if do_md_stop succeed for a real stop (not a read-only stop) then you didn't have to unlock the mddev, otherwise you did. Now you always unlock the mddev after do_md_stop.
-
Neil Brown authored
The mapping from minor number to mddev structure allows for a 'data' that is never used. This patch removes that and explicitly inlines some inline functions that become trivial. mddev_map also becomes completely local to md.c
-
Neil Brown authored
It doesn't really help clarity or brevity.
-
Neil Brown authored
The nb_dev field is not needed. Most uses are the test if it is zero or not, and they can be replaced by tests on the emptiness of the disks list. Other uses are for iterating through devices in numerical order and it makes the code clearer (IMO) to unroll the devices into an array first (which has to be done at some stage anyway) and then walk that array. This makes ITERATE_RDEV_ORDERED un-necessary. Also remove the "name" field which is never used.
-
Neil Brown authored
Discard functions that have been "not yet" for a long time It is not clear what these should do, or if they will ever be needed, so let's clean them out. They can easily be recreated if there is a need.
-
Neil Brown authored
As we now have per-device queues, we don't need a common make_request function that dispatches, we can dispatch directly. Each *_make_request function is changed to take a request_queue_t from which it extract the mddev that it needs, and to deduce the "rw" flag directly from the bio.
-
Neil Brown authored
It currently has several lines of code where one will do.
-
Neil Brown authored
It isn't needed. Only the chunksize is used, and it can be found in the superblock.
-
Neil Brown authored
Several awkard constructs could be replaced by list_del_init, list_for_each or list_empty. Also two bugs fixes: free_device_names was freeing the wrong thing same_set wasn't initialised.
-
Neil Brown authored
-
Neil Brown authored
..as it is nolonger used.
-
Neil Brown authored
We embed a request_queue_t in the mddev structure and so have a separate one for each mddev. This is used for plugging (in raid5). Given this embeded request_queue_t, md_make_request no-longer needs to make from device number to mddev, but can map from the queue to the mddev instead.
-
Neil Brown authored
md: "dev" isn't defined any more. raid5: must report on "bi" before reusing the variable raid5: buffer_head should be bio (not a debugging thing)
-
Neil Brown authored
We embed a request_queue_t in the card structure and so have a separate one for each card. This is used for plugging. Given this embeded request_queue_t, mm_make_request no-longer needs to make from device number to mddev, but can map from the queue to the card instead.
-
Neil Brown authored
Cast to u64 before >>32, incase it was only u32 - thanks to Alan Cox.
-
Neil Brown authored
While this initialisation could be done in individual drivers, it is better to have it central... Init plug_list for make_request_fn devices: blk_queue_make_request should init ->plug_list just like blk_init_queue does.
-