[git pull] FireWire fixes

2014-03-08 Thread Stefan Richter
sion since v3.10. Stefan Richter (2): firewire: net: fix use after free firewire: ohci: fix probe failure with Agere/LSI controllers drivers/firewire/net.c | 6 +++--- drivers/firewire/ohci.c | 15 ++- 2 files changed, 5 insertions(+), 16 deletions(-) Thanks. -- Stef

Re: [PATCH 4/9] firewire: don't use PREPARE_DELAYED_WORK

2014-03-05 Thread Stefan Richter
On Feb 21 Stefan Richter wrote: > On Feb 20 Tejun Heo wrote: > > PREPARE_[DELAYED_]WORK() are being phased out. They have few users > > and a nasty surprise in terms of reentrancy guarantee as workqueue > > considers work items to be different if they don't have

Re: memory-barriers.txt again (was Re: [PATCH 4/9] firewire: don't use PREPARE_DELAYED_WORK)

2014-02-23 Thread Stefan Richter
only the first of these two alternatives can > +occur. ^^^ ...these {,three} alternatives... -- Stefan Richter -=-- --=- ==--- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of

memory-barriers.txt again (was Re: [PATCH 4/9] firewire: don't use PREPARE_DELAYED_WORK)

2014-02-22 Thread Stefan Richter
So rather than deadlock, "the UNLOCK would simply complete". But /why/ does it complete? It is left unclear (to me at least), why it would do so. IOW, what mechanism will make it always proceed to the UNLOCK? Without knowing that, it is left entirely unclear (to me) why the dea

Re: [PATCH 4/9] firewire: don't use PREPARE_DELAYED_WORK

2014-02-21 Thread Stefan Richter
te the users to set the > ->workfn fields instead of overriding work functions using > PREPARE_DELAYED_WORK(). > > It would probably be best to route this with other related updates > through the workqueue tree. > > Compile tested. > > Signed-off-by: Tejun Heo > Cc: Ste

[git pull] FireWire updates post v3.13

2014-01-27 Thread Stefan Richter
debugging documentation firewire: ohci: Turn remote DMA support into a module parameter Stefan Richter (1): firewire: Enable remote DMA above 4 GB Documentation/debugging-via-ohci1394.txt | 41 +++-- Documentation/power/basic-pm-debuggin

[PATCH v2] firewire: Enable remote DMA above 4 GB

2014-01-18 Thread Stefan Richter
ter is opened through firewire-sbp2. Derived from patch "firewire: Enable physical DMA above 4GB" by Peter Hurley from March 27, 2013. Signed-off-by: Stefan Richter --- v2: updated documentation, extended changelog Documentation/debugging-via-ohci1394.txt | 13 + dr

Re: [PATCH] firewire: Enable remote DMA above 4 GB

2014-01-18 Thread Stefan Richter
On Jan 18 Stefan Richter wrote: > This makes all of a machine's memory accessible to remote debugging via > FireWire, using the physical response unit (i.e. RDMA) of OHCI-1394 link > layer controllers. > > This requires actual support by the controller. The only ones currentl

[PATCH] firewire: Enable remote DMA above 4 GB

2014-01-18 Thread Stefan Richter
addresses. Derived from patch "firewire: Enable physical DMA above 4GB" by Peter Hurley from March 27, 2013. Signed-off-by: Stefan Richter --- drivers/firewire/core-transaction.c |6 +++--- drivers/firewire/core.h |3 +++ drivers/firewire/ohci.c |8

Re: [PATCH] docs: Update FireWire debugging documentation

2014-01-12 Thread Stefan Richter
On Dec 23 Stefan Richter wrote: > CC'ing linux1394-devel, quoting in full. (forgot to Cc akpm, who pointed me to the patch in the first place.) > On Dec 22 Lubomir Rintel wrote: > > The old firewire stack is long dead now and a new version firescope has been > > released w

Re: security review needed - Re: [PATCH] ohci: Turn remote DMA support into a module parameter

2014-01-12 Thread Stefan Richter
On Dec 23 Stefan Richter wrote: > On Dec 22 Lubomir Rintel wrote: > > This makes it possible to debug kernel over FireWire without the need to > > recompile it. > > > > Cc: Stefan Richter > > Cc: Dave Hansen > > Signed-off-by: Lubomir Rintel >

security review needed - Re: [PATCH] ohci: Turn remote DMA support into a module parameter

2013-12-23 Thread Stefan Richter
On Dec 22 Lubomir Rintel wrote: > This makes it possible to debug kernel over FireWire without the need to > recompile it. > > Cc: Stefan Richter > Cc: Dave Hansen > Signed-off-by: Lubomir Rintel Looks good to me. A load-time option is preferable over a compile-time option

Re: [PATCH] docs: Update FireWire debugging documentation

2013-12-23 Thread Stefan Richter
gt; Signed-off-by: Lubomir Rintel Reviewed-by: Stefan Richter > --- > Documentation/debugging-via-ohci1394.txt | 24 +--- > Documentation/power/basic-pm-debugging.txt | 2 +- > 2 files changed, 10 insertions(+), 16 deletions(-) Rob, will you be taking this or sho

[git pull] FireWire fixlet

2013-12-22 Thread Stefan Richter
gy targets which could malfunction when being subjected to this command are already sufficiently protected by a scsi_level check in sd + SCSI core. Stefan Richter (1): firewire: sbp2: bring back WRITE SAME support drivers/firewire/sbp2.c | 1 - 1 file changed, 1 deletion

[git pull] FireWire updates post v3.11

2013-09-08 Thread Stefan Richter
deadlocked between transaction completion handling and bus reset handling if the worker pool could not be increased in time. - janitorial updates Backports of the regression fix for -stable and -longterm will be posted after upstream merge. Stefan Richter (3): firewire: core: typecast

Re: Proposed stable release changes

2013-08-24 Thread Stefan Richter
ween 2.5 months ( = a mainline kernel release period) and more than a year ( = two release periods of typical distributions). -- Stefan Richter -=-===-= =--- ==--- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to

Re: [PATCH 4/9] firewire: WQ_NON_REENTRANT is meaningless and going away

2013-07-30 Thread Stefan Richter
o the for-next branch. > -- 8< --- > dbf2576e37 ("workqueue: make all workqueues non-reentrant") made > WQ_NON_REENTRANT no-op and the flag is going away. Remove its usages. > > This patch doesn't introduce any behavior changes. > > Signed-off-by

[git pull] FireWire regression fix

2013-07-29 Thread Stefan Richter
eintroduce the old behaviour for old clients. Tested-by: Stepan Salenikovich Tested-by: Josep Bosch Cc: # 3.4+ Signed-off-by: Clemens Ladisch Signed-off-by: Stefan Richter diff --git a/drivers/firewire/core-cdev.c b/drivers/firewire/core-cdev.c index 7ef316f..ac1b

[git pull] FireWire updates post v3.10

2013-07-07 Thread Stefan Richter
4 protocol drivers. This is especially useful to 1394 audio drivers for model-specific parameters and methods. Stefan Richter (2): firewire: introduce fw_driver.probe and .remove methods firewire: remove support of fw_driver.driver.probe and .remove methods drivers/firewire/core-device.c

Re: [PATCH] firewire: introduce fw_driver.probe and .remove methods

2013-06-09 Thread Stefan Richter
ver.driver.probe and .remove methods". This means that starting from tomorrow, any FireWire driver which goes into linux-next _must_ implement the new methods. -- Stefan Richter -=-===-= -==- -=--= http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe li

Re: [PATCH] firewire: introduce fw_driver.probe and .remove methods

2013-06-02 Thread Stefan Richter
On Jun 02 Greg Kroah-Hartman wrote: > On Mon, Jun 03, 2013 at 12:27:49AM +0200, Stefan Richter wrote: > > Although all in-tree drivers are being converted to the new methods, > > support for the old methods is left in place for the time being, aiming > > to avoid conflicts if

[PATCH] firewire: remove support of fw_driver.driver.probe and .remove methods

2013-06-02 Thread Stefan Richter
After all drivers being converted to bus-specific .probe/.remove methods, remove support of the obsolete generic methods. Signed-off-by: Stefan Richter --- To be merged maybe one release cycle after patch "firewire: introduce fw_driver.probe and .remove methods". drivers/firewire/cor

[PATCH] firewire: introduce fw_driver.probe and .remove methods

2013-06-02 Thread Stefan Richter
error return code. Although all in-tree drivers are being converted to the new methods, support for the old methods is left in place for the time being, aiming to avoid conflicts if/when new drivers are being merged into the mainline via other trees. Signed-off-by: Stefan Richter Cc: Clemens Ladisc

Re: How to get driver_data of struct ieee1394_device_id in kernel driver module?

2013-05-29 Thread Stefan Richter
On May 27 Greg Kroah-Hartman wrote: > Why not pass it in the probe() function, like USB and PCI does? That > way, if the driver wants to save it for that device, it can. I will try that, probably at the weekend. -- Stefan Richter -=-===-= -=-= ===-= http://arcgraph.de/sr/ -- To unsub

Re: How to get driver_data of struct ieee1394_device_id in kernel driver module?

2013-05-26 Thread Stefan Richter
On May 26 Stefan Richter wrote: > (Adding LKML and Greg KH, in case that there are general opinions about > how a bus_type should work.) > > On May 26 Takashi Sakamoto wrote: > > Hi all, > > > > I'm a newbile for Linux Firewire subsystem and not used to IE

Re: How to get driver_data of struct ieee1394_device_id in kernel driver module?

2013-05-26 Thread Stefan Richter
.driver_data is currently totally useless. How about we make it available like in the following patch? Besides being useful to your presently out-of-tree work, the in-tree sound/firewire/speakers.c::fwspk_detect() could be rewritten to use this approach. Maybe I will post an expanded version of

Re: function call fw_iso_resource_mange(..) (core-iso.c) does not return

2013-05-22 Thread Stefan Richter
ituation on our system. [...] I retract my request to get it retested with mainline since a) the analysis has clearly shown that a mainline bug is at the root of the problem, and b) Ralf confirmed now that one of the possible fixes (using two queues) does in fact work for him like the theory s

Re: function call fw_iso_resource_mange(..) (core-iso.c) does not return

2013-05-22 Thread Stefan Richter
lution 3b --- Remove the higher-level worklet's dependency. E.g. reimplement the higher-level worklet such that it is woken by a timer and then aborts or reschedules ( = lets the lower-level worklet bubble up in the queue). -- Stefan Richter -=-===-= -=-= =-==- http://arcgra

Re: function call fw_iso_resource_mange(..) (core-iso.c) does not return

2013-05-22 Thread Stefan Richter
e a timer to cancel outbound transactions if the request-transmit IRQ event was not received before a timeout. We had such a timeout in the older ieee1394 drivers and we also had it in earlier versions of the firewire drivers, at a risk of a race between CPU and OHCI. -- Stefan Richter -=-=

Re: function call fw_iso_resource_mange(..) (core-iso.c) does not return

2013-05-21 Thread Stefan Richter
> takes too long (>50us). > > The point being that I don't think this is contributing to the > > create_worker() +10ms > > delay. > > That depends on. The console on embedded systems (like ours) often runs > on a serial port with 115200 baud. We measured th

Re: function call fw_iso_resource_mange(..) (core-iso.c) does not return

2013-05-21 Thread Stefan Richter
ing PHY register accesses? (Since we can't get rid of the sleeping ones.) If the accesses are not fully serialized, you will get corrupt PHY reg reads or writes. If they are fully serialized, the non-sleeping PHY reg accesses need to go a try-lock route and will be forced to erro

Re: function call fw_iso_resource_mange(..) (core-iso.c) does not return

2013-05-21 Thread Stefan Richter
del in the normal case (no > work running on the rescuer thread). Maybe it's worth to think about > that because some work looks rather heavy to me. > > Regards, > > Stephan There is quite a bit going on in firewire-ohci's and firewire-core's parts of self-ID-c

[git pull] FireWire updates post v3.9

2013-05-09 Thread Stefan Richter
i: Improve bus reset error messages firewire: ohci: dump_stack() for PHY regs read/write failures Stefan Richter (3): firewire: core: remove an always false test firewire: sbp2: replace BUG_ON by WARN_ON firewire: remove unnecessary alloc/OOM messages drivers/firewire

[PATCH] firewire: core: typecast from gfp_t to bool more safely

2013-04-28 Thread Stefan Richter
restricted gfp_t So let's convert from gfp_t bitfield to Boolean explicitly and safely. Signed-off-by: Stefan Richter --- drivers/firewire/core-cdev.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/firewire/core-cdev.c +++ b/drivers/firewire/core-cdev.c @@ -485,7 +

[PATCH v3] firewire: Fix 'failed to read phy reg' on FW643 rev8

2013-04-28 Thread Stefan Richter
) root This failure no longer happens after the patch, without an obvious reason for the failure or the fix.] [Stefan R: Added quirk flag, quirk table entry, and comment.] Reported-by: Tim Jordan Signed-off-by: Peter Hurley Signed-off-by: Stefan Richter --- drivers/firewire/ohci.c

Re: [PATCH v2] firewire: Fix 'failed to read phy reg' on FW643 rev8

2013-04-28 Thread Stefan Richter
ds. So I believe we should white-list the shorter wait after LPS write only for Agere/LSI cards. I will be replying with a respective patch shortly. -- Stefan Richter -=-===-= -=-- ===-- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

Re: [PATCH v2] firewire: Fix 'failed to read phy reg' on FW643 rev8

2013-04-28 Thread Stefan Richter
} This duplicates a mistake (?) that we already did a few times elsewhere in firewire-ohci: msleep(1) may put the thread to sleep for much, much longer than 1 millisecond. I will propose a patch soon. -- Stefan Richter -=-===-= -=-- ===-- http://arcgraph.de/sr/ -- To unsubscribe from this list:

Re: [PATCH] firewire: Enable physical DMA above 4GB

2013-03-29 Thread Stefan Richter
On Mar 26 Stefan Richter wrote: > On Mar 26 Peter Hurley wrote: > > On Tue, 2013-03-26 at 19:56 +0100, Stefan Richter wrote: > > > It has been a long time though since I last checked whether PhyUpperBound > > > is implemented; maybe it has become more wides

Re: [PATCH] firewire: Enable physical DMA above 4GB

2013-03-26 Thread Stefan Richter
On Mar 26 Peter Hurley wrote: > On Tue, 2013-03-26 at 19:56 +0100, Stefan Richter wrote: > > It has been a long time though since I last checked whether PhyUpperBound > > is implemented; maybe it has become more widespread than it was back then. > > > > Or maybe it h

Re: [PATCH] firewire: Enable physical DMA above 4GB

2013-03-26 Thread Stefan Richter
sn't: All OHCI-1394 chips that ever came to market are 32 bit chips anyway. So the few rare ones that do support PhyUpperBound larger than 4 GB cannot in fact use it. Or am I severely behind the times about this? -- Stefan Richter -=-===-= --== ==-=- http://arcgraph.de/sr/ -- To unsu

[PATCH] firewire: remove unnecessary alloc/OOM messages

2013-03-24 Thread Stefan Richter
These are redundant to log messages from the mm core. Signed-off-by: Stefan Richter --- drivers/firewire/core-cdev.c | 24 +--- drivers/firewire/core-device.c |4 +--- drivers/firewire/net.c |7 +-- drivers/firewire/ohci.c|3 --- drivers

[PATCH] firewire: sbp2: replace BUG_ON by WARN_ON

2013-03-24 Thread Stefan Richter
No need to crash and burn if S/G element sizes cannot be set to our liking; just leave a message in the log. Signed-off-by: Stefan Richter --- drivers/firewire/sbp2.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/firewire/sbp2.c +++ b/drivers/firewire/sbp2.c

[PATCH] firewire: core: remove an always false test

2013-03-24 Thread Stefan Richter
struct fw_cdev_allocate_iso_resource.bandwidth is unsigned. Signed-off-by: Stefan Richter --- drivers/firewire/core-cdev.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) --- a/drivers/firewire/core-cdev.c +++ b/drivers/firewire/core-cdev.c @@ -1365,8 +1365,7 @@ static int

Re: Workqueue behaviour - Synchronization within submitted work.

2013-03-24 Thread Stefan Richter
data are indeed never accessed in other contexts besides the workers. - For serialization in non-atomic context, a struct mutex is usually preferred over a struct semaphore. The foremost advantage of mutexes over counting semaphores is that code using the former can be

Re: [PATCH] firewire: Remove two unneeded checks for macros

2013-03-19 Thread Stefan Richter
ewire/sbp2.c > +++ b/drivers/firewire/sbp2.c > @@ -1636,9 +1636,7 @@ MODULE_LICENSE("GPL"); > MODULE_DEVICE_TABLE(ieee1394, sbp2_id_table); > > /* Provide a module alias so root-on-sbp2 initrds don't break. */ > -#ifndef CONFIG_IEEE1394_SBP2_MODULE > MODULE_ALIAS(

Re: [PATCH 14/14] MAINTAINERS: use same pattern for firewire in uapi

2013-03-03 Thread Stefan Richter
4/linux1394.git > S: Maintained > F: drivers/firewire/ > -F: include/linux/firewire.h > +F: include/linux/firewire*.h > F: include/uapi/linux/firewire*.h > F: tools/firewire/ > I prefer "include/linux/firewire.h". -- Stefan Richter -=-===-= --==

Re: [PATCH linux-next v2] firewire: fw_device_init: 'minor' may be used uninitialized

2013-02-19 Thread Stefan Richter
ter 3.8/ before 3.9-rc1, IOW 3.8 is not affected. (I notified akpm who handles the faulty patch.) -- Stefan Richter -=-===-= --=- =--== http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.o

[PATCH linux-next v2] firewire: fw_device_init: 'minor' may be used uninitialized

2013-02-19 Thread Stefan Richter
zed in this function [-Wuninitialized] Signed-off-by: Tim Gardner Signed-off-by: Stefan Richter --- Andrew, please send this on together with the other idr patches, or even merge this into patch "firewire: convert to idr_alloc()" before the latter reaches upstream. drivers/firewire/core-device.

Re: [PATCH v3] firewire: Fix ohci free_irq() warning

2013-02-17 Thread Stefan Richter
- err = ohci_enable(&ohci->card, NULL, 0); > + err = ohci_enable_regs(&ohci->card, NULL, 0); > if (err) > return err; > > + ohci_enable_irq(ohci); > + ohci_enable_run(ohci); > + > ohci_resume_iso_dma(ohci); >

Re: [PATCH v2 13/62] firewire: convert to idr_alloc()

2013-02-04 Thread Stefan Richter
nor upper limit check. > > Signed-off-by: Tejun Heo > Cc: Stefan Richter > Cc: linux1394-de...@lists.sourceforge.net Acked-by: Stefan Richter > --- > drivers/firewire/core-cdev.c | 19 ++- > drivers/firewire/core-device.c |8 +--- > 2

Re: [PATCH 12.5/62] firewire: add minor number range check to fw_device_init()

2013-02-04 Thread Stefan Richter
On Feb 04 Tejun Heo wrote: > fw_device_init() didn't check whether the allocated minor number isn't > too large. Fail if it goes overflows MINORBITS. > > Signed-off-by: Tejun Heo > Suggested-by: Stefan Richter > Cc: sta...@vger.kernel.org > --- > dri

Re: [PATCH 13/62] firewire: convert to idr_alloc()

2013-02-03 Thread Stefan Richter
On Feb 03 Stefan Richter wrote: > On Feb 02 Tejun Heo wrote: > > --- a/drivers/firewire/core-device.c > > +++ b/drivers/firewire/core-device.c > > @@ -1017,13 +1017,12 @@ static void fw_device_init(struct work_struct *work) > > > > fw_device_get(device); &g

Re: [PATCH 13/62] firewire: convert to idr_alloc()

2013-02-03 Thread Stefan Richter
On Feb 02 Tejun Heo wrote: > Convert to the much saner new idr interface. > > Only compile tested. > > Signed-off-by: Tejun Heo > Cc: Stefan Richter > Cc: linux1394-de...@lists.sourceforge.net > --- > This patch depends on an earlier idr changes and I think it wou

Re: [PATCH] firewire: Fix ohci free_irq() warning

2013-02-02 Thread Stefan Richter
On Feb 02 Alan Stern wrote: > On Sat, 2 Feb 2013, Stefan Richter wrote: > > > On Feb 01 Mark Einon wrote: > > > On 1 February 2013 21:09, Peter Hurley wrote: > > > >>>> On Jan 29 Alan Stern wrote: > > > >>>>> Why does the pci

Re: [PATCH] firewire: Fix ohci free_irq() warning

2013-02-02 Thread Stefan Richter
ything to plug into the port. > I assume that any large changes I make are quite capable of breaking > something there... This is a valid assumption. Some years ago I caused a regression in stable kernel branches in exactly this way myself. -- Stefan Richter -=-===-= --=- ---=- http://arcgrap

Re: [PATCH] firewire: Fix ohci free_irq() warning

2013-02-02 Thread Stefan Richter
On Jan 30 Mark Einon wrote: > On 29 January 2013 16:04, Stefan Richter wrote: > > Added Cc: linux-pm. > > > > On Jan 29 Mark Einon wrote: > >> On 28 January 2013 23:01, Stefan Richter wrote: > >> > On Jan 28 Mark Einon wrote: > >> >> This

Re: [PATCH] firewire: Fix ohci free_irq() warning

2013-01-29 Thread Stefan Richter
Added Cc: linux-pm. On Jan 29 Mark Einon wrote: > On 28 January 2013 23:01, Stefan Richter wrote: > > On Jan 28 Mark Einon wrote: > >> This patch fixes the kernel warning generated when putting an MSI MS-1727 > >> GT740 laptop into suspend mode. The call sequence in th

Re: [PATCH] firewire: Fix ohci free_irq() warning

2013-01-28 Thread Stefan Richter
pci_disable_msi(dev); > err = pci_save_state(dev); > if (err) { > @@ -3807,6 +3807,7 @@ static int pci_resume(struct pci_dev *dev) > return err; > > ohci_resume_iso_dma(ohci); > + enable_irq(dev->irq); > > r

Re: [PATCH 04/14] firewire: don't use idr_remove_all()

2013-01-26 Thread Stefan Richter
On Jan 25 Tejun Heo wrote: > idr_destroy() can destroy idr by itself and idr_remove_all() is being > deprecated. Drop its usage. > > Signed-off-by: Tejun Heo > Cc: Stefan Richter > Cc: linux1394-de...@lists.sourceforge.net > --- > This patch depends on an earlier

Re: [PATCH] firewire net: Ensure checksumming in upper layer.

2013-01-20 Thread Stefan Richter
his actually does the job of Indeed neither the device nor the lower drivers check protocol checksums. But the CRCs of the encapsulating 1394 packets are checked in hardware. Shall protocol checksums be verified regardless? -- Stefan Richter -=-===-= ---= =-=-- http://arcgraph.de/sr/ -- To unsubscrib

Re: [PATCH v2 6/6] staging/fwserial: Drop suggestion for helper fn integration

2012-12-15 Thread Stefan Richter
> On Dec 15 Peter Hurley wrote: > > The firewire core does not require or want the suggested helper fns; > > drop suggestion from TODO file. PS, the patch and the other five look good to me. -- Stefan Richter -=-===-- ==-- - http://arcgraph.de/sr/ -- To unsubscribe from

Re: [PATCH v2 6/6] staging/fwserial: Drop suggestion for helper fn integration

2012-12-15 Thread Stefan Richter
e. [That's too many words about a two- or three-line piece of code; but on the on the other hand it is a generally relevant consideration whenever new functionality is to be added somewhere in the driver stack.] -- Stefan Richter -=-===-- ==-- - http://arcgraph.de/sr/ -- To unsubscrib

[PATCH] MAINTAINERS: adjust for UAPI: firewire

2012-12-12 Thread Stefan Richter
Signed-off-by: Stefan Richter --- MAINTAINERS | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 386fce6..9722918 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3071,7 +3071,8 @@ W:http://ieee1394.wiki.kernel.org/ T: git git

[git pull] FireWire updates post v3.7

2012-12-12 Thread Stefan Richter
allow thin-provisioning related commands - trivia Thanks. Stefan Richter (2): tools/firewire: nosy-dump: check for allocation failure firewire: sbp2: allow WRITE SAME and REPORT SUPPORTED OPERATION CODES Stephan Gatzka (1): firewire: net: Fix handling of fragmented multicast/broadca

Re: [PATCH -next] firewire: net: remove unused variable in fwnet_receive_broadcast()

2012-12-03 Thread Stefan Richter
On Dec 03 Wei Yongjun wrote: > The variable card is initialized but never used > otherwise, so remove the unused variable. Committed to linux1394.git. -- Stefan Richter -=-===-- ==-- ---== http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux

Re: [PATCH v2 1/1] staging: fwserial: Add TTY-over-Firewire serial driver

2012-11-27 Thread Stefan Richter
u ignore the cards actual capability and just allocate 4096 bytes. OHCIs that you can buy offer fw_card.max_receive of 1024, or 2048, or 4096 bytes. 1024 bytes is the limit of many but not all 1394a S400 CardBus cards. [Issues of transaction retries and possible loss at session termination to b

Re: [PATCH v2 1/1] staging: fwserial: Add TTY-over-Firewire serial driver

2012-11-13 Thread Stefan Richter
On Nov 13 Peter Hurley wrote: > On Tue, 2012-11-13 at 00:33 +0100, Stefan Richter wrote: > > On Nov 02 Peter Hurley wrote: > > > +2. MAX_ASYNC_PAYLOAD needs to be publicly exposed by core/ohci > > > + - otherwise how will this driver know the max size of address window

Re: [PATCH v2 1/1] staging: fwserial: Add TTY-over-Firewire serial driver

2012-11-12 Thread Stefan Richter
is contiguously mapped) For the particular application "firewire-net", the present AR-Req DMA buffer size has apparently been determined as suitable. Of course if you find a different optimum, that could certainly be changed. I haven't looked closely, but I suppose you

Re: [PATCH 0/1] staging: Add firewire-serial driver

2012-10-24 Thread Stefan Richter
taging/fwserial/dma_fifo.h | 130 ++ > > drivers/staging/fwserial/fwserial.c | 2885 > > +++++++ > > drivers/staging/fwserial/fwserial.h | 355 + drivers/firewire/net.c is 1721 lines, sbp_target is 2868 lines. Why is fwserial bigger? -- Stefan Richter -=

Re: [PATCH 0/1] staging: Add firewire-serial driver

2012-10-23 Thread Stefan Richter
nsactions: http://marc.info/?l=linux1394-devel&m=109128028930225 That's at the physical layer; link layer and application layer performances are less than that of course. -- Stefan Richter -=-===-- =-=- =-=== http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscr

[PATCH] tools/firewire: nosy-dump: check for allocation failure

2012-10-22 Thread Stefan Richter
Behavior of null pointer dereference is undefined in the C language. Portably implement the desired behavior. Reported-by: Yang Yeping Signed-off-by: Stefan Richter --- tools/firewire/nosy-dump.c |4 1 file changed, 4 insertions(+) --- a/tools/firewire/nosy-dump.c +++ b/tools

new "Cputime accounting" kconfig prompt not "make oldconfig" friendly

2012-10-14 Thread Stefan Richter
vel IRQ time accounting (IRQ_TIME_ACCOUNTING) choice[1-2]: ? There is no help available for this option. I wonder if make oldconfig could be made smarter about presenting help for choice prompts. As a complication, this particular prompt's set of items differs from architecture to arch

[git pull] FireWire fix

2012-10-10 Thread Stefan Richter
Linus, please pull from the tag "firewire-fix" at git://git.kernel.org/pub/scm/linux/kernel/git/ieee1394/linux1394.git firewire-fix to get an old bug of the /dev/fw* ioctl ABI fixed. Thanks. Stefan Richter (1): firewire: cdev: fix user memory corruption (i386 userlan

[PATCH] firewire: cdev: fix user memory corruption (i386 userland on amd64 kernel)

2012-10-06 Thread Stefan Richter
truct fw_cdev_event_bus_reset. Reported-by: Simon Kirby Signed-off-by: Stefan Richter Cc: sta...@kernel.org --- drivers/firewire/core-cdev.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/firewire/core-cdev.c +++ b/drivers/firewire/core-cdev.c @@ -473,8 +473,8 @@ stati

Re: Linux 3.6

2012-10-04 Thread Stefan Richter
would I have it? The > description says it's for SELinux, which I do not use. It says it is /among else/ for SELinux. Another user appears to be ConsoleKit, which wants CONFIG_AUDITSYSCALL, which depends on CONFIG_AUDIT. -- Stefan Richter -=-===-- =-=- --=-- http://arcgraph.de

[git pull] FireWire updates post v3.6

2012-10-03 Thread Stefan Richter
k handling of a TI S400B phy, - avoid holding a core lock while calling into highlevel drivers. Thanks. Clemens Ladisch (1): firewire: core: feed /dev/random with devices' GUIDs Peter Hurley (1): firewire: remove global lock around address handlers, convert to RCU St

Re: [PATCH] firewire: remove global lock around address handlers, convert to RCU

2012-09-28 Thread Stefan Richter
On Sep 27 Stefan Richter wrote: > --- a/drivers/firewire/core-transaction.c > +++ b/drivers/firewire/core-transaction.c > @@ -489,7 +489,7 @@ static struct fw_address_handler *lookup > { > struct fw_address_handler *handler; > > - list_for_each_entr

[PATCH] firewire: addendum to address handler RCU conversion

2012-09-27 Thread Stefan Richter
r() and in fw_core_remove_address_handler(), and document that process context is now required for fw_core_add_address_handler(). - Extend the documentation of fw_address_callback_t. Signed-off-by: Stefan Richter --- drivers/firewire/core-transaction.c | 13 - inc

[PATCH] firewire: remove global lock around address handlers, convert to RCU

2012-09-27 Thread Stefan Richter
DMA tasklet for inbound request events.) In other words, address handlers are now called in an RCU read-side critical section instead of from within a spin_lock_bh serialized section. (Changelog rewritten by Stefan R.) Signed-off-by: Peter Hurley Signed-off-by: Stefan Richter --- Peter, are you

Re: [RFC] DMA mapping error check analysis

2012-09-10 Thread Stefan Richter
to fix this when I get some time. -- Stefan Richter -=-===-- =--= -=-=- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [RFC] DMA mapping error check analysis

2012-09-10 Thread Stefan Richter
7;s fail: path is executed outside the client->lock protected section. That might be a problem. I need to look further into it.) -- Stefan Richter -=-===-- =--= -=-=- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a messa

Intel DZ77BH-55K, iTE IT8892 interrupt misrouting (was Re: [Bug 47191] New: Firewire related BUG in kernels 3.3.7 and up on Intel HW.)

2012-09-07 Thread Stefan Richter
530 94268 12725 14161 > 12829 16951 TLB shootdowns > TRM: 0 0 0 0 0 0 > 0 0 Thermal event interrupts > THR: 0 0 0 0 0 0 >

Re: [PATCH] firewire: core: feed /dev/random with devices' GUIDs

2012-08-12 Thread Stefan Richter
evice->config_rom_retries = 0; > > set_broadcast_channel(device, device->generation); > + > + add_device_randomness(&device->config_rom[3], 8); > } > > /* -- Stefan Richter -=-===-- =--- -==-- http://arcgraph.de/sr/ -

new section mismatch: uncore_types_exit() refers to .init.text:uncore_type_exit()

2012-08-05 Thread Stefan Richter
(). This is often because uncore_types_exit lacks a __init annotation or the annotation of uncore_type_exit is wrong. HTH, -- Stefan Richter -=-===-- =--- --=-= http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

[git pull] FireWire updates post v3.5

2012-07-29 Thread Stefan Richter
IR with buffers larger than 2 GB firewire: ohci: initialize multiChanMode bits after reset firewire: core: add is_local sysfs device attribute Stefan Richter (1): firewire: core: document is_local sysfs attribute Documentation/ABI/stable/sysfs-bus-firewire | 11 +++ d

-next build logs

2008-02-26 Thread Stefan Richter
compiler warnings? Thanks, -- Stefan Richter -=-==--- --=- ==-=- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Pleas

Re: 2.6.25-rc2: ohci1394 problem

2008-02-25 Thread Stefan Richter
ake -j..." is supposed to work. I use it all the time and also did runtime tests of -rc2 and -rc3 built in that way, FWIW. -- Stefan Richter -=-==--- --=- ==--= http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a mes

Re: 2.6.25-rc2: ohci1394 problem

2008-02-25 Thread Stefan Richter
Thanks for retesting. -- Stefan Richter -=-==--- --=- ==--= http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [GIT PULL] FireWire updates

2008-02-25 Thread Stefan Richter
ieee1394/sbp2.c | 15 + > drivers/ieee1394/sbp2.h |2 + > 7 files changed, 350 insertions(+), 109 deletions(-) > > Stefan Richter (19): > firewire: fw-sbp2: unsigned int vs. unsigned > firewire: fw-sbp2: fix logout before login retry >

[GIT PULL] FireWire updates

2008-02-25 Thread Stefan Richter
ivers/firewire/fw-device.h |2 +- drivers/firewire/fw-sbp2.c | 358 +- drivers/ieee1394/sbp2.c | 15 + drivers/ieee1394/sbp2.h |2 + 7 files changed, 350 insertions(+), 109 deletions(-) Stefan Richte

[PATCH 5/5] firewire: refactor fw_unit reference counting

2008-02-24 Thread Stefan Richter
Add wrappers for getting and putting a unit. Remove some line breaks. Signed-off-by: Stefan Richter <[EMAIL PROTECTED]> --- drivers/firewire/fw-device.h | 27 +-- drivers/firewire/fw-sbp2.c |4 ++-- 2 files changed, 19 insertions(+), 12 deletions(-) Index:

[PATCH 4/5] firewire: fw-sbp2: fix reference counting

2008-02-24 Thread Stefan Richter
The reference count of the unit dropped too low in an error path in sbp2_probe. Fixed by moving the _get further up. Signed-off-by: Stefan Richter <[EMAIL PROTECTED]> --- drivers/firewire/fw-sbp2.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) Index: linux/drivers/firewire/f

[PATCH 2/5] firewire: fix crash in automatic module unloading

2008-02-24 Thread Stefan Richter
the node is to be shut down. Signed-off-by: Stefan Richter <[EMAIL PROTECTED]> --- drivers/firewire/fw-card.c| 10 +- drivers/firewire/fw-device.c | 21 ++--- drivers/firewire/fw-device.h | 16 ++-- drivers/firewire/fw-sbp2.c

[PATCH 3/5] firewire: remove superfluous reference counting

2008-02-24 Thread Stefan Richter
The card->kref became obsolete since patch "firewire: fix crash in automatic module unloading" added another counter of card users. Signed-off-by: Stefan Richter <[EMAIL PROTECTED]> --- drivers/firewire/fw-card.c| 38 -- drivers

[PATCH 1/5] firewire: invalid pointers used in fw_card_bm_work

2008-02-24 Thread Stefan Richter
out of fw_card_bm_work's sight during shutdown of the card. Signed-off-by: Stefan Richter <[EMAIL PROTECTED]> --- drivers/firewire/fw-card.c | 51 ++--- drivers/firewire/fw-topology.c |1 2 files changed, 35 insertions(+), 17 deletions(-) Index: linux/dr

[PATCH 0/5] firewire: fix crashes in workqueue jobs

2008-02-24 Thread Stefan Richter
bp2.c|9 ++- drivers/firewire/fw-topology.c|1 drivers/firewire/fw-transaction.h |6 -- 7 files changed, 89 insertions(+), 90 deletions(-) -- Stefan Richter -=-==--- --=- ==--- http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe li

Re: 2.6.25-rc2: ohci1394 problem

2008-02-23 Thread Stefan Richter
NFIG_FIREWIRE_OHCI off in the kernel config of 2.6.25-rc2, rebuild and reinstall the modules and reboot, does the problem vanish? -- Stefan Richter -=-==--- --=- =-=== http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a m

Re: [PATCH 1/2] firewire: endianess fix

2008-02-23 Thread Stefan Richter
ets. (Some PL-3507 only accept unified transactions, hence this test needs to be performed with other bridge chips.) -- Stefan Richter -=-==--- --=- =-=== http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

Re: [PATCH 1/2] firewire: endianess fix

2008-02-23 Thread Stefan Richter
evice in should already cause enough generation changes for the code to trip over an endianess bug before or after thhe patch, me thinks. -- Stefan Richter -=-==--- --=- =-=== http://arcgraph.de/sr/ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body

[PATCH 2/2] firewire: endianess annotations

2008-02-23 Thread Stefan Richter
tion.h:93:10:got restricted unsigned int [usertype] drivers/firewire/fw-ohci.c:1490:8: warning: restricted degrades to integer drivers/firewire/fw-ohci.c:1490:35: warning: restricted degrades to integer drivers/firewire/fw-ohci.c:1516:5: warning: cast to restricted type Signed-off-by: St

<    1   2   3   4   5   6   7   8   9   10   >