Messages in this thread | | | Subject | Re: [BK] disconnected operation | From | James Bottomley <> | Date | Sun, 26 Dec 2004 12:25:46 -0600 |
| |
On Sun, 2004-12-26 at 09:19 -0800, Larry McVoy wrote: > For James, could you do a little debugging please? Run the following > when you are plugged in and it works and also when it doesn't: > > bk getuser > bk getuser -r > bk gethost > bk gethost -r > bk dotbk > > We'll track it down and fix it if it is a problem on our end. This stuff > is supposed to work, we certainly haven't intentionally caused a problem.
OK, I cloned a new repository and started applying patches to it. The transcript of what I did is attached. You can see that after I disconnect from the network, I get three emails imported before it spits an error at me.
James
jejb@mulgrave> bk lease renew jejb@mulgrave> PATH=/home/jejb/BK/BK-kernel-tools:$PATH jejb@mulgrave> dotest < ~/tmp.mail bk import -tpatch -CR -yibmvscsi.c: replace schedule_timeout() with msleep() /tmp/patch20138 . Patching... Patching file drivers/scsi/ibmvscsi/ibmvscsi.c Checking for potential renames in /home/jejb/BK/test-2.6 ... Checking in new or modified files in /home/jejb/BK/test-2.6 ... bk commit -y[PATCH] ibmvscsi.c: replace schedule_timeout() with msleep()
Description: Use msleep() instead of schedule_timeout() to guarantee the task delays as expected. Originally submitted to linux-scsi by the janitors, and resubmitted by boutcher (after testing :-)
Signed-off-by: Nishanth Aravamudan <nacc@us.ibm.com> Signed-off-by: Maximilian Attems <janitor@sternwelten.at> Signed-off-by: Dave Boutcher <boutcher@us.ibm.com>
ChangeSet revision 1.2206: +1 -0 = 38195 Sending ChangeSet log ... jejb@mulgrave> bk getuser jejb jejb@mulgrave> bk getuser -r jejb jejb@mulgrave> bk gethost mulgrave.(none) jejb@mulgrave> bk gethost -r mulgrave.(none) jejb@mulgrave> bk dotbk /home/jejb/.bk jejb@mulgrave> dotest < ~/tmp.mail bk import -tpatch -CR -yibmvscsi.c: limit size of I/O requests /tmp/patch20470 . Patching... Patching file drivers/scsi/ibmvscsi/ibmvscsi.c Checking for potential renames in /home/jejb/BK/test-2.6 ... Checking in new or modified files in /home/jejb/BK/test-2.6 ... bk commit -y[PATCH] ibmvscsi.c: limit size of I/O requests
Description: Limit the size of I/O requests sent by the ibmvscsi adapter. With better I/O scheduling (and thus larger requests) we were breaking some servers.
Signed-off-by: Dave Boutcher <boutcher@us.ibm.com>
ChangeSet revision 1.2207: +1 -0 = 38196 Sending ChangeSet log ...
[HERE I FLIP OUT THE WIRELESS CARD TO DISCONNECT]
jejb@mulgrave> dotest < ~/tmp.mail bk import -tpatch -CR -yscsi/aic7xxx/aic79xx_osm.c: remove an unused function /tmp/patch20653 . Patching... Patching file drivers/scsi/aic7xxx/aic79xx_osm.c Checking for potential renames in /home/jejb/BK/test-2.6 ... Checking in new or modified files in /home/jejb/BK/test-2.6 ... bk commit -y[PATCH] scsi/aic7xxx/aic79xx_osm.c: remove an unused function
[ this time without the problems due to a digital signature... ]
The patch below removes an unused function from drivers/scsi/aic7xxx/aic79xx_osm.c
diffstat output: drivers/scsi/aic7xxx/aic79xx_osm.c | 26 -------------------------- 1 files changed, 26 deletions(-)
Signed-off-by: Adrian Bunk <bunk@stusta.de>
ChangeSet revision 1.2208: +1 -0 = 38197 Sending ChangeSet log ... jejb@mulgrave> dotest < ~/tmp.mail bk import -tpatch -CR -yscsi/ahci.c: remove an unused function /tmp/patch20760 . Patching... Patching file drivers/scsi/ahci.c Checking for potential renames in /home/jejb/BK/test-2.6 ... Checking in new or modified files in /home/jejb/BK/test-2.6 ... bk commit -y[PATCH] scsi/ahci.c: remove an unused function
[ this time without the problems due to a digital signature... ]
The patch below removes an unused function from drivers/scsi/ahci.c
diffstat output: drivers/scsi/ahci.c | 9 --------- 1 files changed, 9 deletions(-)
Signed-off-by: Adrian Bunk <bunk@stusta.de>
ChangeSet revision 1.2209: +1 -0 = 38198 Sending ChangeSet log ... jejb@mulgrave> dotest < ~/tmp.mail bk import -tpatch -CR -ygdth: reduce large on-stack locals /tmp/patch20867 . Patching... Patching file drivers/scsi/gdth.c Patching file drivers/scsi/gdth_proc.c Checking for potential renames in /home/jejb/BK/test-2.6 ... Checking in new or modified files in /home/jejb/BK/test-2.6 ... bk commit -y[PATCH] gdth: reduce large on-stack locals
gdth is the fourth-highest stack user (for a single function) in 2.6.10-rc3-bk-recent (sizes on x86-32).
Reduce stack usage in gdth driver: reduce ioc_rescan() from 1564 to 52 bytes; reduce ioc_hdrlist() from 1528 to 24 bytes; reduce gdth_get_info() from 1076 to 300 bytes;
Signed-off-by: Randy Dunlap <rddunlap@osdl.org>
diffstat:= drivers/scsi/gdth.c | 194 ++++++++++++++++++++++++++--------------------- drivers/scsi/gdth_proc.c | 145 ++++++++++++++++++----------------- 2 files changed, 189 insertions(+), 150 deletions(-)
[HERE BK POPS UP A DIALOGUE SAYING:
Unable to obtain permission to use this version of BitKeeper (bk-3.2.3) from lease.openlogging.org. That server issues certificates to use BK for openlogging for 30 days at a time. The bk binary needs to be able to make a http connection to lease.openlogging.org at least once a month.
Look at 'bk help url' if you need to tell 'bk' about a proxy.
AND THE COMMIT FAILS WITH THE FOLLOWING:]
You need to figure out why you have two files with the same ID and correct that situation before this ChangeSet can be created. jejb@mulgrave> bk getuser jejb jejb@mulgrave> bk getuser -r jejb jejb@mulgrave> bk gethost mulgrave.(none) jejb@mulgrave> bk gethost -r mulgrave.(none) jejb@mulgrave> bk dotbk /home/jejb/.bk
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |