All of lore.kernel.org
 help / color / mirror / Atom feed
* Linux 2.6.0-test3
@ 2003-08-09  5:40 ` Linus Torvalds
  2003-08-09  7:49   ` Linux 2.6.0-test3: USB still broken reg
                     ` (6 more replies)
  0 siblings, 7 replies; 46+ messages in thread
From: Linus Torvalds @ 2003-08-09  5:40 UTC (permalink / raw)
  To: Kernel Mailing List


The bulk of the diff by far is various architecture updates, and in 
particular bringing MIPS[64] a bit closer to being up-to-date for 2.6.x  
But there's arm, alpha, h8300 and ia64 updates too. 

Merging the SELinux security architecture also ends up growing the patch, 
even though it may not be all that noticeable for most normal users.

For most x86 users, the CPU frequency updates, network driver updates, and 
some USB updates are most likely to matter.

And this should fix the PCMCIA lock-up that a number of people have seen
happening since 2.5.71 or so. Thanks for people involved in testing and 
fixing that one.

Also, Andrew fixed a read-ahead bug that was introduced in test2 that
could cause (non-readahead) IO failures under load.

Shortlog follows, full details in the BK trees and in the log on the
ftp/web sites.

		Linus

----

Summary of changes from v2.6.0-test2 to v2.6.0-test3
====================================================

<nikkne:hotpop.com>:
  o I2C: fix Kconfig info

Adam Belay:
  o [PNP] Handle unset resources properly
  o [PNP] Remove protocol_data from pnp_dev and pnp_card
  o [PNPBIOS] Move Parsing Functions to the PnPBIOS driver
  o [SOUND] Remove __(dev)initdata from all pnp sound drivers
  o [PNPBIOS] Move low level code into a separate file
  o [PNP] Remove device naming based on id
  o [PNP] Increment Version Number
  o [PCMCIA] Fix PnP Probing in i82365.c

Alan Cox:
  o [NET]: illegal --> invalid
  o remove a dead EXPORT_NO_SYMBOLS
  o more typo fixes
  o dont assume newer cpus have the same magic registers
  o more arch typo/illegal->invalid fixes
  o docs for updated sk98 from vendor
  o kill an old __NO_VERSION__
  o re-enable SiS direct render
  o console on by default if not embedded (save mucho pain)
  o more typo/invalid bits
  o more typo fixes and dead old code removal
  o mouse and keyboard by default if not embedded
  o second block illegal/invalid fixups for isdn
  o keyboard controller by default if not embedded
  o some isdn invalid/illegal fixups
  o fix return on pms change
  o more typo fixes
  o sk98 vendor driver update
  o more net illegal/invalid typo fixes
  o further z85230 fixes
  o idents for all the new skge cards
  o fix pcmcia_cs without ISA
  o fix invalid/illegal and printk formatting for scsi drivers
  o phonedev handles this
  o phonedev has an owner so this is ok too I think
  o fix build of asix usb ethernet
  o vga text console if x86 and not embedded
  o fix invalid/illegal oddments in USB
  o switch escaped 8859-1 symbols inthe kernel to ascii
  o tdfx framebuffer updates
  o fix binfmt_flat typos
  o kill __NO_VERSION__ in intermezzo
  o use invalid not illegal in reiserfs
  o fix 2 byte data leak due to padding
  o typo fix for time.h
  o fix strncpy on generic user platforms
  o maintainer for sk98
  o the rest of the slab.c fix
  o sunrpc doesnt need uaccess.h
  o fix posix compliance for mkcompile_h script
  o allow 2.6 to build on old old setups
  o fix section conflict and typo in ALSA isa
  o further OSS audio updates
  o further sound updates
  o update Changes for NFS changes
  o work around tosh keyboards
  o update for DC395 driver (big)
  o remaining illegal/invalid/separate stuff for scsi
  o ikconfig

Alan Stern:
  o USB:  Proper I/O buffering for the shuttle_usbat subdriver
  o USB: Fix irq problem in hcd_endpoint_disable()
  o USB: Rename usb_connect() to usb_choose_address()
  o USB: Small fixes for usbtest
  o USB: More unusual_devs.h stuff
  o USB: Rename probe and unbind functions
  o USB: Add functions to enable/disable endpoints, interfaces
  o USB: Use the new enable/disable routines
  o USB: Remove usb_set_maxpacket()
  o USB: usb-storage: Move static string out of initializers.h

Alex Williamson:
  o ia64: New CMC/CPE polling
  o ia64: Update to CMC/CPE polling
  o PCI: trivial 2.4/2.6 PCI name change/addition

Alexey Kuznetsov:
  o [IPV4]: IP options were not updated while forwarding multicasts
  o [PKT_SCHED]: More reasonable PSCHED_JSCALE for various values of HZ

Andi Kleen:
  o x86-64 merge
  o Make x86-64 compile again

Andrew Morton:
  o misc fixes
  o selinux merge
  o re-slabify i386 pgd's and pmd's
  o buffer.c debugging
  o update to speedstep-centrino.c
  o 3c59x suspend/resume fix
  o dev_t printing
  o non-MII 3c59x fix
  o kwsapd can free too much memory
  o unlock_buffer() needs a barrier
  o Interface to invalidate regions of mmaps
  o Fix vmtruncate race and distributed filesystem race
  o fix bogus IO error messages
  o rework readahead for congested queues
  o ext3: avoid reading empty inode blocks
  o Fix race in ext3_getblk
  o ext3: don't start a commit in write_super()
  o fix alloc_bootmem_low_pages
  o soundcard.c devfs fix
  o 6PACK asumes HZ=100
  o devfs_lookup stack corruption fix rework
  o use mark_page_accessed() in the write() path
  o fix kswapd throttling
  o vmscan: decaying average of zone pressure
  o vmscan: use zone_pressure for page unmapping
  o direct-io support for XFS unwritten extents
  o Fix ipt_helper compilation
  o fix select() with an xoffed tty
  o fix ip_conntrack_core.h compile error
  o ext3: fix commit assertion failure
  o fix read_dir()
  o Move the special_file() definition
  o uidhash init-time locking
  o com20020_cs.c doesn't compile
  o pc300_drv build fix
  o binfmt_script argv[0] fix
  o Fix dac960 for devfs
  o quota typo fix
  o i810fb oops fix
  o export agp_memory_reserved
  o IPMI build fix
  o generic HDLC updates
  o dscc4 compile fix for hdlc
  o serial drivers are not experimental
  o ftl.c warning fix
  o Watchdog: use module_param
  o export install_page() to modules
  o cpu_idle() startup race fix
  o fix 64-bit architectures for the binprm change
  o remove PF_READAHEAD
  o ax8817x.c build fix for older gcc's
  o dm: don't use MODULE_PARM
  o dm: remove blk.h include
  o dm: decimal device num sscanf
  o dm: 64 bit ioctl fixes
  o dm: missing #include
  o [patch 7/25 dm: use sector_div()
  o dm: resume() name clash
  o reiserfs: fix savelinks on bigendian arches
  o reiserfs: fix problem when fs is out of space
  o reiserfs: fix races between link and unlink on same
  o remount_fs needs lock_kernel()
  o move_one_page() atomicity fix
  o ide-cd error handling oops fix
  o might_sleep() checks for x86 usercopy functions
  o Don't trigger NMI watchdog for panic delay
  o export lookup_create()
  o fix free_all_bootmem_core for virtual memmap
  o NMI watchdog documentation for x86-64
  o Add do_setitimer prototype to linux/time.h
  o itimer resolution and rounding fixes
  o ext3: handle aborted journals in
  o declare struct irq_desc
  o mtrr race fix
  o initialise page->private

Andries E. Brouwer:
  o osf partition numbering

Andy Grover:
  o ia64 fix for ACPI

Angelo Dell'Aera:
  o [NET]: Convert tc35815.c to spinlocks
  o [NET]: Trivial fix for 82596 driver

Anton Blanchard:
  o ppc64: remove all request_irq calls that occur before kmalloc is
    set up
  o ppc64: Fix any_online_cpu for > 32 cpus, mask must be unsigned long
  o ppc64: clear error_code in do_page_fault on an SLB miss
  o ppc64: add div64 for 32bit boot wrapper
  o ppc64: Add semtimedop
  o ppc64: Add posix timers, tgkill, utimes, statfs64
  o ppc64: add asm/sections.h
  o ppc64: defconfig update
  o [NET]: Add missing memory barriors for __LINK_STATE_RX_SCHED
    handling
  o ppc64: Reserve 8042 IO space on machines with no keyboard
  o __initdata must not be marked const

Arnaldo Carvalho de Melo:
  o wl3501: introduce iw_mgmt_info_element & associate functions and enums
  o wl3501: use iw_mgmt_info_element for phy_pset (now ds_parameter_set)
  o wl3501: fix set_essid wireless extension, using the flags for any
  o wl3501: fix bug in iw_mgmt_info_element id field and more
  o wl3501: introduce iw_mgmt_ibss_pset
  o wl3501: introduce struct iw_mgmt_cf_pset
  o wl3501: introduce iw_mgmt_data_rset and rate labels enum
  o wl3501: implement {get,set}_scan wireless extensions
  o wl3501: add a first cut, lazy scan triggering for set_scan
  o MAINTAINERS: add acme as wl3501 maintainer

Arun Sharma:
  o ia64: make ia32 core dumps work
  o ia64: epoll support for ia32 applications
  o ia64: compat_ioctl.c support

Bart De Schuymer:
  o [NETFILTER]: Fix use after free of skb in nf_reinject()
  o [BRIDGE]: Only build br_netfilter when INET is enabled

Bartlomiej Zolnierkiewicz:
  o ide: kill no longer used CONFIG_BLK_DEV_IDE_MODES
  o ide: kill no longer used CONFIG_BLK_DEV_NFORCE
  o ide: kill no longer used CONFIG_BLK_DEV_PDC202XX
  o ide: kill surplus menu in Kconfig
  o ide: fix ordering in Kconfig
  o ide: clean BLK_DEV_IDE dependencies in Kconfig
  o ide: clean IDE_CHIPSETS dependencies in Kconfig
  o ide: clean BLK_DEV_IDEDMA_PCI dependencies in Kconfig
  o ide: use def_bool in Kconfig
  o move "config SOUND" to sound/Kconfig
  o do not scan partitions twice for removable devices
  o fix ata_probe() driver autoloading
  o ide_unregister() fixes
  o fix return value for idedisk_*_max_address_* functions
  o capacity related fixes
  o init_idedisk_capacity() cleanup()
  o fix CONFIG_IDEDISK_STROKE support in ide-disk.c

Ben Collins:
  o Update IEEE1394 (r1023)

Benjamin Herrenschmidt:
  o I2C: timer clean up for i2c-keywest.c
  o [NET]: Do not call request_irq with spinlock held in sungem.c
  o PPC32: Properly register CPUs
  o PPC32: Update pmac_cpufreq driver back to working
  o Fix mdelay's use of 'msec' name

Bjorn Helgaas:
  o ia64: IOSAPIC .weak symbol cleanup
  o ia64: more ACPI/IOSAPIC cleanup

Chas Williams:
  o [ATM]: Update LANAI driver to modern PCI and DMA APIs (from
    mitch@sfgoth.com)
  o [ATM]: remove EXACT_TS remove from zatm (untested)
  o [ATM]: use set_current_state(x) (from bellucda@tiscali.it)
  o [ATM]: fix a typo in net/atm/Makefile (from eguaj@free.fr)
  o [ATM]: Convert cli() to spinlock in ZATM driver
  o [ATM]: update links to new site (from <mitch@sfgoth.com)
  o [ATM]: Minor LANAI fixes from mitch@sfgoth.com

Chip Salzenberg:
  o Require nfs-utils 1.0.5; document where to get it

Christoph Hellwig:
  o [netdrvr pcmcia] remove the release timer from all pcmcia net
    drivers

Daniel Ritz:
  o [IPV6]: Fix memory leak in esp6_input
  o [PCMCIA] don't call functions twice

Daniele Bellucci:
  o USB: Audit usb_register in drivers/usb/class/audio.c
  o USB: Audit usb_register() in drivers/usb/misc/emi26.c
  o USB: Audit usb_register() in drivers/usb/input/wacom.c
  o USB: Audit usb_register in drivers/usb/input/xpad.c
  o USB: Audit usb_register() in drivers/usb/input/usbkbd.c
  o USB: Audit usb_register() in drivers/usb/input/aiptek.c
  o USB: Audit usb_register in usbmouse_init()
  o USB: Audit usb_register() in drivers/usb/net/catc.c
  o I2C: fixed a little memory leak in i2c-ali15x3.c

Danny ter Haar:
  o CREDITS change

Dave Jones:
  o [CPUFREQ] update Documentation
  o [CPUFREQ] Fix compilation of speedstep-ich.c if SPEEDSTEP_DEBUG is
    set
  o [CPUFREQ] Longrun validate can fail
  o [CPUFREQ] Locking fixes [1/11] First part in a series of patches
    from Dominik which clean up the locking in cpufreq. It sort of
    worked, but is full of races. But to keep it working at least as
    well it works now, add a new spinlock cpufreq_driver_lock which
    will be what cpufreq_driver_sem intended to be -- but it can indeed
    be a spinlock instead of a semaphore. 
  o [CPUFREQ] Locking fixes [2/11] As the per-CPU initialization can
    fail, we need to deal with this situation in cpufreq_remove_dev as
    the sysdev core doesn't (yet) do this for us. The solution: add a
    "struct cpufreq_policy
  o [CPUFREQ] Fix locking [3/11] Change the return value of
    cpufreq_cpu_get from a nondescriptent int to struct
    cpufreq_policy*. This will allow for cleaner code - and later it
    will be expected that anyone who grabs a reference by calling
    cpufreq_cpu_get must return this struct cpufreq_policy* to
    cpufreq_cpu_put.
  o [CPUFREQ] Fix locking [4/11] Make cpufreq_remove_dev a bit more
    readable. Also, assure that the GOV_STOP and the
    cpufreq_driver->exit() call are indeed the last calls done.
  o [CPUFREQ] Fix locking [5/11] Make cpufreq_set_policy a bit more
    readable by storing the CPU's cpufreq_policy into a pointer.
  o [CPUFREQ] Fix locking [6/11] Change the function parameter of
    cpufreq_cpu_put to struct cpufreq_policy *.
  o [CPUFREQ] Fix locking [7/11] Finally implement the two different
    cpufreq_driver_target callbacks -- the one called while the per-CPU
    lock is held, the other while not. While we're at it, clean up
    cpufreq_governor.
  o [CPUFREQ] Fix locking [8/11] Split up cpufreq_governor into a
    locking and non-locking variant.
  o [CPUFREQ] Fix locking [9/11] Use the per-CPU policy->lock to lock
    the per-CPU "policy" data. Also, use it to serialize the setting of
    new frequency policies on each CPU.
  o [CPUFREQ] Fix locking [10/11] Remove the final instances of the
    now-deprecated cpufreq_driver_sem. Also, the previous
    "one-frequency change at one moment" limitation is gone. If it's
    needed by the cpufreq driver, it should be implemented in its
    cpufreq_driver->{target,setpolicy} callback.
  o [CPUFREQ] Fix locking [11/11] Implement per-CPU memory allocation.
    Is a bit cleaner than the previous "once and for all" approach.
  o [CPUFREQ] speedstep_ICH new frequency notification fix
  o [CPUFREQ] Sanity checks for voltage scaling in longhaul driver
  o [AGPGART] Handle Intel chipsets whose BIOS has 'forgotten' to
    assign resources
  o [AGPGART] Fix Nforce modular compilation
  o [CPUFREQ] Update MAINTAINERS/CREDITS Dominik doesn't want
    maintainership any more (besides him still doing lots of great work
    on it. 8-)
  o [CPUFREQ] Sparse __user annotations for acpi driver
  o [CPUFREQ] Fix sparse warnings in powernow-k7
  o [CPUFREQ] sparse __user annotation for cpufreq userspace interface
  o [CPUFREQ] longhaul printk cleanups
  o [CPUFREQ] Better handling of revisionKey bits on Longhaul
  o [CPUFREQ] Move MSRs from powernow-k7.h to include/asm-i386/msr.h
  o [CPUFREQ] Remove CVS idents
  o [CPUFREQ] Move rdmsrl/wrmsrl to include/asm-i386/msr.h
  o [CPUFREQ] Clean up longhaul MSR usage using bitfields
  o [CPUFREQ] Convert Longhaul v1 over to bitfields too
  o Cset exclude:
    davej@codemonkey.org.uk|ChangeSet|20030715115826|58980
  o [CPUFREQ] Fix detection of Transmeta CPUs. Makes longrun work again
  o [CPUFREQ] Fix compiler warnings in msr macros
  o [CPUFREQ] Use standard types in wrmsrl
  o [CPUFREQ] Update documentation
  o [CPUFREQ] Remove deprecated email address
  o Athlon Machine Check fix

Dave Kleikamp:
  o JFS: Missed some non-negative error return codes
  o JFS: fine-grained xattr locking
  o JFS: write_super_lockfs should mark superblock clean

David Brownell:
  o USB: usb audio, remove garbage warning
  o USB: ehci needs a readb() on IDP425 PCI (ARM)
  o USB: usbnet: zaurus c-750, motorola
  o USB: ehci-hcd, TT fixup
  o USB: usb_gadget.h doc fix
  o USB: usb_unlink_urb() kerneldoc
  o USB: ehci-hcd, show microframe schedules
  o USB: ehci-hcd and period=1frame hs interrupts
  o USB: usb root hubs need longer timeout
  o USB: hcd initialization fix
  o USB: ohci-hcd, minor d3cold resume fix
  o USB: usb_new_device() updates
  o USB: disable both sides of usb device ep0 at once
  o USB: usbnet, prevent exotic rtnl deadlock

David Mosberger:
  o ia64: Two small fixes to make ia64 build & work again
  o ia64: Add forgotton <asm-ia64/sections.h>
  o Many files
  o ia64: Define fsid_t for kernel purposes
  o xtalk.h, sn_ksyms.c, sn2_smp.c, cache.c, shuberror.c, shub.c,
    iomv.c, hcl.c
  o ia64: Clean up trailing whitespace
  o ia64: Make elfcore32.h compile for 2.6.0-test2
  o modify data types in efi.h

David S. Miller:
  o [SPARC64]: Mark more things __init in kernel/pci.c
  o [SPARC64]: Make sure to reject all PCI DAC dma masks
  o [SPARC64]: In schizo driver, if virtual-dma property exists,
    respect it
  o [ATM]: Remove -g option from driver directory CFLAGS
  o [NET]: Need to export secpath_dup to modules
  o [SPARC64]: Use i2c/media Kconfigs instead of hardcoding
  o [I2C]: ELV and VELLEMAN depend upon ISA
  o [DVB]: 64-bit fixes for skystar2.c
  o [DVB]: Fix pointer-->int cast in dvb_net.c
  o [DVB]: Fix pointer-->int cast in budget-patch.c
  o [DVB]: saa7134.h needs asm/io.h
  o [MEDIA]: Fix pointer-->int cast in saa5249.c
  o [MEDIA]: Make read operations return correct ssize_t in
    {c-qcam,bw-qcam,w9966}.c
  o [MEDIA]: Make VIDEO_PMS depend upon ISA
  o [MEDIA]: Make read method return ssize_t in cpia.c
  o [MEDIA]: bttvp.h needs asm/io.h
  o [MEDIA]: Fix u64 printing in bttv driver
  o [MEDIA]: bttv-cards.c needs linux/vmalloc.h
  o [SPARC64]: Update defconfig
  o [SPARC64]: Use Makefile to control dec_and_lock.o building not
    in-file ifdefs
  o [SCSI]: Make dc395x.c build again
  o [SPARC64]: Add some more symbol debugging in register dumps
  o [STRING]: Fix bug in generic strncpy() change
  o [SPARC64]: Propagate bprm->interp changes to sparc 32-bit compat
    layer
  o [SPARC64]: More tomatillo PCI controller fixes
  o [TG3]: More Sun onboard 5704 fixes
  o [SPARC64]: Update defconfig
  o [TG3]: Only call tg3_init_rings() after hardware has been reset
  o [SPARC64]: Fix AFSR error reporting for Cheetah+/Jalapeno
  o [SPARC64]: Missing cheetah+ ASI defines

David T. Hollis:
  o USB: AX8817x mii/ethtool fixes among others
  o USB: ax8817x.c - Fix flags to greatly increase rx performance

Eric Sandeen:
  o [XFS] Use C99 initializers on sysctl structs
  o [XFS] Catch read-only filesystems in xfs_setattr, and return EROFS

Felipe Damasio:
  o drivers/char/stallion.c: devfs_mk_cdev fix

François Romieu:
  o fix typo in drivers/net/arcnet/com20020-isa.c

Geert Uytterhoeven:
  o [IPSEC]: ipcomp.c does not neet net/esp.h

Gerd Knorr:
  o v4l: sysfs'ify videodev
  o v4l: bttv driver update

Greg Kroah-Hartman:
  o I2C: minor cleanups to the i2c-nforce2 driver
  o I2C: consolidate the i2c delay functions
  o USB: fix stupid kobject coding error with regards to struct
    usb_interface
  o USB: core cleanups for struct usb_interface changes
  o USB: changes due to struct usb_interface changing from a pointer to
    an array of pointers
  o USB: remove improper use of devinitdata markings for device ids
  o USB: Compile AX8817x driver
  o USB: AX8817x (USB ethernet) problem in 2.6.0-test1
  o USB: fix bug if open() fails in usb-serial device
  o USB: remove funny characters from visor driver after much prodding
  o USB: bluetty: remove write_urb_pool logic, fixing locking issues
  o Cset exclude: greg@kroah.com|ChangeSet|20030730200104|44589
  o USB: fix memory leak in auerswald driver
  o USB: added support for TIOCM_RI and TIOCM_CD to pl2303 driver and
    fix stupid bug
  o PCI: pci_device_id can not be marked __devinitdata.  Fixes up
    sound/*
  o PCI: pci_device_id can not be marked __devinitdata
  o PCI: pci_device_id can not be marked __devinitdata
  o PCI: pci_device_id can not be marked __devinitdata
  o PCI: pci_device_id can not be marked __devinitdata
  o PCI: pci_device_id can not be marked __devinitdata
  o I2C: remove devinitdata marking from i2c-nforce2.c as it's wrong
  o USB: fix up ALSA merge due to struct usb_interface changes
  o PCI: merge fixups
  o USB: remove some vendor specific stuff from the pl2303 driver to
    get other devices to work
  o USB: remove all struct device.name usage from the USB code
  o PCI: remove all struct device.name usage from the PCI core code
  o PCI: make eepro100 driver use pci_name() instead of dev.name
  o USB: remove dev.name usage from gadget code
  o I2C: move the name field back into the i2c_client and i2c_adapter
    structures
  o I2C: fix up driver model programming error

Harald Welte:
  o [NETFILTER]: Use in-kernel IPSEC structures in iptables ah (by
    Patrick McHardy)
  o [NETFILTER]: Use in-kernel IPSEC structures in iptables esp (by
    Patrick McHardy)

Herbert Xu:
  o [IPSEC]: Use per-SA flag to control ECN propagation
  o [IPSEC]: Fix SKB secpath refcounting
  o [XFRM]: Fix OOPS in xfrm_state_update
  o [XFRM_USER]: Pass correct args to xfrm_find_acq()
  o [XFRM_USER]: Fix spurious NLMSG_ERROR messages

Hirofumi Ogawa:
  o Fix the vfat dentry handling fix
  o uses ->i_pos instead of ->i_ino on fat_fs_panic()
  o Consolidate dot-stripping code
  o Inline vfat_strnicmp()
  o more use fat_get_short_entry()

Ian Abbott:
  o USB: ftdi_sio - additional pids
  o USB: ftdi_sio - VID/PID for ID TECH IDT1221U USB to RS-232 adapter

Ian Wienand:
  o ia64: hardirq.h should include smp_lock.h when preempt enabled

Ivan Kokshaysky:
  o alpha: pci domains update
  o PCI: pci_enable_device vs bridges bugs

Jan Dittmer:
  o I2C: convert via686a temp_* to milli degree celsius

Jan Kara:
  o Fix old quota format locking

Jaroslav Kysela:
  o ALSA update 0.9.5
  o ALSA 0.9.6 update
  o ALSA update

Javier Achirica:
  o [wireless airo] fix Tx race
  o [wireless airo] safer shutdown sequence
  o [wireless airo] eliminate infinite loop
  o [wireless airo] makes the card passive when entering monitor mode
  o [wireless airo] adds support for noise level reporting (if
    available)
  o [netdrvr airo] Missing defines (only for documentation)
  o [netdrvr airo] MAC type changed to unsigned
  o [netdrvr airo] add missing lines for Wireless Extensions 16
  o [netdrvr airo] MIC support with newer firmware
  o [netdrvr airo] safer unload code
  o [netdrvr airo] Fix adhoc config

Jean Tourrilhes:
  o Donauboe probe fix
  o IrLAP retry count
  o irda-usb probe fix
  o tekram-sir driver fix

Jeff Garzik:
  o [netdrvr bonding] update docs
  o [netdrvr bonding] fix ifenslave build on ia64
  o [tokenring ibmtr_cs] fix build, due to missing ibmtr.c build
  o Cset exclude: jgarzik@redhat.com|ChangeSet|20030731201437|53548
  o intel ich5 irq router, pci ids
  o [netdrvr airo] now that it builds, re-enable wireless_ext
  o [irda] Add VIA FIR driver, via-ircc

Jens Axboe:
  o fix broken blk_start_queue behavior
  o get rid of unused request_queue field queue_wait
  o AS barrier bug
  o Fix bio RW_AHEAD test
  o Proper block queue reference counting
  o scsi_ioctl reference counting
  o floppy smp fix

Jesse Barnes:
  o ia64: Kconfig cleanup
  o ia64: sn2 pci fixes (among others)

Jones Desougi:
  o [IGMP]: linux/igmp.h needs asm/byteorder.h

Judd Montgomery:
  o USB: visor.h[c] USB device IDs documentation

Kai Germaschewski:
  o kbuild: Fix 'make -jN' warning
  o kbuild: [PATCH] fix in-kernel genksyms for parisc symbols
  o kbuild: Fix .config dependency generation
  o kbuild: List deps on generated files in drivers/video/logo/Makefile
  o kbuild: Add relocation information into .lst file

Kochi Takayoshi:
  o ia64: Interrupt polarity fix

Linus Torvalds:
  o Mark the SiS DRM driver as depending on the SiS FBCON support. It
    won't even compile without it.
  o Merge i830 IRQ handler cleanups from DRI CVS tree
  o Update r128 drm driver from DRI CVS tree
  o Fix up AGP merge: agp_memory_reserved got exported twice
  o Document non-standard do_div() calling convention, since it does
    not work like a C function.
  o Fix up ieee1394 compile
  o Make "static const" initializer to zero explicit, since truly
    constant values should not end up in the BSS.
  o "kernel_locked()" needs <linux/smp_lock.h>
  o Fix vfsmnt leak: make autofs4 mount point expiry release the entry
    that we got through the lookup_mnt().
  o Don't try to signal already-zombied threads in zap_threads
  o Merge with DRI CVS tree: fix use-after-free bug in DRM(takedown)
  o Remove bogus temporary file from revision control

Maciej W. Rozycki:
  o defxx: Maintenance + DMA API fixes

Martin Devera:
  o [NET]: Fix bugs in sch_htb packet scheduler

Michael Hunold:
  o Kconfig and Makefile updates
  o DVB core update
  o mt312 DVB frontend update
  o Update MAC handling for various DVB PCI cards
  o TTUSB-DEC driver update
  o Hexium saa7146 driver update

Michal Sojka:
  o USB: fixes for usb-skeleton.c

Miles Bader:
  o Add new include files for v850

Mitchell Blank Jr.:
  o PCI: Trivial DMA-mapping.txt fix
  o PCI: add 2 entries to pci_ids.h
  o [SPARC64]: Control debuglocks.c compilation using Makefile rules

Nathan Scott:
  o [XFS] Ensure the VFS doesn't rip the inode out from beneath us when
    doing unwritten extent conversion
  o [XFS] Avoid doing the page->mapping->host dereferences twice on
    writepage
  o [XFS] Fix couple of issues in max file size calculation, print big
    fs setting too
  o [XFS] Cleanup empty/noaddr pagebuf initialisation; particularly for
    buffers used for log IO - no longer allocate buffers for data
    device then reset target, gets it right from the start
  o [XFS] Max file size tweak for LBD - if LBD enabled on 32 bit
    platforms slightly bigger files are possible
  o [XFS] Fix the resvsp/truncate interaction problem that mikes@av.com
    reported
  o [XFS] Change any references to legal/illegal into valid/invalid -
    apparently this was bad, mkaay?
  o [XFS] Minor cleanups for unwritten extent fix, using one less
    variable on the stack
  o [XFS] Forward port 2.4 pagebuf locking to 2.5, based on Steve's
    analysis - to fix the infamous pagebuf IO completion buglet

Neil Brown:
  o Make sure sunrpc/cache doesn't confuse writers with readers
  o Change NFSEXP_CROSSMNT to NFSEXP_CROSSMOUNT
  o Remove some unused export flags and reserve a new one
  o Protect against NFS requests to create symlinks bigger than one
    page
  o Change atomic_inc to cache_get twice
  o Create a mountpoint for the nfsd filesystem
  o nfsv4 page boundary handling fixes
  o Fix the sunrpc cache/reader management properly

Nemosoft Unv.:
  o USB: PWC 8.11

Oliver Neukum:
  o USB: error return codes in usblp
  o USB: cleanup of usblp (release and poll)
  o USB: fix race condition in usblp_write
  o USB: remove GFP_DMA from pegasus
  o USB: DMA coherency issue with rtl8150
  o USB: use of __devinit in st5481
  o USB: dvb usb driver sleeping in interrupt

Olof Johansson:
  o [RANDOM]: Fix SMP deadlock in __check_and_rekey()

Otto Solares:
  o [SPARC]: Add local.h and sections.h headers

Patrick Dreker:
  o I2C: add ncforce2 i2c bus driver

Patrick McHardy:
  o [NETFILTER]: Fix logging of AH spis

Patrick Mochel:
  o [power] Create drivers/power/
  o [power] Move process suspension functions to their own file
  o [power] Move saved_context_* variables
  o [power] Make CONFIG_ACPI_SLEEP and CONFIG_SOFTWARE_SUSPEND
    independent
  o [power] Remove duplicate target
  o [power] Move drivers/power/ to kernel/power/
  o [power] Kill some unnessecary printk()s
  o [power] Fix up refrigerator to work with ^Z-ed processes
  o [power] Minor swsusp cleanups
  o [power] Divorce suspend console code from swsusp
  o [power] Make sure ACPI prepares a console during S3
  o [power] Fix compilation when CONFIG_PM=n
  o [power] Fix up compiler warnings
  o [power] Remove unused function reset_videobios_after_s3()

Paul Mackerras:
  o PPC32: Compile fix for arch/ppc/kernel/setup.c
  o PPC32: Fix compilation when CPU_FREQ is defined
  o PPC32: Make strncpy clear the unused part of the destination
  o PPC32: Fix the help message for the SANDPOINT config option

Peter Chubb:
  o ia64: Kill a warning if \!CONFIG_SMP

Petr Vandrovec:
  o Remove write-only palette variable from matroxfb. Now it is
    possible to build matroxfb without fbcon support.
  o matroxfb: Initialize fbcon's cmap
  o Add support for panning at vertical blanking to the matroxfb. Now
    mplayer output looks much better on primary output (secondary
    output is always synced with vbl).

Ralf Bächle:
  o MIPS update

Randy Dunlap:
  o janitor: don't init static data (arm26)
  o janitor: add MODULE_LICENSE() in 2 drivers
  o janitor: make serio init/exit static
  o janitor: use char arrays for strings
  o jantior: return -EFAULT on copy_user error
  o janitor: don't init statics to 0
  o don't init statics to 0 (fs/)
  o janitor: convert to pci_name()
  o [NET]: janitor: don't init statics to 0
  o Fix incorrect umem iounmap() call

Richard Henderson:
  o [ALPHA] Fix remaining pci bus conversion
  o [ALPHA] Compressed kernel bootp images
  o [ALPHA] Disable Wildfire, Titan, Marvel if !LEGACY_START_ADDRESS
  o [ALPHA] IRQ updates to match x86 changes
  o [ALPHA] Fix RTC init for LYNX
  o [ALPHA] Enhanced EV6/EV7 error management

Rik van Riel:
  o CREDITS update

Rob Radez:
  o remove unnecessary stubs from watchdog drivers

Roland McGrath:
  o spurious SIGCHLD from dying thread group leader
  o fix spinlock deadlock in ptrace-reaping of detached thread
  o fixes to zap_other_threads fix

Russell King:
  o [ARM] Update ARM ethernet drivers
  o [PCMCIA] Fix suspend for yenta_socket and i82092
  o [PCMCIA] Convert cs.c and rsrc_mgr.c to use named initialisers
  o [ARM] gcc 3.2.x for ARM miscompiles jffs2
  o [ARM] Automatically select compiler options for ARMv5/Xscale CPUs
  o [ARM] Add missing include for clps711x irq handling
  o [ARM] Make PXA compile
  o [ARM] Add generic sections.h for ARM
  o [ARM] Fix do_settimeofday()
  o [ARM] Remove old ARC and A5K machine class
  o [ARM] Remove unused get_hackkit_scr prototype
  o [ARM] Fix SSP IRQ handler return type
  o [ARM] Update machine types file
  o [ARM] Remove more static register accesses from sa1111.c
  o [ARM] Fix two Acorn SCSI drivers
  o [ARM] Fix platform_add_devices() prototype
  o [ARM] Add asm-arm/local.h
  o [netdrvr ARM] alloc_etherdev updates
  o [PCMCIA] Fix cardbus init failure paths
  o [PCMCIA] Disable IRQ steering and don't change the IRQ MUX register
  o [PCMCIA] Report subsystem vendor/device IDs

Sam Ravnborg:
  o usr: Create objectfile for usr filesystem using .incbin
  o fs/Makefile: Eliminate ifneq
  o bk: Ignore drivers/scsi/aic7xxx/aic7xxx_reg_print.c
  o kbuild/doc: Grammatical errors corrected
  o kbuild: Added CONFIG_DEBUG_INFO
  o arch/um: Renamed CONFIG_DEBUGSYM to CONFIG_DEBUG_INFO
  o kbuild/doc: Delete obsolete documentation files
  o docbook: Added support for generating man files
  o kbuild/doc: Delete instructions about running make dep
  o kbuild: Let 'make help' point to README
  o kbuild: Updated/moved around comments in top-level Makefile
  o kbuild: Allow architectures to change CPPFLAGS

Samuel Thibault:
  o [PPP]: Fix ppp_async xon/xoff handling

Sean Estabrooks:
  o I2C: Additional P4B subsystem id for hidden asus smbus

Sridhar Samudrala:
  o [SCTP] ADDIP basic infrastructure support. (Ardelle.Fan)
  o [SCTP] Fix to avoid large kmalloc failures on 64-bit platforms

Stephen Hemminger:
  o [BRIDGE]: Mailing list is at osdl.org now
  o [VLAN]: Allow it to compile with VLAN_DEBUG enabled
  o [VLAN]: Convert VLAN procfs stubs to no-ops
  o [VLAN]: Convert over to seq_file
  o [NET]: Fix rtnetlink symbol exports when INET is disabled
  o [NET]: Add likely/unlikely to pskb_may_pull
  o [BRIDGE]: Fix bridge notification processing
  o [NET]: Convert lp486e over to dynamic allocation
  o [NET] Dynamically allocate net_device structures for ROSE
  o [NET] Convert ROSE to seq_file
  o [NET] Fix use after free in AX.25
  o [NET] Fix X.25 use after free
  o [NET] X.25 async net_device fixup

Stephen Lord:
  o Do not include linux/version.h, avoid massive recompilations
  o Correct the maxbytes sb value for non-pagesize filesystem block
    sizes, and all uses of max file size constant within XFS
  o [XFS] remove extra locking which was folded into the xfs extended
    attribute calls - we do not need this.
  o Encapsulate the setting and clearing of PF_FSTRANS around xfs
    transactions. This is used to protect against nested transactions
    which can cause deadlocks. Also ensures we keep the state set for
    the duration of a chain of rolling transactions.
  o Add an IO completion handler to the direct_IO path to allow the
    initiator to take an action at completion time. XFS uses this to 
  o Rework XFS read/write path so that there is one common read and one
    common write path for all the different I/O variants. This means we
    can now support true async I/O.
  o Change XFS to support sector aligned O_DIRECT rather than
    filesystem block alignment.
  o [XFS] Scale default number of log buffers based on memory size
  o Make the permission operation in xfs conditional on ACLs being
    compiled in
  o Only add bytes read into the stats if we did not get an error
  o Use i_size_read and i_size_write instead of direct access to the
    i_size field.
  o Fix a couple of pagebuf end cases, in particular, deal with block
    device which is not correctly initialized and do not submit a bio
    to it - that trips a BUG.
  o Move from fsid_t to __kernel_fsid_t, fixes ia64 build
  o Do not include linux/version.h - no need for it

Steve French:
  o lookup intents part 2
  o cifs enablement for lookup intents (new 2.5 create/lookup flags)
    final part
  o fix create/open/lookup to use namei intent flags.  Enable oplock
    and packet signing
  o Fix oops in cifs mkdir when server fails to return inode info after
    successful mkdir
  o Fix oops in cifs reopen files (restoring state after server comes
    back from failure)
  o Oplock flag reversed in create path
  o Fix for file size handling for locally cached files
  o Do not leave files created with mknod open on server. Fix related
    oops.  Fix timeout on open to be at least server oplock break
    timeout to avoid timing out successful opens when secondary client
    hung
  o Fix blocksize and allocation size miscalculation
  o Fix blocksize and allocation size mismatch

Theodore Y. T'so:
  o Fix 32/64-bit bug in ext3

Tomas Szepe:
  o [NETFILTER]: Put netfilter Kconfig options into dedicated submenu

Ville Nuorvala:
  o [IPV6]: Use correct hop-limit in ip6_push_pending_frames()

Wensong Zhang:
  o [IPV4] IPVS missing stats locking in estimation_timer()
  o [IPV4] IPVS: sanity check of threshold setting and code tidy up

Wiktor Wodecki:
  o I2C: i2c sysfs rant

Wim Van Sebroeck:
  o [WATCHDOG] I810_TCO info in Kconfig
  o [WATCHDOG] Cleanup of Kconfig file for the watchdog drivers
  o [WATCHDOG] make wdt_pci.c independant of wdt.c
  o [WATCHDOG] advantechwdt patches

Yoshinori Sato:
  o h8300: config updates
  o h8300: interrupt management update
  o h8300: H8S architecture support
  o h8300: header file updates
  o h8300: arch update
  o h8300: build error fixes



^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: USB still broken.
  2003-08-09  5:40 ` Linux 2.6.0-test3 Linus Torvalds
@ 2003-08-09  7:49   ` reg
  2003-08-13 23:53     ` Greg KH
  2003-08-09 14:53   ` [Bug 973] Re: Linux 2.6.0-test3: Presario laptop panic Thomas Molina
                     ` (5 subsequent siblings)
  6 siblings, 1 reply; 46+ messages in thread
From: reg @ 2003-08-09  7:49 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: Kernel Mailing List, reg


The USB subsystem is still broken in test3.
everything seems to build w/o errors, but late in the boot the screen
is filled with the line

	drivers/usb/input/hid-core.c: control queue full

which repeats a a high rate and is VERY hard to break free from.
Not good.
-- 
                                        Reg.Clemens
                                        reg@dwf.com



^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: [Bug 973] Re: Linux 2.6.0-test3:  Presario laptop panic
  2003-08-09 14:53   ` [Bug 973] Re: Linux 2.6.0-test3: Presario laptop panic Thomas Molina
@ 2003-08-09 13:52     ` Zwane Mwaikambo
  0 siblings, 0 replies; 46+ messages in thread
From: Zwane Mwaikambo @ 2003-08-09 13:52 UTC (permalink / raw)
  To: Thomas Molina; +Cc: Linux Kernel Mailing List

On Sat, 9 Aug 2003, Thomas Molina wrote:

> Please see Bugzilla for the gory details.  Synopsis is that my Preario 
> laptop panics on boot:
> 
> Unable to handle kernel pagin request at virtual address c035800
> 
> EIP 0060:[<c014CE95>]
> EIP is at store_stackinfo+0x85/0xc0
> 
> This behaviour continues since 2.5.74-bk1.  

Thanks for the update, i'll debug it during the course of the weekend.

	Zwane


^ permalink raw reply	[flat|nested] 46+ messages in thread

* [Bug 973] Re: Linux 2.6.0-test3:  Presario laptop panic
  2003-08-09  5:40 ` Linux 2.6.0-test3 Linus Torvalds
  2003-08-09  7:49   ` Linux 2.6.0-test3: USB still broken reg
@ 2003-08-09 14:53   ` Thomas Molina
  2003-08-09 13:52     ` Zwane Mwaikambo
  2003-08-09 16:00   ` Linux 2.6.0-test3: logo patch Thomas Molina
                     ` (4 subsequent siblings)
  6 siblings, 1 reply; 46+ messages in thread
From: Thomas Molina @ 2003-08-09 14:53 UTC (permalink / raw)
  To: Linux Kernel Mailing List; +Cc: Zwane Mwaikambo

Please see Bugzilla for the gory details.  Synopsis is that my Preario 
laptop panics on boot:

Unable to handle kernel pagin request at virtual address c035800

EIP 0060:[<c014CE95>]
EIP is at store_stackinfo+0x85/0xc0

This behaviour continues since 2.5.74-bk1.  



^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: logo patch
  2003-08-09 16:00   ` Linux 2.6.0-test3: logo patch Thomas Molina
@ 2003-08-09 15:38     ` Russell King
  2003-08-11  6:24       ` Martin Schlemmer
  2003-08-09 16:28     ` ismail (cartman) donmez
  1 sibling, 1 reply; 46+ messages in thread
From: Russell King @ 2003-08-09 15:38 UTC (permalink / raw)
  To: Thomas Molina, James Simmons; +Cc: Linus Torvalds, Kernel Mailing List

On Sat, Aug 09, 2003 at 11:00:57AM -0500, Thomas Molina wrote:
> The following patch has been floating around forever.  Can we get it in 
> mainstream sometime in the near future?
> 
> --- linux-2.5-tm/drivers/video/cfbimgblt.c.orig	2003-08-08 17:42:16.000000000 -0500
> +++ linux-2.5-tm/drivers/video/cfbimgblt.c	2003-08-08 17:42:30.000000000 -0500
> @@ -325,7 +325,7 @@
>  		else 
>  			slow_imageblit(image, p, dst1, fgcolor, bgcolor,
>  					start_index, pitch_index);
> -	} else if (image->depth == bpp) 
> +	} else if (image->depth <= bpp) 
>  		color_imageblit(image, p, dst1, start_index, pitch_index);
>  }
>  

Is this patch _still_ not in the kernel.

Linus - please merge this patch - its required for several ARM framebuffer
drivers, and several other drivers.  James has indicated that this is the
correct fix back in May:

On Tue, May 13, 2003 at 11:41:34PM +0100, James Simmons wrote:
> At the very bottom of cfbimgblt.c change
>
> } else if (image->depth == bpp)
>
> to
>
> } else if (image->depth <= bpp)
>
> and tell me if this works.


-- 
Russell King (rmk@arm.linux.org.uk)                The developer of ARM Linux
             http://www.arm.linux.org.uk/personal/aboutme.html


^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: logo patch
  2003-08-09  5:40 ` Linux 2.6.0-test3 Linus Torvalds
  2003-08-09  7:49   ` Linux 2.6.0-test3: USB still broken reg
  2003-08-09 14:53   ` [Bug 973] Re: Linux 2.6.0-test3: Presario laptop panic Thomas Molina
@ 2003-08-09 16:00   ` Thomas Molina
  2003-08-09 15:38     ` Russell King
  2003-08-09 16:28     ` ismail (cartman) donmez
  2003-08-09 21:14   ` Linux 2.6.0-test3 Jacek Kawa
                     ` (3 subsequent siblings)
  6 siblings, 2 replies; 46+ messages in thread
From: Thomas Molina @ 2003-08-09 16:00 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: Kernel Mailing List

The following patch has been floating around forever.  Can we get it in 
mainstream sometime in the near future?

--- linux-2.5-tm/drivers/video/cfbimgblt.c.orig	2003-08-08 17:42:16.000000000 -0500
+++ linux-2.5-tm/drivers/video/cfbimgblt.c	2003-08-08 17:42:30.000000000 -0500
@@ -325,7 +325,7 @@
 		else 
 			slow_imageblit(image, p, dst1, fgcolor, bgcolor,
 					start_index, pitch_index);
-	} else if (image->depth == bpp) 
+	} else if (image->depth <= bpp) 
 		color_imageblit(image, p, dst1, start_index, pitch_index);
 }
 


^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: logo patch
  2003-08-09 16:00   ` Linux 2.6.0-test3: logo patch Thomas Molina
  2003-08-09 15:38     ` Russell King
@ 2003-08-09 16:28     ` ismail (cartman) donmez
  2003-08-09 18:52       ` Andrew Morton
  1 sibling, 1 reply; 46+ messages in thread
From: ismail (cartman) donmez @ 2003-08-09 16:28 UTC (permalink / raw)
  To: Thomas Molina; +Cc: Kernel Mailing List, akpm

Patch somehow got out of -mm tree too. Andrew can you please apply this  too ?
> The following patch has been floating around forever.  Can we get it in
> mainstream sometime in the near future?
>
> --- linux-2.5-tm/drivers/video/cfbimgblt.c.orig	2003-08-08
> 17:42:16.000000000 -0500 +++
> linux-2.5-tm/drivers/video/cfbimgblt.c	2003-08-08 17:42:30.000000000 -0500
> @@ -325,7 +325,7 @@
>  		else
>  			slow_imageblit(image, p, dst1, fgcolor, bgcolor,
>  					start_index, pitch_index);
> -	} else if (image->depth == bpp)
> +	} else if (image->depth <= bpp)
>  		color_imageblit(image, p, dst1, start_index, pitch_index);
>  }
>
>


^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: logo patch
  2003-08-09 16:28     ` ismail (cartman) donmez
@ 2003-08-09 18:52       ` Andrew Morton
  2003-08-12 17:36         ` James Simmons
  0 siblings, 1 reply; 46+ messages in thread
From: Andrew Morton @ 2003-08-09 18:52 UTC (permalink / raw)
  To: ismail (cartman) donmez; +Cc: tmolina, linux-kernel, James Simmons

"ismail (cartman) donmez" <kde@myrealbox.com> wrote:
>
> Patch somehow got out of -mm tree too. Andrew can you please apply this  too ?
> > The following patch has been floating around forever.  Can we get it in
> > mainstream sometime in the near future?
> >
> > --- linux-2.5-tm/drivers/video/cfbimgblt.c.orig	2003-08-08
> > 17:42:16.000000000 -0500 +++
> > linux-2.5-tm/drivers/video/cfbimgblt.c	2003-08-08 17:42:30.000000000 -0500
> > @@ -325,7 +325,7 @@
> >  		else
> >  			slow_imageblit(image, p, dst1, fgcolor, bgcolor,
> >  					start_index, pitch_index);
> > -	} else if (image->depth == bpp)
> > +	} else if (image->depth <= bpp)
> >  		color_imageblit(image, p, dst1, start_index, pitch_index);
> >  }
> >
> >

This change was also present in James's fbdev update, so I dropped it.  But
then James's fbdev update broke so I dropped that too.

James was going to send me an updated patch.  Hint. :)



^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3
  2003-08-09  5:40 ` Linux 2.6.0-test3 Linus Torvalds
                     ` (2 preceding siblings ...)
  2003-08-09 16:00   ` Linux 2.6.0-test3: logo patch Thomas Molina
@ 2003-08-09 21:14   ` Jacek Kawa
  2003-08-09 21:27     ` Jacek Kawa
  2003-08-10 10:40   ` Geert Uytterhoeven
                     ` (2 subsequent siblings)
  6 siblings, 1 reply; 46+ messages in thread
From: Jacek Kawa @ 2003-08-09 21:14 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: Kernel Mailing List, linux-usb-devel, oliver

Linus Torvalds wrote:

[...]

> Oliver Neukum:
>   o USB: error return codes in usblp
>   o USB: cleanup of usblp (release and poll)
>   o USB: fix race condition in usblp_write

[...]

Well, I cannot finish printing anything now and I believe, that one of
above is responsible. 
It looks as if printer gets some amount of data but then communication 
'stops' and printer waits for more data (usually I have about 2cm 
of page printed, paper locked and later printer error leds start
blinking happily :)

Not sure, if it's directly related, but system seems to be less 
responsive later.

Version of usblp.c from 2.6.0-test2 compiled with 2.6.0-test3 does
not work as I expected (I can load module, but printer is not 
detected - no usual 'usblp0: USB Unidirectional printer dev 3 if 0 alt 1
proto 2 vid 0x03F0 pid' message in logs).

I traced my problem down to -test2-bk2 (with bk1 everything works
correctly).

o Printer is connected to /dev/usb/lp0; it's HP840c
o ver_linux output: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/ver_linux
o config: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/config
o dmesg: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/dmesg
o /proc/bus/usb/devices: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/devices
o some related logs: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/logs

Any suggestions are welcome :)

jk

-- 
Jacek Kawa

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3
  2003-08-09 21:14   ` Linux 2.6.0-test3 Jacek Kawa
@ 2003-08-09 21:27     ` Jacek Kawa
  0 siblings, 0 replies; 46+ messages in thread
From: Jacek Kawa @ 2003-08-09 21:27 UTC (permalink / raw)
  To: Linus Torvalds, Kernel Mailing List, linux-usb-devel, oliver

I wrote:

> o Printer is connected to /dev/usb/lp0; it's HP840c
> o ver_linux output: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/ver_linux
> o config: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/config
> o dmesg: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/dmesg
> o /proc/bus/usb/devices: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/devices
> o some related logs: http://zeus.polsl.gliwice.pl/kernel/2.6.0-test3/logs

aieee:
http://zeus.polsl.gliwice.pl/~jfk/kernel/2.6.0-test3/config
http://zeus.polsl.gliwice.pl/~jfk/kernel/2.6.0-test3/devices
http://zeus.polsl.gliwice.pl/~jfk/kernel/2.6.0-test3/dmesg
http://zeus.polsl.gliwice.pl/~jfk/kernel/2.6.0-test3/logs
http://zeus.polsl.gliwice.pl/~jfk/kernel/2.6.0-test3/ver_linux

and one more thing:

PID  USER      PR  NI  VIRT  RES  SHR S %CPU %MEM    TIME+  COMMAND           
2788 root      25   0  2096  576 1888 R 97.7  0.2  22:32.75 usb

> Any suggestions are welcome :)

Sorry!

-- 
Jacek Kawa

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3
  2003-08-09  5:40 ` Linux 2.6.0-test3 Linus Torvalds
                     ` (3 preceding siblings ...)
  2003-08-09 21:14   ` Linux 2.6.0-test3 Jacek Kawa
@ 2003-08-10 10:40   ` Geert Uytterhoeven
  2003-08-10 21:24   ` [patch] 2.6.0-test3: typo in hd.c Adrian Bunk
  2003-08-11 23:07   ` Linux 2.6.0-test3 (compile statistics) John Cherry
  6 siblings, 0 replies; 46+ messages in thread
From: Geert Uytterhoeven @ 2003-08-10 10:40 UTC (permalink / raw)
  To: Stephen Smalley, Andrew Morton, Linus Torvalds; +Cc: Kernel Mailing List

On Fri, 8 Aug 2003, Linus Torvalds wrote:
> Merging the SELinux security architecture also ends up growing the patch, 
> even though it may not be all that noticeable for most normal users.

I need these patches to make it compile for m68k:

--- linux-2.6.0-test3/security/selinux/avc.c	Sat Aug  9 21:43:41 2003
+++ linux-m68k-2.6.0-test3/security/selinux/avc.c	Sun Aug 10 11:09:44 2003
@@ -16,6 +16,7 @@
 #include <linux/slab.h>
 #include <linux/fs.h>
 #include <linux/dcache.h>
+#include <linux/init.h>
 #include <linux/skbuff.h>
 #include <net/sock.h>
 #include <linux/un.h>
--- linux-2.6.0-test3/security/selinux/ss/global.h	Sat Aug  9 21:43:41 2003
+++ linux-m68k-2.6.0-test3/security/selinux/ss/global.h	Sun Aug 10 11:22:59 2003
@@ -7,7 +7,7 @@
 #include <linux/ctype.h>
 #include <linux/in.h>
 #include <linux/spinlock.h>
-#include <asm/semaphore.h>
+#include <linux/sched.h>
 
 #include "flask.h"
 #include "avc.h"

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds


^ permalink raw reply	[flat|nested] 46+ messages in thread

* [patch] 2.6.0-test3: typo in hd.c
  2003-08-09  5:40 ` Linux 2.6.0-test3 Linus Torvalds
                     ` (4 preceding siblings ...)
  2003-08-10 10:40   ` Geert Uytterhoeven
@ 2003-08-10 21:24   ` Adrian Bunk
  2003-08-11 23:07   ` Linux 2.6.0-test3 (compile statistics) John Cherry
  6 siblings, 0 replies; 46+ messages in thread
From: Adrian Bunk @ 2003-08-10 21:24 UTC (permalink / raw)
  To: Jens Axboe; +Cc: Kernel Mailing List

On Fri, Aug 08, 2003 at 10:40:37PM -0700, Linus Torvalds wrote:
>...
> Summary of changes from v2.6.0-test2 to v2.6.0-test3
> ====================================================
>...
> Jens Axboe:
>...
>   o Proper block queue reference counting
>...

This contains a typo that is fixed with the following patch:

--- linux-2.6.0-test3-not-full/drivers/ide/legacy/hd.c.old	2003-08-10 23:21:04.000000000 +0200
+++ linux-2.6.0-test3-not-full/drivers/ide/legacy/hd.c	2003-08-10 23:21:56.000000000 +0200
@@ -715,7 +715,7 @@
 
 	hd_queue = blk_init_queue(do_hd_request, &hd_lock);
 	if (!hd_queue) {
-		unegister_blkdev(MAJOR_NR,"hd");
+		unregister_blkdev(MAJOR_NR,"hd");
 		return -ENOMEM;
 	}
 


cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed


^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: logo patch
  2003-08-09 15:38     ` Russell King
@ 2003-08-11  6:24       ` Martin Schlemmer
  2003-08-11 17:25         ` James Simmons
  0 siblings, 1 reply; 46+ messages in thread
From: Martin Schlemmer @ 2003-08-11  6:24 UTC (permalink / raw)
  To: James Simmons
  Cc: Russell King, Thomas Molina, Linus Torvalds, Kernel Mailing List

On Sat, 2003-08-09 at 17:38, Russell King wrote:
> On Sat, Aug 09, 2003 at 11:00:57AM -0500, Thomas Molina wrote:
> > The following patch has been floating around forever.  Can we get it in 
> > mainstream sometime in the near future?
> > 
> > --- linux-2.5-tm/drivers/video/cfbimgblt.c.orig	2003-08-08 17:42:16.000000000 -0500
> > +++ linux-2.5-tm/drivers/video/cfbimgblt.c	2003-08-08 17:42:30.000000000 -0500
> > @@ -325,7 +325,7 @@
> >  		else 
> >  			slow_imageblit(image, p, dst1, fgcolor, bgcolor,
> >  					start_index, pitch_index);
> > -	} else if (image->depth == bpp) 
> > +	} else if (image->depth <= bpp) 
> >  		color_imageblit(image, p, dst1, start_index, pitch_index);
> >  }
> >  
> 
> Is this patch _still_ not in the kernel.
> 
> Linus - please merge this patch - its required for several ARM framebuffer
> drivers, and several other drivers.  James has indicated that this is the
> correct fix back in May:
> 

Right, this fixes the 'no logo' issue - anybody have any idea why
I now have two logo's ? :)

If need be, I can give dmesg/etc/etc, but not at home right now.

> On Tue, May 13, 2003 at 11:41:34PM +0100, James Simmons wrote:
> > At the very bottom of cfbimgblt.c change
> >
> > } else if (image->depth == bpp)
> >
> > to
> >
> > } else if (image->depth <= bpp)
> >
> > and tell me if this works.
-- 
Martin Schlemmer
Gentoo Linux Developer, Desktop Team
Cape Town, South Africa


^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: logo patch
  2003-08-11  6:24       ` Martin Schlemmer
@ 2003-08-11 17:25         ` James Simmons
  2003-08-11 20:24           ` Martin Schlemmer
  0 siblings, 1 reply; 46+ messages in thread
From: James Simmons @ 2003-08-11 17:25 UTC (permalink / raw)
  To: Martin Schlemmer; +Cc: Kernel Mailing List


> Right, this fixes the 'no logo' issue - anybody have any idea why
> I now have two logo's ? :)

Do you have two CPUs ?


^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: logo patch
  2003-08-11 17:25         ` James Simmons
@ 2003-08-11 20:24           ` Martin Schlemmer
  0 siblings, 0 replies; 46+ messages in thread
From: Martin Schlemmer @ 2003-08-11 20:24 UTC (permalink / raw)
  To: James Simmons; +Cc: Kernel Mailing List

[-- Attachment #1: Type: text/plain, Size: 303 bytes --]

On Mon, 2003-08-11 at 19:25, James Simmons wrote:
> > Right, this fixes the 'no logo' issue - anybody have any idea why
> > I now have two logo's ? :)
> 
> Do you have two CPUs ?

Right, this is what you get when you live in a third world country :P


Thanks,

-- 

Martin Schlemmer




[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 189 bytes --]

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3 (compile statistics)
  2003-08-09  5:40 ` Linux 2.6.0-test3 Linus Torvalds
                     ` (5 preceding siblings ...)
  2003-08-10 21:24   ` [patch] 2.6.0-test3: typo in hd.c Adrian Bunk
@ 2003-08-11 23:07   ` John Cherry
  2003-08-12  0:03     ` Mike Fedyk
  2003-08-13 19:26     ` Adrian Bunk
  6 siblings, 2 replies; 46+ messages in thread
From: John Cherry @ 2003-08-11 23:07 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: Kernel Mailing List

Compile statistics: 2.6.0-test3
Compiler: gcc 3.2.2
Script: http://developer.osdl.org/~cherry/compile/compregress.sh

               bzImage       bzImage        modules
             (defconfig)  (allmodconfig) (allmodconfig)

2.6.0-test3  0 warnings     7 warnings    984 warnings
             0 errors       9 errors       42 errors

2.6.0-test2  0 warnings     7 warnings   1201 warnings
             0 errors       9 errors       43 errors

2.6.0-test1  0 warnings     8 warnings   1319 warnings
             0 errors       9 errors       38 errors



Compile statistics for 2.5 kernels and 2.6 kernels are at:
http://developer.osdl.org/~cherry/compile/


Failure summary:

   drivers/block: 2 warnings, 1 errors
   drivers/char: 191 warnings, 4 errors
   drivers/isdn: 212 warnings, 9 errors
   drivers/media: 17 warnings, 6 errors
   drivers/mtd: 42 warnings, 1 errors
   drivers/net: 214 warnings, 5 errors
   drivers/net: 27 warnings, 5 errors
   drivers/scsi/aic7xxx: 0 warnings, 1 errors
   drivers/scsi: 102 warnings, 10 errors
   drivers/video: 13 warnings, 3 errors
   sound/oss: 51 warnings, 2 errors
   sound: 2 warnings, 2 errors

Warning summary:

   drivers/atm: 12 warnings, 0 errors
   drivers/bluetooth: 2 warnings, 0 errors
   drivers/cdrom: 26 warnings, 0 errors
   drivers/i2c: 3 warnings, 0 errors
   drivers/ide: 29 warnings, 0 errors
   drivers/ieee1394: 1 warnings, 0 errors
   drivers/md: 2 warnings, 0 errors
   drivers/message: 1 warnings, 0 errors
   drivers/pcmcia: 3 warnings, 0 errors
   drivers/scsi/aacraid: 1 warnings, 0 errors
   drivers/scsi/pcmcia: 4 warnings, 0 errors
   drivers/scsi/sym53c8xx_2: 1 warnings, 0 errors
   drivers/serial: 1 warnings, 0 errors
   drivers/telephony: 5 warnings, 0 errors
   drivers/video/aty: 3 warnings, 0 errors
   drivers/video/matrox: 5 warnings, 0 errors
   drivers/video/sis: 1 warnings, 0 errors
   fs/afs: 1 warnings, 0 errors
   fs/intermezzo: 1 warnings, 0 errors
   fs/jffs: 1 warnings, 0 errors
   fs/smbfs: 2 warnings, 0 errors
   net: 35 warnings, 0 errors
   sound/isa: 3 warnings, 0 errors

John




^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3 (compile statistics)
  2003-08-11 23:07   ` Linux 2.6.0-test3 (compile statistics) John Cherry
@ 2003-08-12  0:03     ` Mike Fedyk
  2003-08-13 19:26     ` Adrian Bunk
  1 sibling, 0 replies; 46+ messages in thread
From: Mike Fedyk @ 2003-08-12  0:03 UTC (permalink / raw)
  To: Kernel Mailing List

On Mon, Aug 11, 2003 at 04:07:08PM -0700, John Cherry wrote:
> Compile statistics: 2.6.0-test3
> Compiler: gcc 3.2.2
> Script: http://developer.osdl.org/~cherry/compile/compregress.sh
> 
>                bzImage       bzImage        modules
>              (defconfig)  (allmodconfig) (allmodconfig)
> 
> 2.6.0-test3  0 warnings     7 warnings    984 warnings
> 2.6.0-test2  0 warnings     7 warnings   1201 warnings
                                           ^^^^
					   
Over 200 warnings removed.  Nice.					   

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: logo patch
  2003-08-09 18:52       ` Andrew Morton
@ 2003-08-12 17:36         ` James Simmons
  0 siblings, 0 replies; 46+ messages in thread
From: James Simmons @ 2003-08-12 17:36 UTC (permalink / raw)
  To: Andrew Morton; +Cc: ismail (cartman) donmez, tmolina, linux-kernel


> This change was also present in James's fbdev update, so I dropped it.  But
> then James's fbdev update broke so I dropped that too.
> 
> James was going to send me an updated patch.  Hint. :)

Here you go. 

http://phoenix.infradead.org/~jsimmons/fbdev.diff.gz

Its against 2.6.0-test3. 



^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3 (compile statistics)
  2003-08-11 23:07   ` Linux 2.6.0-test3 (compile statistics) John Cherry
  2003-08-12  0:03     ` Mike Fedyk
@ 2003-08-13 19:26     ` Adrian Bunk
  2003-08-14  0:41       ` John Cherry
  1 sibling, 1 reply; 46+ messages in thread
From: Adrian Bunk @ 2003-08-13 19:26 UTC (permalink / raw)
  To: John Cherry; +Cc: Linus Torvalds, Kernel Mailing List

On Mon, Aug 11, 2003 at 04:07:08PM -0700, John Cherry wrote:
> Compile statistics: 2.6.0-test3
> Compiler: gcc 3.2.2
> Script: http://developer.osdl.org/~cherry/compile/compregress.sh
> 
>                bzImage       bzImage        modules
>              (defconfig)  (allmodconfig) (allmodconfig)
> 
> 2.6.0-test3  0 warnings     7 warnings    984 warnings
>...                          ^^^^^^^^^^

This number is misleading.

As a result of the "uniq" in your script the warning

  include/linux/mca-legacy.h:10:2: warning: #warning "MCA legacy - 
  please move your driver to the new sysfs api"

that occurs in 8 different files is counted only once although this 
problem is in eight different files.

> John

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed


^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3: USB still broken.
  2003-08-09  7:49   ` Linux 2.6.0-test3: USB still broken reg
@ 2003-08-13 23:53     ` Greg KH
  0 siblings, 0 replies; 46+ messages in thread
From: Greg KH @ 2003-08-13 23:53 UTC (permalink / raw)
  To: reg; +Cc: Kernel Mailing List, reg

On Sat, Aug 09, 2003 at 01:49:01AM -0600, reg@dwf.com wrote:
> 
> The USB subsystem is still broken in test3.

Works for me :)

> everything seems to build w/o errors, but late in the boot the screen
> is filled with the line
> 
> 	drivers/usb/input/hid-core.c: control queue full
> 
> which repeats a a high rate and is VERY hard to break free from.
> Not good.

Can you tell the people on the linux-usb-devel mailing list about this,
and file a bug at bugzilla.kernel.org for it?

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: Linux 2.6.0-test3 (compile statistics)
  2003-08-13 19:26     ` Adrian Bunk
@ 2003-08-14  0:41       ` John Cherry
  0 siblings, 0 replies; 46+ messages in thread
From: John Cherry @ 2003-08-14  0:41 UTC (permalink / raw)
  To: Adrian Bunk; +Cc: Linus Torvalds, Kernel Mailing List

I agree that the MCA legacy warnings are a bit misleading.  Fortunately,
we don't have many multiple line warnings in the build.  I'll see what I
can do to update the script for these kind of warnings.

John

On Wed, 2003-08-13 at 12:26, Adrian Bunk wrote:
> On Mon, Aug 11, 2003 at 04:07:08PM -0700, John Cherry wrote:
> > Compile statistics: 2.6.0-test3
> > Compiler: gcc 3.2.2
> > Script: http://developer.osdl.org/~cherry/compile/compregress.sh
> > 
> >                bzImage       bzImage        modules
> >              (defconfig)  (allmodconfig) (allmodconfig)
> > 
> > 2.6.0-test3  0 warnings     7 warnings    984 warnings
> >...                          ^^^^^^^^^^
> 
> This number is misleading.
> 
> As a result of the "uniq" in your script the warning
> 
>   include/linux/mca-legacy.h:10:2: warning: #warning "MCA legacy - 
>   please move your driver to the new sysfs api"
> 
> that occurs in 8 different files is counted only once although this 
> problem is in eight different files.
> 
> > John
> 
> cu
> Adrian


^ permalink raw reply	[flat|nested] 46+ messages in thread

* ~/.git/config ?
@ 2006-05-26 15:28 Anand Kumria
  2006-05-26 16:33 ` Timo Hirvonen
                   ` (2 more replies)
  0 siblings, 3 replies; 46+ messages in thread
From: Anand Kumria @ 2006-05-26 15:28 UTC (permalink / raw)
  To: git

Hi,

git is unable to construct a reasonable default email address in my
current environment.  So, I use GIT_AUTHOR_EMAIL and GIT_COMMITTER_EMAIL
to override things.

This has worked well but, now, I need to vary the email address for some
repositories.  Unfortunately the environment variables override
.git/config.

It would be good if things were like:
	- try to construct one automagically
	- use ~/.git/config (if available)
	- use .git/config
	- use environment variables

That way I could set my default email address in ~/.git/config and
override it as required for those repositories that need it.

Thanks,
Anand

-- 
 `When any government, or any church for that matter, undertakes to say to
  its subjects, "This you may not read, this you must not see, this you are
  forbidden to know," the end result is tyranny and oppression no matter how
  holy the motives' -- Robert A Heinlein, "If this goes on --"

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-26 15:28 ~/.git/config ? Anand Kumria
@ 2006-05-26 16:33 ` Timo Hirvonen
  2006-05-26 16:37   ` Jakub Narebski
  2006-05-26 16:38   ` Petr Baudis
  2006-05-26 17:38 ` Junio C Hamano
  2006-05-28 22:26 ` [PATCH] Read configuration also from ~/.gitrc Petr Baudis
  2 siblings, 2 replies; 46+ messages in thread
From: Timo Hirvonen @ 2006-05-26 16:33 UTC (permalink / raw)
  To: Anand Kumria; +Cc: git

Anand Kumria <wildfire@progsoc.uts.edu.au> wrote:

> Hi,
> 
> git is unable to construct a reasonable default email address in my
> current environment.  So, I use GIT_AUTHOR_EMAIL and GIT_COMMITTER_EMAIL
> to override things.
> 
> This has worked well but, now, I need to vary the email address for some
> repositories.  Unfortunately the environment variables override
> .git/config.
> 
> It would be good if things were like:
> 	- try to construct one automagically
> 	- use ~/.git/config (if available)
> 	- use .git/config
> 	- use environment variables
> 
> That way I could set my default email address in ~/.git/config and
> override it as required for those repositories that need it.

I backup my $HOME using git, so there's a .git directory in ~.  I don't
think a global config file is really needed but it would be nice if
.git/config would override the environment variables, not the other way
around.

-- 
http://onion.dynserv.net/~timo/

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-26 16:33 ` Timo Hirvonen
@ 2006-05-26 16:37   ` Jakub Narebski
  2006-05-26 16:38   ` Petr Baudis
  1 sibling, 0 replies; 46+ messages in thread
From: Jakub Narebski @ 2006-05-26 16:37 UTC (permalink / raw)
  To: git

Timo Hirvonen wrote:

> Anand Kumria <wildfire@progsoc.uts.edu.au> wrote:
> 
>> Hi,
>> 
>> git is unable to construct a reasonable default email address in my
>> current environment.  So, I use GIT_AUTHOR_EMAIL and GIT_COMMITTER_EMAIL
>> to override things.
>> 
>> This has worked well but, now, I need to vary the email address for some
>> repositories.  Unfortunately the environment variables override
>> .git/config.
>> 
>> It would be good if things were like:
>>      - try to construct one automagically
>>      - use ~/.git/config (if available)
>>      - use .git/config
>>      - use environment variables
>> 
>> That way I could set my default email address in ~/.git/config and
>> override it as required for those repositories that need it.
> 
> I backup my $HOME using git, so there's a .git directory in ~.  I don't
> think a global config file is really needed but it would be nice if
> .git/config would override the environment variables, not the other way
> around.

Well, I'm not sure if environmental variables overriding wouldn't make
invocations like 'GIT_DIR=something git command' possible.

There are templates, also for config. Currently git lacks user (not
repository) config file, e.g. ~/.gitconfig (common for all repositories).

-- 
Jakub Narebski
Warsaw, Poland

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-26 16:33 ` Timo Hirvonen
  2006-05-26 16:37   ` Jakub Narebski
@ 2006-05-26 16:38   ` Petr Baudis
  2006-05-26 17:05     ` Timo Hirvonen
  2006-05-27  2:36     ` Pavel Roskin
  1 sibling, 2 replies; 46+ messages in thread
From: Petr Baudis @ 2006-05-26 16:38 UTC (permalink / raw)
  To: Timo Hirvonen; +Cc: Anand Kumria, git

Dear diary, on Fri, May 26, 2006 at 06:33:25PM CEST, I got a letter
where Timo Hirvonen <tihirvon@gmail.com> said that...
> I backup my $HOME using git, so there's a .git directory in ~.

Then it should be called ~/.gitconfig. :-)

> I don't think a global config file is really needed but it would be
> nice if .git/config would override the environment variables, not the
> other way around.

Then you have no other way to override .git/config e.g. when committing
patches submitted by other people.

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
A person is just about as big as the things that make them angry.

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-26 16:38   ` Petr Baudis
@ 2006-05-26 17:05     ` Timo Hirvonen
  2006-05-26 17:11       ` Petr Baudis
  2006-05-27  2:36     ` Pavel Roskin
  1 sibling, 1 reply; 46+ messages in thread
From: Timo Hirvonen @ 2006-05-26 17:05 UTC (permalink / raw)
  To: Petr Baudis; +Cc: wildfire, git

Petr Baudis <pasky@suse.cz> wrote:

> Dear diary, on Fri, May 26, 2006 at 06:33:25PM CEST, I got a letter
> where Timo Hirvonen <tihirvon@gmail.com> said that...
> > I backup my $HOME using git, so there's a .git directory in ~.
> 
> Then it should be called ~/.gitconfig. :-)

I just wanted to point the fact that ~/.git/ could not be used :)

> > I don't think a global config file is really needed but it would be
> > nice if .git/config would override the environment variables, not the
> > other way around.
> 
> Then you have no other way to override .git/config e.g. when committing
> patches submitted by other people.

git commit --author "name <email>"

-- 
http://onion.dynserv.net/~timo/

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-26 17:05     ` Timo Hirvonen
@ 2006-05-26 17:11       ` Petr Baudis
  0 siblings, 0 replies; 46+ messages in thread
From: Petr Baudis @ 2006-05-26 17:11 UTC (permalink / raw)
  To: Timo Hirvonen; +Cc: wildfire, git

Dear diary, on Fri, May 26, 2006 at 07:05:26PM CEST, I got a letter
where Timo Hirvonen <tihirvon@gmail.com> said that...
> > > I don't think a global config file is really needed but it would be
> > > nice if .git/config would override the environment variables, not the
> > > other way around.
> > 
> > Then you have no other way to override .git/config e.g. when committing
> > patches submitted by other people.
> 
> git commit --author "name <email>"

Except that this just sets the environment variables for you. :-)

Now, you could do some really funny stuff with overriding the
environment variables at git commit's entry point with .git/config
stuff, then possibly setting them again in case --author was passed, but
I seriously think such a confusion is not worth it.

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
A person is just about as big as the things that make them angry.

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-26 15:28 ~/.git/config ? Anand Kumria
  2006-05-26 16:33 ` Timo Hirvonen
@ 2006-05-26 17:38 ` Junio C Hamano
  2006-05-28 22:26 ` [PATCH] Read configuration also from ~/.gitrc Petr Baudis
  2 siblings, 0 replies; 46+ messages in thread
From: Junio C Hamano @ 2006-05-26 17:38 UTC (permalink / raw)
  To: Anand Kumria; +Cc: git

Anand Kumria <wildfire@progsoc.uts.edu.au> writes:

> It would be good if things were like:
> 	- try to construct one automagically
> 	- use ~/.git/config (if available)
> 	- use .git/config
> 	- use environment variables
>
> That way I could set my default email address in ~/.git/config and
> override it as required for those repositories that need it.

If you mean by the above "do all of these and take the last
value that was available", that sounds sane.  Except perhaps I
would suggest to use ~/.git-config instead.  Some people seem to
want to track their home directory with git, and that way, your
personal fallback default file can be version controlled.

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-26 16:38   ` Petr Baudis
  2006-05-26 17:05     ` Timo Hirvonen
@ 2006-05-27  2:36     ` Pavel Roskin
  2006-05-27  2:57       ` Petr Baudis
  2006-05-27  8:50       ` ~/.git/config ? Nikolai Weibull
  1 sibling, 2 replies; 46+ messages in thread
From: Pavel Roskin @ 2006-05-27  2:36 UTC (permalink / raw)
  To: Petr Baudis; +Cc: Timo Hirvonen, Anand Kumria, git

On Fri, 2006-05-26 at 18:38 +0200, Petr Baudis wrote:
> Dear diary, on Fri, May 26, 2006 at 06:33:25PM CEST, I got a letter
> where Timo Hirvonen <tihirvon@gmail.com> said that...
> > I backup my $HOME using git, so there's a .git directory in ~.
> 
> Then it should be called ~/.gitconfig. :-)

No, make it .gitrc for compatibility with .cvsrc, .lynxrc and others.

I know, it's becoming a bikeshed issue :-)

-- 
Regards,
Pavel Roskin

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  2:36     ` Pavel Roskin
@ 2006-05-27  2:57       ` Petr Baudis
  2006-05-27  3:20         ` Linus Torvalds
  2006-05-27  8:50       ` ~/.git/config ? Nikolai Weibull
  1 sibling, 1 reply; 46+ messages in thread
From: Petr Baudis @ 2006-05-27  2:57 UTC (permalink / raw)
  To: Pavel Roskin; +Cc: Timo Hirvonen, Anand Kumria, git

Dear diary, on Sat, May 27, 2006 at 04:36:22AM CEST, I got a letter
where Pavel Roskin <proski@gnu.org> said that...
> On Fri, 2006-05-26 at 18:38 +0200, Petr Baudis wrote:
> > Dear diary, on Fri, May 26, 2006 at 06:33:25PM CEST, I got a letter
> > where Timo Hirvonen <tihirvon@gmail.com> said that...
> > > I backup my $HOME using git, so there's a .git directory in ~.
> > 
> > Then it should be called ~/.gitconfig. :-)
> 
> No, make it .gitrc for compatibility with .cvsrc, .lynxrc and others.

~/.gitrc might get useful for actually doing what ~/.cvsrc or ~/.cgrc
does, that is providing default options for git commands. ~/.gitconfig
would just give you per-user defaults for the repository config file.

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
A person is just about as big as the things that make them angry.

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  2:57       ` Petr Baudis
@ 2006-05-27  3:20         ` Linus Torvalds
  2006-05-27  3:36           ` Linus Torvalds
                             ` (3 more replies)
  0 siblings, 4 replies; 46+ messages in thread
From: Linus Torvalds @ 2006-05-27  3:20 UTC (permalink / raw)
  To: Petr Baudis; +Cc: Pavel Roskin, Timo Hirvonen, Anand Kumria, git



On Sat, 27 May 2006, Petr Baudis wrote:
> 
> ~/.gitrc might get useful for actually doing what ~/.cvsrc or ~/.cgrc
> does, that is providing default options for git commands. ~/.gitconfig
> would just give you per-user defaults for the repository config file.

I don't think the two are necessarily any different.

I do think that
 (a) we might as well use the same syntax. There's no point in having 
     different syntax for the files, even if they end up having slightly 
     different usage.
 (b) a "user-wide" config file would tend to have different things in it 
     than a per-repository one, but some of the things it would have in it 
     are the things that we currently put in the per-repository one. 
     Notably exactly the "user.name" and "user.email" values.
 (c) having a user- (and perhaps a system-) wide config file would make 
     some things that we do not _yet_ support in the .git/config file 
     format more natural, but that doesn't necessarily mean that having 
     them on a per-repo basis would be wrong either.

As an example of (c), let's say that somebody wants to use the CVS aliases 
with git. They've used cvs for years, and as a result their brain has 
atrophied, and they have a really hard time teaching their fingers to 
write "git commit" when they want to write "cvs ci".

So they would do "alias cvs git" in a desperate attempt to save themselves 
from CVS, and then add

	[alias "co"]
		cmd = commit -a

to their .gitrc file. Now, the only thing we'd need to do is to teach 
"git.c" to parse that simple "alias.$1.command" variable, and that really 
sounds pretty damn easy, no?

And that really does make more sense in a user-wide ".gitrc" file. But at 
the same time, there's really no reason to _disallow_ it from the 
repo-private .git/config file either. For example, you could do

	[alias "publish"]
		cmd = push public.site.com:/pub/scm/my-public-repo

and that is somethign that actually makes sense as a per-repository 
command alias, so that "git publish" does the right thing for just _that_ 
repository.

So I would argue that yes, ".gitrc" makes sense, and no, we shouldn't have 
a separate ".gitrc" and ".gitconfig", because I think what we really would 
want is a way to do default .git/config entries, and that it is _also_ a 
very natural way to add some things that we don't do yet.

			Linus

PS. I really like my [alias "cmd"] idea. _All_ my ideas are great, of 
course, but this one seems even better than some others. No?

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  3:20         ` Linus Torvalds
@ 2006-05-27  3:36           ` Linus Torvalds
  2006-05-27  4:43             ` Junio C Hamano
  2006-05-27  3:40           ` Jakub Narebski
                             ` (2 subsequent siblings)
  3 siblings, 1 reply; 46+ messages in thread
From: Linus Torvalds @ 2006-05-27  3:36 UTC (permalink / raw)
  To: Petr Baudis; +Cc: Pavel Roskin, Timo Hirvonen, Anand Kumria, git



On Fri, 26 May 2006, Linus Torvalds wrote:
> 
> So they would do "alias cvs git" in a desperate attempt to save themselves 
> from CVS, and then add
> 
> 	[alias "co"]
> 		cmd = commit -a

That, of course, would be confusing.

It should be "ci" for commit, and "co" for "checkout".

Duh. Chalk it up to me not having used cvs in the last three or four years 
or so (we used it at transmeta, although I can hope that they've since 
seen the light ;).

			Linus

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  3:20         ` Linus Torvalds
  2006-05-27  3:36           ` Linus Torvalds
@ 2006-05-27  3:40           ` Jakub Narebski
  2006-05-27  9:11           ` Junio C Hamano
  2006-05-27 12:52           ` Horst von Brand
  3 siblings, 0 replies; 46+ messages in thread
From: Jakub Narebski @ 2006-05-27  3:40 UTC (permalink / raw)
  To: git

Linus Torvalds wrote:

> PS. I really like my [alias "cmd"] idea. _All_ my ideas are great, of 
> course, but this one seems even better than some others. No?

So, would we use it also for default options of commands, like

        [alias "log"]
                cmd = log -p -M -C

(we could implement "git whatchanges" as an alias, keeping it's man page, or
sharing it with "git log").

-- 
Jakub Narebski
Warsaw, Poland

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  3:36           ` Linus Torvalds
@ 2006-05-27  4:43             ` Junio C Hamano
  0 siblings, 0 replies; 46+ messages in thread
From: Junio C Hamano @ 2006-05-27  4:43 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: git

Linus Torvalds <torvalds@osdl.org> writes:

>> 	[alias "co"]
>> 		cmd = commit -a
>
> That, of course, would be confusing.
>
> It should be "ci" for commit, and "co" for "checkout".
>
> Duh. Chalk it up to me not having used cvs in the last three or four years 
> or so (we used it at transmeta, although I can hope that they've since 
> seen the light ;).
>
> 			Linus

You are lucky ;-).

The alias would be a nice addition -- I can get rid of the hack
I have for show-branch.  We would need an argument splitter, but
that should be trivial.

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  2:36     ` Pavel Roskin
  2006-05-27  2:57       ` Petr Baudis
@ 2006-05-27  8:50       ` Nikolai Weibull
  2006-05-27  9:09         ` Timo Hirvonen
  1 sibling, 1 reply; 46+ messages in thread
From: Nikolai Weibull @ 2006-05-27  8:50 UTC (permalink / raw)
  To: Pavel Roskin; +Cc: Petr Baudis, Timo Hirvonen, Anand Kumria, git

On 5/27/06, Pavel Roskin <proski@gnu.org> wrote:
> On Fri, 2006-05-26 at 18:38 +0200, Petr Baudis wrote:
> > Dear diary, on Fri, May 26, 2006 at 06:33:25PM CEST, I got a letter
> > where Timo Hirvonen <tihirvon@gmail.com> said that...
> > > I backup my $HOME using git, so there's a .git directory in ~.
> >
> > Then it should be called ~/.gitconfig. :-)
>
> No, make it .gitrc for compatibility with .cvsrc, .lynxrc and others.
>
> I know, it's becoming a bikeshed issue :-)

I want to paint part of it as well!

Wouldn't we be futureproofing ourselves by stuffing it in a
subdirectory instead?  What if we want to add more files later?

Also, can we please make sure to make the location of the file (or
directory) configurable through an environment variable, e.g.,
GIT_CONFIG, GIT_USER_CONFIG, GIT_HOME, or GIT_USER_HOME?

Peace.

  nikolai

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  8:50       ` ~/.git/config ? Nikolai Weibull
@ 2006-05-27  9:09         ` Timo Hirvonen
  0 siblings, 0 replies; 46+ messages in thread
From: Timo Hirvonen @ 2006-05-27  9:09 UTC (permalink / raw)
  To: Nikolai Weibull; +Cc: proski, pasky, wildfire, git

"Nikolai Weibull" <now@bitwi.se> wrote:

> On 5/27/06, Pavel Roskin <proski@gnu.org> wrote:
> > On Fri, 2006-05-26 at 18:38 +0200, Petr Baudis wrote:
> > >
> > > Then it should be called ~/.gitconfig. :-)
> >
> > No, make it .gitrc for compatibility with .cvsrc, .lynxrc and others.
> >
> > I know, it's becoming a bikeshed issue :-)
> 
> I want to paint part of it as well!
> 
> Wouldn't we be futureproofing ourselves by stuffing it in a
> subdirectory instead?  What if we want to add more files later?

Makes sense, we may want to put templates to ~/.gitsomething/templates/.

-- 
http://onion.dynserv.net/~timo/

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  3:20         ` Linus Torvalds
  2006-05-27  3:36           ` Linus Torvalds
  2006-05-27  3:40           ` Jakub Narebski
@ 2006-05-27  9:11           ` Junio C Hamano
  2006-05-27 12:52           ` Horst von Brand
  3 siblings, 0 replies; 46+ messages in thread
From: Junio C Hamano @ 2006-05-27  9:11 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: git

Linus Torvalds <torvalds@osdl.org> writes:

> On Sat, 27 May 2006, Petr Baudis wrote:
>> 
>> ~/.gitrc might get useful for actually doing what ~/.cvsrc or ~/.cgrc
>> does, that is providing default options for git commands. ~/.gitconfig
>> would just give you per-user defaults for the repository config file.
>
> I don't think the two are necessarily any different.

Me neither.  Default options to fall back on if repository
configuration does not specify them and when environments do not
override them -- that's exactly "per-user defaults", nothing else.

Earlier I suggested ~/.git-config, but anything other than
~/.git/config is OK.  ~/.gitrc is fine.  And the order suggested
by Anand sounded fine.

 - use environment (if the value is obtained, stop here)
 - use .git/config (if the value is obtained, stop here)
 - use ~/.git-config (if the value is obtained, stop here)
 - do whatever the program does by default.

>  (a) we might as well use the same syntax. There's no point in having 
>      different syntax for the files, even if they end up having slightly 
>      different usage.

Absolutely.

>  (b) a "user-wide" config file would tend to have different things in it 
>      than a per-repository one, but some of the things it would have in it 
>      are the things that we currently put in the per-repository one. 
>      Notably exactly the "user.name" and "user.email" values.

Yes.  Also "preference" kind of things as opposed to "per
project policy" such as apply.whitespace.

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: ~/.git/config ?
  2006-05-27  3:20         ` Linus Torvalds
                             ` (2 preceding siblings ...)
  2006-05-27  9:11           ` Junio C Hamano
@ 2006-05-27 12:52           ` Horst von Brand
  2006-05-28 21:59             ` [PATCH] Support for configurable git command aliases Petr Baudis
  3 siblings, 1 reply; 46+ messages in thread
From: Horst von Brand @ 2006-05-27 12:52 UTC (permalink / raw)
  To: Linus Torvalds
  Cc: Petr Baudis, Pavel Roskin, Timo Hirvonen, Anand Kumria, git

Linus Torvalds <torvalds@osdl.org> wrote:
> On Sat, 27 May 2006, Petr Baudis wrote:
> > 
> > ~/.gitrc might get useful for actually doing what ~/.cvsrc or ~/.cgrc
> > does, that is providing default options for git commands. ~/.gitconfig
> > would just give you per-user defaults for the repository config file.

Yep.

[...]

> So they would do "alias cvs git" in a desperate attempt to save themselves 
> from CVS, and then add
> 
> 	[alias "co"]
> 		cmd = commit -a

I don't like this syntax. What other stuff (beside "cmd") would be under
"[alias "co"]? Why not simply:

        [alias]
		co = commit -a
		publish = push public.site.com:/pub/scm/my-public-repo
-- 
Dr. Horst H. von Brand                   User #22616 counter.li.org
Departamento de Informatica                     Fono: +56 32 654431
Universidad Tecnica Federico Santa Maria              +56 32 654239
Casilla 110-V, Valparaiso, Chile                Fax:  +56 32 797513

^ permalink raw reply	[flat|nested] 46+ messages in thread

* [PATCH] Support for configurable git command aliases
  2006-05-27 12:52           ` Horst von Brand
@ 2006-05-28 21:59             ` Petr Baudis
  2006-05-28 22:57               ` Jakub Narebski
  2006-05-29  2:01               ` Junio C Hamano
  0 siblings, 2 replies; 46+ messages in thread
From: Petr Baudis @ 2006-05-28 21:59 UTC (permalink / raw)
  To: Horst von Brand
  Cc: Linus Torvalds, Pavel Roskin, Timo Hirvonen, Anand Kumria, git

Dear diary, on Sat, May 27, 2006 at 02:52:35PM CEST, I got a letter
where Horst von Brand <vonbrand@inf.utfsm.cl> said that...
> > So they would do "alias cvs git" in a desperate attempt to save themselves 
> > from CVS, and then add
> > 
> > 	[alias "co"]
> > 		cmd = commit -a
> 
> I don't like this syntax. What other stuff (beside "cmd") would be under
> "[alias "co"]? Why not simply:
> 
>         [alias]
> 		co = commit -a
> 		publish = push public.site.com:/pub/scm/my-public-repo

Nice, I like this.

Well, the following isn't exactly the nicest code I have ever written...
But it seems to work. ;-)

---

This patch adds support for configurable aliases for git commands -
"alias.WHATEVER = which ever" will kick in when you do "git WHATEVER"
and substitute WHATEVER with "which ever" (splitted to arguments at
whitespaces).

Signed-off-by: Petr Baudis <pasky@suse.cz>
---

 Documentation/config.txt |    5 ++++
 Documentation/git.txt    |    3 +++
 git.c                    |   52 ++++++++++++++++++++++++++++++++++++++++++++--
 3 files changed, 58 insertions(+), 2 deletions(-)

diff --git a/Documentation/config.txt b/Documentation/config.txt
index d1a4bec..ce616e3 100644
--- a/Documentation/config.txt
+++ b/Documentation/config.txt
@@ -83,6 +83,11 @@ core.warnAmbiguousRefs::
 	If true, git will warn you if the ref name you passed it is ambiguous
 	and might match multiple refs in the .git/refs/ tree. True by default.
 
+alias.*::
+	Command aliases for the gitlink:git[1] command wrapper - e.g.
+	after defining "alias.last = cat-file commit HEAD", the invocation
+	"git last" is equivalent to "git cat-file commit HEAD".
+
 apply.whitespace::
 	Tells `git-apply` how to handle whitespaces, in the same way
 	as the '--whitespace' option. See gitlink:git-apply[1].
diff --git a/Documentation/git.txt b/Documentation/git.txt
index 24ca55d..e474bdf 100644
--- a/Documentation/git.txt
+++ b/Documentation/git.txt
@@ -21,6 +21,9 @@ link:everyday.html[Everyday Git] for a u
 "man git-commandname" for documentation of each command.  CVS users may
 also want to read link:cvs-migration.html[CVS migration].
 
+The COMMAND is either a name of a Git command (see below) or an alias
+as defined in the configuration file (see gitlink:git-repo-config[1]).
+
 OPTIONS
 -------
 --version::
diff --git a/git.c b/git.c
index 10ea934..0d9cd0e 100644
--- a/git.c
+++ b/git.c
@@ -10,6 +10,7 @@ #include <limits.h>
 #include <stdarg.h>
 #include "git-compat-util.h"
 #include "exec_cmd.h"
+#include "cache.h" /* setup_git_directory_gently() */
 
 #include "builtin.h"
 
@@ -87,13 +88,27 @@ static void handle_internal_command(int 
 	}
 }
 
+static const char *cmd;
+static char *cmdalias;
+
+int git_alias_config(const char *var, const char *value)
+{
+	if (strncmp(var, "alias.", 6))
+		return 0;
+	var += /* strlen("alias.") */ 6;
+	if (!strcmp(var, cmd))
+		cmdalias = strdup(value);
+	return 0;
+}
+
 int main(int argc, const char **argv, char **envp)
 {
-	const char *cmd = argv[0];
-	char *slash = strrchr(cmd, '/');
+	char *slash = strrchr(argv[0], '/');
 	char git_command[PATH_MAX + 1];
 	const char *exec_path = NULL;
 
+	cmd = argv[0];
+
 	/*
 	 * Take the basename of argv[0] as the command
 	 * name, and the dirname as the default exec_path
@@ -165,6 +180,39 @@ int main(int argc, const char **argv, ch
 	}
 	argv[0] = cmd;
 
+	/* Is this an alias? */
+	{
+		/* XXX: We do a redundant git directory detection. */
+		int nongit = 0;
+		const char *subdir = setup_git_directory_gently(&nongit);
+
+		if (!nongit) {
+			git_config(git_alias_config);
+			if (cmdalias) {
+				/* More than the worst case: */
+				const char **argv2 = malloc((strlen(cmdalias) + argc) * sizeof(char*));
+				int argc2 = 0, i = 1;
+
+				while (cmdalias && *cmdalias) {
+					argv2[argc2++] = strsep(&cmdalias, " \t");
+					if (cmdalias)
+						while (*cmdalias == ' ' || *cmdalias == '\t')
+							cmdalias++;
+				}
+				while (i < argc) {
+					argv2[argc2++] = argv[i++];
+				}
+				argv2[argc2] = NULL;
+				argv = argv2;
+				argc = argc2;
+			}
+		}
+
+		/* Go back so that the commands start with clean table */
+		if (subdir)
+			chdir(subdir);
+	}
+
 	/*
 	 * We search for git commands in the following order:
 	 *  - git_exec_path()


-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
A person is just about as big as the things that make them angry.

^ permalink raw reply related	[flat|nested] 46+ messages in thread

* [PATCH] Read configuration also from ~/.gitrc
  2006-05-26 15:28 ~/.git/config ? Anand Kumria
  2006-05-26 16:33 ` Timo Hirvonen
  2006-05-26 17:38 ` Junio C Hamano
@ 2006-05-28 22:26 ` Petr Baudis
  2006-05-29  7:20   ` Johannes Schindelin
  2 siblings, 1 reply; 46+ messages in thread
From: Petr Baudis @ 2006-05-28 22:26 UTC (permalink / raw)
  To: Anand Kumria; +Cc: git

  Hi,

Dear diary, on Fri, May 26, 2006 at 05:28:37PM CEST, I got a letter
where Anand Kumria <wildfire@progsoc.uts.edu.au> said that...
> git is unable to construct a reasonable default email address in my
> current environment.  So, I use GIT_AUTHOR_EMAIL and GIT_COMMITTER_EMAIL
> to override things.
> 
> This has worked well but, now, I need to vary the email address for some
> repositories.  Unfortunately the environment variables override
> .git/config.
> 
> It would be good if things were like:
> 	- try to construct one automagically
> 	- use ~/.git/config (if available)
> 	- use .git/config
> 	- use environment variables
> 
> That way I could set my default email address in ~/.git/config and
> override it as required for those repositories that need it.

  hmm, might it be as simple as this?

---

This command makes Git read configuration from ~/.gitrc in addition
to the per-repository .git/config configuration file, and updates
the documentation accordingly (and also expands it a little).

Idea by Anand Kumria.

Signed-off-by: Petr Baudis <pasky@suse.cz>
---

 Documentation/git-commit-tree.txt |    9 +++++----
 Documentation/git-repo-config.txt |    6 +++---
 Documentation/git-var.txt         |    5 +++--
 Documentation/git.txt             |   15 +++++++++++----
 config.c                          |    6 +++++-
 5 files changed, 27 insertions(+), 14 deletions(-)

diff --git a/Documentation/git-commit-tree.txt b/Documentation/git-commit-tree.txt
index 41d1a1c..1b0d102 100644
--- a/Documentation/git-commit-tree.txt
+++ b/Documentation/git-commit-tree.txt
@@ -49,8 +49,9 @@ A commit encapsulates:
 - committer name and email and the commit time.
 
 If not provided, "git-commit-tree" uses your name, hostname and domain to
-provide author and committer info. This can be overridden by
-either `.git/config` file, or using the following environment variables.
+provide author and committer info. This can be overridden by either the
+`~/.gitrc` file, the `.git/config` file, or using the following
+environment variables.
 
 	GIT_AUTHOR_NAME
 	GIT_AUTHOR_EMAIL
@@ -60,8 +61,8 @@ either `.git/config` file, or using the 
 
 (nb "<", ">" and "\n"s are stripped)
 
-In `.git/config` file, the following items are used for GIT_AUTHOR_NAME and
-GIT_AUTHOR_EMAIL:
+In the configuration file, the following items are used for GIT_AUTHOR_NAME
+and GIT_AUTHOR_EMAIL:
 
 	[user]
 		name = "Your Name"
diff --git a/Documentation/git-repo-config.txt b/Documentation/git-repo-config.txt
index 660c18f..bb7f81f 100644
--- a/Documentation/git-repo-config.txt
+++ b/Documentation/git-repo-config.txt
@@ -3,7 +3,7 @@ git-repo-config(1)
 
 NAME
 ----
-git-repo-config - Get and set options in .git/config
+git-repo-config - Get and set git runtime configuration options
 
 
 SYNOPSIS
@@ -37,7 +37,7 @@ no checks or transformations are perform
 
 This command will fail if:
 
-. The .git/config file is invalid,
+. The configuration file is invalid,
 . Can not write to .git/config,
 . no section was provided,
 . the section or key is invalid,
@@ -70,7 +70,7 @@ OPTIONS
 	Remove all matching lines from .git/config.
 
 -l, --list::
-	List all variables set in .git/config.
+	List all variables set in ~/.gitrc or .git/config.
 
 
 EXAMPLE
diff --git a/Documentation/git-var.txt b/Documentation/git-var.txt
index a5b1a0d..4679aef 100644
--- a/Documentation/git-var.txt
+++ b/Documentation/git-var.txt
@@ -18,8 +18,9 @@ OPTIONS
 -------
 -l::
 	Cause the logical variables to be listed. In addition, all the
-	variables of the git configuration file .git/config are listed
-	as well. (However, the configuration variables listing functionality
+	variables of the git configuration files `~/.gitrc` and `.git/config`
+	are listed as well.
+	(However, the configuration variables listing functionality
 	is deprecated in favor of `git-repo-config -l`.)
 
 EXAMPLE
diff --git a/Documentation/git.txt b/Documentation/git.txt
index e474bdf..f4e5df5 100644
--- a/Documentation/git.txt
+++ b/Documentation/git.txt
@@ -116,7 +116,7 @@ gitlink:git-read-tree[1]::
 	Reads tree information into the index.
 
 gitlink:git-repo-config[1]::
-	Get and set options in .git/config.
+	Get and set git runtime configuration options.
 
 gitlink:git-unpack-objects[1]::
 	Unpacks objects out of a packed archive.
@@ -473,8 +473,7 @@ gitlink:gitk[1]::
 Configuration Mechanism
 -----------------------
 
-Starting from 0.99.9 (actually mid 0.99.8.GIT), `.git/config` file
-is used to hold per-repository configuration options.  It is a
+You can adjust the Git behaviour by a configuration file.  It is a
 simple text file modelled after `.ini` format familiar to some
 people.  Here is an example:
 
@@ -496,7 +495,15 @@ #
 ------------
 
 Various commands read from the configuration file and adjust
-their operation accordingly.
+their operation accordingly. See gitlink:git-repo-config[1]
+for details and list of options.
+
+Git first reads the per-user global configuration from `~/.gitrc`
+and then per-repository configuration from the `.git/config` file.
+Either of these files may be missing; the per-repository configuration
+wins in case of a conflict. Some behaviour can be also tweaked using
+environment variables; in general, they take precedence over configuration
+options.
 
 
 Identifier Terminology
diff --git a/config.c b/config.c
index 0248c6d..8a98865 100644
--- a/config.c
+++ b/config.c
@@ -312,7 +312,11 @@ int git_config_from_file(config_fn_t fn,
 
 int git_config(config_fn_t fn)
 {
-	return git_config_from_file(fn, git_path("config"));
+	int ret = 0;
+	if (getenv("HOME"))
+		ret += git_config_from_file(fn, mkpath("%s/.gitrc", getenv("HOME")));
+	ret += git_config_from_file(fn, git_path("config"));
+	return ret;
 }
 
 /*


-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
A person is just about as big as the things that make them angry.

^ permalink raw reply related	[flat|nested] 46+ messages in thread

* Re: [PATCH] Support for configurable git command aliases
  2006-05-28 21:59             ` [PATCH] Support for configurable git command aliases Petr Baudis
@ 2006-05-28 22:57               ` Jakub Narebski
  2006-05-29  3:58                 ` Jeff King
  2006-05-29  2:01               ` Junio C Hamano
  1 sibling, 1 reply; 46+ messages in thread
From: Jakub Narebski @ 2006-05-28 22:57 UTC (permalink / raw)
  To: git

Petr Baudis wrote:

> Dear diary, on Sat, May 27, 2006 at 02:52:35PM CEST, I got a letter
> where Horst von Brand <vonbrand@inf.utfsm.cl> said that...
>>
>> I don't like this syntax. What other stuff (beside "cmd") would be under
>> "[alias "co"]? Why not simply:
>> 
>>         [alias]
>>              co = commit -a
>>              publish = push public.site.com:/pub/scm/my-public-repo
> 
> Nice, I like this.
> 
> Well, the following isn't exactly the nicest code I have ever written...
> But it seems to work. ;-)

Well, if [alias] would be used also for giving default options for git
commands, e.g.

        [alias]
            log = log --pretty

(from what Petr "Pasky" Baudis said on IRC, currently on the right side
there should be only true git commands, which eliminates nicely problems
with recursion ;-), it would be better to ensure that aliases are checked
*only* for interactive sessions - otherwise using aliases for default
arguments would/could mess the scripts.

-- 
Jakub Narebski
Warsaw, Poland

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: [PATCH] Support for configurable git command aliases
  2006-05-28 21:59             ` [PATCH] Support for configurable git command aliases Petr Baudis
  2006-05-28 22:57               ` Jakub Narebski
@ 2006-05-29  2:01               ` Junio C Hamano
  2006-05-29  8:02                 ` Petr Baudis
  1 sibling, 1 reply; 46+ messages in thread
From: Junio C Hamano @ 2006-05-29  2:01 UTC (permalink / raw)
  To: Petr Baudis; +Cc: git

Petr Baudis <pasky@ucw.cz> writes:

>> I don't like this syntax. What other stuff (beside "cmd") would be under
>> "[alias "co"]? Why not simply:
>> 
>>         [alias]
>> 		co = commit -a
>> 		publish = push public.site.com:/pub/scm/my-public-repo
>
> Nice, I like this.

Sorry, I don't.  The left hand side of '=' does not allow
anything but alnum and squashes the case.  Please stick to
[alias "co"] syntax.

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: [PATCH] Support for configurable git command aliases
  2006-05-28 22:57               ` Jakub Narebski
@ 2006-05-29  3:58                 ` Jeff King
  0 siblings, 0 replies; 46+ messages in thread
From: Jeff King @ 2006-05-29  3:58 UTC (permalink / raw)
  To: git

On Mon, May 29, 2006 at 12:57:26AM +0200, Jakub Narebski wrote:

> Well, if [alias] would be used also for giving default options for git
> commands, e.g.
> 
>         [alias]
>             log = log --pretty

A funny side effect of that would be that "git log" and "git-log" would
now have completely different defaults (unless the alias mechanism
starts intercepting direct calls to git-*, which seems invasive). I
wonder if it might be simpler to just add 
  core.defaults.log = "--pretty"
or similar.

-Peff

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: [PATCH] Read configuration also from ~/.gitrc
  2006-05-28 22:26 ` [PATCH] Read configuration also from ~/.gitrc Petr Baudis
@ 2006-05-29  7:20   ` Johannes Schindelin
  2006-05-29 14:00     ` Jakub Narebski
  0 siblings, 1 reply; 46+ messages in thread
From: Johannes Schindelin @ 2006-05-29  7:20 UTC (permalink / raw)
  To: Petr Baudis; +Cc: Anand Kumria, git

Hi,

On Mon, 29 May 2006, Petr Baudis wrote:

> diff --git a/config.c b/config.c
> index 0248c6d..8a98865 100644
> --- a/config.c
> +++ b/config.c
> @@ -312,7 +312,11 @@ int git_config_from_file(config_fn_t fn,
>  
>  int git_config(config_fn_t fn)
>  {
> -	return git_config_from_file(fn, git_path("config"));
> +	int ret = 0;
> +	if (getenv("HOME"))
> +		ret += git_config_from_file(fn, mkpath("%s/.gitrc", getenv("HOME")));
> +	ret += git_config_from_file(fn, git_path("config"));
> +	return ret;
>  }
>  
>  /*


But would this not break for the normal case? If you override one key in 
the repository's config, with this patch, repo-config will barf. The 
normal case is that you do not expect multiple values for the same key. 
Your patch reads both ~/.gitrc and $GIT_DIR/config, and if a key has a 
value in both (even if they are identical), repo-config will error out.

Further, storing a key will no longer work. This is an obscure side 
effect of this patch not caring about storing anything in ~/.gitrc: If you 
find the key section (or the key) in ~/.gitrc, the offset will be stored, 
_and used on $GIT_DIR/config_!

I agree it is nice to have a global git configuration, but I have it: I 
use templates.

Ciao,
Dscho

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: [PATCH] Support for configurable git command aliases
  2006-05-29  2:01               ` Junio C Hamano
@ 2006-05-29  8:02                 ` Petr Baudis
  0 siblings, 0 replies; 46+ messages in thread
From: Petr Baudis @ 2006-05-29  8:02 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

Dear diary, on Mon, May 29, 2006 at 04:01:48AM CEST, I got a letter
where Junio C Hamano <junkio@cox.net> said that...
> Petr Baudis <pasky@ucw.cz> writes:
> 
> >> I don't like this syntax. What other stuff (beside "cmd") would be under
> >> "[alias "co"]? Why not simply:
> >> 
> >>         [alias]
> >> 		co = commit -a
> >> 		publish = push public.site.com:/pub/scm/my-public-repo
> >
> > Nice, I like this.
> 
> Sorry, I don't.  The left hand side of '=' does not allow
> anything but alnum and squashes the case.

Does that really matter that much? Perhaps we might support something
like

	"!ooOk" = commit -a

(and it will probably not do what the user expects if he sticks
whitespaces in).

> Please stick to [alias "co"] syntax.

That sucks, e.g. because it's alias-specific, and it's inconsistent.
I can't have anything like "cgalias" then.

Let's make some effort to keep the syntax clean...

-- 
				Petr "Pasky" Baudis
Stuff: http://pasky.or.cz/
A person is just about as big as the things that make them angry.

^ permalink raw reply	[flat|nested] 46+ messages in thread

* Re: [PATCH] Read configuration also from ~/.gitrc
  2006-05-29  7:20   ` Johannes Schindelin
@ 2006-05-29 14:00     ` Jakub Narebski
  0 siblings, 0 replies; 46+ messages in thread
From: Jakub Narebski @ 2006-05-29 14:00 UTC (permalink / raw)
  To: git

Johannes Schindelin wrote:

> But would this not break for the normal case? If you override one key in 
> the repository's config, with this patch, repo-config will barf. The 
> normal case is that you do not expect multiple values for the same key. 
> Your patch reads both ~/.gitrc and $GIT_DIR/config, and if a key has a 
> value in both (even if they are identical), repo-config will error out.

So the patch was to simplistic. Values from user's configuration file
~/.gitrc should be marked, to be overridden by $GIT_DIR/config per
repository configuration file.
 
> Further, storing a key will no longer work. This is an obscure side 
> effect of this patch not caring about storing anything in ~/.gitrc: If you 
> find the key section (or the key) in ~/.gitrc, the offset will be stored, 
> _and used on $GIT_DIR/config_!

I think that storing a key should (unless new option --user-config or
--global is used) should store it in $GIT_DIR/config file; of course index
has to be found there, and if not found it key should be created. Per
configuration file offsets?

> I agree it is nice to have a global git configuration, but I have it: I 
> use templates.

There are system-wide templates. git-init-db(1) doesn't show default
directory for _user_ templates...


And I guess that these are the issues why Junio C Hamano wrote:

> * The 'pu' branch, in addition, has these.
> 
>  - $HOME/.gitrc (Petr Baudis)
>       Read configuration also from ~/.gitrc
> 
>    * I like this but it breaks the tests big time.  Not "next"
>      material yet, unfortunately.
 

-- 
Jakub Narebski
Warsaw, Poland

^ permalink raw reply	[flat|nested] 46+ messages in thread

end of thread, other threads:[~2006-05-29 14:01 UTC | newest]

Thread overview: 46+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-05-26 15:28 ~/.git/config ? Anand Kumria
2006-05-26 16:33 ` Timo Hirvonen
2006-05-26 16:37   ` Jakub Narebski
2006-05-26 16:38   ` Petr Baudis
2006-05-26 17:05     ` Timo Hirvonen
2006-05-26 17:11       ` Petr Baudis
2006-05-27  2:36     ` Pavel Roskin
2006-05-27  2:57       ` Petr Baudis
2006-05-27  3:20         ` Linus Torvalds
2006-05-27  3:36           ` Linus Torvalds
2006-05-27  4:43             ` Junio C Hamano
2006-05-27  3:40           ` Jakub Narebski
2006-05-27  9:11           ` Junio C Hamano
2006-05-27 12:52           ` Horst von Brand
2006-05-28 21:59             ` [PATCH] Support for configurable git command aliases Petr Baudis
2006-05-28 22:57               ` Jakub Narebski
2006-05-29  3:58                 ` Jeff King
2006-05-29  2:01               ` Junio C Hamano
2006-05-29  8:02                 ` Petr Baudis
2006-05-27  8:50       ` ~/.git/config ? Nikolai Weibull
2006-05-27  9:09         ` Timo Hirvonen
2006-05-26 17:38 ` Junio C Hamano
2006-05-28 22:26 ` [PATCH] Read configuration also from ~/.gitrc Petr Baudis
2006-05-29  7:20   ` Johannes Schindelin
2006-05-29 14:00     ` Jakub Narebski
     [not found] <torvalds@osdl.org>
2003-08-09  5:40 ` Linux 2.6.0-test3 Linus Torvalds
2003-08-09  7:49   ` Linux 2.6.0-test3: USB still broken reg
2003-08-13 23:53     ` Greg KH
2003-08-09 14:53   ` [Bug 973] Re: Linux 2.6.0-test3: Presario laptop panic Thomas Molina
2003-08-09 13:52     ` Zwane Mwaikambo
2003-08-09 16:00   ` Linux 2.6.0-test3: logo patch Thomas Molina
2003-08-09 15:38     ` Russell King
2003-08-11  6:24       ` Martin Schlemmer
2003-08-11 17:25         ` James Simmons
2003-08-11 20:24           ` Martin Schlemmer
2003-08-09 16:28     ` ismail (cartman) donmez
2003-08-09 18:52       ` Andrew Morton
2003-08-12 17:36         ` James Simmons
2003-08-09 21:14   ` Linux 2.6.0-test3 Jacek Kawa
2003-08-09 21:27     ` Jacek Kawa
2003-08-10 10:40   ` Geert Uytterhoeven
2003-08-10 21:24   ` [patch] 2.6.0-test3: typo in hd.c Adrian Bunk
2003-08-11 23:07   ` Linux 2.6.0-test3 (compile statistics) John Cherry
2003-08-12  0:03     ` Mike Fedyk
2003-08-13 19:26     ` Adrian Bunk
2003-08-14  0:41       ` John Cherry

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.