Hello, On Wed, Oct 25, 2017 at 03:52:56PM +0530, Rameshwar Prasad Sahu wrote: > @@ -288,8 +289,8 @@ > ), > > TP_fast_assign( > - __entry->ata_port = dev->link->ap->print_id; > - __entry->ata_dev = dev->link->pmp + dev->devno; > + __entry->ata_port = link->ap->print_id; > + __entry->ata_dev = link->pmp + link->device->devno;
The above is wrong if there are multiple devices on the link. It probably should take both link and dev and use dev iff it's not NULL. Thanks. -- tejun
